I agree. This bug has manifested for a while before I fixed it partially in "[PATCH] net/gve: fix dqo bug for chained descriptors" However, for higher queue counts (> 13); we still see this behavior. I'll add a statistic.
On Mon, Feb 19, 2024, 10:56 PM Stephen Hemminger <[email protected]> wrote: > On Mon, 19 Feb 2024 02:44:35 +0000 > Rushil Gupta <[email protected]> wrote: > > > This was causing failure for testpmd runs (for queues >=15) > > presumably due to flooding of logs due to descriptor ring being > > overwritten. > > > > Fixes: a01854 ("net/gve: fix dqo bug for chained descriptors") > > Cc: [email protected] > > > > Signed-off-by: Rushil Gupta <[email protected]> > > Reviewed-by: Joshua Washington <[email protected]> > > Isn't this still an error. What about the descriptor overwritten is there > an mbuf leak? > Maybe a statistic would be better than a message? >
