Skip to content

Commit 1f02f20

Browse files
Gang Baalexdeucher
authored andcommitted
drm/amdgpu: Avoid extra evict-restore process.
If vm belongs to another process, this is fclose after fork, wait may enable signaling KFD eviction fence and cause parent process queue evicted. [677852.634569] amdkfd_fence_enable_signaling+0x56/0x70 [amdgpu] [677852.634814] __dma_fence_enable_signaling+0x3e/0xe0 [677852.634820] dma_fence_wait_timeout+0x3a/0x140 [677852.634825] amddma_resv_wait_timeout+0x7f/0xf0 [amdkcl] [677852.634831] amdgpu_vm_wait_idle+0x2d/0x60 [amdgpu] [677852.635026] amdgpu_flush+0x34/0x50 [amdgpu] [677852.635208] filp_flush+0x38/0x90 [677852.635213] filp_close+0x14/0x30 [677852.635216] do_close_on_exec+0xdd/0x130 [677852.635221] begin_new_exec+0x1da/0x490 [677852.635225] load_elf_binary+0x307/0xea0 [677852.635231] ? srso_alias_return_thunk+0x5/0xfbef5 [677852.635235] ? ima_bprm_check+0xa2/0xd0 [677852.635240] search_binary_handler+0xda/0x260 [677852.635245] exec_binprm+0x58/0x1a0 [677852.635249] bprm_execve.part.0+0x16f/0x210 [677852.635254] bprm_execve+0x45/0x80 [677852.635257] do_execveat_common.isra.0+0x190/0x200 Suggested-by: Christian König <[email protected]> Signed-off-by: Gang Ba <[email protected]> Reviewed-by: Christian König <[email protected]> Signed-off-by: Alex Deucher <[email protected]> Cc: [email protected]
1 parent 284d4df commit 1f02f20

File tree

1 file changed

+2
-4
lines changed

1 file changed

+2
-4
lines changed

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

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -2414,13 +2414,11 @@ void amdgpu_vm_adjust_size(struct amdgpu_device *adev, uint32_t min_vm_size,
24142414
*/
24152415
long amdgpu_vm_wait_idle(struct amdgpu_vm *vm, long timeout)
24162416
{
2417-
timeout = dma_resv_wait_timeout(vm->root.bo->tbo.base.resv,
2418-
DMA_RESV_USAGE_BOOKKEEP,
2419-
true, timeout);
2417+
timeout = drm_sched_entity_flush(&vm->immediate, timeout);
24202418
if (timeout <= 0)
24212419
return timeout;
24222420

2423-
return dma_fence_wait_timeout(vm->last_unlocked, true, timeout);
2421+
return drm_sched_entity_flush(&vm->delayed, timeout);
24242422
}
24252423

24262424
static void amdgpu_vm_destroy_task_info(struct kref *kref)

0 commit comments

Comments
 (0)