Thread (4 messages) 4 messages, 3 authors, 2013-01-30

Re: [PATCH net-next] net: disallow drivers with buggy VLAN accel to register_netdevice()

From: Vlad Yasevich <hidden>
Date: 2013-01-30 13:36:16

On 01/29/2013 10:56 PM, David Miller wrote:
From: Vlad Yasevich <redacted>
Date: Tue, 29 Jan 2013 20:17:45 -0500
quoted
On 01/29/2013 08:14 PM, Michał Mirosław wrote:
quoted
Instead of jumping aroung bugs that are easily fixed just don't let
them in:
affected drivers should be either fixed or have NETIF_F_HW_VLAN_FILTER
removed from advertised features.

Quick grep in drivers/net shows two drivers that have
NETIF_F_HW_VLAN_FILTER
but not ndo_vlan_rx_add/kill_vid(), but those are false-positives
(features
are commented out).

OTOH two drivers have ndo_vlan_rx_add/kill_vid() implemented but don't
advertise NETIF_F_HW_VLAN_FILTER. Those are:
I am a bit hesitant to completely disable drivers, but I guess it
would force people to fix their drivers if this is applied.
I think an improperly implemented driver should be BUG trapped as soon
as possible.

I've applied this patch, thanks.
OK,  I'll rebase my series on top of this and get rid the buggy calls.

Thanks
-vlad
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help