[PATCH 2/8] drm/amdgpu: always set and check dma addresses in the VM code

[Christian König]

Clean that up a bit and allow to always have the DMA addresses around.

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

diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c 
b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c
index c8f0e4ca05fb..2d61d74d32a8 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c
@@ -84,6 +84,13 @@ struct amdgpu_pte_update_params {
         */
        struct amdgpu_vm *vm;
 
+       /**
+        * @pages_addr:
+        *
+        * DMA addresses to use for mapping
+        */
+       dma_addr_t *pages_addr;
+
        /**
         * @src: address where to copy page table entries from
         */
@@ -101,12 +108,6 @@ struct amdgpu_pte_update_params {
                     struct amdgpu_bo *bo, uint64_t pe,
                     uint64_t addr, unsigned count, uint32_t incr,
                     uint64_t flags);
-       /**
-        * @pages_addr:
-        *
-        * DMA addresses to use for mapping, used during VM update by CPU
-        */
-       dma_addr_t *pages_addr;
 };
 
 /**
@@ -1573,7 +1574,7 @@ static void amdgpu_vm_fragment(struct 
amdgpu_pte_update_params *params,
                max_frag = 31;
 
        /* system pages are non continuously */
-       if (params->src) {
+       if (params->pages_addr) {
                *frag = 0;
                *frag_end = end;
                return;
@@ -1753,16 +1754,13 @@ static int amdgpu_vm_bo_update_mapping(struct 
amdgpu_device *adev,
        memset(&params, 0, sizeof(params));
        params.adev = adev;
        params.vm = vm;
+       params.pages_addr = pages_addr;
 
        /* sync to everything except eviction fences on unmapping */
        if (!(flags & AMDGPU_PTE_VALID))
                owner = AMDGPU_FENCE_OWNER_KFD;
 
        if (vm->use_cpu_for_update) {
-               /* params.src is used as flag to indicate system Memory */
-               if (pages_addr)
-                       params.src = ~0;
-
                /* Wait for PT BOs to be idle. PTs share the same resv. object
                 * as the root PD BO
                 */
@@ -1778,7 +1776,6 @@ static int amdgpu_vm_bo_update_mapping(struct 
amdgpu_device *adev,
                }
 
                params.func = amdgpu_vm_cpu_set_ptes;
-               params.pages_addr = pages_addr;
                return amdgpu_vm_update_ptes(&params, start, last + 1,
                                             addr, flags);
        }
-- 
2.17.1

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