From: Tatsuyuki Ishi <ishitatsuy...@gmail.com>

These are considered map operations rather than unmap, and there is no
point of doing implicit synchronization here.

Signed-off-by: Tatsuyuki Ishi <ishitatsuy...@gmail.com>
---
 drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c 
b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c
index bcb729094521..a4fc24dff04d 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c
@@ -994,7 +994,7 @@ int amdgpu_vm_update_range(struct amdgpu_device *adev, 
struct amdgpu_vm *vm,
        /* Implicitly sync to command submissions in the same VM before
         * unmapping. Sync to moving fences before mapping.
         */
-       if (!(flags & AMDGPU_PTE_VALID))
+       if (!(flags & (AMDGPU_PTE_VALID | AMDGPU_PTE_PRT)))
                sync_mode = AMDGPU_SYNC_EQ_OWNER;
        else
                sync_mode = AMDGPU_SYNC_EXPLICIT;
--
2.46.0

Reply via email to