From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jesse Gross Subject: Re: [patch net-next-2.6 v2] net: vlan: make non-hw-accel rx path similar to hw-accel Date: Sun, 22 May 2011 09:11:01 -0700 Message-ID: References: <1302241713-3637-1-git-send-email-jpirko@redhat.com> <20110412.141645.112604563.davem@davemloft.net> <20110521072925.GA2588@jirka.orion> <4DD7BB61.9050200@gmail.com> <4DD87C25.4030701@gmail.com> <20110522062915.GA2611@jirka.orion> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Cc: Jiri Pirko , =?UTF-8?Q?Nicolas_de_Peslo=C3=BCan?= , Changli Gao , David Miller , netdev@vger.kernel.org, shemminger@linux-foundation.org, kaber@trash.net, fubar@us.ibm.com, eric.dumazet@gmail.com, andy@greyhouse.net To: "Eric W. Biederman" Return-path: Received: from mail-vw0-f46.google.com ([209.85.212.46]:45196 "EHLO mail-vw0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752141Ab1EVQLW (ORCPT ); Sun, 22 May 2011 12:11:22 -0400 Received: by vws1 with SMTP id 1so3613917vws.19 for ; Sun, 22 May 2011 09:11:21 -0700 (PDT) In-Reply-To: Sender: netdev-owner@vger.kernel.org List-ID: On Sat, May 21, 2011 at 11:34 PM, Eric W. Biederman wrote: > Jiri Pirko writes: > >> Sun, May 22, 2011 at 04:59:49AM CEST, nicolas.2p.debian@gmail.com wrote: >> >> >>> >>>And because some setups may still require the skb not to be untagged, >>>may be we need the ability to re-tag the skb in some situations... >>>When a protocol handler or rx_handler is explicitly registered on a >>>net_device which expect to receive tagged skb, we should deliver >>>tagged skb to it... Arguably, this may sound incredible for the >>>general case, but may be required for not-so-special cases like >>>bridge or protocol analyzer. >> >> Wait, what setups/code require the skb not to be untagged? If there's >> such, it should be fixed. > > tcpdump on the non-vlan interface for one. There are some drivers still using the old vlan model that will drop tags or packets when no vlan group is configured but that's a driver problem, not one with networking core or tcpdump.