From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: Re: [PATCH 1/3] pmd_ring: remove deprecated functions Date: Wed, 17 Jun 2015 01:37:32 +0200 Message-ID: <1777667.uJIdOE8Rh4@xps13> References: <1434387073-16951-1-git-send-email-stephen@networkplumber.org> <20150616135216.GA9780@bricha3-MOBL3> <20150616160512.03873f22@urahara> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Cc: dev@dpdk.org, Stephen Hemminger To: Stephen Hemminger Return-path: Received: from mail-wg0-f42.google.com (mail-wg0-f42.google.com [74.125.82.42]) by dpdk.org (Postfix) with ESMTP id D55BBC37A for ; Wed, 17 Jun 2015 01:38:31 +0200 (CEST) Received: by wgv5 with SMTP id 5so23644479wgv.1 for ; Tue, 16 Jun 2015 16:38:31 -0700 (PDT) In-Reply-To: <20150616160512.03873f22@urahara> 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" 2015-06-16 16:05, Stephen Hemminger: > On Tue, 16 Jun 2015 14:52:16 +0100 > Bruce Richardson wrote: > > > On Mon, Jun 15, 2015 at 09:51:11AM -0700, Stephen Hemminger wrote: > > > From: Stephen Hemminger > > > > > > These were deprecated in 2.0 so remove them from 2.1 > > > > > > Signed-off-by: Stephen Hemminger > > > --- > > > drivers/net/ring/rte_eth_ring.c | 55 ------------------------------- > > > drivers/net/ring/rte_eth_ring_version.map | 4 +-- > > > 2 files changed, 1 insertion(+), 58 deletions(-) > > > > > [..snip..] > > > diff --git a/drivers/net/ring/rte_eth_ring_version.map b/drivers/net/ring/rte_eth_ring_version.map > > > index 8ad107d..5ee55d9 100644 > > > --- a/drivers/net/ring/rte_eth_ring_version.map > > > +++ b/drivers/net/ring/rte_eth_ring_version.map > > > @@ -1,9 +1,7 @@ > > > -DPDK_2.0 { > > > +DPDK_2.1 { > > > global: > > > > > > rte_eth_from_rings; > > > - rte_eth_ring_pair_attach; > > > - rte_eth_ring_pair_create; > > > > > > local: *; > > > }; > > > > [ABI newbie question] Is this how deprecating a fn is done? We no longer have any DPDK_2.0 > > version listings in the .map file? > > Notice the version # changed as well, so linker will generate a new version. > The function was marked deprecated in last version. What happens if you load the 2.1 lib with an app built for 2.0? Shouldn't we keep the DPDK_2.0 block?