On Tue, Feb 07, 2023 at 09:46:36AM +0100, Robin Jarry wrote:
> The doc string was left to its previous definition. Make it explicit
> that RTE_TEL_INT_VAL is a 64-bit signed integer.
> 
> Fixes: 8e639c7c50cc ("telemetry: use 64-bit signed values in API")
> 
> Signed-off-by: Robin Jarry <rja...@redhat.com>
> ---
>  lib/telemetry/rte_telemetry.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/lib/telemetry/rte_telemetry.h b/lib/telemetry/rte_telemetry.h
> index ad712732be04..d8ea97f17395 100644
> --- a/lib/telemetry/rte_telemetry.h
> +++ b/lib/telemetry/rte_telemetry.h
> @@ -43,7 +43,7 @@ struct rte_tel_data;
>   */
>  enum rte_tel_value_type {
>       RTE_TEL_STRING_VAL, /** a string value */
> -     RTE_TEL_INT_VAL,    /** a signed 32-bit int value */
> +     RTE_TEL_INT_VAL,    /** a signed 64-bit int value */
>       RTE_TEL_UINT_VAL,  /** an unsigned 64-bit int value */
>       RTE_TEL_CONTAINER, /** a container struct */
>  };
Thanks for catching this.

Acked-by: Bruce Richardson <bruce.richard...@intel.com>

Reply via email to