diff options
author | Christian König <christian.koenig@amd.com> | 2017-09-11 10:54:59 -0400 |
---|---|---|
committer | Alex Deucher <alexander.deucher@amd.com> | 2017-09-13 12:10:06 -0400 |
commit | 4e55eb3879fea6d8c7d414cebaa5bff1da58b4a1 (patch) | |
tree | 40faee87cf8636d6162e89f6ecd1db8839cdb46c /drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c | |
parent | 4a00f21db800bc64264bb6764c3d0d0878e9f4c4 (diff) |
drm/amdgpu: fix amdgpu_vm_handle_moved as well v2
There is no guarantee that the last BO_VA actually needed an update.
Additional to that all command submissions must wait for moved BOs to
be cleared, not just the first one.
v2: Don't overwrite any newer fence.
Signed-off-by: Christian König <christian.koenig@amd.com>
Reviewed-by: Chunming Zhou <david1.zhou@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Diffstat (limited to 'drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c')
-rw-r--r-- | drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c index ff61073b7181..9f1202a4182f 100644 --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c | |||
@@ -814,7 +814,7 @@ static int amdgpu_bo_vm_update_pte(struct amdgpu_cs_parser *p) | |||
814 | 814 | ||
815 | } | 815 | } |
816 | 816 | ||
817 | r = amdgpu_vm_handle_moved(adev, vm, &p->job->sync); | 817 | r = amdgpu_vm_handle_moved(adev, vm); |
818 | if (r) | 818 | if (r) |
819 | return r; | 819 | return r; |
820 | 820 | ||