Revert "i965/tex_image: Reference the renderbuffer miptree in setTexBuffer2"
authorKenneth Graunke <kenneth@whitecape.org>
Mon, 16 Oct 2017 22:30:47 +0000 (15:30 -0700)
committerKenneth Graunke <kenneth@whitecape.org>
Mon, 16 Oct 2017 23:02:53 +0000 (16:02 -0700)
commit9e779e59b20ea9510801a4484613d33d8a8e4c3e
tree6633fd5dc2669359ae5ed0bed0b8ceda1188cb66
parentc6e584f194ea0d475e0fe1564c5491a2a535146a
Revert "i965/tex_image: Reference the renderbuffer miptree in setTexBuffer2"

This reverts commit d80cbbeaff9329fdc78ae3d97097c1e65dfcdd61.

It turns out that formats do matter - the framebuffer's miptree has an
sRGB format, and the one we created did not.  This broke rendering when
using KWin compositing, GNOME Terminal Fedora (with a transparent
background), and Qt menu rendering in general, to name a few.

It's been a month and this hasn't been fixed, and I'm sick of reverting
this patch or applying NAK'd hacks and restarting various programs at
random times every day, multiple times a day, to keep my desktop
environment functional.

The only benefit of this patch was to prepare the way for modifiers,
which AFAIK aren't finished yet anyway, so there's really no downside
to reverting it.

Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=102924
src/mesa/drivers/dri/i965/intel_tex_image.c