i965: Ensure FS execution in presence of atomic buffers
authorIago Toral Quiroga <itoral@igalia.com>
Fri, 18 Dec 2015 09:18:00 +0000 (10:18 +0100)
committerSamuel Iglesias Gonsálvez <siglesias@igalia.com>
Tue, 22 Dec 2015 11:38:48 +0000 (12:38 +0100)
commit9bbdd0eda47677d27b339095913b7c1ee64b5f07
treee755cec2d881c2c3f1c0361eddf2590b198a1889
parent1a95b87dad341e73e355b244a275d9b4d60537af
i965: Ensure FS execution in presence of atomic buffers

On Haswell we need to set the UAV_ONLY WM state bit when there are no colour
or depth buffer writes and on all hardware we should set the early
depth/stencil control field to PSEXEC unless early fragment tests are enabled
to make sure that the fragment shader is executed regardless of whether
per-fragment tests pass or not as the spec requires.

So far we have been doing this for images only, but we should apply the same
treatment to all side effectful scenarios. Suggested by Curro.

This is not strictly required for compliance with the original
ARB_shader_atomic_counters extension, it's only necessary to get the execution
semantics specified in GL4.2+ right.

v2:
- Mark active_fs_has_side_effects as constant. (Curro)
- Mention that this is only only necessary to get the execution semantics
specified in GL4.2+ right. (Curro)

Reviewed-by: Francisco Jerez <currojerez@riseup.net>
src/mesa/drivers/dri/i965/gen7_wm_state.c
src/mesa/drivers/dri/i965/gen8_ps_state.c