drm/amdgpu: explicitly sync VM update to PDs/PTs
authorChristian König <christian.koenig@amd.com>
Mon, 16 Dec 2019 13:22:35 +0000 (14:22 +0100)
committerAlex Deucher <alexander.deucher@amd.com>
Wed, 5 Feb 2020 04:30:39 +0000 (23:30 -0500)
Explicitly sync VM updates to the moving fence in PDs and PTs.

Signed-off-by: Christian König <christian.koenig@amd.com>
Reviewed-by: Felix Kuehling <Felix.Kuehling@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/amdgpu/amdgpu_vm_cpu.c
drivers/gpu/drm/amd/amdgpu/amdgpu_vm_sdma.c

index 73fec7a..68b013b 100644 (file)
@@ -86,6 +86,13 @@ static int amdgpu_vm_cpu_update(struct amdgpu_vm_update_params *p,
 {
        unsigned int i;
        uint64_t value;
+       int r;
+
+       if (bo->tbo.moving) {
+               r = dma_fence_wait(bo->tbo.moving, true);
+               if (r)
+                       return r;
+       }
 
        pe += (unsigned long)amdgpu_bo_kptr(bo);
 
index 19b7f80..4bbd8ff 100644 (file)
@@ -208,6 +208,11 @@ static int amdgpu_vm_sdma_update(struct amdgpu_vm_update_params *p,
        uint64_t *pte;
        int r;
 
+       /* Wait for PD/PT moves to be completed */
+       r = amdgpu_sync_fence(&p->job->sync, bo->tbo.moving, false);
+       if (r)
+               return r;
+
        do {
                ndw = p->num_dw_left;
                ndw -= p->job->ibs->length_dw;