From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by dpdk.space (Postfix) with ESMTP id 62694A0471 for ; Thu, 20 Jun 2019 20:26:02 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 2D8331D434; Thu, 20 Jun 2019 20:26:02 +0200 (CEST) Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) by dpdk.org (Postfix) with ESMTP id EF63F1D42D for ; Thu, 20 Jun 2019 20:26:00 +0200 (CEST) Received: from smtp.corp.redhat.com (int-mx03.intmail.prod.int.phx2.redhat.com [10.5.11.13]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id B777E13AA9; Thu, 20 Jun 2019 18:25:46 +0000 (UTC) Received: from dhcp-25.97.bos.redhat.com (unknown [10.18.25.84]) by smtp.corp.redhat.com (Postfix) with ESMTPS id C6F246090E; Thu, 20 Jun 2019 18:25:42 +0000 (UTC) From: Aaron Conole To: Gage Eads Cc: dev@dpdk.org, jerinj@marvell.com, harry.van.haaren@intel.com, nikhil.rao@intel.com, erik.g.carrillo@intel.com, nhorman@tuxdriver.com, Bruce Richardson , Pablo de Lara References: <20190620164206.3972-1-gage.eads@intel.com> Date: Thu, 20 Jun 2019 14:25:42 -0400 In-Reply-To: <20190620164206.3972-1-gage.eads@intel.com> (Gage Eads's message of "Thu, 20 Jun 2019 11:42:06 -0500") Message-ID: User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/26.2 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain X-Scanned-By: MIMEDefang 2.79 on 10.5.11.13 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.29]); Thu, 20 Jun 2019 18:26:00 +0000 (UTC) Subject: Re: [dpdk-dev] [PATCH] eal: promote some service core functions to stable 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: , Errors-To: dev-bounces@dpdk.org Sender: "dev" Gage Eads writes: > The functions rte_service_may_be_active(), rte_service_lcore_attr_get(), > and rte_service_attr_reset_all() were introduced nearly a year ago in DPDK > 18.08. They can be considered non-experimental for the 19.08 release. > > rte_service_may_be_active() is used by eventdev and the sw PMD, and this > commit allows them to not need any experimental API. > > Signed-off-by: Gage Eads > --- > drivers/event/sw/Makefile | 1 - > drivers/event/sw/meson.build | 1 - > lib/librte_eal/common/include/rte_service.h | 15 +++------------ > lib/librte_eal/common/rte_service.c | 6 +++--- > lib/librte_eal/rte_eal_version.map | 6 +++--- > lib/librte_eventdev/Makefile | 1 - > lib/librte_eventdev/meson.build | 1 - > 7 files changed, 9 insertions(+), 22 deletions(-) > > diff --git a/drivers/event/sw/Makefile b/drivers/event/sw/Makefile > index 81236a392..c6600e836 100644 > --- a/drivers/event/sw/Makefile > +++ b/drivers/event/sw/Makefile > @@ -7,7 +7,6 @@ include $(RTE_SDK)/mk/rte.vars.mk > LIB = librte_pmd_sw_event.a > > # build flags > -CFLAGS += -DALLOW_EXPERIMENTAL_API > CFLAGS += -O3 > CFLAGS += $(WERROR_FLAGS) > # for older GCC versions, allow us to initialize an event using > diff --git a/drivers/event/sw/meson.build b/drivers/event/sw/meson.build > index 30d221647..985012219 100644 > --- a/drivers/event/sw/meson.build > +++ b/drivers/event/sw/meson.build > @@ -1,7 +1,6 @@ > # SPDX-License-Identifier: BSD-3-Clause > # Copyright(c) 2017 Intel Corporation > > -allow_experimental_apis = true I don't think you can remove these. There are still some experimental APIs (f.e. the rename for rte_cryptodev_sym_session_get_private_data marked that function as experimental and it will cause build breakage). Maybe I'm mis understanding it? It would be good to get verification from Bruce whether that API should not be marked as experimental (it was just a rename, so not sure...) - maybe that's a follow up for this patch? See: https://travis-ci.com/ovsrobot/dpdk/jobs/209722145 for an example The odd thing is I only see it on the clang builds - perhaps it's a missing definition for the clang compiler. > sources = files('sw_evdev_scheduler.c', > 'sw_evdev_selftest.c', > 'sw_evdev_worker.c', > diff --git a/lib/librte_eal/common/include/rte_service.h b/lib/librte_eal/common/include/rte_service.h > index bf25aec35..d8701dd4c 100644 > --- a/lib/librte_eal/common/include/rte_service.h > +++ b/lib/librte_eal/common/include/rte_service.h > @@ -162,9 +162,6 @@ int32_t rte_service_runstate_set(uint32_t id, uint32_t runstate); > int32_t rte_service_runstate_get(uint32_t id); > > /** > - * @warning > - * @b EXPERIMENTAL: this API may change, or be removed, without prior notice > - * > * This function returns whether the service may be currently executing on > * at least one lcore, or definitely is not. This function can be used to > * determine if, after setting the service runstate to stopped, the service > @@ -178,7 +175,7 @@ int32_t rte_service_runstate_get(uint32_t id); > * @retval 0 Service is not running on any lcore > * @retval -EINVAL Invalid service id > */ > -int32_t __rte_experimental > +int32_t > rte_service_may_be_active(uint32_t id); > > /** > @@ -389,9 +386,6 @@ int32_t rte_service_attr_reset_all(uint32_t id); > #define RTE_SERVICE_LCORE_ATTR_LOOPS 0 > > /** > - * @warning > - * @b EXPERIMENTAL: this API may change without prior notice > - * > * Get an attribute from a service core. > * > * @param lcore Id of the service core. > @@ -401,14 +395,11 @@ int32_t rte_service_attr_reset_all(uint32_t id); > * -EINVAL Invalid lcore, attr_id or attr_value was NULL. > * -ENOTSUP lcore is not a service core. > */ > -int32_t __rte_experimental > +int32_t > rte_service_lcore_attr_get(uint32_t lcore, uint32_t attr_id, > uint64_t *attr_value); > > /** > - * @warning > - * @b EXPERIMENTAL: this API may change without prior notice > - * > * Reset all attribute values of a service core. > * > * @param lcore The service core to reset all the statistics of > @@ -416,7 +407,7 @@ rte_service_lcore_attr_get(uint32_t lcore, uint32_t attr_id, > * -EINVAL Invalid service id provided > * -ENOTSUP lcore is not a service core. > */ > -int32_t __rte_experimental > +int32_t > rte_service_lcore_attr_reset_all(uint32_t lcore); > > #ifdef __cplusplus > diff --git a/lib/librte_eal/common/rte_service.c b/lib/librte_eal/common/rte_service.c > index 5f75e5a53..c3653ebae 100644 > --- a/lib/librte_eal/common/rte_service.c > +++ b/lib/librte_eal/common/rte_service.c > @@ -378,7 +378,7 @@ service_run(uint32_t i, int lcore, struct core_state *cs, uint64_t service_mask) > return 0; > } > > -int32_t __rte_experimental > +int32_t > rte_service_may_be_active(uint32_t id) > { > uint32_t ids[RTE_MAX_LCORE] = {0}; > @@ -754,7 +754,7 @@ rte_service_attr_get(uint32_t id, uint32_t attr_id, uint64_t *attr_value) > } > } > > -int32_t __rte_experimental > +int32_t > rte_service_lcore_attr_get(uint32_t lcore, uint32_t attr_id, > uint64_t *attr_value) > { > @@ -814,7 +814,7 @@ rte_service_attr_reset_all(uint32_t id) > return 0; > } > > -int32_t __rte_experimental > +int32_t > rte_service_lcore_attr_reset_all(uint32_t lcore) > { > struct core_state *cs; > diff --git a/lib/librte_eal/rte_eal_version.map b/lib/librte_eal/rte_eal_version.map > index 824edf0ff..fc26b9503 100644 > --- a/lib/librte_eal/rte_eal_version.map > +++ b/lib/librte_eal/rte_eal_version.map > @@ -292,6 +292,9 @@ DPDK_19.08 { > > rte_lcore_index; > rte_lcore_to_socket_id; > + rte_service_lcore_attr_get; > + rte_service_lcore_attr_reset_all; > + rte_service_may_be_active; > > } DPDK_19.05; > > @@ -383,9 +386,6 @@ EXPERIMENTAL { > rte_mp_sendmsg; > rte_option_register; > rte_realloc_socket; > - rte_service_lcore_attr_get; > - rte_service_lcore_attr_reset_all; > - rte_service_may_be_active; > > # added in 19.08 > rte_lcore_cpuset; > diff --git a/lib/librte_eventdev/Makefile b/lib/librte_eventdev/Makefile > index 53079f4c2..fdaec7d06 100644 > --- a/lib/librte_eventdev/Makefile > +++ b/lib/librte_eventdev/Makefile > @@ -11,7 +11,6 @@ LIB = librte_eventdev.a > LIBABIVER := 6 > > # build flags > -CFLAGS += -DALLOW_EXPERIMENTAL_API > CFLAGS += -O3 > CFLAGS += $(WERROR_FLAGS) > ifeq ($(CONFIG_RTE_EXEC_ENV_LINUX),y) > diff --git a/lib/librte_eventdev/meson.build b/lib/librte_eventdev/meson.build > index 6cfe60e1f..f623d74f8 100644 > --- a/lib/librte_eventdev/meson.build > +++ b/lib/librte_eventdev/meson.build > @@ -2,7 +2,6 @@ > # Copyright(c) 2017 Intel Corporation > > version = 6 > -allow_experimental_apis = true > > if is_linux > cflags += '-DLINUX'