> -----Original Message-----
> From: Jie Hai <[email protected]>
> Sent: Tuesday, May 9, 2023 9:29 AM
> To: Honnappa Nagarahalli <[email protected]>; Konstantin Ananyev
> <[email protected]>; Ruifeng Wang <[email protected]>; Gavin
> Hu
> <[email protected]>; Olivier Matz <[email protected]>; Dharmik Jayesh
> Thakkar
> <[email protected]>
> Cc: [email protected]; [email protected]
> Subject: [PATCH v5 1/3] ring: fix unmatched type definition and usage
>
> Field 'flags' of struct rte_ring is defined as int type. However, it is used
> as unsigned
> int. To ensure consistency, change the type of flags to unsigned int. Since
> these two
> types has the same byte size, this change is not an ABI change.
>
> Fixes: cc4b218790f6 ("ring: support configurable element size")
The change looks good.
However, I think the fix line is not accurate.
I suppose it fixes af75078fece3 ("first public release").
>
> Signed-off-by: Jie Hai <[email protected]>
> Acked-by: Konstantin Ananyev <[email protected]>
> Acked-by: Chengwen Feng <[email protected]>
> ---
> lib/ring/rte_ring_core.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/lib/ring/rte_ring_core.h b/lib/ring/rte_ring_core.h index
> 82b237091b71..1c809abeb531 100644
> --- a/lib/ring/rte_ring_core.h
> +++ b/lib/ring/rte_ring_core.h
> @@ -120,7 +120,7 @@ struct rte_ring_hts_headtail { struct rte_ring {
> char name[RTE_RING_NAMESIZE] __rte_cache_aligned;
> /**< Name of the ring. */
> - int flags; /**< Flags supplied at creation. */
> + uint32_t flags; /**< Flags supplied at creation. */
> const struct rte_memzone *memzone;
> /**< Memzone, if any, containing the rte_ring */
> uint32_t size; /**< Size of ring. */
> --
> 2.33.0
