Re: [PATCH 2.6.36] vlan: Avoid hwaccel vlan packets when vid notused

From: Eric Dumazet
Date: Thu Jan 06 2011 - 22:04:57 EST


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 ;)

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 ?



--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/