Re: in kernel 2.6.x, tun/tap nic supports vlan packets

[zhuyj]

On 04/23/2014 01:53 AM, Ben Hutchings wrote:
> On Thu, 2014-04-17 at 07:02 +0200, Willy Tarreau wrote:
> > Hi Zhu,
> >
> > On Thu, Apr 17, 2014 at 11:35:58AM +0800, zhuyj wrote:
> > > Hi, all
> > >
> > > In kernel 2.6.x, linux depends on nic vlan hardware acceleration to
> > > insert/extract
> > > vlan tag.
Hi, Ben

Thanks for your reply.
> This is a gross overstatement.
>
> The problem I know of is that prior to Linux 2.6.37 the RX path behaved
> differently for VLAN-tagged packets depending on whether they were
> extracted by the driver/hardware.
Yes. You are right. So I backported 
0002-vlan-Centralize-handling-of-hardware-acceleration.patch to fix this 
problem.
> If you put a bridge (or bond) and VLAN device on top of a single
> physical device that doesn't do VLAN tag extraction, the VLAN device
> didn't get any packets because the bridge packet handler was called
> first.  Whereas, if the driver called the 'VLAN accelerated' RX path,
> the VLAN packet handler was called first.  (Linux 2.6.37 actually
> standardised on the former behaviour, and 3.2 fixed it to be the
> latter.)
Yes. So I made a patch 
"0001-tun-tap-add-the-feature-of-vlan-rx-extraction.patch" to make tap 
driver extract vlan tag.
> I don't know whether that's the problem zhuyj has run into.
>
> [...]
> > Well, 2.6.32.x is in deep freeze mode and it receives only critical fixes
> > once in a while. While I can appreciate that the patch above might solve
> > the issue you're facing, I'm wondering if there are not any acceptable
> > workarounds for such a deep freeze kernel. You patch is not huge,
> I think it's huge by the standards of 2.6.32.y.
>
> > but it
> > definitely affects a working driver, and I wouldn't like risking to break
> > the tap driver for other users, and I reall don't have the skills to audit
> > it completely to ensure this is not the case. And if it breaks, I'll have
> > to revert it or seek for some help on netdev.
> >
> > So I'd say that I'd rather not merge it unless I get an Acked-by from some
> > netdev people who are willing to help in case of any future regression,
> > which is unlikely but still possible.
> [...]
>
> For what it's worth, I would recommend against applying this.  I don't
> think even Red Hat has backported the VLAN changes, and they have been
> quite aggressive about backporting features to RHEL 6.
If we do not merge these patches, maybe RHEL 6 can not make tap driver 
support vlan well.

Best Regards!
Zhu Yanjun
> Ben.

--
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/