From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from stargate.chelsio.com (stargate.chelsio.com [12.32.117.8]) by dpdk.org (Postfix) with ESMTP id CA2C8324D for ; Wed, 28 Mar 2018 09:40:26 +0200 (CEST) Received: from localhost (scalar.blr.asicdesigners.com [10.193.185.94]) by stargate.chelsio.com (8.13.8/8.13.8) with ESMTP id w2S7eMkP019941; Wed, 28 Mar 2018 00:40:23 -0700 Date: Wed, 28 Mar 2018 13:09:34 +0530 From: Rahul Lakkireddy To: Shahaf Shuler Cc: Ferruh Yigit , "dev@dpdk.org" , Kumar A S , Nirranjan Kirubaharan , Indranil Choudhury Message-ID: <20180328073933.GA19600@chelsio.com> References: <20180327070155.GA17402@chelsio.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.24 (2015-08-30) Subject: Re: [dpdk-dev] [PATCH 00/13] cxgbe: add CXGBE VF PMD and updates X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 28 Mar 2018 07:40:27 -0000 On Wednesday, March 03/28/18, 2018 at 10:19:17 +0530, Shahaf Shuler wrote: > Tuesday, March 27, 2018 10:02 AM, Rahul Lakkireddy: > > [...] > > > > > > > > > Patch 13 adds compile time option to keep outer vlan tag in Q-in-Q > > > > packet. > > > > > > We are trying to reduce the config options, is it possible to provide > > > this functionality with a runtime option (devargs) ? > > > > > > > Thank you for pointing to this. It seems like a good option. > > > > > Or there is already an offload option DEV_RX_OFFLOAD_QINQ_STRIP, I > > > guess this is different (is it?), if so does it make sense to have > > > another offload option to cover your case? > > > > > > > > > > Yes, this is different. Here, its about stripping or preserving Outer-VLAN tag > > from double-vlan in Rx. We have few customers who need this for their use- > > case. So, adding another offload option would also help. Let us know which is > > preferred - either one, devargs OR another offload, seems fine. > > What exactly is the offload you try to expose? To strip the inner VLAN while keeping the outer one? > Sorry about that, I should have provided more details on this earlier. Chelsio hardware supports below for double-vlans: 1) Dropping Outer VLAN in hw; inner vlan is stripped and available as packet metadata. 2) Preserving Outer VLAN so, Outer VLAN is stripped and available as packet metadata. Inner VLAN is available as part of packet itself. So, the configurability here is with respect to Outer VLAN and subsequently what is available as part of packet metadata (outer or inner vlan). Inner VLAN gets retained in both cases. However, based on how Outer VLAN is configured, Inner VLAN will be available either as packet metadata or as part of packet itself. We are not very sure if DEV_RX_OFFLOAD_QINQ_STRIP could be used here. If it holds the same meaning then we will use the same. Thanks, Rahul