https://bugs.freedesktop.org/show_bug.cgi?id=104214

--- Comment #27 from Evangelos Foutras <evange...@foutrelis.com> ---
(In reply to Thomas Hellström from comment #25)
> Created attachment 136645 [details] [review]
> Patch to fix a potential problem with renderbuffer freeing

Doesn't fix it for me (testing with mpv; probably same for dota).

With the printf()s from bug 104376, and printing !!buf->busy just before the
dri3_free_render_buffer() call in dri3_handle_present_event(), I get this just
before it crashes:

==================
buf->busy = 0
dri3_handle_present_event() freed 0x7fffbc8d6380
dri3_get_buffer() freeing buffer = 0x7fffbc8d6380; draw->buffers[buf_id] =
(nil)
==================

I'm totally unfamiliar with gpu driver code but I'm wondering why the
"buf->pixmap == ie->pixmap" check isn't used anymore; why didn't commit
15e208c4cc just stick "draw->cur_blit_source != b" into the second if
conditional and leave the rest as it was?

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to