<div dir="ltr"><div>Hi,</div><div><br></div><div>I opened MR to fix this issue and started a discussion about the proper solution:</div><div><a href="https://gitlab.freedesktop.org/mesa/mesa/merge_requests/1258">https://gitlab.freedesktop.org/mesa/mesa/merge_requests/1258</a></div><div><br></div><div>Thanks,</div><div>Andrii.<br> </div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Jul 24, 2019 at 10:32 AM Olivier Fourdan <<a href="mailto:fourdan@gmail.com">fourdan@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi,<br>
<br>On Wed, May 22, 2019 at 4:01 AM Ian Romanick <<a href="mailto:idr@freedesktop.org" target="_blank">idr@freedesktop.org</a>> wrote:<br>> > On Thu, Jul 19, 2018 at 12:08 PM andrey simiklit<br>> ><br>> > 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>> Looking at the patch and the "Simple reproducer" in the bug, I think<br>> this just papers over the issue.  It seems like the problem is somewhere<br>> down inside the driver's handling of glXBindTexImageEXT.  My best guess<br>> is that the texture is GL_TEXTURE_2D but the miptree backing it is<br>> GL_TEXTURE_RECTANGLE.  It seems that the glXBindTexImageEXT handling<br>> should mark the miptree as GL_TEXTURE_2D when binding the image to a<br>> texture that is GL_TEXTURE_2D.  Or is that not possible for some<br>> non-obvious reason?<br>
<br>Sorry to be a pain, but I still get bug reports in xfce for this, for<br>everyone using a Mesa build with debug enabled (or a pre-release for<br>that matter) on Intel, the `assert()` are enabled and this bug occurs<br>with the xfce compositor.<br>
<br>Maybe Andrii's patch is just hiding the issue, but that's already the<br>case without the `assert()` enabled (i.e. all stable releases of<br>Mesa), so I guess it's not such a big deal anyway.<br>
<br>Can we agree on a fix on this?<br>
<br>Cheers,<br>Olivier<br>
</blockquote></div></div>