Mesa (refs/keep-around/41dabdc47538fb7660f7063d9dd423473eaa2515): radv: Fix output for sparse MRTs.
GitLab Mirror
gitlab-mirror at kemper.freedesktop.org
Mon Jul 9 17:28:23 UTC 2018
Module: Mesa
Branch: refs/keep-around/41dabdc47538fb7660f7063d9dd423473eaa2515
Commit: 41dabdc47538fb7660f7063d9dd423473eaa2515
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=41dabdc47538fb7660f7063d9dd423473eaa2515
Author: Bas Nieuwenhuizen <bas at basnieuwenhuizen.nl>
Date: Wed Jun 13 23:31:54 2018 +0200
radv: Fix output for sparse MRTs.
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>
Reviewed-by: Samuel Pitoiset <samuel.pitoiset at gmail.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 b8b425aca9..6eeedc65a3 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
More information about the mesa-commit
mailing list