Re: [PATCH 1/2] vlan: only create special VLAN 0 once
From: David Miller <davem@davemloft.net>
Date: 2011-06-09 00:05:47
From: Jesse Gross <redacted> Date: Tue, 7 Jun 2011 18:25:23 -0700
No, it's not true. All drivers store the registered vlan filters in some way so that they can restore them when the device is reset. This is currently done in one of two ways: storing a bitmap or iterating over the devices currently registered in a group. The vlan code is moving away from directly accessing groups and no new drivers do this. In fact, once all drivers are converted over groups will not even be registered on devices. This is because otherwise there is quite a bit of vlan code in each driver, which leads to inconsistent behavior and bugs. Really, all a driver needs to know is whether it should add a given vlan to its table, not what the upper layers plan to do with it. So when ndo_vlan_rx_add_vid() is called it should add it to its CAM table and store it if it is needed to restore behavior after a reset, just as is done with all other configuration state.
Thanks for clearing all of this up Jesse.