drm/nouveau/mmu/nv44: implement vmm on top of new base
authorBen Skeggs <bskeggs@redhat.com>
Tue, 31 Oct 2017 17:56:19 +0000 (03:56 +1000)
committerBen Skeggs <bskeggs@redhat.com>
Thu, 2 Nov 2017 03:32:25 +0000 (13:32 +1000)
Signed-off-by: Ben Skeggs <bskeggs@redhat.com>
drivers/gpu/drm/nouveau/include/nvkm/subdev/mmu.h
drivers/gpu/drm/nouveau/nvkm/subdev/mmu/Kbuild
drivers/gpu/drm/nouveau/nvkm/subdev/mmu/base.c
drivers/gpu/drm/nouveau/nvkm/subdev/mmu/nv04.c
drivers/gpu/drm/nouveau/nvkm/subdev/mmu/nv04.h [deleted file]
drivers/gpu/drm/nouveau/nvkm/subdev/mmu/nv44.c
drivers/gpu/drm/nouveau/nvkm/subdev/mmu/priv.h
drivers/gpu/drm/nouveau/nvkm/subdev/mmu/vmm.c
drivers/gpu/drm/nouveau/nvkm/subdev/mmu/vmm.h
drivers/gpu/drm/nouveau/nvkm/subdev/mmu/vmmnv44.c [new file with mode: 0644]

index 8ff71e7f2f5b3aa3fe1006f127365de6d5c8ba6f..f171e7a82f57241e7aabfbadd904f550d2a76e59 100644 (file)
@@ -48,6 +48,9 @@ struct nvkm_vm {
 
        bool bootstrapped;
        atomic_t engref[NVKM_SUBDEV_NR];
+
+       dma_addr_t null;
+       void *nullp;
 };
 
 int  nvkm_vm_new(struct nvkm_device *, u64 offset, u64 length, u64 mm_offset,
index 4b3e403a76f34e6095494e21b371572e3b1387c4..f8a46f9a37a08d2d1e3ab71713533ca93d697e79 100644 (file)
@@ -15,3 +15,4 @@ nvkm-y += nvkm/subdev/mmu/gp10b.o
 nvkm-y += nvkm/subdev/mmu/vmm.o
 nvkm-y += nvkm/subdev/mmu/vmmnv04.o
 nvkm-y += nvkm/subdev/mmu/vmmnv41.o
+nvkm-y += nvkm/subdev/mmu/vmmnv44.o
index d55ec0e85dca76de2c77c4f227280fa6d879f8b3..0b4cb7b6a81fafb8b257dc0f84d3b2b45167e6bb 100644 (file)
@@ -786,14 +786,11 @@ static void *
 nvkm_mmu_dtor(struct nvkm_subdev *subdev)
 {
        struct nvkm_mmu *mmu = nvkm_mmu(subdev);
-       void *data = mmu;
 
-       if (mmu->func->dtor)
-               data = mmu->func->dtor(mmu);
        nvkm_vm_ref(NULL, &mmu->vmm, NULL);
 
        nvkm_mmu_ptc_fini(mmu);
-       return data;
+       return mmu;
 }
 
 static const struct nvkm_subdev_func
index fc4390de345e2ed65a94386406ba2fed51ec2f0b..3e603643175408cd1eaedf679cfbccd3ef99ab5c 100644 (file)
@@ -21,7 +21,6 @@
  *
  * Authors: Ben Skeggs
  */
-#include "nv04.h"
 #include "vmm.h"
 
 #include <nvif/class.h>
@@ -81,32 +80,6 @@ nv04_mmu_oneinit(struct nvkm_mmu *mmu)
        return 0;
 }
 
