On Thu, Sep 03, 2026 at 01:06:11PM -0600, Paolo Abeni wrote: > > > On 9/3/26 1:38 AM, [email protected] 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 >
makes sense, I will address these in v7 of this patchset. Best, Juanlu
