| 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
| ||
|
Message-ID: <AANLkTikKkAesHc4Fi4SwAkV0e65D0TQRztrRYoDNu3MP@mail.gmail.com> Date: Mon, 6 Dec 2010 12:04:48 -0800 From: Jesse Gross <jesse@...ira.com> To: Michael Leun <lkml20101129@...ton.leun.net> Cc: Eric Dumazet <eric.dumazet@...il.com>, David Miller <davem@...emloft.net>, Ben Greear <greearb@...delatech.com>, linux-kernel@...r.kernel.org, netdev@...r.kernel.org Subject: Re: [PATCH 2.6.36] vlan: Avoid hwaccel vlan packets when vid not used On Mon, Dec 6, 2010 at 11:34 AM, Michael Leun <lkml20101129@...ton.leun.net> wrote: > On Mon, 6 Dec 2010 10:14:55 -0800 > Jesse Gross <jesse@...ira.com> wrote: > >> On Sun, Dec 5, 2010 at 2:44 AM, Michael Leun >> <lkml20101129@...ton.leun.net> wrote: >> > Hi Jesse, >> > >> > On Sun, 5 Dec 2010 10:55:28 +0100 >> > Michael Leun <lkml20101129@...ton.leun.net> wrote: >> > >> >> On Sun, 05 Dec 2010 09:03:53 +0100 >> >> Eric Dumazet <eric.dumazet@...il.com> wrote: >> >> >> >> > > But on >> >> > > >> >> > > hpdl320g5:/home/ml # lspci | grep Eth >> >> > > 03:04.0 Ethernet controller: Broadcom Corporation NetXtreme >> >> > > BCM5714 Gigabit Ethernet (rev a3) 03:04.1 Ethernet controller: >> >> > > Broadcom Corporation NetXtreme BCM5714 Gigabit Ethernet (rev >> >> > > a3) >> >> > > >> >> > > the good message is that it also does not crash, but with >> >> > > tcpdump I see vlan tags when no vlan devices configured on the >> >> > > respective eth, if so I do not see tags anymore vlan tags on >> >> > > the trunk interface. >> >> > > >> >> > >> >> > For all these very specific needs, you'll have to try 2.6.37 I am >> >> > afraid. Jesse did huge changes to exactly make this working, we >> >> > wont backport this to 2.6.36, but only avoid crashes. >> >> >> >> OK, I'm perfectly fine with that, of course, actually nice to hear >> >> that the issue already is addressed. >> >> >> >> Likely I'll give some rc an shot on this machine (maybe over >> >> christmas), but it is an production machine (acutally testing other >> >> devices is the "product" produced on this machine), so >> >> unfortunately I'm not that free in when and what I can do (but the >> >> possibility to, for example, bridge the trunk interface would make >> >> testing easier, that justifies something...). >> >> >> >> Thank you all very much for your work. >> > >> > Are these changes already in 2.6.37-rc4? Or, if not are they >> > somewhere publically available already? >> > >> > I looked into various changelogs but have some difficulties to >> > identify them... >> > >> > Maybe I have some time next days to give them an try... >> >> Yes, all of the existing vlan changes are in 2.6.37-rc4. There were a >> number of patches but the main one was >> 3701e51382a026cba10c60b03efabe534fba4ca4 > > Then, I'm afraid, this (seeing vlan tags even if vlan interfaces are > configured) does not work on HP DL320G5 (for exact description and > examples please see my mail a few days ago). What driver are you using? Is it tg3? The vlan changes that I made unfortunately require updating drivers to get the full benefit. I've been busy lately so tg3 hasn't yet been updated. I know that tg3 does some things differently depending on whether a vlan group is configured, so that would likely be the cause of what you are seeing. I'd have to look at it in more detail to be sure though. You said that everything works on the other Broadcom NIC that you tested? Maybe it uses bnx2 instead? -- 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
Powered by blists - more mailing lists