From 47e01e77d8b658606527f048cda786440f7fbe85 Mon Sep 17 00:00:00 2001 From: Gert Wollny Date: Mon, 10 Sep 2018 12:39:44 +0200 Subject: [PATCH] mesa/texture: Also check for LA texture when querying intensity component size MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit Gallium may pick L16A16_FLOAT to represent GL_INTENSITY16F if no intensity format is provided by the driver. However, when calling glGetTexLevelParameteriv(..., GL_TEXTURE_INTENSITY_SIZE, ...) mesa will return a zero size because the actually used format has no intensity channel and as a fallback only the sizes of the red/green channels are checked. Also checking for LA sizes in the allocated texture resolves this problem. v2: Only check alpha channel size and return it (Marek) L and A size are always the same in this case. Fixes (on virgl): ext_framebuffer_multisample-fast-clear GL_ARB_texture_float * Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=107832 Signed-off-by: Gert Wollny Reviewed-by: Marek Olšák --- src/mesa/main/texparam.c | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/src/mesa/main/texparam.c b/src/mesa/main/texparam.c index b5d86d64d5b..a3ec7241986 100644 --- a/src/mesa/main/texparam.c +++ b/src/mesa/main/texparam.c @@ -1426,6 +1426,11 @@ get_tex_level_parameter_image(struct gl_context *ctx, _mesa_get_format_bits(texFormat, GL_TEXTURE_GREEN_SIZE)); } + if (*params == 0 && pname == GL_TEXTURE_INTENSITY_SIZE) { + /* Gallium may store intensity as LA */ + *params = _mesa_get_format_bits(texFormat, + GL_TEXTURE_ALPHA_SIZE); + } } else { *params = 0; -- 2.30.2