On Sat, 2015-11-07 at 13:35 -0500, David Miller wrote: > From: Eric Dumazet <[email protected]> > Date: Sat, 07 Nov 2015 04:42:56 -0800 > > > The if (!packet_use_direct_xmit(po)) test looks dubious. > > > > Setting transport header has nothing to do with bypassing qdisc ? > > > > This might lead to hard to debug problems, for drivers expecting > > transport header being set ? > > Do we have any such drivers that need it in this scenerio?
Well, imagine following scenario (a real one, as I use it all of time, thus how I discovered all trafgen traffic ends up on one slave only) Even if qdisc is bypassed on the bond0, the current handling does not prevent going to the slave qdiscs. So it is not clear to me why we do a selective probe depending on the bypass of first qdisc. -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
