Thread (13 messages) flat view 13 messages, 5 authors, 2011-06-11

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.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help