[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date: Tue, 17 May 2011 08:48:23 +0300
From: "Michael S. Tsirkin" <mst@...hat.com>
To: Herbert Xu <herbert@...dor.hengli.com.au>
Cc: David Miller <davem@...emloft.net>, mirqus@...il.com,
shanwei@...fujitsu.com, mirq-linux@...e.qmqm.pl,
netdev@...r.kernel.org, bhutchings@...arflare.com
Subject: Re: tap/bridge: Dropping NETIF_F_GSO/NETIF_F_SG
On Tue, May 17, 2011 at 03:24:01PM +1000, Herbert Xu wrote:
> On Tue, May 17, 2011 at 08:18:45AM +0300, Michael S. Tsirkin wrote:
> >
> > Hmm, we get the warnings about bits dropped on each set_offload
> > call:
> > netdev_update_features is called,
> > that calls netdev_fix_features
> >
> > No?
>
> I presume this has changed recently as the current Linus tree
> does not contain a call to netdev_update_features, it only calls
> netdev_features_change.
>
> If so then this does ensure that the feature bits will be sane.
>
> I still think the warnings should be retained at their previous
> level in the call paths other than set_offload. All we have to
> do is add a parameter to netdev_update_features and co.
>
> Cheers,
You mean like bool quiet?
--
MST
--
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