<div dir="ltr"><div dir="ltr"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi all,<br>
<br>
On Thu, Jul 19, 2018 at 12:08 PM andrey simiklit<br>
<<a href="mailto:asimiklit.work@gmail.com" target="_blank">asimiklit.work@gmail.com</a>> wrote:<br>
> > Ugh... not so good.  According to Oliver on the bug, this just make the assert go away and doesn't actually fix anything.  Likely this is needed but not sufficient.<br>
><br>
> So as far as I understand Oliver found the bad commit in xorg glamor:<br>
> <a href="https://bugs.freedesktop.org/show_bug.cgi?id=107287" rel="noreferrer" target="_blank">https://bugs.freedesktop.org/show_bug.cgi?id=107287</a><br>
><br>
> So at the moment we should fix just this "assertion" issue for Intel because "rendering" issue came from xorg/glamor and there is no "rendering" issue in Intel part.<br>
> Please correct me if I incorrect.<br>
<br>
Reviving an old thread/patch here.<br>
<br>
Andrey, I reckon your patch here is still much needed as it fixes the<br>
assert() issue:<br>
<br>
intel_mipmap_tree.c:1301: intel_miptree_match_image: Assertion<br>
`image->TexObject->Target == mt->target' failed.<br>
<br>
Which is still occurring even with current master.<br>
<br>
My patch was to fix the rendering issue (landed a while ago before<br>
18.1 iirc), but yours was never merged and is still needed, I can<br>
reproduce the assert() at will with the reproducer from<br>
<a href="https://bugs.freedesktop.org/show_bug.cgi?id=107117" rel="noreferrer" target="_blank">https://bugs.freedesktop.org/show_bug.cgi?id=107117</a><br>
<br></blockquote><div><div><br></div><div>I could re-create this changes as MR instead of patch unless nobody against it.</div><div><br></div><div>I fixed a bit  the style of this patch to avoid new local variables:</div><div><a href="https://gitlab.freedesktop.org/asimiklit/mesa/commit/c11aae1cd403dde240c3877974f6a4b467cc83f5">https://gitlab.freedesktop.org/asimiklit/mesa/commit/c11aae1cd403dde240c3877974f6a4b467cc83f5</a></div><div><br></div></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
Jason, can we reconsider Andrii's patch? It still applies cleanly<br>
(<a href="https://patchwork.freedesktop.org/patch/237490/" rel="noreferrer" target="_blank">https://patchwork.freedesktop.org/patch/237490/</a>)<br>
<br>
Cheers,<br>
Olivier<br>
</blockquote></div></div></div>