[Mesa-stable] [Mesa-dev] [PATCH] radv: Fix output for sparse MRTs.

[Dave Airlie]

On 14 June 2018 at 07:35, Bas Nieuwenhuizen <bas at basnieuwenhuizen.nl> wrote:
> We need to init the cb_shader_format correctly with the changed
> col_format, so this moves the col_format adjustment to before the
> adjustment to before the cb_shader_mask gets generated.
>
> Fixes: 06d3c650980 "radv: fix a GPU hang when MRTs are sparse"
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=106903
> CC: 18.1 <mesa-stable at lists.freedesktop.org>

Reviewed-by: Dave Airlie <airlied at redhat.com>

> ---
>  src/amd/vulkan/radv_pipeline.c | 19 ++++++++++---------
>  1 file changed, 10 insertions(+), 9 deletions(-)
>
> diff --git a/src/amd/vulkan/radv_pipeline.c b/src/amd/vulkan/radv_pipeline.c
> index b8b425aca9f..6eeedc65a39 100644
> --- a/src/amd/vulkan/radv_pipeline.c
> +++ b/src/amd/vulkan/radv_pipeline.c
> @@ -524,20 +524,21 @@ radv_pipeline_compute_spi_color_formats(struct radv_pipeline *pipeline,
>                 col_format |= cf << (4 * i);
>         }
>
> -       blend->cb_shader_mask = ac_get_cb_shader_mask(col_format);
> -
> -       if (blend->mrt0_is_dual_src)
> -               col_format |= (col_format & 0xf) << 4;
> -       blend->spi_shader_col_format = col_format;
> -
>         /* If the i-th target format is set, all previous target formats must
>          * be non-zero to avoid hangs.
>          */
> -       num_targets = (util_last_bit(blend->spi_shader_col_format) + 3) / 4;
> +       num_targets = (util_last_bit(col_format) + 3) / 4;
>         for (unsigned i = 0; i < num_targets; i++) {
> -               if (!(blend->spi_shader_col_format & (0xf << (i * 4))))
> -                       blend->spi_shader_col_format |= V_028714_SPI_SHADER_32_R << (i * 4);
> +               if (!(col_format & (0xf << (i * 4)))) {
> +                       col_format |= V_028714_SPI_SHADER_32_R << (i * 4);
> +               }
>         }
> +
> +       blend->cb_shader_mask = ac_get_cb_shader_mask(col_format);
> +
> +       if (blend->mrt0_is_dual_src)
> +               col_format |= (col_format & 0xf) << 4;
> +       blend->spi_shader_col_format = col_format;
>  }
>
>  static bool
> --
> 2.17.0
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev