Note that WAVES_PER_SH should be 0x3ff on the compute ring for the ring priorities to be applied. I don't know if you need to do the same thing for the gfx ring. You can ask Andres for more info.
Marek On Tue, Aug 14, 2018 at 12:10 PM Samuel Pitoiset <samuel.pitoi...@gmail.com> wrote: > > The last parameter of radeon_set_sh_reg_seq() is the number of > dwords to emit. We were lucky because WAVES_PER_SH(0x3) is 3 but > it was initialized to 0. > > COMPUTE_RESOURCE_LIMITS is correctly set when generating > compute pipelines, so we don't need to initialize it. > > Signed-off-by: Samuel Pitoiset <samuel.pitoi...@gmail.com> > --- > src/amd/vulkan/si_cmd_buffer.c | 4 +--- > 1 file changed, 1 insertion(+), 3 deletions(-) > > diff --git a/src/amd/vulkan/si_cmd_buffer.c b/src/amd/vulkan/si_cmd_buffer.c > index 2337036c67..2cfa7f4c2c 100644 > --- a/src/amd/vulkan/si_cmd_buffer.c > +++ b/src/amd/vulkan/si_cmd_buffer.c > @@ -88,9 +88,7 @@ si_emit_compute(struct radv_physical_device > *physical_device, > radeon_emit(cs, 0); > radeon_emit(cs, 0); > > - radeon_set_sh_reg_seq(cs, R_00B854_COMPUTE_RESOURCE_LIMITS, > - S_00B854_WAVES_PER_SH(0x3)); > - radeon_emit(cs, 0); > + radeon_set_sh_reg_seq(cs, R_00B858_COMPUTE_STATIC_THREAD_MGMT_SE0, 2); > /* R_00B858_COMPUTE_STATIC_THREAD_MGMT_SE0 / SE1 */ > radeon_emit(cs, S_00B858_SH0_CU_EN(0xffff) | > S_00B858_SH1_CU_EN(0xffff)); > radeon_emit(cs, S_00B85C_SH0_CU_EN(0xffff) | > S_00B85C_SH1_CU_EN(0xffff)); > -- > 2.18.0 > > _______________________________________________ > mesa-dev mailing list > mesa-dev@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/mesa-dev _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev
