On Fri, 2007-11-02 at 14:08 -0400, Dave Johnson wrote:

quoted

*2:  If chip supports 'ASF', tag is always stripped (see *1 above).
     Looks ok if ASF is not supported. 

Michael,

These changes seems to cause this issue:

quoted

 [BNX2]: Fix VLAN on ASF
 
 Always set up the device to strip incoming VLAN tags when ASF is
 enabled. ASF firmware will not parse packets correctly if VLAN tags
 are not stripped.
 
 Signed-off-by: Michael Chan [off-list ref]
 Signed-off-by: David S. Miller [off-list ref]
 
 GIT: e29054f92d7d575631691865c1b95bee5bc974cc

and

quoted

ChangeSet@1.1371.72.2, 2003-12-02 02:34:13-08:00, davem@nuts.ninka.net +1 -0
  [TG3]: Do not set RX_MODE_KEEP_VLAN_TAG when ASF is enabled.

Could you elaborate if this is really needed, if so is there some
workaround that could be done instead?

This is needed for management firmware to work properly.  Management
firmware expects any VLAN tags to be stripped.  Unfortunately, VLAN
stripping cannot be done independently between the driver and the
firmware.  The workaround is to disable management firmware.

Simply removing the check seemed to work for me, but I'm unsure if
this is actually a valid thing to do with these MACs.

Most of these on-board devices are shipped with management firmware
enabled.  Removing the check will make the firmware not functional.

We realize this VLAN limitation is causing problems to many users and we
are looking for ways to address it.