Commit 808c2052 authored by Prike Liang's avatar Prike Liang Committed by Alex Deucher
Browse files

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



This reverts commit 820b3d37.

It’s better to validate VM TLB flushes in the flush‑TLB backend
rather than in the generic VM layer.

Reverting this patch depends on
commit fa7c231fc2b0 ("drm/amdgpu: validate the flush_gpu_tlb_pasid()")
being present in the tree.

Signed-off-by: default avatarPrike Liang <Prike.Liang@amd.com>
Reviewed-by: default avatarChristian König <christian.koenig@amd.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
(cherry picked from commit 9163fe4d)
parent 0bea77b1

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

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



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

	if (!params->unlocked &&

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

	    params->adev->family > AMDGPU_FAMILY_SI) {

	if (!params->unlocked) {

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



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