From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wr1-f67.google.com (mail-wr1-f67.google.com [209.85.221.67]) by dpdk.org (Postfix) with ESMTP id 38BD75B3C for ; Mon, 18 Feb 2019 21:27:58 +0100 (CET) Received: by mail-wr1-f67.google.com with SMTP id v13so19767393wrw.5 for ; Mon, 18 Feb 2019 12:27:58 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=ZX1ZReCGUWI1SEy1KI/Hbj/hxS9yj56hXjrnfMacb1Q=; b=lU+KD30Fm7X8VxZuTJf0TnpFfLtD55oNw7bx5UZnNdU8fmAerEYiWYUhtFKqDGuHvh Yj0Vv5ITnGaFvMfSuL/a1zccoASthJV8ZEpmuktFnc7e/niwOWLBWwYBiRyT/geIhfG1 X72VG3XuZ/dsh1DaOncYpNj1oamerhR3aCkwvdtOZEsU/z5n99gzTAPJkMpui0JCEhLz fB+olYwLbHakkGV5U5YP4fs01WlgC/qu5uFTsl4+a3MZvFQ5wpWyMM8yMHVoC8E301xZ XCdWBYBPv4yJzY7FT2njTkJW8b9O6bY7fmEiWULix6GQfjkV2yaO2OM1qtgH9RmAyOK/ MTyQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=ZX1ZReCGUWI1SEy1KI/Hbj/hxS9yj56hXjrnfMacb1Q=; b=FUvUza1Mft/OWaF1vr71mvHtVUXnQOsbKD9rAemKFw3yQLATft6ZXhCK2rEafUQGfR F384JURqqEHYm2w2AhuGq8ny1xtzDawedQk7olXaPQ+PGnuQrEnyWSHa4EO4Ur9KfZBy G7WJ8nXjo1oiHoVDvp4/7wYTGbH/D93XBW39+dHwxapGedsl3ROXegHRbZ8w4SPHbbAA wi5VQk/Nz8KJWWuM+zsF2vF0ZJsItKNtHpX1lAMVih31xEopC0qdMkDzNQPKee7hETKQ yDzLTKN964JcmP4XLkD9wQTrRhRNxEWx9qLtXFXBWbAiygVEWYUFW25LHdk+qA3DkN2F Cfjg== X-Gm-Message-State: AHQUAubK94KwMC5O0hUfGYpWUWpfYW228YryUUOKmLvclaKpqlO0CORZ BeEVRZnNHTEH236Yw5qUQF8NDGIc3QczbHniUWI= X-Google-Smtp-Source: AHgI3IZOzTyOBPGMn67h1523eBeuTAwjJyLm7hp1JEDtrsEsL+SrNKxTDkHHRhDDqWoFCBGUUPEUUsDQrF8rP3ynZJo= X-Received: by 2002:adf:f80c:: with SMTP id s12mr17175313wrp.150.1550521677773; Mon, 18 Feb 2019 12:27:57 -0800 (PST) MIME-Version: 1.0 References: <1547906489-71240-1-git-send-email-nikhil.rao@intel.com> <1550226554-147562-1-git-send-email-nikhil.rao@intel.com> In-Reply-To: <1550226554-147562-1-git-send-email-nikhil.rao@intel.com> From: Rami Rosen Date: Mon, 18 Feb 2019 22:27:46 +0200 Message-ID: To: Nikhil Rao Cc: harry.van.haaren@intel.com, Ferruh Yigit , dev Content-Type: text/plain; charset="UTF-8" X-Content-Filtered-By: Mailman/MimeDel 2.1.15 Subject: Re: [dpdk-dev] [PATCH v3] service: fix parameter type 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 Feb 2019 20:27:58 -0000 Reviewed-by: Rami Rosen On Fri, Feb 15, 2019 at 12:29 PM Nikhil Rao wrote: > The type of value parameter to rte_service_attr_get > should be uint64_t *, since the attributes > are of type uint64_t. > > Fixes: 4d55194d76a4 ("service: add attribute get function") > > Reviewed-by: Gage Eads > Signed-off-by: Nikhil Rao > Acked-by: Harry van Haaren > --- > lib/librte_eal/common/include/rte_service.h | 2 +- > lib/librte_eal/common/rte_service.c | 2 +- > test/test/test_service_cores.c | 2 +- > doc/guides/rel_notes/deprecation.rst | 4 ---- > doc/guides/rel_notes/release_19_05.rst | 5 ++++- > lib/librte_eal/bsdapp/eal/Makefile | 2 +- > lib/librte_eal/linuxapp/eal/Makefile | 2 +- > lib/librte_eal/meson.build | 2 +- > 8 files changed, 10 insertions(+), 11 deletions(-) > > v2: > * Update release notes. > * Update library version in makefiles. > > v3: > * Remove deprecation notice. > > diff --git a/lib/librte_eal/common/include/rte_service.h > b/lib/librte_eal/common/include/rte_service.h > index 34b41af..670c89a 100644 > --- a/lib/librte_eal/common/include/rte_service.h > +++ b/lib/librte_eal/common/include/rte_service.h > @@ -372,7 +372,7 @@ int32_t rte_service_run_iter_on_app_lcore(uint32_t id, > * -EINVAL Invalid id, attr_id or attr_value was NULL. > */ > int32_t rte_service_attr_get(uint32_t id, uint32_t attr_id, > - uint32_t *attr_value); > + uint64_t *attr_value); > > /** > * Reset all attribute values of a service. > diff --git a/lib/librte_eal/common/rte_service.c > b/lib/librte_eal/common/rte_service.c > index 03fde97..5f75e5a 100644 > --- a/lib/librte_eal/common/rte_service.c > +++ b/lib/librte_eal/common/rte_service.c > @@ -734,7 +734,7 @@ int32_t rte_service_run_iter_on_app_lcore(uint32_t id, > } > > int32_t > -rte_service_attr_get(uint32_t id, uint32_t attr_id, uint32_t *attr_value) > +rte_service_attr_get(uint32_t id, uint32_t attr_id, uint64_t *attr_value) > { > struct rte_service_spec_impl *s; > SERVICE_VALID_GET_OR_ERR_RET(id, s, -EINVAL); > diff --git a/test/test/test_service_cores.c > b/test/test/test_service_cores.c > index ec31882..82bb2ce 100644 > --- a/test/test/test_service_cores.c > +++ b/test/test/test_service_cores.c > @@ -259,7 +259,7 @@ static int32_t dummy_mt_safe_cb(void *args) > rte_service_set_stats_enable(id, 1); > > uint32_t attr_id = UINT32_MAX; > - uint32_t attr_value = 0xdead; > + uint64_t attr_value = 0xdead; > /* check error return values */ > TEST_ASSERT_EQUAL(-EINVAL, rte_service_attr_get(id, attr_id, > &attr_value), > diff --git a/doc/guides/rel_notes/deprecation.rst > b/doc/guides/rel_notes/deprecation.rst > index 1b4fcb7..93ed31f 100644 > --- a/doc/guides/rel_notes/deprecation.rst > +++ b/doc/guides/rel_notes/deprecation.rst > @@ -20,10 +20,6 @@ Deprecation Notices > * kvargs: The function ``rte_kvargs_process`` will get a new parameter > for returning key match count. It will ease handling of no-match case. > > -* eal: The ``attr_value`` parameter of ``rte_service_attr_get()`` > - will be changed from ``uint32_t *`` to ``uint64_t *`` > - as the attributes are of type ``uint64_t``. > - > * eal: both declaring and identifying devices will be streamlined in > v18.11. > New functions will appear to query a specific port from buses, classes > of > device and device drivers. Device declaration will be made coherent > with the > diff --git a/doc/guides/rel_notes/release_19_05.rst > b/doc/guides/rel_notes/release_19_05.rst > index 2b0f60d..b8ed3d3 100644 > --- a/doc/guides/rel_notes/release_19_05.rst > +++ b/doc/guides/rel_notes/release_19_05.rst > @@ -94,6 +94,9 @@ API Changes > Also, make sure to start the actual text at the margin. > ========================================================= > > +eal: as shown in the 19.02 deprecation notice, the type of the > ``attr_value`` > + parameter of the function ``rte_service_attr_get()`` has been changed > + from ``uint32_t *`` to ``uint64_t *``. > > ABI Changes > ----------- > @@ -143,7 +146,7 @@ The libraries prepended with a plus sign were > incremented in this version. > librte_compressdev.so.1 > librte_cryptodev.so.6 > librte_distributor.so.1 > - librte_eal.so.9 > + +librte_eal.so.10 > librte_efd.so.1 > librte_ethdev.so.11 > librte_eventdev.so.6 > diff --git a/lib/librte_eal/bsdapp/eal/Makefile > b/lib/librte_eal/bsdapp/eal/Makefile > index bfeddaa..4bc8555 100644 > --- a/lib/librte_eal/bsdapp/eal/Makefile > +++ b/lib/librte_eal/bsdapp/eal/Makefile > @@ -22,7 +22,7 @@ LDLIBS += -lrte_kvargs > > EXPORT_MAP := ../../rte_eal_version.map > > -LIBABIVER := 9 > +LIBABIVER := 10 > > # specific to bsdapp exec-env > SRCS-$(CONFIG_RTE_EXEC_ENV_BSDAPP) := eal.c > diff --git a/lib/librte_eal/linuxapp/eal/Makefile > b/lib/librte_eal/linuxapp/eal/Makefile > index 51deb57..db6aca3 100644 > --- a/lib/librte_eal/linuxapp/eal/Makefile > +++ b/lib/librte_eal/linuxapp/eal/Makefile > @@ -10,7 +10,7 @@ ARCH_DIR ?= $(RTE_ARCH) > EXPORT_MAP := ../../rte_eal_version.map > VPATH += $(RTE_SDK)/lib/librte_eal/common/arch/$(ARCH_DIR) > > -LIBABIVER := 9 > +LIBABIVER := 10 > > VPATH += $(RTE_SDK)/lib/librte_eal/common > > diff --git a/lib/librte_eal/meson.build b/lib/librte_eal/meson.build > index a18f3a8..030171e 100644 > --- a/lib/librte_eal/meson.build > +++ b/lib/librte_eal/meson.build > @@ -21,7 +21,7 @@ else > error('unsupported system type "@0@ > "'.format(host_machine.system())) > endif > > -version = 9 # the version of the EAL API > +version = 10 # the version of the EAL API > allow_experimental_apis = true > deps += 'compat' > deps += 'kvargs' > -- > 1.8.3.1 > > -- regards, Rami Rosen