From mboxrd@z Thu Jan 1 00:00:00 1970 From: Kevin Traynor Subject: Re: Stable ABI status of rte_meter_[t|s]rtcm_profile_config Date: Wed, 1 Aug 2018 11:47:52 +0100 Message-ID: References: <43ec6b34-b68c-618e-ab14-1f1b85f3315c@warmcat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Cc: Andy Green , "Singh, Jasvinder" To: dev@dpdk.org, "Dumitrescu, Cristian" Return-path: Received: from mx1.redhat.com (mx3-rdu2.redhat.com [66.187.233.73]) by dpdk.org (Postfix) with ESMTP id 978B058C4 for ; Wed, 1 Aug 2018 12:47:55 +0200 (CEST) In-Reply-To: <43ec6b34-b68c-618e-ab14-1f1b85f3315c@warmcat.com> Content-Language: en-US List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On 05/28/2018 04:31 AM, Andy Green wrote: > Hi - > > Between 18.02 and the putative 18.05 there were changes in the way the > meter stuff deals with its config. > > I updated the related code in lagopus, but I get warnings about using > the new APIs (it's the same for rte_meter_trtcm_profile_config()) > > ./dpdk/meter.c: In function 'dpdk_register_meter': > ./dpdk/meter.c:119:7: warning: 'rte_meter_srtcm_profile_config' is > deprecated: Symbol is not yet part of stable ABI > [-Wdeprecated-declarations] > rte_meter_srtcm_profile_config(&lband->sp, ¶m); > ^ > In file included from ./dpdk/meter.c:27:0: > /home/agreen/lagopus/src/dpdk/build/include/rte_meter.h:86:1: note: > declared here > rte_meter_srtcm_profile_config(struct rte_meter_srtcm_profile *p, > ^ > ./dpdk/meter.c:132:7: warning: 'rte_meter_srtcm_profile_config' is > deprecated: Symbol is not yet part of stable ABI > [-Wdeprecated-declarations] > rte_meter_srtcm_profile_config(&lband->sp, ¶m); > ^ > In file included from ./dpdk/meter.c:27:0: > /home/agreen/lagopus/src/dpdk/build/include/rte_meter.h:86:1: note: > declared here > rte_meter_srtcm_profile_config(struct rte_meter_srtcm_profile *p, > Hi Cristian, Are these API still to be considered experimental in 18.08, or the tags can be removed? Kevin. > > As far as I can see this api change is not optional, it changes the > parameters for related apis to require a struct prepared with these new > apis. > > -Andy