<div dir="auto"><div><br><br><div class="gmail_quote"><div dir="ltr">On Fri, 1 Feb 2019, 00:23 Pete Heist <<a href="mailto:pete@heistp.net">pete@heistp.net</a> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
> On Feb 1, 2019, at 12:18 AM, Toke Høiland-Jørgensen <<a href="mailto:toke@redhat.com" target="_blank" rel="noreferrer">toke@redhat.com</a>> wrote:<br>
> <br>
> (crossed streams; but yeah, basically the same conclusion)<br>
<br>
Ok, crossed-crossed streams references it is… :)<br>
<br>
>> Do you think this is a correct patch?<br>
> <br>
> Well, the "correct patch" would be to get upstream fixed :)<br>
> <br>
> But yeah, I don't think it would hurt to carry this as a workaround<br>
> patch until that happens...<br>
> <br>
> I guess you could try running it for a while and see if anything else<br>
> blows up? You seem to have quite a knack for getting it to fail... ;)<br>
<br>
That sounds reasonable, let me see if I can break it in some different cases.<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">Cool.</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
My guess is, past qdiscs didn’t manage to fill the tailroom space and cause this problem, but I’m not sure…<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">Wouldn't be the first bug we've exposed in this code: <a href="https://git.kernel.org/pub/scm/linux/kernel/git/davem/net-next.git/commit/?id=d5a672ac9f48f81b20b1cad1d9ed7bbf4e418d4c">https://git.kernel.org/pub/scm/linux/kernel/git/davem/net-next.git/commit/?id=d5a672ac9f48f81b20b1cad1d9ed7bbf4e418d4c</a></div><div dir="auto"><br></div><div dir="auto">-Toke</div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
</blockquote></div></div></div>