On 2/4/2025 3:13 AM, Alex Deucher wrote: > In begin_use/end_use use amdgpu_gfx_off_ctrl_immediate() > rather than amdgpu_gfx_off_ctrl() as we don't need the > extra delay before we allow gfxoff again. > > Signed-off-by: Alex Deucher <alexander.deucher@xxxxxxx> Won't this cause unnecessary GFX allows since sdma jobs could also be used for clearing the buffers? Thanks, Lijo > --- > drivers/gpu/drm/amd/amdgpu/sdma_v5_2.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/gpu/drm/amd/amdgpu/sdma_v5_2.c b/drivers/gpu/drm/amd/amdgpu/sdma_v5_2.c > index b1818e87889a2..7f2e1962b5755 100644 > --- a/drivers/gpu/drm/amd/amdgpu/sdma_v5_2.c > +++ b/drivers/gpu/drm/amd/amdgpu/sdma_v5_2.c > @@ -1882,7 +1882,7 @@ static void sdma_v5_2_ring_begin_use(struct amdgpu_ring *ring) > * doorbells when entering PG. If you remove this, update > * sdma_v5_2_ring_set_wptr() as well! > */ > - amdgpu_gfx_off_ctrl(adev, false); > + amdgpu_gfx_off_ctrl_immediate(adev, false); > } > > static void sdma_v5_2_ring_end_use(struct amdgpu_ring *ring) > @@ -1893,7 +1893,7 @@ static void sdma_v5_2_ring_end_use(struct amdgpu_ring *ring) > * disallow GFXOFF in some cases leading to > * hangs in SDMA. Allow GFXOFF when SDMA is complete. > */ > - amdgpu_gfx_off_ctrl(adev, true); > + amdgpu_gfx_off_ctrl_immediate(adev, true); > } > > static void sdma_v5_2_print_ip_state(struct amdgpu_ip_block *ip_block, struct drm_printer *p)
