Mesa (main): radv/winsys: remove useless has_sparse_vm_mappings checks

GitLab Mirror gitlab-mirror at kemper.freedesktop.org
Wed Dec 15 11:05:56 UTC 2021


Module: Mesa
Branch: main
Commit: 782474782be87a4e1fba47b7143ec6081a7a806e
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=782474782be87a4e1fba47b7143ec6081a7a806e

Author: Samuel Pitoiset <samuel.pitoiset at gmail.com>
Date:   Mon Dec  6 18:32:46 2021 +0100

radv/winsys: remove useless has_sparse_vm_mappings checks

Sparse is only exposed on GFX8+, so this is always TRUE.

Signed-off-by: Samuel Pitoiset <samuel.pitoiset at gmail.com>
Reviewed-by: Bas Nieuwenhuizen <bas at basnieuwenhuizen.nl>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/14116>

---

 src/amd/vulkan/winsys/amdgpu/radv_amdgpu_bo.c | 6 ------
 1 file changed, 6 deletions(-)

diff --git a/src/amd/vulkan/winsys/amdgpu/radv_amdgpu_bo.c b/src/amd/vulkan/winsys/amdgpu/radv_amdgpu_bo.c
index 3bea2f38028..63002e27c9d 100644
--- a/src/amd/vulkan/winsys/amdgpu/radv_amdgpu_bo.c
+++ b/src/amd/vulkan/winsys/amdgpu/radv_amdgpu_bo.c
@@ -73,9 +73,6 @@ radv_amdgpu_winsys_virtual_map(struct radv_amdgpu_winsys *ws, struct radv_amdgpu
    assert(range->size);
 
    if (!range->bo) {
-      if (!ws->info.has_sparse_vm_mappings)
-         return;
-
       internal_flags |= AMDGPU_VM_PAGE_PRT;
    } else
       p_atomic_inc(&range->bo->ref_count);
@@ -94,9 +91,6 @@ radv_amdgpu_winsys_virtual_unmap(struct radv_amdgpu_winsys *ws, struct radv_amdg
    assert(range->size);
 
    if (!range->bo) {
-      if (!ws->info.has_sparse_vm_mappings)
-         return;
-
       /* Even though this is an unmap, if we don't set this flag,
          AMDGPU is going to complain about the missing buffer. */
       internal_flags |= AMDGPU_VM_PAGE_PRT;



More information about the mesa-commit mailing list