From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stephen Hemminger Subject: Re: [PATCH 2/5] mbuf: use the reserved 16 bits for double vlan Date: Tue, 26 May 2015 08:35:05 -0700 Message-ID: <20150526083505.247e5431@urahara> References: <1430793143-3610-1-git-send-email-helin.zhang@intel.com> <1432629400-25303-1-git-send-email-helin.zhang@intel.com> <1432629400-25303-3-git-send-email-helin.zhang@intel.com> <20150526075515.1bc42ae1@urahara> <2601191342CEEE43887BDE71AB97725821431C8F@irsmsx105.ger.corp.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Cc: "dev@dpdk.org" To: "Ananyev, Konstantin" Return-path: Received: from mail-pd0-f171.google.com (mail-pd0-f171.google.com [209.85.192.171]) by dpdk.org (Postfix) with ESMTP id 187689A92 for ; Tue, 26 May 2015 17:35:03 +0200 (CEST) Received: by pdbqa5 with SMTP id qa5so93359202pdb.0 for ; Tue, 26 May 2015 08:35:02 -0700 (PDT) In-Reply-To: <2601191342CEEE43887BDE71AB97725821431C8F@irsmsx105.ger.corp.intel.com> List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On Tue, 26 May 2015 15:02:51 +0000 "Ananyev, Konstantin" wrote: > Hi Stephen, > > > -----Original Message----- > > From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Stephen Hemminger > > Sent: Tuesday, May 26, 2015 3:55 PM > > To: Zhang, Helin > > Cc: dev@dpdk.org > > Subject: Re: [dpdk-dev] [PATCH 2/5] mbuf: use the reserved 16 bits for double vlan > > > > On Tue, 26 May 2015 16:36:37 +0800 > > Helin Zhang wrote: > > > > > Use the reserved 16 bits in rte_mbuf structure for the outer vlan, > > > also add QinQ offloading flags for both RX and TX sides. > > > > > > Signed-off-by: Helin Zhang > > > > Yet another change that is much needed, but breaks ABI compatibility. > > Why do you think it breaks ABI compatibility? > As I can see, it uses field that was reserved. > Konstantin Because an application maybe assuming something or reusing the reserved fields. Yes, it would be dumb of application to do that but from absolute ABI point of view it is a change.