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 <step...@networkplumber.org>
wrote:
> On Mon, 19 Feb 2024 02:44:35 +0000
> Rushil Gupta <rush...@google.com> 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: sta...@dpdk.org
> >
> > Signed-off-by: Rushil Gupta <rush...@google.com>
> > Reviewed-by: Joshua Washington <joshw...@google.com>
>
> 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?
>
