On Tue, Jan 14, 2025 at 5:32 AM Lin.Cao <lincao12@xxxxxxx> 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 <lincao12@xxxxxxx>
Reviewed-by: Alex Deucher <alexander.deucher@xxxxxxx>
> ---
> 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
>
