On Thu, Dec 22, 2016 at 12:32 PM, Maxim Uvarov <maxim.uva...@linaro.org> wrote: > checked that code compiles on that broken compiler. So it will be nice > to merge once approved.
Good to know. I think the changes are good and will enable further optimization, but I'd like clarification about the proposed semantics changes. It's not good to make these sort of changes under the guise of a bug fix. > > Maxim. > > On 12/22/16 17:33, Petri Savolainen wrote: >> Optimized concat operation to avoid packet copy when destination >> packet has room to link source packet segments. Extend tail/head was >> also modified to handle variable sized segments. New validation tests >> were added to improve test coverage of various segmentation scenarios. >> Also couple of bugs were corrected. >> >> Petri Savolainen (7): >> api: packet: src and dst packet must not be the same >> linux-gen: packet: fix bug in tailroom calculation >> linux-gen: packet: improve packet print >> validation: packet: concat-split test bug fix >> linux-gen: packet: optimize concat >> validation: packet: add new concat and extend tests >> linux-gen: config: increase max num of segments >> >> include/odp/api/spec/packet.h | 6 +- >> .../linux-generic/include/odp_buffer_internal.h | 2 +- >> .../linux-generic/include/odp_config_internal.h | 2 +- >> platform/linux-generic/odp_packet.c | 542 >> +++++++++++++++++---- >> test/common_plat/validation/api/packet/packet.c | 500 ++++++++++++++++++- >> test/common_plat/validation/api/packet/packet.h | 5 + >> 6 files changed, 967 insertions(+), 90 deletions(-) >> >
