From bed54fa4029e6d98becd2dafb4bed982ac94663b Mon Sep 17 00:00:00 2001 From: Olivia Lee Date: Wed, 14 May 2025 18:55:39 -0700 Subject: [PATCH] panfrost: fix assertion failure compiling image conversion shaders In 59a3e12039c, we changed the UBO->push optimization in panfrost to only push UBOs that are available in a CPU buffer. We require first_ubo_is_default_ubo, to ensure that UBO0 will be a user buffer. We weren't setting this flag for the image conversion shaders, so got an assertion failure compiling them. This can be triggered by the panvk_force_afbc_packing driconf option. The conversion shader info UBO isn't exactly a "default" UBO in the sense of being lowered from uniforms, but it is a user buffer, so setting the flag should be fine. Fixes: 59a3e12039c ("panfrost: do not push "true" UBOs") Signed-off-by: Olivia Lee Reviewed-by: Boris Brezillon Reviewed-by: Eric R. Smith Part-of: --- src/gallium/drivers/panfrost/pan_mod_conv_cso.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/gallium/drivers/panfrost/pan_mod_conv_cso.c b/src/gallium/drivers/panfrost/pan_mod_conv_cso.c index 2c58b6df55f..319dfb4b496 100644 --- a/src/gallium/drivers/panfrost/pan_mod_conv_cso.c +++ b/src/gallium/drivers/panfrost/pan_mod_conv_cso.c @@ -474,6 +474,9 @@ panfrost_get_mod_convert_shaders(struct panfrost_context *ctx, nir_shader *nir = \ panfrost_create_##name##_shader(screen, __VA_ARGS__); \ nir->info.num_ubos = 1; \ + /* "default" UBO is maybe not correct here, but in panfrost we're */ \ + /* using this as an indicator for whether UBO0 is a user UBO */ \ + nir->info.first_ubo_is_default_ubo = true; \ shader->name##_cso = pipe_shader_from_nir(pctx, nir); \ }