Thread (4 messages) 4 messages, 2 authors, 2015-01-04

Re: [PATCH net-next] net: More vlan tests before registering netdevice

From: David Miller <davem@davemloft.net>
Date: 2015-01-02 20:56:23

From: Yuval Mintz <redacted>
Date: Wed, 31 Dec 2014 08:35:36 +0200
When register_netdevice() is called, netdevice's vlan filtering feature
and supplied callbacks are checked to see the vlan implementation is
not buggy.
This adds an additional test - see that the vlan_features were filled
correctly, as the vlan devices inherits those as its own features;
Incorrect values set there would later prevent the vlan interface from being
registered itself [as it doesn't implement the filtering ndos].

Signed-off-by: Yuval Mintz <redacted>
 ...
Not sure why take such a defensive approach regarding this feature.
Perhaps it would have been better to simply remove these checks altogether.
I guess this is fine as a defensive test, so I'll apply this.

But do you actually know of any devices which have violated this rule
either now or in the past?  I quickly tried to audit the entire tree
for this right now and found no problems.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help