From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from smtp.tuxdriver.com (charlotte.tuxdriver.com [70.61.120.58]) by dpdk.org (Postfix) with ESMTP id 0C78A271 for ; Mon, 18 Dec 2017 15:44:19 +0100 (CET) Received: from cpe-2606-a000-111b-423c-e874-da8e-c543-d863.dyn6.twc.com ([2606:a000:111b:423c:e874:da8e:c543:d863] helo=localhost) by smtp.tuxdriver.com with esmtpsa (TLSv1:AES256-SHA:256) (Exim 4.63) (envelope-from ) id 1eQweG-0006ch-IS; Mon, 18 Dec 2017 09:44:10 -0500 Date: Mon, 18 Dec 2017 09:43:36 -0500 From: Neil Horman To: "Wiles, Keith" Cc: Hemant Agrawal , "olivier.matz@6wind.com" , "dev@dpdk.org" Message-ID: <20171218144336.GC24106@hmswarspite.think-freely.org> References: <1513333251-4147-1-git-send-email-hemant.agrawal@nxp.com> <1513334482-4788-1-git-send-email-hemant.agrawal@nxp.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: User-Agent: Mutt/1.9.1 (2017-09-22) X-Spam-Score: -2.9 (--) X-Spam-Status: No Subject: Re: [dpdk-dev] [PATCH v2] doc: announce ABI change for pktmbuf pool create API 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: Mon, 18 Dec 2017 14:44:20 -0000 On Mon, Dec 18, 2017 at 01:51:52PM +0000, Wiles, Keith wrote: > > > > On Dec 15, 2017, at 4:41 AM, Hemant Agrawal wrote: > > > > Introduce a new argument ops_name in rte_mempool_set_ops_byname > > for allowing the application to optionally specify the mempool ops. > > > > Signed-off-by: Hemant Agrawal > > --- > > v2: fix checkpatch error > > > > doc/guides/rel_notes/deprecation.rst | 3 +++ > > 1 file changed, 3 insertions(+) > > > > diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst > > index 13e8543..968ca14 100644 > > --- a/doc/guides/rel_notes/deprecation.rst > > +++ b/doc/guides/rel_notes/deprecation.rst > > @@ -53,3 +53,6 @@ Deprecation Notices > > > > * librte_meter: The API will change to accommodate configuration profiles. > > Most of the API functions will have an additional opaque parameter. > > + > > +* librte_mbuf: a new optional parameter for representing name of mempool_ops > > + will be added to the API ``rte_pktmbuf_pool_create``. > > > Sorry, for the late response I was on vacation. > > My question is why do we need to change rte_pktmbuf_pool_create ABI yet again, why could we not add a new API to just set the name of the pool after it is created. This would allow all current applications to work without any ABI breakage and only require adding a new API call for anyone that wants the name. The rte_pktmbuf_pool_create() routine could assign a default name or some incrementing style name as the default. e.g. ‘pktmbuf_%d’ with a static incrementing variable or whatever you like. > > Sorry if this was asked and answered before. +1, that seems like the more flexible solution. Neil > > > -- > > 2.7.4 > > > > Regards, > Keith >