On Sat, Sep 26, 2015 at 6:44 PM, Aaron Conole <[email protected]> wrote: > Greetings. > > Tolga Ceylan <[email protected]> writes: >> +#define SO_REUSEPORT_LISTEN_OFF 51 >> + > For all of these, I think the space should be tab. > >> unsigned char skc_reuseport:1; >>+ unsigned char skc_reuseport_listen_off:1; >> unsigned char skc_ipv6only:1; > The spacing here is wrong. > >> @@ -224,10 +224,13 @@ begin: >> phash = inet_ehashfn(net, daddr, hnum, >> saddr, sport); >> matches = 1; >> + if (sk->sk_reuseport_listen_off) >> + result = NULL; > I am concerned here. I think you need to reset hiscore and matches as > well, not just result.
By keeping hiscore/matches as is, I'm trying to keep the hashing consistent. Otherwise, this would behave similar to removing a listener which drops connections. -- 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
