On 11/23/2015 06:44 AM, Cong Wang wrote: > On Sat, Nov 21, 2015 at 6:57 AM, Nikolay Aleksandrov > <ra...@blackwall.org> wrote: >> net/ipv4/ipmr.c | 191 >> +++++++++++++++++++++++++++++++------------------------- >> 1 file changed, 107 insertions(+), 84 deletions(-) > > Does this really simplify the code? :-/ > Did I really say it does ? :-) Now, to the point it just makes it much easier to reason about this setsockopt which was doing conditional locking in some of the cases before, and some were left out, also "v" was sometimes signed and sometimes unsigned, it's clearer now which type is used. I've left a comment why the only special case needs to unlock rtnl (MRT_DONE).
-- 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