On 2023/5/9 14:23, Ruifeng Wang wrote:
-----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").
Thanks for your review. Sorry for quoting the wrong commit.
This issue was indeed introduced by commit af75078fece3 ("first public
release").
I will fix this in the next version.
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
.
