diff options
author | Marek Olšák <marek.olsak@amd.com> | 2020-09-17 19:45:14 -0400 |
---|---|---|
committer | Marek Olšák <marek.olsak@amd.com> | 2020-09-25 04:37:23 -0400 |
commit | 8be46d6558e04f5dc9b8bebd31a36b1f3d593aa6 (patch) | |
tree | d504dcd80a079870f29a127796f27cce91071e33 | |
parent | a7e3cc7a0eafc1076a2f7775f754e74584fc3537 (diff) |
radeonsi: fix indirect dispatches with variable block sizes
The block size input was uninitialized.
Fixes: 77c81164bc1c "radeonsi: support ARB_compute_variable_group_size"
Acked-by: Pierre-Eric Pelloux-Prayer <pierre-eric.pelloux-prayer@amd.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/6782>
-rw-r--r-- | src/gallium/drivers/radeonsi/si_compute.c | 21 |
1 files changed, 10 insertions, 11 deletions
diff --git a/src/gallium/drivers/radeonsi/si_compute.c b/src/gallium/drivers/radeonsi/si_compute.c index ee28f82a165..f7f6c15c800 100644 --- a/src/gallium/drivers/radeonsi/si_compute.c +++ b/src/gallium/drivers/radeonsi/si_compute.c @@ -697,27 +697,26 @@ static void si_setup_nir_user_data(struct si_context *sctx, const struct pipe_gr 12 * sel->info.uses_grid_size; unsigned cs_user_data_reg = block_size_reg + 12 * program->reads_variable_block_size; - if (info->indirect) { - if (sel->info.uses_grid_size) { + if (sel->info.uses_grid_size) { + if (info->indirect) { for (unsigned i = 0; i < 3; ++i) { si_cp_copy_data(sctx, sctx->gfx_cs, COPY_DATA_REG, NULL, (grid_size_reg >> 2) + i, COPY_DATA_SRC_MEM, si_resource(info->indirect), info->indirect_offset + 4 * i); } - } - } else { - if (sel->info.uses_grid_size) { + } else { radeon_set_sh_reg_seq(cs, grid_size_reg, 3); radeon_emit(cs, info->grid[0]); radeon_emit(cs, info->grid[1]); radeon_emit(cs, info->grid[2]); } - if (program->reads_variable_block_size) { - radeon_set_sh_reg_seq(cs, block_size_reg, 3); - radeon_emit(cs, info->block[0]); - radeon_emit(cs, info->block[1]); - radeon_emit(cs, info->block[2]); - } + } + + if (program->reads_variable_block_size) { + radeon_set_sh_reg_seq(cs, block_size_reg, 3); + radeon_emit(cs, info->block[0]); + radeon_emit(cs, info->block[1]); + radeon_emit(cs, info->block[2]); } if (program->num_cs_user_data_dwords) { |