From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <ramirose@gmail.com>
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 <dev@dpdk.org>; Mon, 18 Feb 2019 21:27:58 +0100 (CET)
Received: by mail-wr1-f67.google.com with SMTP id v13so19767393wrw.5
 for <dev@dpdk.org>; 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 <ramirose@gmail.com>
Date: Mon, 18 Feb 2019 22:27:46 +0200
Message-ID: <CAHLOa7QfcY2WU5KpVDAG4kPPxSfo+=8xcueJ23VF2+eRh9qFJQ@mail.gmail.com>
To: Nikhil Rao <nikhil.rao@intel.com>
Cc: harry.van.haaren@intel.com, Ferruh Yigit <ferruh.yigit@intel.com>,
 dev <dev@dpdk.org>
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 <dev.dpdk.org>
List-Unsubscribe: <https://mails.dpdk.org/options/dev>,
 <mailto:dev-request@dpdk.org?subject=unsubscribe>
List-Archive: <http://mails.dpdk.org/archives/dev/>
List-Post: <mailto:dev@dpdk.org>
List-Help: <mailto:dev-request@dpdk.org?subject=help>
List-Subscribe: <https://mails.dpdk.org/listinfo/dev>,
 <mailto:dev-request@dpdk.org?subject=subscribe>
X-List-Received-Date: Mon, 18 Feb 2019 20:27:58 -0000

Reviewed-by: Rami Rosen <ramirose@gmail.com>

On Fri, Feb 15, 2019 at 12:29 PM Nikhil Rao <nikhil.rao@intel.com> 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 <gage.eads@intel.com>
> Signed-off-by: Nikhil Rao <nikhil.rao@intel.com>
> Acked-by: Harry van Haaren <harry.van.haaren@intel.com>
> ---
>  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