drm/amdgpu: use a single linked list for amdgpu_vm_bo_base
authorChristian König <christian.koenig@amd.com>
Mon, 10 Sep 2018 18:02:46 +0000 (20:02 +0200)
committerAlex Deucher <alexander.deucher@amd.com>
Thu, 13 Sep 2018 20:14:12 +0000 (15:14 -0500)
Instead of the double linked list. Gets the size of amdgpu_vm_pt down to
64 bytes again.

We could even reduce it down to 32 bytes, but that would require some
rather extreme hacks.

Signed-off-by: Christian König <christian.koenig@amd.com>
Reviewed-by: Chunming Zhou <david1.zhou@amd.com>
Acked-by: Felix Kuehling <Felix.Kuehling@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/amdgpu/amdgpu_object.c
drivers/gpu/drm/amd/amdgpu/amdgpu_object.h
drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c
drivers/gpu/drm/amd/amdgpu/amdgpu_vm.h

index de990bdcdd6cb1a865374725d3041217e5033c3a..e6909252aefabec5594cc92eaa2cc7f5acade34e 100644 (file)
@@ -448,7 +448,7 @@ static int amdgpu_bo_do_create(struct amdgpu_device *adev,
                return -ENOMEM;
        drm_gem_private_object_init(adev->ddev, &bo->gem_base, size);
        INIT_LIST_HEAD(&bo->shadow_list);
-       INIT_LIST_HEAD(&bo->va);
+       bo->vm_bo = NULL;
        bo->preferred_domains = bp->preferred_domain ? bp->preferred_domain :
                bp->domain;
        bo->allowed_domains = bo->preferred_domains;
index 907fdf46d895387fc8effd42613b61f01602ee38..64337ff2ad63d06ae576ff5ca7d51e746d0f0c06 100644 (file)
@@ -89,8 +89,8 @@ struct amdgpu_bo {
        void                            *metadata;
        u32                             metadata_size;
        unsigned                        prime_shared_count;
-       /* list of all virtual address to which this bo is associated to */
-       struct list_head                va;
+       /* per VM structure for page tables and with virtual addresses */
+       struct amdgpu_vm_bo_base        *vm_bo;
        /* Constant after initialization */
        struct drm_gem_object           gem_base;
        struct amdgpu_bo                *parent;
index 234764ac58cfa74eeb658c3b883c09144d017034..a7f9aaa47c492748b4bd19eb47d522d8615f5228 100644 (file)
@@ -309,12 +309,13 @@ static void amdgpu_vm_bo_base_init(struct amdgpu_vm_bo_base *base,
 {
        base->vm = vm;
        base->bo = bo;
-       INIT_LIST_HEAD(&base->bo_list);
+       base->next = NULL;
        INIT_LIST_HEAD(&base->vm_status);
 
        if (!bo)
                return;
-       list_add_tail(&base->bo_list, &bo->va);
+       base->next = bo->vm_bo;
+       bo->vm_bo = base;
 
        if (bo->tbo.resv != vm->root.base.bo->tbo.resv)
                return;
@@ -352,7 +353,7 @@ static struct amdgpu_vm_pt *amdgpu_vm_pt_parent(struct amdgpu_vm_pt *pt)
        if (!parent)
                return NULL;
 
-       return list_first_entry(&parent->va, struct amdgpu_vm_pt, base.bo_list);
+       return container_of(parent->vm_bo, struct amdgpu_vm_pt, base);
 }
 
 /**
@@ -954,7 +955,7 @@ static void amdgpu_vm_free_pts(struct amdgpu_device *adev,
        for_each_amdgpu_vm_pt_dfs_safe(adev, vm, cursor, entry) {
 
                if (entry->base.bo) {
-                       list_del(&entry->base.bo_list);
+                       entry->base.bo->vm_bo = NULL;
                        list_del(&entry->base.vm_status);
                        amdgpu_bo_unref(&entry->base.bo->shadow);
                        amdgpu_bo_unref(&entry->base.bo);
@@ -1162,12 +1163,13 @@ int amdgpu_vm_flush(struct amdgpu_ring *ring, struct amdgpu_job *job, bool need_
 struct amdgpu_bo_va *amdgpu_vm_bo_find(struct amdgpu_vm *vm,
                                       struct amdgpu_bo *bo)
 {
-       struct amdgpu_bo_va *bo_va;
+       struct amdgpu_vm_bo_base *base;
 
-       list_for_each_entry(bo_va, &bo->va, base.bo_list) {
-               if (bo_va->base.vm == vm) {
-                       return bo_va;
-               }
+       for (base = bo->vm_bo; base; base = base->next) {
+               if (base->vm != vm)
+                       continue;
+
+               return container_of(base, struct amdgpu_bo_va, base);
        }
        return NULL;
 }
@@ -2728,11 +2730,21 @@ void amdgpu_vm_bo_rmv(struct amdgpu_device *adev,
        struct amdgpu_bo_va_mapping *mapping, *next;
        struct amdgpu_bo *bo = bo_va->base.bo;
        struct amdgpu_vm *vm = bo_va->base.vm;
+       struct amdgpu_vm_bo_base **base;
 
-       if (bo && bo->tbo.resv == vm->root.base.bo->tbo.resv)
-               vm->bulk_moveable = false;
+       if (bo) {
+               if (bo->tbo.resv == vm->root.base.bo->tbo.resv)
+                       vm->bulk_moveable = false;
 
-       list_del(&bo_va->base.bo_list);
+               for (base = &bo_va->base.bo->vm_bo; *base;
+                    base = &(*base)->next) {
+                       if (*base != &bo_va->base)
+                               continue;
+
+                       *base = bo_va->base.next;
+                       break;
+               }
+       }
 
        spin_lock(&vm->invalidated_lock);
        list_del(&bo_va->base.vm_status);
@@ -2774,7 +2786,7 @@ void amdgpu_vm_bo_invalidate(struct amdgpu_device *adev,
        if (bo->parent && bo->parent->shadow == bo)
                bo = bo->parent;
 
-       list_for_each_entry(bo_base, &bo->va, bo_list) {
+       for (bo_base = bo->vm_bo; bo_base; bo_base = bo_base->next) {
                struct amdgpu_vm *vm = bo_base->vm;
 
                if (evicted && bo->tbo.resv == vm->root.base.bo->tbo.resv) {
index 12d21eec4568d3a43d759f863876632a5a33e11b..2a8898d19c8b58076dd88526a4304e3104a67866 100644 (file)
@@ -129,7 +129,7 @@ struct amdgpu_vm_bo_base {
        struct amdgpu_bo                *bo;
 
        /* protected by bo being reserved */
-       struct list_head                bo_list;
+       struct amdgpu_vm_bo_base        *next;
 
        /* protected by spinlock */
        struct list_head                vm_status;