[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <BANLkTi=X_LvQ1-SA5-P6rs3akY2tGbHryQ@mail.gmail.com>
Date: Fri, 22 Apr 2011 13:39:28 +0200
From: Michał Mirosław <mirqus@...il.com>
To: Shan Wei <shanwei@...fujitsu.com>
Cc: David Miller <davem@...emloft.net>,
netdev <netdev@...r.kernel.org>,
Patrick McHardy <kaber@...sh.net>, rusty@...tcorp.com.au,
mst@...hat.com, Eric Dumazet <eric.dumazet@...il.com>,
krkumar2@...ibm.com, mirq-linux@...e.qmqm.pl, joe@...ches.com
Subject: Re: [PATCH net-next 2/3] net: fullly using NETIF_F_GSO_SOFTWARE
macros and NETIF_F_SOFT_FEATURES
2011/4/22 Shan Wei <shanwei@...fujitsu.com>:
> Fullly using NETIF_F_GSO_SOFTWARE and NETIF_F_SOFT_FEATURES.
> And some code style tuning. Just compile test.
[...]
Plese don't replace feature sets with masks of them like that. The
values are the same, but it will confuse readers as to the meaning of
the code.
Best Regards,
Michał Mirosław
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists