Skip to content

Commit eb296c0

Browse files
committed
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: f3854e0 ("drm/amdgpu: attach tlb fence to the PTs update") Reviewed-by: Timur Kristóf <timur.kristof@gmail.com> Signed-off-by: Alex Deucher <alexander.deucher@amd.com> (cherry picked from commit 820b3d3)
1 parent 3c41114 commit eb296c0

1 file changed

Lines changed: 3 additions & 1 deletion

File tree

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

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1069,7 +1069,9 @@ amdgpu_vm_tlb_flush(struct amdgpu_vm_update_params *params,
10691069
}
10701070

10711071
/* Prepare a TLB flush fence to be attached to PTs */
1072-
if (!params->unlocked) {
1072+
if (!params->unlocked &&
1073+
/* SI doesn't support pasid or KIQ/MES */
1074+
params->adev->family > AMDGPU_FAMILY_SI) {
10731075
amdgpu_vm_tlb_fence_create(params->adev, vm, fence);
10741076

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

0 commit comments

Comments
 (0)