RE: [PATCH] drm/amdgpu: Mark amdgpu_bo as invalid after moved

2024-07-11 Thread YuanShang Mao (River)
invalidated BO. Agree with Trigger. Seems kfd eviction should been synced to move notify, not the move action. Thanks River -Original Message- From: Christian König Sent: Thursday, July 11, 2024 8:39 PM To: Huang, Trigger ; YuanShang Mao (River) ; amd-gfx@lists.freedesktop.org Subject: Re

RE: [PATCH] drm/amdgpu: Mark amdgpu_bo as invalid after moved

2024-07-12 Thread YuanShang Mao (River)
er is still accessing the "invalidated" BO. I am not familiar with amdgpu_vm_bo state machine, so I don’t know if it is an code error or an design error. Thanks River -Original Message- From: YuanShang Mao (River) Sent: Friday, July 12, 2024 10:55 AM To: Christian

RE: [PATCH] drm/amdgpu: Mark amdgpu_bo as invalid after moved

2024-07-19 Thread YuanShang Mao (River)
[AMD Official Use Only - AMD Internal Distribution Only] Same issue on CPU page table update. -Original Message- From: Kuehling, Felix Sent: Thursday, July 18, 2024 12:28 AM To: Christian König ; YuanShang Mao (River) ; Huang, Trigger ; amd-gfx@lists.freedesktop.org; cao, lin Subject

RE: [PATCH v3] drm/amd/amdgpu: Update RLC_SPM_MC_CNT by ring wreg

2024-01-15 Thread YuanShang Mao (River)
[AMD Official Use Only - General] Ping... -Original Message- From: YuanShang Mao (River) Sent: Saturday, January 13, 2024 2:58 PM To: amd-gfx@lists.freedesktop.org Cc: YuanShang Mao (River) ; YuanShang Mao (River) Subject: [PATCH v3] drm/amd/amdgpu: Update RLC_SPM_MC_CNT by ring wreg

RE: [PATCH v2] drm/amdgpu: fix the memleak caused by fence not released

2025-02-14 Thread YuanShang Mao (River)
[AMD Official Use Only - AMD Internal Distribution Only] Better to put the fence outside amdgpu_gem_va_update_vm. Since it is passed to the caller, and the caller must keep one reference at least until this fence is no longer needed. Thanks River -Original Message- From: amd-gfx On Be

[PATCH] drm/amdgpu: delete function amdgpu_flush

2025-06-26 Thread YuanShang Mao (River)
[AMD Official Use Only - AMD Internal Distribution Only] Currently, amdgpu_flush is used to prevent new jobs from being submitted in the same context when a file descriptor is closed and to wait for existing jobs to complete. Additionally, if the current process is in an exit state and the late

RE: [PATCH] drm/amdgpu: delete function amdgpu_flush

2025-06-30 Thread YuanShang Mao (River)
lush, can >be removed too. Sure. If we decide to remove amdgpu_flush. @Koenig, Christian @Deucher, Alexander, do you have any concern on removal of amdgpu_flush? Thanks River -Original Message- From: Yang, Philip Sent: Friday, June 27, 2025 10:44 PM To: YuanShang Mao (River) ; amd-g

RE: [PATCH] drm/amdgpu: keep job->vm in amdgpu_job_prepare_job

2025-07-28 Thread YuanShang Mao (River)
dly. Thanks River -Original Message- From: YuanShang Mao (River) Sent: Wednesday, July 23, 2025 5:06 PM To: amd-gfx@lists.freedesktop.org Cc: Koenig, Christian ; YuanShang Mao (River) Subject: [PATCH] drm/amdgpu: keep job->vm in amdgpu_job_prepare_job job->vm is used in function amdgp

RE: [PATCH] drm/amdgpu: keep job->vm in amdgpu_job_prepare_job

2025-07-28 Thread YuanShang Mao (River)
d_entity_error(&vm->delayed)) ++result; return result; } Thanks River -Original Message- From: Alex Deucher Sent: Tuesday, July 29, 2025 2:10 AM To: YuanShang Mao (River) Cc: Deucher, Alexander ; Koenig, Christian ; amd-gfx@lists.freedesktop.org; cao, lin ; Zhang, Tian

RE: [PATCH] drm/amdgpu: keep job->vm in amdgpu_job_prepare_job

2025-07-29 Thread YuanShang Mao (River)
d be skipped. Signed-off-by: YuanShang Reviewed-by: Alex Deucher Thanks River -Original Message- From: Alex Deucher Sent: Tuesday, July 29, 2025 9:08 PM To: YuanShang Mao (River) Cc: Deucher, Alexander ; Koenig, Christian ; amd-gfx@lists.freedesktop.org; cao, lin ; Zhang, Tiantian (Celi