login
Header Space

 
 

Re: [PATCH 2/2] [e1000 VLAN] Disable vlan hw accel when promiscuous mode

Score:
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: David Miller <davem@...>
Cc: <kaber@...>, <joonwpark81@...>, <w@...>, <cfriesen@...>, <auke-jan.h.kok@...>, <netdev@...>, <djohnson+linux-kernel@...>, <linux-kernel@...>, <e1000-devel@...>
Date: Tuesday, November 13, 2007 - 8:03 am

On Tue, Nov 13, 2007 at 03:36:11AM -0800, David Miller wrote:

Or perhaps we should just teach everyone to always run tcpdump
with -p, like me :)

Of course this would still have a negative impact on those who
have to be in promiscuous mode all the time (heh) due to multiple
unicast MAC addresses and such.  However, we should able to
communicate that fact to the driver and the driver can then elect
to not disable VLAN acceleration unless we really want to be in
promiscuous mode.

In other words we can make it so that nobody is in promiscuous
mode and therefore have to disable VLAN acceleration *unless*
they really want to be in that state.  In which case it would
imply that they wish to see everything and therefore we should
disable VLAN acceleration.

So that means I'd like to see our current core/driver interface
enhanced so that whether the user has requested us to be in
promiscuous mode can be differentiated from whether the network
stack wants us to be.

Once we have that then I would think that such a patch would be
less controversial.

Cheers,
-- 
Visit Openswan at http://www.openswan.org/
Email: Herbert Xu ~{PmV>HI~} <herbert@gondor.apana.org.au>
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
Re: [PATCH 2/2] [e1000 VLAN] Disable vlan hw accel when prom..., Stephen Hemminger, (Tue Nov 13, 4:43 pm)
Re: [PATCH 2/2] [e1000 VLAN] Disable vlan hw accel when prom..., Herbert Xu, (Tue Nov 13, 8:03 am)
speck-geostationary