Re: [PATCH] bond: remove ol flags from transmit policy checks

[Thomas Monjalon <thomas.monjalon-pdR9zngts4EAvxtiuMwx3w@public.gmane.org>]

2015-02-02 10:57, Pawel Wodkowski:
> On 2015-01-30 18:02, Declan Doherty wrote:
> > The Link bonding library is incorrectly using receive packet type flags
> > in the transmit policy hashing functions, which would cause packets
> > generated locally to be incorrectly distributed across the slave
> > devices. This patch completely removes the dependency on the packet
> > type flags and uses the ether_type from either the Ethernet header or
> > the VLAN headers for branching.
> >
> > This patch also includes the associate changes in the test suite and in
> > the packet_burst_generator code to remove the dependences on the packet
> > type flags.
> >
> > Signed-off-by: Declan Doherty <declan.doherty-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
> > ---
> >   app/test/packet_burst_generator.c      | 24 +++++++++++-------------
> >   app/test/packet_burst_generator.h      |  3 ++-
> >   app/test/test_link_bonding.c           | 16 ++++++++--------
> >   app/test/test_pmd_perf.c               |  2 +-
> >   lib/librte_pmd_bond/rte_eth_bond_pmd.c | 26 +++++++++++++++-----------
> >   5 files changed, 37 insertions(+), 34 deletions(-)
> >
> 
> Reviewed-by: Wodkowski Pawel <pawelx.wodkowski-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>

Applied

Thanks
-- 
Thomas