i965: Fix predicated-send-based discards with MRT.
authorEric Anholt <eric@anholt.net>
Tue, 4 Mar 2014 23:12:40 +0000 (15:12 -0800)
committerEric Anholt <eric@anholt.net>
Wed, 5 Mar 2014 21:37:33 +0000 (13:37 -0800)
commit171ec9585f214c2dff3cfc97acebf5286bb5001d
treefc01dd218467da96f8ae6da35ea6cf84f5e6c4b5
parent9856d658ced3382060f4d805f3e4fbdc2b618e64
i965: Fix predicated-send-based discards with MRT.

We need the header setup to not be predicated on which pixels are
undiscarded.  I'm not sure originally if I had thought that the mask
disable implied predicate disable, or if I had just misread the mask
disable as predicate disable.  Either way, I know I had spent more time
thinking about this in the gen8 generator than the gen7 generator.

Plus, it turns out that I had mis-implemented the "the GPU will use the
predicate unless this header is present" comment, by skipping setting up
the pixel mask when the header was present.

Fixes GPU hangs in piglit glsl-fs-discard-mrt, Trine, Trine 2 and
preusmably MLL.

Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=75207
Tested-by: Tapani Pälli <tapani.palli@intel.com>
Reviewed-by: Kenneth Graunke <kenneth@whitecape.org>
src/mesa/drivers/dri/i965/brw_fs_generator.cpp