From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga11.intel.com (mga11.intel.com [192.55.52.93]) by dpdk.org (Postfix) with ESMTP id 81734B418 for ; Tue, 21 Jun 2016 13:28:15 +0200 (CEST) Received: from fmsmga004.fm.intel.com ([10.253.24.48]) by fmsmga102.fm.intel.com with ESMTP; 21 Jun 2016 04:28:14 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.26,503,1459839600"; d="scan'208";a="125914513" Received: from bricha3-mobl3.ger.corp.intel.com ([10.237.220.108]) by fmsmga004.fm.intel.com with SMTP; 21 Jun 2016 04:28:12 -0700 Received: by (sSMTP sendmail emulation); Tue, 21 Jun 2016 12:28:11 +0025 Date: Tue, 21 Jun 2016 12:28:11 +0100 From: Bruce Richardson To: Panu Matilainen Cc: Beilei Xing , jingjing.wu@intel.com, dev@dpdk.org, thomas.monjalon@6wind.com, nhorman@tuxdriver.com Message-ID: <20160621112811.GA19572@bricha3-MOBL3> References: <1464247695-4694-1-git-send-email-beilei.xing@intel.com> <1465805012-2907-1-git-send-email-beilei.xing@intel.com> <20160621102930.GA21016@bricha3-MOBL3> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Organization: Intel Research and =?iso-8859-1?Q?De=ACvel?= =?iso-8859-1?Q?opment?= Ireland Ltd. User-Agent: Mutt/1.5.23 (2014-03-12) Subject: Re: [dpdk-dev] [PATCH v2] i40e: modify the meaning of single VLAN type X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 21 Jun 2016 11:28:15 -0000 On Tue, Jun 21, 2016 at 02:06:38PM +0300, Panu Matilainen wrote: > On 06/21/2016 01:29 PM, Bruce Richardson wrote: > >On Mon, Jun 13, 2016 at 04:03:32PM +0800, Beilei Xing wrote: > >>In current i40e codebase, if single VLAN header is added in a packet, > >>it's treated as inner VLAN. Generally, a single VLAN header is > >>treated as the outer VLAN header. So change corresponding register > >>for single VLAN. > >>At the meanwhile, change the meanings of inner VLAN and outer VLAN. > >> > >>Signed-off-by: Beilei Xing > > > >This patch changes the ABI, since an app written to the original API as specified > >e.g. to set a single vlan header, would no longer work with this change. > >Therefore, even though the original behaviour was inconsistent with other drivers > >it may still need to be preserved. > > > >I'm thinking that we may need to provide appropriately versioned copies of the > >vlan_offload_set and vlan_tpid_set functions for backward compatibility with > >the old ABI. > > > >Any other comments or thoughts on this? > >Neil, Thomas, Panu - is this fix something that we need to provide backward > >version-compatibility for, or given that the functions are being called through > >a generic ethdev API mean that this can just go in as a straight bug-fix? > > Since it's currently inconsistent with everything else, I'd just call it a > bug-fix and leave it at that. > Yep, makes sense. > Besides, I dont think you could version it via the ordinary means even if > you wanted to, due to the way its called through eth_dev_ops etc. > Good point, never thought of that! :-( > - Panu - Thanks for the guidance. /Bruce