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

--- Comment #28 from Thomas Hellström <thellst...@vmware.com> ---
(In reply to Evangelos Foutras from comment #27)

> 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?

Because then we would potentially leak buffers since we'd have nothing that
freed that buffer if freeing had been skipped because draw->cur_blit_source !=
b.

Anyway, I think I know what might be going on now. Updated patch soon.

/Thomas

-- 
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