On Tue, 2016-01-05 at 15:08 -0500, Craig Gallek wrote: > From: Craig Gallek <kr...@google.com> > > This socket-lookup path did not pass along the skb in question > in my original BPF-based socket selection patch. The skb in the > udpN_lib_lookup2 path can be used for BPF-based socket selection just > like it is in the 'traditional' udpN_lib_lookup path. > > udpN_lib_lookup2 kicks in when there are greater than 10 sockets in > the same hlist slot. Coincidentally, I chose 10 sockets per > reuseport group in my functional test, so the lookup2 path was not > excersised. This adds an additional set of tests with 20 sockets. > > Fixes: 538950a1b752 ("soreuseport: setsockopt SO_ATTACH_REUSEPORT_[CE]BPF") > Fixes: 3ca8e4029969 ("soreuseport: BPF selection functional test") > Suggested-by: Eric Dumazet <eric.duma...@gmail.com> > Signed-off-by: Craig Gallek <kr...@google.com> > ---
Great. I'll send the patch to fix udp_dump_one() so that we can remove the test against skb in fast path. Acked-by: Eric Dumazet <eduma...@google.com> -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html