diff options
author | Michel Dänzer <michel.daenzer@amd.com> | 2017-04-28 04:28:14 -0400 |
---|---|---|
committer | Alex Deucher <alexander.deucher@amd.com> | 2017-04-28 17:33:16 -0400 |
commit | c81a1a74037f61c578f6b8218b079aa434e300b4 (patch) | |
tree | 8176c83897e467d913a7059e1a652a9adb7598a7 /drivers/gpu/drm/amd/amdgpu/amdgpu_kms.c | |
parent | 8b9242eddd51f17b8306d6c96172fd68ef1106c6 (diff) |
drm/amdgpu: Make amdgpu_bo_reserve use uninterruptible waits for cleanup
Some of these paths probably cannot be interrupted by a signal anyway.
Those that can would fail to clean up things if they actually got
interrupted.
Reviewed-by: Christian König <christian.koenig@amd.com>
Signed-off-by: Michel Dänzer <michel.daenzer@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Diffstat (limited to 'drivers/gpu/drm/amd/amdgpu/amdgpu_kms.c')
-rw-r--r-- | drivers/gpu/drm/amd/amdgpu/amdgpu_kms.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_kms.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_kms.c index 21f616cdb279..4f2b6679f72f 100644 --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_kms.c +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_kms.c | |||
@@ -821,7 +821,7 @@ void amdgpu_driver_postclose_kms(struct drm_device *dev, | |||
821 | 821 | ||
822 | if (amdgpu_sriov_vf(adev)) { | 822 | if (amdgpu_sriov_vf(adev)) { |
823 | /* TODO: how to handle reserve failure */ | 823 | /* TODO: how to handle reserve failure */ |
824 | BUG_ON(amdgpu_bo_reserve(adev->virt.csa_obj, false)); | 824 | BUG_ON(amdgpu_bo_reserve(adev->virt.csa_obj, true)); |
825 | amdgpu_vm_bo_rmv(adev, fpriv->vm.csa_bo_va); | 825 | amdgpu_vm_bo_rmv(adev, fpriv->vm.csa_bo_va); |
826 | fpriv->vm.csa_bo_va = NULL; | 826 | fpriv->vm.csa_bo_va = NULL; |
827 | amdgpu_bo_unreserve(adev->virt.csa_obj); | 827 | amdgpu_bo_unreserve(adev->virt.csa_obj); |