On 23.03.2016 02:27, Marek Olšák wrote:
> From: Marek Olšák <marek.ol...@amd.com>
> 
> ---
>  src/gallium/drivers/radeonsi/si_state.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/src/gallium/drivers/radeonsi/si_state.c 
> b/src/gallium/drivers/radeonsi/si_state.c
> index b9bdd47..b8fde00 100644
> --- a/src/gallium/drivers/radeonsi/si_state.c
> +++ b/src/gallium/drivers/radeonsi/si_state.c
> @@ -2993,7 +2993,8 @@ si_make_texture_descriptor(struct si_screen *screen,
>       if (type == V_008F1C_SQ_RSRC_IMG_1D_ARRAY) {
>               height = 1;
>               depth = res->array_size;
> -     } else if (type == V_008F1C_SQ_RSRC_IMG_2D_ARRAY) {
> +     } else if (type == V_008F1C_SQ_RSRC_IMG_2D_ARRAY ||
> +                type == V_008F1C_SQ_RSRC_IMG_2D_MSAA_ARRAY) {
>               if (sampler || res->target != PIPE_TEXTURE_3D)
>                       depth = res->array_size;
>       } else if (type == V_008F1C_SQ_RSRC_IMG_CUBE)
> 

Reviewed-and-Tested-by: Michel Dänzer <michel.daen...@amd.com>


P.S. The incorrect code actually seemed to "fix" the
"spec@ext_framebuffer_multisample_blit_scaled@blit-scaled
samples={2,4,6,8} with gl_texture_2d_multisample_array" tests for me,
i.e. those tests were originally failing, then passing since Nicolai's
image changes, and now failing again with this fix. Might be worth
looking into, or maybe they were just passing by chance with the
incorrect code?


-- 
Earthling Michel Dänzer               |               http://www.amd.com
Libre software enthusiast             |             Mesa and X developer
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to