<div dir="auto">I agree. <div dir="auto"><div dir="auto">This bug has manifested for a while before I fixed it partially in "[PATCH] net/gve: fix dqo bug for chained descriptors"<div dir="auto">However, for higher queue counts (> 13); we still see this behavior. I'll add a statistic.</div></div></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Feb 19, 2024, 10:56 PM Stephen Hemminger <<a href="mailto:stephen@networkplumber.org">stephen@networkplumber.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Mon, 19 Feb 2024 02:44:35 +0000<br>
Rushil Gupta <<a href="mailto:rushilg@google.com" target="_blank" rel="noreferrer">rushilg@google.com</a>> wrote:<br>
<br>
> This was causing failure for testpmd runs (for queues >=15)<br>
> presumably due to flooding of logs due to descriptor ring being<br>
> overwritten.<br>
> <br>
> Fixes: a01854 ("net/gve: fix dqo bug for chained descriptors")<br>
> Cc: <a href="mailto:stable@dpdk.org" target="_blank" rel="noreferrer">stable@dpdk.org</a><br>
> <br>
> Signed-off-by: Rushil Gupta <<a href="mailto:rushilg@google.com" target="_blank" rel="noreferrer">rushilg@google.com</a>><br>
> Reviewed-by: Joshua Washington <<a href="mailto:joshwash@google.com" target="_blank" rel="noreferrer">joshwash@google.com</a>><br>
<br>
Isn't this still an error. What about the descriptor overwritten is there an mbuf leak?<br>
Maybe a statistic would be better than a message?<br>
</blockquote></div>