| 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
| ||
|
Date: Thu, 6 Jan 2011 23:38:27 -0500 From: Jesse Gross <jesse@...ira.com> To: Matt Carlson <mcarlson@...adcom.com> Cc: Eric Dumazet <eric.dumazet@...il.com>, Michael Leun <lkml20101129@...ton.leun.net>, Michael Chan <mchan@...adcom.com>, David Miller <davem@...emloft.net>, Ben Greear <greearb@...delatech.com>, "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>, "netdev@...r.kernel.org" <netdev@...r.kernel.org> Subject: Re: [PATCH 2.6.36] vlan: Avoid hwaccel vlan packets when vid not used On Thu, Jan 6, 2011 at 10:41 PM, Matt Carlson <mcarlson@...adcom.com> wrote: > On Thu, Jan 06, 2011 at 07:04:46PM -0800, Eric Dumazet wrote: >> Le jeudi 06 janvier 2011 ?? 18:59 -0800, Matt Carlson a ??crit : >> > On Thu, Jan 06, 2011 at 06:43:22PM -0800, Eric Dumazet wrote: >> > > Le vendredi 07 janvier 2011 ?? 03:41 +0100, Eric Dumazet a ??crit : >> > > > Le jeudi 06 janvier 2011 ?? 18:29 -0800, Matt Carlson a ??crit : >> > > > >> > > > > Hi Eric. Sorry for the delay. I was under the impression that your >> > > > > problems were software related and that you just needed a revised >> > > > > version of these VLAN patches I was sending to Michael. Is this not >> > > > > true? >> > > > > >> > > > > Having a hardware stat increment suggests this is a new problem. >> > > > > Maybe I missed it, but I didn't see what hardware you are working >> > > > > with and whether or not management firmware was enabled. Could you tell >> > > > > me that info? >> > > > > >> > > > >> > > > Hi Matt >> > > > >> > > > I started a bisection, because I couldnt sleep tonight anyway :( >> > > > >> > > > 14:04.0 Ethernet controller: Broadcom Corporation NetXtreme BCM5715S >> > > > Gigabit Ethernet (rev a3) >> > > > Subsystem: Hewlett-Packard Company NC326m PCIe Dual Port Adapter >> > > > Flags: bus master, 66MHz, medium devsel, latency 64, IRQ 43 >> > > > Memory at fdff0000 (64-bit, non-prefetchable) [size=64K] >> > > > Memory at fdfe0000 (64-bit, non-prefetchable) [size=64K] >> > > > [virtual] Expansion ROM at fdbe0000 [disabled] [size=128K] >> > > > Capabilities: [40] PCI-X non-bridge device >> > > > Capabilities: [48] Power Management version 2 >> > > > Capabilities: [50] Vital Product Data >> > > > Capabilities: [58] MSI: Enable+ Count=1/8 Maskable- 64bit+ >> > > > Kernel driver in use: tg3 >> > > > Kernel modules: tg3 >> > > > >> > > > >> > > >> > > $ ethtool -i eth2 >> > > driver: tg3 >> > > version: 3.115 >> > > firmware-version: 5715s-v3.28 >> > > bus-info: 0000:14:04.0 >> > > $ dmesg | grep ASF >> > > [ 6.220577] tg3 0000:14:04.0: eth2: RXcsums[1] LinkChgREG[0] MIirq[0] >> > > ASF[0] TSOcap[1] >> > > [ 6.228586] tg3 0000:14:04.1: eth3: RXcsums[1] LinkChgREG[0] MIirq[0] >> > > ASF[0] TSOcap[1] >> > >> > Thanks. So management firmware is disabled. This should be >> > straightforward case. >> > >> > I'm wondering if I'm misunderstanding something though. You said earlier >> > that VLAN tagging doesn't work unless you applied my patch. Is this no >> > longer true? >> > >> >> I dont apply your patch because Jesse said it was not a good patch ;) > > Oh. > >> Maybe I missed something and it must be applied ? Problem is : current >> Linus tree now includes net-next-2.6 and vlan doesnt work. You should >> resubmit it perhaps ? > > Yes, something needs to be submitted. I want to make sure we aren't > chasing the same problem though. If the patch(es) fix your problem, > then I can concentrate on finalizing the patch. > > I can combine my last patch (the one that always enabled VLAN tag > stripping) and the previous patch (that implements all your comments so > far) into one patch, but that still leaves the behavior Michael noted > unaddressed. Just to clarify, I think there are three separate things going on here: * The patch, which independent of the separately reported issues, is good because it moves tg3 to the new vlan model. However, I don't think we should always disable vlan stripping as is done because it is probably useful in the majority of cases. Maybe in some situations it needs to be disabled but those are independent and should affect both the patched and unpatched versions. * Eric's issue. It sounds like the commit that bisect turned up has some interaction with stripping. The patch fixes this because it always disables stripping but that doesn't seem like the right solution because previous versions worked with stripping enabled. * Michael's issue. Not clear what the cause is but disabling stripping fixes it. It has different symptoms from Eric's though (missing tags vs missing packets). The patch changes behavior a little bit because it changes when stripping is enabled but doesn't fix the underlying cause. So each needs to be tracked down separately. Unfortunately, the fixed patch will no longer solve Eric's issue... -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists