The root PD can also contain huge PDEs.

Signed-off-by: Christian König <christian.koe...@amd.com>
---
 drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c | 25 +++++--------------------
 1 file changed, 5 insertions(+), 20 deletions(-)

diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c 
b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c
index 6844ba7467a6..23729fdd34fb 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c
@@ -94,23 +94,17 @@ struct amdgpu_prt_cb {
 static unsigned amdgpu_vm_level_shift(struct amdgpu_device *adev,
                                      unsigned level)
 {
-       unsigned shift = 0xff;
-
        switch (level) {
        case AMDGPU_VM_PDB2:
        case AMDGPU_VM_PDB1:
        case AMDGPU_VM_PDB0:
-               shift = 9 * (AMDGPU_VM_PDB0 - level) +
+               return 9 * (AMDGPU_VM_PDB0 - level) +
                        adev->vm_manager.block_size;
-               break;
        case AMDGPU_VM_PTB:
-               shift = 0;
-               break;
+               return 0;
        default:
-               dev_err(adev->dev, "the level%d isn't supported.\n", level);
+               return ~0;
        }
-
-       return shift;
 }
 
 /**
@@ -1432,13 +1426,6 @@ static int amdgpu_vm_update_ptes(struct 
amdgpu_vm_update_params *params,
 
                pt = cursor.entry->base.bo;
 
-               /* The root level can't be a huge page */
-               if (cursor.level == adev->vm_manager.root_level) {
-                       if (!amdgpu_vm_pt_descendant(adev, &cursor))
-                               return -ENOENT;
-                       continue;
-               }
-
                shift = amdgpu_vm_level_shift(adev, cursor.level);
                parent_shift = amdgpu_vm_level_shift(adev, cursor.level - 1);
                if (adev->asic_type < CHIP_VEGA10 &&
@@ -1457,11 +1444,9 @@ static int amdgpu_vm_update_ptes(struct 
amdgpu_vm_update_params *params,
                        if (!amdgpu_vm_pt_descendant(adev, &cursor))
                                return -ENOENT;
                        continue;
-               } else if (frag >= parent_shift &&
-                          cursor.level - 1 != adev->vm_manager.root_level) {
+               } else if (frag >= parent_shift) {
                        /* If the fragment size is even larger than the parent
-                        * shift we should go up one level and check it again
-                        * unless one level up is the root level.
+                        * shift we should go up one level and check it again.
                         */
                        if (!amdgpu_vm_pt_ancestor(&cursor))
                                return -ENOENT;
-- 
2.17.1

_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

Reply via email to