Commit 69277985 authored by Alex Deucher's avatar Alex Deucher

drm/amdgpu: remove VM shadow WARN_ONs

Printing a warning into the logs that we will certainly run into a BUG() is
completely nonsense, the BUG() is more than noisy enough.
Signed-off-by: default avatarChristian König <christian.koenig@amd.com>
Reviewed-by: default avatarFelix Kuehling <Felix.Kuehling@amd.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent a33cab7a
...@@ -1044,8 +1044,7 @@ static int amdgpu_vm_update_level(struct amdgpu_device *adev, ...@@ -1044,8 +1044,7 @@ static int amdgpu_vm_update_level(struct amdgpu_device *adev,
params.adev = adev; params.adev = adev;
shadow = parent->bo->shadow; shadow = parent->bo->shadow;
WARN_ON(vm->use_cpu_for_update && shadow); if (vm->use_cpu_for_update) {
if (vm->use_cpu_for_update && !shadow) {
r = amdgpu_bo_kmap(parent->bo, (void **)&pd_addr); r = amdgpu_bo_kmap(parent->bo, (void **)&pd_addr);
if (r) if (r)
return r; return r;
...@@ -1310,9 +1309,6 @@ static int amdgpu_vm_update_ptes(struct amdgpu_pte_update_params *params, ...@@ -1310,9 +1309,6 @@ static int amdgpu_vm_update_ptes(struct amdgpu_pte_update_params *params,
r = amdgpu_bo_kmap(pt, (void *)&pe_start); r = amdgpu_bo_kmap(pt, (void *)&pe_start);
if (r) if (r)
return r; return r;
WARN_ONCE(pt->shadow,
"CPU VM update doesn't support shadow pages");
} else { } else {
if (pt->shadow) { if (pt->shadow) {
pe_start = amdgpu_bo_gpu_offset(pt->shadow); pe_start = amdgpu_bo_gpu_offset(pt->shadow);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment