netdev - RE: [PATCH 2.6.24 1/2]S2io: Strip the vlan tag if the vlan group is not NULL

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:	Sat, 24 Nov 2007 13:11:58 -0500
From:	"Ramkrishna Vepa" <Ramkrishna.Vepa@...erion.com>
To:	"Jeff Garzik" <jeff@...zik.org>
Cc:	<netdev@...r.kernel.org>, "support" <support@...erion.com>
Subject: RE: [PATCH 2.6.24 1/2]S2io: Strip the vlan tag if the vlan group is not NULL

Jeff,

I don't think this functionality is present in ethtool. I only see
#ifdef code in most of the drivers. What functionality do you mean?

We created this patch in response to the following thread with subject
line -
RE: [PATCH 2/2] NET: Re-add VLAN tag for devices incapable of keeping it

Since the Xframe hardware can dynamically turn Vlan stripping on and
off, we gave an option to the user to leave it in the default state
(strip the Vlan tag if a vlan group is present, else no stripping) or to
always turn OFF stripping (if say in promiscuous mode) or always turn it
ON (always strip vlan tags).

Ram
> -----Original Message-----
> From: Jeff Garzik [mailto:jeff@...zik.org]
> Sent: Friday, November 23, 2007 7:04 PM
> To: ram.vepa@...erion.com
> Cc: netdev@...r.kernel.org; support
> Subject: Re: [PATCH 2.6.24 1/2]S2io: Strip the vlan tag if the vlan
group
> is not NULL
> 
> Ramkrishna Vepa wrote:
> > - Updated the vlan tag stripping code as per Dave Johnson's patch
> >   <djohnson@...starentnetworks.com>
> >   Below is the driver behaviour for vlan_tag_strip loadable
paramter,
> > 	vlan_tag_strip - 0: Don't strip the vlan tag
> > 	vlan_tag_strip - 1: Always strip the vlan tag
> > 	vlan_tag_strip - 2 (default): strip the vlan tag if the
> > 	  vlan group is not NULL.
> >
> > Signed-off-by: Santoshkumar Rastapur <santosh.rastapur@...erion.com>
> > Signed-off-by: Ramkrishna Vepa <ram.vepa@...erion.com>
> > ---
> > diff -Nurp 2.0.26.6/drivers/net/s2io.c 2.0.26.8P1/drivers/net/s2io.c
> > --- 2.0.26.6/drivers/net/s2io.c	2007-11-15 18:19:42.000000000
-0800
> > +++ 2.0.26.8P1/drivers/net/s2io.c	2007-11-15 18:15:00.000000000
-0800
> > @@ -46,10 +46,10 @@
> >   *     Possible values '1' for enable and '0' for disable. Default
is
> '1'
> >   * ufo: This parameter used to enable/disable UDP Fragmentation
> Offload(UFO)
> >   *      Possible values '1' for enable and '0' for disable. Default
is
> '0'
> > - * vlan_tag_strip: This can be used to enable or disable vlan
> stripping.
> > - *                 Possible values '1' for enable , '0' for
disable.
> > - *                 Default is '2' - which means disable in promisc
mode
> > - *                 and enable in non-promiscuous mode.
> > + * vlan_tag_strip: This can be used to enable or disable vlan tag
> stripping.
> > + *      Possible values '2' for driver default, '1' for enable and
> > + *      '0' for disable
> > + *      Default is '2' - VLAN tag stripping enabled if vlan group
> present
> >
>
************************************************************************
/
> 
> this should be controlled via ethtool and behave like other net
drivers
> 

-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html