On Sun, Nov 05, 2017 at 01:38:47PM -0800, Eric Dumazet wrote: > > diff --git a/drivers/net/bonding/bond_main.c > > b/drivers/net/bonding/bond_main.c > > index c99dc59..728fa08 100644 > > --- a/drivers/net/bonding/bond_main.c > > +++ b/drivers/net/bonding/bond_main.c > > @@ -3237,7 +3237,7 @@ u32 bond_xmit_hash(struct bonding *bond, struct > > sk_buff *skb) > > > > if (bond->params.xmit_policy == BOND_XMIT_POLICY_ENCAP34 && > > skb->l4_hash) > > - return skb->hash; > > + return skb->hash >> 1; > > Why are you changing this part ? > > The l4 hash provided by local TCP stack does not use a pathological > XOR based on ports/addresses, > but a random value with pretty good entropy. >
Oh, my bad. I only tested the patch works, but not carefully check the skb->hash when skb->l4_hash is 1. Will send a v2 patch to fix this. Regards Hangbin
