Re: [net-next,v6,5/6] selftests: net: add multithread server support to iou-zcrx

From: Paolo Abeni

Date: Thu Sep 03 2026 - 07:13:37 EST




On 9/3/26 1:38 AM, netdev-bot+sashiko@xxxxxxxxxx wrote:
> Thank you for your contribution! Sashiko AI review found 6 potential
> issue(s) to consider:
>
> Critical: 0 · High: 1 · Medium: 2 (3) · Low: 3
>
> - [High] listen() is now executed before the workers have registered
> their zcrx ifqs, breaking the ordering that two existing in-tree tests…
> - [Medium] A worker thread that is dispatched zero connections reports
> success without receiving or verifying a single byte.
> - [Medium] The new CQ sizing removes the implicit bound that kept the
> zcrx refill-queue producer inside one ring.

My understanding is that the above 3 concerns may affect tests stability
and should be addressed before merging.

Thanks,

Paolo