From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga01.intel.com (mga01.intel.com [192.55.52.88]) by dpdk.org (Postfix) with ESMTP id 375D88E7E for ; Tue, 24 Nov 2015 16:45:52 +0100 (CET) Received: from fmsmga001.fm.intel.com ([10.253.24.23]) by fmsmga101.fm.intel.com with ESMTP; 24 Nov 2015 07:45:51 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.20,338,1444719600"; d="scan'208";a="846154297" Received: from bricha3-mobl3.ger.corp.intel.com ([10.237.208.62]) by fmsmga001.fm.intel.com with SMTP; 24 Nov 2015 07:45:49 -0800 Received: by (sSMTP sendmail emulation); Tue, 24 Nov 2015 15:45:48 +0025 Date: Tue, 24 Nov 2015 15:45:48 +0000 From: Bruce Richardson To: Thomas Monjalon Message-ID: <20151124154548.GA17696@bricha3-MOBL3> References: <1446552059-5446-1-git-send-email-bruce.richardson@intel.com> <20151117075309.7f05dc3a@samsung9> <20151124145630.GA19796@bricha3-MOBL3> <2448231.qRa78uBjDE@xps13> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <2448231.qRa78uBjDE@xps13> Organization: Intel Shannon Ltd. User-Agent: Mutt/1.5.23 (2014-03-12) Cc: dev@dpdk.org Subject: Re: [dpdk-dev] [PATCH v4 2/2] ethdev: add sanity checks to functions 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, 24 Nov 2015 15:45:52 -0000 On Tue, Nov 24, 2015 at 04:29:12PM +0100, Thomas Monjalon wrote: > 2015-11-24 14:56, Bruce Richardson: > > On Tue, Nov 17, 2015 at 07:53:09AM -0800, Stephen Hemminger wrote: > > > On Tue, 17 Nov 2015 12:21:07 +0000 > > > Bruce Richardson wrote: > > > > -static inline uint32_t > > > > +static inline int > > Are we talking about this change only? > Or the move in the first patch from .c to .h? > The move is the ABI breaker. > [...] > > > This breaks ABI since older application built with debug will try > > > and find the shared library entry for the routine. > > > > Ok, so assuming we care about the ABI for debug builds, > > The return type is not only for debug build? > > > is it enough to just push a patch with a deprecation notice for this for 2.2, > > The ABI is already broken for ethdev in 2.2. > So the symbol move should not hurt more. > And the API change (return type) should not be a big deal, > but at least an API change notification is required in the release notes. > Other opinion? Ok, it makes sense. > > > or do I need to see about doing a new patchset with the NEXT_ABI macros > > included in it? My preference is obviously for the former. > > No NEXT_ABI is required when ABI is already broken IMHO. If ethdev ABI is already broken, then sure, this additional break for debug build is no big deal, I think. I can do a respin of these two patches to include an API note for release notes. However, I see now that I also need to remove the functions from the map file. I could do with some help to make sure I do this correctly though. Reading through the doc on ABI versionning, it looks like I should completely move all existing functions from the existing release versions and move them to a new 2.2 section, dropping the four now-inline functions along the way. Is this the correct thing to do? /Bruce