On 6/21/21 5:44 PM, Dumitrescu, Cristian wrote:

quoted

-----Original Message-----
From: Thierry Herbelot <redacted>
Sent: Thursday, June 17, 2021 8:12 AM
To: Thomas Monjalon <redacted>; Dumitrescu, Cristian
[off-list ref]
Cc: dev@dpdk.org; stable@dpdk.org
Subject: Re: [dpdk-stable] [PATCH v2] table: fix typo

On 6/17/21 9:01 AM, Thomas Monjalon wrote:

quoted

quoted

quoted

Check Bucket key for all possible indices.

Fixes: d0a00966618ba ("table: add exact match SWX table")
Cc: stable@dpdk.org
Cc: Cristian Dumitrescu <redacted>

Signed-off-by: Thierry Herbelot <redacted>

[...]

quoted

quoted

-	return (!bkt->sig[0] && !bkt->sig[1] && !bkt->sig[2] && !bkt->sig[2])
+	return (!bkt->sig[0] && !bkt->sig[1] && !bkt->sig[2] && !bkt->sig[3])

Acked-by: Cristian Dumitrescu <redacted>

Thierry, thanks again, you could have actually added my ack from V1 :)

The root cause looks to be a typo indeed,
but the impact or the scope of the issue
should stated in few words in the title please.
No need to send a v3, please suggest a title and I will apply.

table: check a bucket is empty for all bucket keys

	Thierry

Hi Thierry,

I think Thomas is asking for you to send a new version of this patch with the title updated. Is this OK with you?

Hello Cristian

Sure !

What do you think of this title:

"table: check a bucket is empty for all bucket keys"

	Best regards

	Thierry

Thanks,
Cristian

quoted

quoted

--
Thierry Herbelot
Senior Software Engineer
Tel: +33 1 39 30 92 61
http://www.6wind.com/

Follow us:
https://www.linkedin.com/company/6wind/
https://twitter.com/6WINDsoftware
https://www.youtube.com/user/6windsoftware

-- 
Thierry Herbelot
Senior Software Engineer
Tel: +33 1 39 30 92 61
http://www.6wind.com/

Follow us:
https://www.linkedin.com/company/6wind/
https://twitter.com/6WINDsoftware
https://www.youtube.com/user/6windsoftware