Re: [PATCH] drm/amdkfd: remove unnecessary cpu domain validation

2025-03-04 Thread Felix Kuehling
On 2025-03-04 5:38, Christian König wrote: > > Am 03.03.25 um 22:01 schrieb Felix Kuehling: >> On 2025-03-03 13:48, Christian König wrote: >>> Am 03.03.25 um 19:45 schrieb James Zhu: before move to GTT domain. >>> That might not be unnecessary. We sometimes intentionally move BOs to the >>> C

Re: [PATCH] drm/amdkfd: remove unnecessary cpu domain validation

2025-03-04 Thread Christian König
Am 03.03.25 um 22:01 schrieb Felix Kuehling: > On 2025-03-03 13:48, Christian König wrote: >> Am 03.03.25 um 19:45 schrieb James Zhu: >>> before move to GTT domain. >> That might not be unnecessary. We sometimes intentionally move BOs to the >> CPU domain to invalidate all VM mappings. > We dis

Re: [PATCH] drm/amdkfd: remove unnecessary cpu domain validation

2025-03-03 Thread Felix Kuehling
On 2025-03-03 13:48, Christian König wrote: > Am 03.03.25 um 19:45 schrieb James Zhu: >> before move to GTT domain. > That might not be unnecessary. We sometimes intentionally move BOs to the CPU > domain to invalidate all VM mappings. We discussed this in our VM sync meeting this morning, and I

Re: [PATCH] drm/amdkfd: remove unnecessary cpu domain validation

2025-03-03 Thread Christian König
Am 03.03.25 um 19:45 schrieb James Zhu: > before move to GTT domain. That might not be unnecessary. We sometimes intentionally move BOs to the CPU domain to invalidate all VM mappings. Christian. > > Signed-off-by: James Zhu > --- > drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gpuvm.c | 6 --

[PATCH] drm/amdkfd: remove unnecessary cpu domain validation

2025-03-03 Thread James Zhu
before move to GTT domain. Signed-off-by: James Zhu --- drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gpuvm.c | 6 -- 1 file changed, 6 deletions(-) diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gpuvm.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gpuvm.c index 62ca12e94581..2ac6d4fa0601