[PATCH] drm/amdgpu: Revert "Fix IB allocation sizes for page table updates"

Kuehling, Felix Felix.Kuehling at amd.com
Thu Aug 31 13:42:48 UTC 2017


Reviewed-by: Felix Kuehling <Felix.Kuehling at amd.com>

Sorry, I didn't think this one through. I'll keep looking for the real problem.
________________________________________
From: amd-gfx <amd-gfx-bounces at lists.freedesktop.org> on behalf of Christian König <deathsimple at vodafone.de>
Sent: Thursday, August 31, 2017 3:45:48 AM
To: amd-gfx at lists.freedesktop.org
Subject: [PATCH] drm/amdgpu: Revert "Fix IB allocation sizes for page table updates"

From: Christian König <christian.koenig at amd.com>

The IB allocation sizes where correct.

This reverts commit d1551e7c7211fa135c4d79f09e237f5f9779e421.

Signed-off-by: Christian König <christian.koenig at amd.com>
---
 drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c
index 473273d..b08f031 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c
@@ -1037,7 +1037,7 @@ static int amdgpu_vm_update_level(struct amdgpu_device *adev,
                ndw = 64;

                /* assume the worst case */
-               ndw += parent->last_entry_used * 10;
+               ndw += parent->last_entry_used * 6;

                pd_addr = amdgpu_bo_gpu_offset(parent->base.bo);

@@ -1542,7 +1542,7 @@ static int amdgpu_vm_bo_update_mapping(struct amdgpu_device *adev,
        ndw = 64;

        /* one PDE write for each huge page */
-       ndw += ((nptes >> adev->vm_manager.block_size) + 1) * 10;
+       ndw += ((nptes >> adev->vm_manager.block_size) + 1) * 6;

        if (pages_addr) {
                /* copy commands needed */
--
2.7.4

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


More information about the amd-gfx mailing list