This patch allows specifying the vm_block_size even when multi level
page directories are active.

Signed-off-by: Christian König <[email protected]>
---
 drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 14 +-------------
 drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c     | 15 +++++++++------
 2 files changed, 10 insertions(+), 19 deletions(-)

diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c 
b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c
index 0212919a6dfa..abd7fe077bdc 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c
@@ -1162,20 +1162,8 @@ static void amdgpu_check_block_size(struct amdgpu_device 
*adev)
        if (amdgpu_vm_block_size < 9) {
                dev_warn(adev->dev, "VM page table size (%d) too small\n",
                         amdgpu_vm_block_size);
-               goto def_value;
+               amdgpu_vm_block_size = -1;
        }
-
-       if (amdgpu_vm_block_size > 24 ||
-           (amdgpu_vm_size * 1024) < (1ull << amdgpu_vm_block_size)) {
-               dev_warn(adev->dev, "VM page table size (%d) too large\n",
-                        amdgpu_vm_block_size);
-               goto def_value;
-       }
-
-       return;
-
-def_value:
-       amdgpu_vm_block_size = -1;
 }
 
 static void amdgpu_check_vm_size(struct amdgpu_device *adev)
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c 
b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c
index 44430c4820cc..24e93370115a 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c
@@ -2600,18 +2600,21 @@ void amdgpu_vm_adjust_size(struct amdgpu_device *adev, 
uint32_t vm_size,
        adev->vm_manager.max_pfn = (uint64_t)vm_size << 18;
 
        tmp = roundup_pow_of_two(adev->vm_manager.max_pfn);
+       if (amdgpu_vm_block_size != -1)
+               tmp >>= amdgpu_vm_block_size - 9;
        tmp = DIV_ROUND_UP(fls64(tmp) - 1, 9) - 1;
        adev->vm_manager.num_level = min(max_level, (unsigned)tmp);
 
        /* block size depends on vm size and hw setup*/
-       if (adev->vm_manager.num_level > 1)
-               /* Use fixed block_size for multi level page tables */
-               adev->vm_manager.block_size = 9;
-       else if (amdgpu_vm_block_size == -1)
+       if (amdgpu_vm_block_size != -1)
                adev->vm_manager.block_size =
-                       amdgpu_vm_get_block_size(vm_size);
+                       min(amdgpu_vm_block_size, max_bits
+                           - AMDGPU_GPU_PAGE_SHIFT
+                           - 9 * adev->vm_manager.num_level);
+       else if (adev->vm_manager.num_level > 1)
+               adev->vm_manager.block_size = 9;
        else
-               adev->vm_manager.block_size = amdgpu_vm_block_size;
+               adev->vm_manager.block_size = amdgpu_vm_get_block_size(tmp);
 
        if (amdgpu_vm_fragment_size == -1)
                adev->vm_manager.fragment_size = fragment_size_default;
-- 
2.11.0

_______________________________________________
amd-gfx mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

Reply via email to