On Tue, Jan 14, 2025 at 5:32 AM Lin.Cao <[email protected]> wrote: > > 'commit 6e66dc05b54f ("drm/amdgpu: set the VM pointer to NULL in > amdgpu_job_prepare")' set job->vm as NULL if there is no fence. It will > cause emit switch buffer be skippen if job->vm set as NULL. > > Check job rather than vm could solve this problem. > > Signed-off-by: Lin.Cao <[email protected]>
Reviewed-by: Alex Deucher <[email protected]> > --- > drivers/gpu/drm/amd/amdgpu/amdgpu_ib.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ib.c > b/drivers/gpu/drm/amd/amdgpu/amdgpu_ib.c > index e0bc37557d2c..2ea98ec60220 100644 > --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_ib.c > +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_ib.c > @@ -297,7 +297,7 @@ int amdgpu_ib_schedule(struct amdgpu_ring *ring, unsigned > int num_ibs, > amdgpu_ring_patch_cond_exec(ring, cond_exec); > > ring->current_ctx = fence_ctx; > - if (vm && ring->funcs->emit_switch_buffer) > + if (job && ring->funcs->emit_switch_buffer) > amdgpu_ring_emit_switch_buffer(ring); > > if (ring->funcs->emit_wave_limit && > -- > 2.46.1 >