-void *
-nv04_mmu_dtor(struct nvkm_mmu *base)
-{
-       struct nv04_mmu *mmu = nv04_mmu(base);
-       struct nvkm_device *device = mmu->base.subdev.device;
-       if (mmu->base.vmm)
-               nvkm_memory_unref(&mmu->base.vmm->pgt[0].mem[0]);
-       if (mmu->nullp) {
-               dma_free_coherent(device->dev, 16 * 1024,
-                                 mmu->nullp, mmu->null);
-       }
-       return mmu;
-}
-
-int
-nv04_mmu_new_(const struct nvkm_mmu_func *func, struct nvkm_device *device,
-             int index, struct nvkm_mmu **pmmu)
-{
-       struct nv04_mmu *mmu;
-       if (!(mmu = kzalloc(sizeof(*mmu), GFP_KERNEL)))
-               return -ENOMEM;
-       *pmmu = &mmu->base;
-       nvkm_mmu_ctor(func, device, index, &mmu->base);
-       return 0;
-}
-
 const struct nvkm_mmu_func
 nv04_mmu = {
        .oneinit = nv04_mmu_oneinit,
diff --git a/drivers/gpu/drm/nouveau/nvkm/subdev/mmu/nv04.h b/drivers/gpu/drm/nouveau/nvkm/subdev/mmu/nv04.h
deleted file mode 100644 (file)
index 6b8f14c..0000000
+++ /dev/null
@@ -1,17 +0,0 @@
-#ifndef __NV04_MMU_PRIV__
-#define __NV04_MMU_PRIV__
-#define nv04_mmu(p) container_of((p), struct nv04_mmu, base)
-#include "priv.h"
-
-struct nv04_mmu {
-       struct nvkm_mmu base;
-       dma_addr_t null;
-       void *nullp;
-};
-
-int nv04_mmu_new_(const struct nvkm_mmu_func *, struct nvkm_device *,
-                 int index, struct nvkm_mmu **);
-void *nv04_mmu_dtor(struct nvkm_mmu *);
-
-extern const struct nvkm_mmu_func nv04_mmu;
-#endif
index cc97500a09010779669997b6003ff3435fb474cc..48ca0cdf2acfa55e3acfdb75e37acc669849d5c0 100644 (file)
  *
  * Authors: Ben Skeggs
  */
-#include "nv04.h"
+#include "vmm.h"
 
-#include <core/gpuobj.h>
 #include <core/option.h>
 #include <subdev/timer.h>
 
+#include <nvif/class.h>
+
 #define NV44_GART_SIZE (512 * 1024 * 1024)
 #define NV44_GART_PAGE (  4 * 1024)
 
@@ -84,7 +85,6 @@ static void
 nv44_vm_map_sg(struct nvkm_vma *vma, struct nvkm_memory *pgt,
               struct nvkm_mem *mem, u32 pte, u32 cnt, dma_addr_t *list)
 {
-       struct nv04_mmu *mmu = nv04_mmu(vma->vm->mmu);
        u32 tmp[4];
        int i;
 
@@ -92,7 +92,7 @@ nv44_vm_map_sg(struct nvkm_vma *vma, struct nvkm_memory *pgt,
        if (pte & 3) {
                u32  max = 4 - (pte & 3);
                u32 part = (cnt > max) ? max : cnt;
-               nv44_vm_fill(pgt, mmu->null, list, pte, part);
+               nv44_vm_fill(pgt, vma->vm->null, list, pte, part);
                pte  += part;
                list += part;
                cnt  -= part;
@@ -109,20 +109,18 @@ nv44_vm_map_sg(struct nvkm_vma *vma, struct nvkm_memory *pgt,
        }
 
        if (cnt)
-               nv44_vm_fill(pgt, mmu->null, list, pte, cnt);
+               nv44_vm_fill(pgt, vma->vm->null, list, pte, cnt);
        nvkm_done(pgt);
 }
 
 static void
 nv44_vm_unmap(struct nvkm_vma *vma, struct nvkm_memory *pgt, u32 pte, u32 cnt)
 {
-       struct nv04_mmu *mmu = nv04_mmu(vma->vm->mmu);
-
        nvkm_kmap(pgt);
        if (pte & 3) {
                u32  max = 4 - (pte & 3);
                u32 part = (cnt > max) ? max : cnt;
-               nv44_vm_fill(pgt, mmu->null, NULL, pte, part);
+               nv44_vm_fill(pgt, vma->vm->null, NULL, pte, part);
                pte  += part;
                cnt  -= part;
        }
@@ -136,16 +134,15 @@ nv44_vm_unmap(struct nvkm_vma *vma, struct nvkm_memory *pgt, u32 pte, u32 cnt)
        }
 
        if (cnt)
-               nv44_vm_fill(pgt, mmu->null, NULL, pte, cnt);
+               nv44_vm_fill(pgt, vma->vm->null, NULL, pte, cnt);
        nvkm_done(pgt);
 }
 
 static void
 nv44_vm_flush(struct nvkm_vm *vm)
 {
-       struct nv04_mmu *mmu = nv04_mmu(vm->mmu);
-       struct nvkm_device *device = mmu->base.subdev.device;
-       nvkm_wr32(device, 0x100814, mmu->base.limit - NV44_GART_PAGE);
+       struct nvkm_device *device = vm->mmu->subdev.device;
+       nvkm_wr32(device, 0x100814, vm->mmu->limit - NV44_GART_PAGE);
        nvkm_wr32(device, 0x100808, 0x00000020);
        nvkm_msec(device, 2000,
                if (nvkm_rd32(device, 0x100808) & 0x00000001)
@@ -159,38 +156,18 @@ nv44_vm_flush(struct nvkm_vm *vm)
  ******************************************************************************/
 
 static int
-nv44_mmu_oneinit(struct nvkm_mmu *base)
+nv44_mmu_oneinit(struct nvkm_mmu *mmu)
 {
-       struct nv04_mmu *mmu = nv04_mmu(base);
-       struct nvkm_device *device = mmu->base.subdev.device;
-       int ret;
-
-       mmu->nullp = dma_alloc_coherent(device->dev, 16 * 1024,
-                                       &mmu->null, GFP_KERNEL);
-       if (!mmu->nullp) {
-               nvkm_warn(&mmu->base.subdev, "unable to allocate dummy pages\n");
-               mmu->null = 0;
-       }
-
-       ret = nvkm_vm_create(&mmu->base, 0, NV44_GART_SIZE, 0, 4096, NULL,
-                            &mmu->base.vmm);
-       if (ret)
-               return ret;
-
-       ret = nvkm_memory_new(device, NVKM_MEM_TARGET_INST,
-                             (NV44_GART_SIZE / NV44_GART_PAGE) * 4,
-                             512 * 1024, true,
-                             &mmu->base.vmm->pgt[0].mem[0]);
-       mmu->base.vmm->pgt[0].refcount[0] = 1;
-       return ret;
+       mmu->vmm->pgt[0].mem[0] = mmu->vmm->pd->pt[0]->memory;
+       mmu->vmm->pgt[0].refcount[0] = 1;
+       return 0;
 }
 
 static void
-nv44_mmu_init(struct nvkm_mmu *base)
+nv44_mmu_init(struct nvkm_mmu *mmu)
 {
-       struct nv04_mmu *mmu = nv04_mmu(base);
-       struct nvkm_device *device = mmu->base.subdev.device;
-       struct nvkm_memory *gart = mmu->base.vmm->pgt[0].mem[0];
+       struct nvkm_device *device = mmu->subdev.device;
+       struct nvkm_memory *gart = mmu->vmm->pgt[0].mem[0];
        u32 addr;
 
        /* calculate vram address of this PRAMIN block, object must be
@@ -201,7 +178,7 @@ nv44_mmu_init(struct nvkm_mmu *base)
        addr -= ((nvkm_memory_addr(gart) >> 19) + 1) << 19;
 
        nvkm_wr32(device, 0x100850, 0x80000000);
-       nvkm_wr32(device, 0x100818, mmu->null);
+       nvkm_wr32(device, 0x100818, mmu->vmm->null);
        nvkm_wr32(device, 0x100804, NV44_GART_SIZE);
        nvkm_wr32(device, 0x100850, 0x00008000);
        nvkm_mask(device, 0x10008c, 0x00000200, 0x00000200);
@@ -212,7 +189,6 @@ nv44_mmu_init(struct nvkm_mmu *base)
 
 static const struct nvkm_mmu_func
 nv44_mmu = {
-       .dtor = nv04_mmu_dtor,
        .oneinit = nv44_mmu_oneinit,
        .init = nv44_mmu_init,
        .limit = NV44_GART_SIZE,
@@ -223,6 +199,7 @@ nv44_mmu = {
        .map_sg = nv44_vm_map_sg,
        .unmap = nv44_vm_unmap,
        .flush = nv44_vm_flush,
+       .vmm = {{ -1, -1, NVIF_CLASS_VMM_NV04}, nv44_vmm_new, true },
 };
 
 int
@@ -232,5 +209,5 @@ nv44_mmu_new(struct nvkm_device *device, int index, struct nvkm_mmu **pmmu)
            !nvkm_boolopt(device->cfgopt, "NvPCIE", true))
                return nv04_mmu_new(device, index, pmmu);
 
-       return nv04_mmu_new_(&nv44_mmu, device, index, pmmu);
+       return nvkm_mmu_new_(&nv44_mmu, device, index, pmmu);
 }
index 3d8d7e103f208b1bc3423457af7b5155452ca8af..bbb9c9cca27db87ca229a578c9c69478e559220e 100644 (file)
@@ -9,7 +9,6 @@ int nvkm_mmu_new_(const struct nvkm_mmu_func *, struct nvkm_device *,
                  int index, struct nvkm_mmu **);
 
 struct nvkm_mmu_func {
-       void *(*dtor)(struct nvkm_mmu *);
        int (*oneinit)(struct nvkm_mmu *);
        void (*init)(struct nvkm_mmu *);
 
index 93c8398b36a5de00c0ed6e4a752f5b7f268d9943..25e86ce3d29c8eddcb37cfcd6202b2c5d339d5f4 100644 (file)
@@ -70,6 +70,11 @@ nvkm_vmm_pt_new(const struct nvkm_vmm_desc *desc, bool sparse,
 void
 nvkm_vmm_dtor(struct nvkm_vmm *vmm)
 {
+       if (vmm->nullp) {
+               dma_free_coherent(vmm->mmu->subdev.device->dev, 16 * 1024,
+                                 vmm->nullp, vmm->null);
+       }
+
        if (vmm->pd) {
                nvkm_mmu_ptc_put(vmm->mmu, true, &vmm->pd->pt[0]);
                nvkm_vmm_pt_del(&vmm->pd);
index 16d4a4c7aae4cf5f1b682c3e78093e554d19b8f6..bade0237c10210ed8c5234ae716fd5c278cc3bbe 100644 (file)
@@ -110,4 +110,6 @@ int nv04_vmm_new(struct nvkm_mmu *, u64, u64, void *, u32,
                 struct lock_class_key *, const char *, struct nvkm_vmm **);
 int nv41_vmm_new(struct nvkm_mmu *, u64, u64, void *, u32,
                 struct lock_class_key *, const char *, struct nvkm_vmm **);
+int nv44_vmm_new(struct nvkm_mmu *, u64, u64, void *, u32,
+                struct lock_class_key *, const char *, struct nvkm_vmm **);
 #endif
diff --git a/drivers/gpu/drm/nouveau/nvkm/subdev/mmu/vmmnv44.c b/drivers/gpu/drm/nouveau/nvkm/subdev/mmu/vmmnv44.c
new file mode 100644 (file)
index 0000000..2b57042
--- /dev/null
@@ -0,0 +1,65 @@
+/*
+ * Copyright 2017 Red Hat Inc.
+ *
+ * Permission is hereby granted, free of charge, to any person obtaining a
+ * copy of this software and associated documentation files (the "Software"),
+ * to deal in the Software without restriction, including without limitation
+ * the rights to use, copy, modify, merge, publish, distribute, sublicense,
+ * and/or sell copies of the Software, and to permit persons to whom the
+ * Software is furnished to do so, subject to the following conditions:
+ *
+ * The above copyright notice and this permission notice shall be included in
+ * all copies or substantial portions of the Software.
+ *
+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+ * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+ * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.  IN NO EVENT SHALL
+ * THE COPYRIGHT HOLDER(S) OR AUTHOR(S) BE LIABLE FOR ANY CLAIM, DAMAGES OR
+ * OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE,
+ * ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR
+ * OTHER DEALINGS IN THE SOFTWARE.
+ */
+#include "vmm.h"
+
+static const struct nvkm_vmm_desc_func
+nv44_vmm_desc_pgt = {
+};
+
+static const struct nvkm_vmm_desc
+nv44_vmm_desc_12[] = {
+       { PGT, 17, 4, 0x80000, &nv44_vmm_desc_pgt },
+       {}
+};
+
+static const struct nvkm_vmm_func
+nv44_vmm = {
+       .page = {
+               { 12, &nv44_vmm_desc_12[0], NVKM_VMM_PAGE_HOST },
+               {}
+       }
+};
+
+int
+nv44_vmm_new(struct nvkm_mmu *mmu, u64 addr, u64 size, void *argv, u32 argc,
+            struct lock_class_key *key, const char *name,
+            struct nvkm_vmm **pvmm)
+{
+       struct nvkm_subdev *subdev = &mmu->subdev;
+       struct nvkm_vmm *vmm;
+       int ret;
+
+       ret = nv04_vmm_new_(&nv44_vmm, mmu, 0, addr, size,
+                           argv, argc, key, name, &vmm);
+       *pvmm = vmm;
+       if (ret)
+               return ret;
+
+       vmm->nullp = dma_alloc_coherent(subdev->device->dev, 16 * 1024,
+                                       &vmm->null, GFP_KERNEL);
+       if (!vmm->nullp) {
+               nvkm_warn(subdev, "unable to allocate dummy pages\n");
+               vmm->null = 0;
+       }
+
+       return 0;
+}