iris: Stop advertising MSAA storage images by mistake
authorKenneth Graunke <kenneth@whitecape.org>
Mon, 22 Jul 2019 23:18:40 +0000 (16:18 -0700)
committerKenneth Graunke <kenneth@whitecape.org>
Tue, 23 Jul 2019 00:30:13 +0000 (17:30 -0700)
commit2f1c7fae9e476fe3914749b33964bc66a9f48f92
tree97dcf3b9896dbf0b6db1f73fc32688f260c8aa54
parent82607f8a900796871470ac4f1a04e154392e4898
iris: Stop advertising MSAA storage images by mistake

st_extensions.c sets const->MaxImageSamples (GL_MAX_IMAGE_SAMPLES) by
looping over [16, 15, .. 1x] MSAA modes, and RGBA/BGRA/ARGB/ABGR 8888
color formats, calling pipe->is_format_supported() for each, with
the usage set to PIPE_BIND_SHADER_IMAGE.  If any are supported, it
selects that number of samples.

We were checking if sample_count <= 1, which meant that we were getting
a value of 1x MSAA, rather than the expected 0x (feature doesn't exist).

But, only on Icelake because Gen11 adds support for typed read messages
for R8G8B8A8_UNORM.  The lack of typed read messages for these formats
was tricking the check on Gen9 to say no correctly.  This caused some
Icelake conformance failures, because we don't implement this feature.

Just check for sample_count == 0 instead.
src/gallium/drivers/iris/iris_formats.c