>-----Original Message-----
>From: Dharmik Thakkar [mailto:[email protected]]
>Sent: Thursday, May 9, 2019 10:19 AM
>To: Wang, Yipeng1 <[email protected]>; Gobriel, Sameh
><[email protected]>; Richardson, Bruce
><[email protected]>; De Lara Guarch, Pablo
><[email protected]>
>Cc: [email protected]; [email protected]; [email protected];
>Dharmik Thakkar <[email protected]>;
>[email protected]
>Subject: [PATCH v4 1/3] hash: fix position bug in 'free key with position'
>
>Currently, in rte_hash_free_key_with_position(), the position returned
>to the ring of free_slots leads to an unexpected conflict with a key
>already in use.
>
>This patch fixes incorrect position returned to the ring of free_slots.
>
>Bugzilla ID: 261
>Fixes: 9d033dac7d7c ("hash: support no free on delete")
>Cc: [email protected]
>Cc: [email protected]
>
>Reported-by: Linfan <[email protected]>
>Suggested-by: Linfan <[email protected]>
>Signed-off-by: Dharmik Thakkar <[email protected]>
>---
> lib/librte_hash/rte_cuckoo_hash.c | 13 ++++++++-----
> 1 file changed, 8 insertions(+), 5 deletions(-)
>
>diff --git a/lib/librte_hash/rte_cuckoo_hash.c
>b/lib/librte_hash/rte_cuckoo_hash.c
>index 261267b7fd3d..5029f9f61fae 100644
>--- a/lib/librte_hash/rte_cuckoo_hash.c
>+++ b/lib/librte_hash/rte_cuckoo_hash.c
>@@ -1587,14 +1587,17 @@ int __rte_experimental
> rte_hash_free_key_with_position(const struct rte_hash *h,
> const int32_t position)
> {
>- RETURN_IF_TRUE(((h == NULL) || (position == EMPTY_SLOT)), -EINVAL);
>+ /* Key index where key is stored, adding the first dummy index*/
[Wang, Yipeng] Minor issue, missing a space at the end.
Acked-by: Yipeng Wang <[email protected]>
