Two minor aspect related nits, but this looks good to me. Reviewed-and-tested-by: Ciprian Barbu <[email protected]>
On Fri, Apr 10, 2015 at 6:52 PM, Bill Fischofer <[email protected]> wrote: > v4 changes: > - Removed RFC status, patch is now ready for API-NEXT > - Added implementation of approved APIs > - Added user metadata tests to packet validation test > > v3 changes: > - Renamed odp_packet_user_metadata() to odp_packet_user_data() > - Split addr/size return, adding odp_packet_user_data_size() > - Moved udata_size to pkt structure within odp_pool_param_t > > v2 changes: > - Moved udata_size to odp_pool_param_t > - Renamed odp_packet_udata() to odp_packet_user_metadata() > - Removed odp_buffer_udata(). User metadata is for packets only > > RFC for proposed minimal API set for user metadata support > based on today's discussions. Note that all initialization > and management of user metadata contents is the responsibility of > the ODP application. ODP APIs that copy system metadata will also > copy any associated user metadata as part of that operation, but > ODP will otherwise ignore these bytes. > > Bill Fischofer (6): > api: packet: add user metadata APIs > api: pool: add user metadata APIs > linux-generic: buffer: restructure user mdatadata fields > linux-generic: pool: add user metadata support > linux-generic: packet: add user metadata support > validation: packet: add user metadata tests > > include/odp/api/packet.h | 20 ++++++++ > include/odp/api/pool.h | 4 ++ > .../linux-generic/include/odp_buffer_internal.h | 4 +- > platform/linux-generic/include/odp_pool_internal.h | 2 +- > platform/linux-generic/odp_packet.c | 34 ++++++++++++++ > platform/linux-generic/odp_pool.c | 34 +++++--------- > test/validation/odp_packet.c | 54 > ++++++++++++++++++++++ > 7 files changed, 126 insertions(+), 26 deletions(-) > > -- > 2.1.0 > > _______________________________________________ > lng-odp mailing list > [email protected] > https://lists.linaro.org/mailman/listinfo/lng-odp _______________________________________________ lng-odp mailing list [email protected] https://lists.linaro.org/mailman/listinfo/lng-odp
