Commit 820b3d37 authored by Alex Deucher's avatar Alex Deucher
Browse files

drm/amdgpu: don't attach the tlb fence for SI 

SI hardware doesn't support pasids, user mode queues, or
KIQ/MES so there is no need for this.  Doing so results in
a segfault as these callbacks are non-existent for SI.

Closes: https://gitlab.freedesktop.org/drm/amd/-/issues/4744


Fixes: f3854e04 ("drm/amdgpu: attach tlb fence to the PTs update")
Reviewed-by: default avatarTimur Kristóf <timur.kristof@gmail.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent 8acdad93

drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c

+3 −1
Original line number Diff line number Diff line
@@ -1069,7 +1069,9 @@ amdgpu_vm_tlb_flush(struct amdgpu_vm_update_params *params, 
	}



	/* Prepare a TLB flush fence to be attached to PTs */

	if (!params->unlocked) {

	if (!params->unlocked &&

	    /* SI doesn't support pasid or KIQ/MES */

	    params->adev->family > AMDGPU_FAMILY_SI) {

		amdgpu_vm_tlb_fence_create(params->adev, vm, fence);



		/* Makes sure no PD/PT is freed before the flush */