Thread (15 messages) flat view 15 messages, 4 authors, 2d ago

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

From: Paolo Abeni <pabeni@redhat.com>
Date: 2026-09-03 11:06:17
Also in: io-uring, linux-kselftest, lkml


On 9/3/26 1:38 AM, netdev-bot+sashiko@kernel.org 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
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help