i915g: fix incorrect gl_FragCoord value
authorNicholas Bishop <nicholasbishop@gmail.com>
Thu, 25 Aug 2016 23:31:53 +0000 (19:31 -0400)
committerStéphane Marchesin <marcheu@chromium.org>
Tue, 11 Oct 2016 01:32:36 +0000 (18:32 -0700)
commit64435fd888ec5932024292d87b0afcecbe5fb818
treefcdb263ded5a468768a11133d3eb81d063fcf512
parentc10dcb2ce837922c6ee4e191e6d6202098a5ee10
i915g: fix incorrect gl_FragCoord value

On Intel Pineview M hardware, the i915 gallium driver doesn't output
the correct gl_FragCoord. It seems to always have an X coord of 0.0
and a Y coord of the window's height in pixels, e.g. 600.0f or such.

I believe this is a regression caused in part by this commit:
afa035031ff9e0c07a2297d864e46c76f7bfff58

The old behavior used the output at index zero, while the new behavior
uses actual zeroes. In the case of gl_FragCoord the output at index
zero happened to be the correct one, so the behavior appeared correct
although the code already had a bug.

Fixed by checking for I915_SEMANTIC_POS when setting up texCoords. If
the generic_mapping is I915_SEMANTIC_POS, look for the
TGSI_SEMANTIC_POSITION instead of a TGSI_SEMANTIC_GENERIC output.

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

Reviewed-by: Stéphane Marchesin <marcheu@chromium.org>
Tested-by: Stéphane Marchesin <marcheu@chromium.org>
src/gallium/drivers/i915/i915_state_derived.c