drm/nouveau/fifo: directly use instmem for runlists and polling areas

Signed-off-by: Ben Skeggs <bskeggs@redhat.com>

diff --git a/drivers/gpu/drm/nouveau/include/nvkm/core/gpuobj.h b/drivers/gpu/drm/nouveau/include/nvkm/core/gpuobj.h
index f854b8098804c9878dd9f6b77a777e1a156e125b..7613107c05fa6c4561b5fd53918c6ce0f5e84909 100644 (file)
--- a/drivers/gpu/drm/nouveau/include/nvkm/core/gpuobj.h
+++ b/drivers/gpu/drm/nouveau/include/nvkm/core/gpuobj.h
@@ -55,7 +55,6 @@ int  nvkm_gpuobj_new(struct nvkm_object *, struct nvkm_object *, u32 size,
                     u32 align, u32 flags, struct nvkm_gpuobj **);
 int  nvkm_gpuobj_dup(struct nvkm_object *, struct nvkm_memory *,
                     struct nvkm_gpuobj **);
-int  nvkm_gpuobj_map(struct nvkm_gpuobj *, u32 acc, struct nvkm_vma *);
 int  nvkm_gpuobj_map_vm(struct nvkm_gpuobj *, struct nvkm_vm *, u32 access,
                        struct nvkm_vma *);
 void nvkm_gpuobj_unmap(struct nvkm_vma *);

diff --git a/drivers/gpu/drm/nouveau/nvkm/core/gpuobj.c b/drivers/gpu/drm/nouveau/nvkm/core/gpuobj.c
index ada5b6114a40f24f0bf78ab176fa09239076cf9d..c14469c3a861e75a44dca28ae557253177aefc97 100644 (file)
--- a/drivers/gpu/drm/nouveau/nvkm/core/gpuobj.c
+++ b/drivers/gpu/drm/nouveau/nvkm/core/gpuobj.c
@@ -263,22 +263,6 @@ nvkm_gpuobj_new(struct nvkm_object *parent, struct nvkm_object *pargpu,
                               (struct nvkm_object **)pgpuobj);
 }
 
-int
-nvkm_gpuobj_map(struct nvkm_gpuobj *gpuobj, u32 access, struct nvkm_vma *vma)
-{
-       struct nvkm_memory *memory = gpuobj->memory;
-       struct nvkm_bar *bar = nvkm_bar(gpuobj);
-       int ret = -EINVAL;
-
-       if (bar && bar->umap) {
-               ret = bar->umap(bar, gpuobj->size, 12, vma);
-               if (ret == 0)
-                       nvkm_memory_map(memory, vma, 0);
-       }
-
-       return ret;
-}
-
 int
 nvkm_gpuobj_map_vm(struct nvkm_gpuobj *gpuobj, struct nvkm_vm *vm,
                   u32 access, struct nvkm_vma *vma)

diff --git a/drivers/gpu/drm/nouveau/nvkm/engine/fifo/g84.c b/drivers/gpu/drm/nouveau/nvkm/engine/fifo/g84.c
index c6150eaf16f081cd0aa9d76f720533170c6f1a41..96c6835e9f149b2ed05d1405c907f5fa6d28b819 100644 (file)
--- a/drivers/gpu/drm/nouveau/nvkm/engine/fifo/g84.c
+++ b/drivers/gpu/drm/nouveau/nvkm/engine/fifo/g84.c
@@ -455,6 +455,7 @@ g84_fifo_ctor(struct nvkm_object *parent, struct nvkm_object *engine,
              struct nvkm_oclass *oclass, void *data, u32 size,
              struct nvkm_object **pobject)
 {
+       struct nvkm_device *device = (void *)parent;
        struct nv50_fifo *fifo;
        int ret;
 
@@ -463,13 +464,13 @@ g84_fifo_ctor(struct nvkm_object *parent, struct nvkm_object *engine,
        if (ret)
                return ret;
 
-       ret = nvkm_gpuobj_new(nv_object(fifo), NULL, 128 * 4, 0x1000, 0,
-                             &fifo->playlist[0]);
+       ret = nvkm_memory_new(device, NVKM_MEM_TARGET_INST, 128 * 4, 0x1000,
+                             false, &fifo->playlist[0]);
        if (ret)
                return ret;
 
-       ret = nvkm_gpuobj_new(nv_object(fifo), NULL, 128 * 4, 0x1000, 0,
-                             &fifo->playlist[1]);
+       ret = nvkm_memory_new(device, NVKM_MEM_TARGET_INST, 128 * 4, 0x1000,
+                             false, &fifo->playlist[1]);
        if (ret)
                return ret;
 

diff --git a/drivers/gpu/drm/nouveau/nvkm/engine/fifo/gf100.c b/drivers/gpu/drm/nouveau/nvkm/engine/fifo/gf100.c
index 763a2db7603a5f3e87cef5a71d16535363df0279..cbb8d249e7fbde41deda58556d5907a275aea172 100644 (file)
--- a/drivers/gpu/drm/nouveau/nvkm/engine/fifo/gf100.c
+++ b/drivers/gpu/drm/nouveau/nvkm/engine/fifo/gf100.c
@@ -27,6 +27,7 @@
 #include <core/engctx.h>
 #include <core/enum.h>
 #include <core/handle.h>
+#include <subdev/bar.h>
 #include <subdev/fb.h>
 #include <subdev/mmu.h>
 #include <subdev/timer.h>
@@ -42,13 +43,13 @@ struct gf100_fifo {
        u64 mask;
 
        struct {
-               struct nvkm_gpuobj *mem[2];
+               struct nvkm_memory *mem[2];
                int active;
                wait_queue_head_t wait;
        } runlist;
 
        struct {
-               struct nvkm_gpuobj *mem;
+               struct nvkm_memory *mem;
                struct nvkm_vma bar;
        } user;
        int spoon_nr;
@@ -78,7 +79,7 @@ gf100_fifo_runlist_update(struct gf100_fifo *fifo)
 {
        struct nvkm_subdev *subdev = &fifo->base.engine.subdev;
        struct nvkm_device *device = subdev->device;
-       struct nvkm_gpuobj *cur;
+       struct nvkm_memory *cur;
        int i, p;
 
        mutex_lock(&nv_subdev(fifo)->mutex);
@@ -96,7 +97,7 @@ gf100_fifo_runlist_update(struct gf100_fifo *fifo)
        }
        nvkm_done(cur);
 
-       nvkm_wr32(device, 0x002270, cur->addr >> 12);
+       nvkm_wr32(device, 0x002270, nvkm_memory_addr(cur) >> 12);
        nvkm_wr32(device, 0x002274, 0x01f00000 | (p >> 3));
 
        if (wait_event_timeout(fifo->runlist.wait,
@@ -238,10 +239,11 @@ gf100_fifo_chan_ctor(struct nvkm_object *parent, struct nvkm_object *engine,
        for (i = 0; i < 0x1000; i += 4)
                nvkm_wo32(fifo->user.mem, usermem + i, 0x00000000);
        nvkm_done(fifo->user.mem);
+       usermem = nvkm_memory_addr(fifo->user.mem) + usermem;
 
        nvkm_kmap(ramfc);
-       nvkm_wo32(ramfc, 0x08, lower_32_bits(fifo->user.mem->addr + usermem));
-       nvkm_wo32(ramfc, 0x0c, upper_32_bits(fifo->user.mem->addr + usermem));
+       nvkm_wo32(ramfc, 0x08, lower_32_bits(usermem));
+       nvkm_wo32(ramfc, 0x0c, upper_32_bits(usermem));
        nvkm_wo32(ramfc, 0x10, 0x0000face);
        nvkm_wo32(ramfc, 0x30, 0xfffff902);
        nvkm_wo32(ramfc, 0x48, lower_32_bits(ioffset));
@@ -879,6 +881,8 @@ gf100_fifo_ctor(struct nvkm_object *parent, struct nvkm_object *engine,
                struct nvkm_oclass *oclass, void *data, u32 size,
                struct nvkm_object **pobject)
 {
+       struct nvkm_device *device = (void *)parent;
+       struct nvkm_bar *bar = device->bar;
        struct gf100_fifo *fifo;
        int ret;
 
@@ -889,28 +893,29 @@ gf100_fifo_ctor(struct nvkm_object *parent, struct nvkm_object *engine,
 
        INIT_WORK(&fifo->fault, gf100_fifo_recover_work);
 
-       ret = nvkm_gpuobj_new(nv_object(fifo), NULL, 0x1000, 0x1000, 0,
-                             &fifo->runlist.mem[0]);
+       ret = nvkm_memory_new(device, NVKM_MEM_TARGET_INST, 0x1000, 0x1000,
+                             false, &fifo->runlist.mem[0]);
        if (ret)
                return ret;
 
-       ret = nvkm_gpuobj_new(nv_object(fifo), NULL, 0x1000, 0x1000, 0,
-                             &fifo->runlist.mem[1]);
+       ret = nvkm_memory_new(device, NVKM_MEM_TARGET_INST, 0x1000, 0x1000,
+                             false, &fifo->runlist.mem[1]);
        if (ret)
                return ret;
 
        init_waitqueue_head(&fifo->runlist.wait);
 
-       ret = nvkm_gpuobj_new(nv_object(fifo), NULL, 128 * 0x1000, 0x1000, 0,
-                             &fifo->user.mem);
+       ret = nvkm_memory_new(device, NVKM_MEM_TARGET_INST, 128 * 0x1000,
+                             0x1000, false, &fifo->user.mem);
        if (ret)
                return ret;
 
-       ret = nvkm_gpuobj_map(fifo->user.mem, NV_MEM_ACCESS_RW,
-                             &fifo->user.bar);
+       ret = bar->umap(bar, 128 * 0x1000, 12, &fifo->user.bar);
        if (ret)
                return ret;
 
+       nvkm_memory_map(fifo->user.mem, &fifo->user.bar, 0);
+
        ret = nvkm_event_init(&gf100_fifo_uevent_func, 1, 1, &fifo->base.uevent);
        if (ret)
                return ret;
@@ -927,10 +932,10 @@ gf100_fifo_dtor(struct nvkm_object *object)
 {
        struct gf100_fifo *fifo = (void *)object;
 
-       nvkm_gpuobj_unmap(&fifo->user.bar);
-       nvkm_gpuobj_ref(NULL, &fifo->user.mem);
-       nvkm_gpuobj_ref(NULL, &fifo->runlist.mem[0]);
-       nvkm_gpuobj_ref(NULL, &fifo->runlist.mem[1]);
+       nvkm_vm_put(&fifo->user.bar);
+       nvkm_memory_del(&fifo->user.mem);
+       nvkm_memory_del(&fifo->runlist.mem[0]);
+       nvkm_memory_del(&fifo->runlist.mem[1]);
 
        nvkm_fifo_destroy(&fifo->base);
 }

diff --git a/drivers/gpu/drm/nouveau/nvkm/engine/fifo/gk104.c b/drivers/gpu/drm/nouveau/nvkm/engine/fifo/gk104.c
index d1238aa7bec5509180adba185c06e105551ce7b2..0e8356d5fa02e038d429e3268758e24be4955c88 100644 (file)
--- a/drivers/gpu/drm/nouveau/nvkm/engine/fifo/gk104.c
+++ b/drivers/gpu/drm/nouveau/nvkm/engine/fifo/gk104.c
@@ -27,6 +27,7 @@
 #include <core/engctx.h>
 #include <core/enum.h>
 #include <core/handle.h>
+#include <subdev/bar.h>
 #include <subdev/fb.h>
 #include <subdev/mmu.h>
 #include <subdev/timer.h>
@@ -53,7 +54,7 @@ static const struct {
 #define FIFO_ENGINE_NR ARRAY_SIZE(fifo_engine)
 
 struct gk104_fifo_engn {
-       struct nvkm_gpuobj *runlist[2];
+       struct nvkm_memory *runlist[2];
        int cur_runlist;
        wait_queue_head_t wait;
 };
@@ -66,7 +67,7 @@ struct gk104_fifo {
 
        struct gk104_fifo_engn engine[FIFO_ENGINE_NR];
        struct {
-               struct nvkm_gpuobj *mem;
+               struct nvkm_memory *mem;
                struct nvkm_vma bar;
        } user;
        int spoon_nr;
@@ -98,7 +99,7 @@ gk104_fifo_runlist_update(struct gk104_fifo *fifo, u32 engine)
        struct gk104_fifo_engn *engn = &fifo->engine[engine];
        struct nvkm_subdev *subdev = &fifo->base.engine.subdev;
        struct nvkm_device *device = subdev->device;
-       struct nvkm_gpuobj *cur;
+       struct nvkm_memory *cur;
        int i, p;
 
        mutex_lock(&nv_subdev(fifo)->mutex);
@@ -116,7 +117,7 @@ gk104_fifo_runlist_update(struct gk104_fifo *fifo, u32 engine)
        }
        nvkm_done(cur);
 
-       nvkm_wr32(device, 0x002270, cur->addr >> 12);
+       nvkm_wr32(device, 0x002270, nvkm_memory_addr(cur) >> 12);
        nvkm_wr32(device, 0x002274, (engine << 20) | (p >> 3));
 
        if (wait_event_timeout(engn->wait, !(nvkm_rd32(device, 0x002284 +
@@ -296,10 +297,11 @@ gk104_fifo_chan_ctor(struct nvkm_object *parent, struct nvkm_object *engine,
        for (i = 0; i < 0x200; i += 4)
                nvkm_wo32(fifo->user.mem, usermem + i, 0x00000000);
        nvkm_done(fifo->user.mem);
+       usermem = nvkm_memory_addr(fifo->user.mem) + usermem;
 
        nvkm_kmap(ramfc);
-       nvkm_wo32(ramfc, 0x08, lower_32_bits(fifo->user.mem->addr + usermem));
-       nvkm_wo32(ramfc, 0x0c, upper_32_bits(fifo->user.mem->addr + usermem));
+       nvkm_wo32(ramfc, 0x08, lower_32_bits(usermem));
+       nvkm_wo32(ramfc, 0x0c, upper_32_bits(usermem));
        nvkm_wo32(ramfc, 0x10, 0x0000face);
        nvkm_wo32(ramfc, 0x30, 0xfffff902);
        nvkm_wo32(ramfc, 0x48, lower_32_bits(ioffset));
@@ -1107,12 +1109,12 @@ gk104_fifo_dtor(struct nvkm_object *object)
        struct gk104_fifo *fifo = (void *)object;
        int i;
 
-       nvkm_gpuobj_unmap(&fifo->user.bar);
-       nvkm_gpuobj_ref(NULL, &fifo->user.mem);
+       nvkm_vm_put(&fifo->user.bar);
+       nvkm_memory_del(&fifo->user.mem);
 
        for (i = 0; i < FIFO_ENGINE_NR; i++) {
-               nvkm_gpuobj_ref(NULL, &fifo->engine[i].runlist[1]);
-               nvkm_gpuobj_ref(NULL, &fifo->engine[i].runlist[0]);
+               nvkm_memory_del(&fifo->engine[i].runlist[1]);
+               nvkm_memory_del(&fifo->engine[i].runlist[0]);
        }
 
        nvkm_fifo_destroy(&fifo->base);
@@ -1123,6 +1125,8 @@ gk104_fifo_ctor(struct nvkm_object *parent, struct nvkm_object *engine,
                struct nvkm_oclass *oclass, void *data, u32 size,
                struct nvkm_object **pobject)
 {
+       struct nvkm_device *device = (void *)parent;
+       struct nvkm_bar *bar = device->bar;
        struct gk104_fifo_impl *impl = (void *)oclass;
        struct gk104_fifo *fifo;
        int ret, i;
@@ -1136,29 +1140,33 @@ gk104_fifo_ctor(struct nvkm_object *parent, struct nvkm_object *engine,
        INIT_WORK(&fifo->fault, gk104_fifo_recover_work);
 
        for (i = 0; i < FIFO_ENGINE_NR; i++) {
-               ret = nvkm_gpuobj_new(nv_object(fifo), NULL, 0x8000, 0x1000,
-                                     0, &fifo->engine[i].runlist[0]);
+               ret = nvkm_memory_new(device, NVKM_MEM_TARGET_INST,
+                                     0x8000, 0x1000, false,
+                                     &fifo->engine[i].runlist[0]);
                if (ret)
                        return ret;
 
-               ret = nvkm_gpuobj_new(nv_object(fifo), NULL, 0x8000, 0x1000,
-                                     0, &fifo->engine[i].runlist[1]);
+               ret = nvkm_memory_new(device, NVKM_MEM_TARGET_INST,
+                                     0x8000, 0x1000, false,
+                                     &fifo->engine[i].runlist[1]);
                if (ret)
                        return ret;
 
                init_waitqueue_head(&fifo->engine[i].wait);
        }
 
-       ret = nvkm_gpuobj_new(nv_object(fifo), NULL, impl->channels * 0x200,
-                             0x1000, NVOBJ_FLAG_ZERO_ALLOC, &fifo->user.mem);
+       ret = nvkm_memory_new(device, NVKM_MEM_TARGET_INST,
+                             impl->channels * 0x200, 0x1000,
+                             true, &fifo->user.mem);
        if (ret)
                return ret;
 
-       ret = nvkm_gpuobj_map(fifo->user.mem, NV_MEM_ACCESS_RW,
-                             &fifo->user.bar);
+       ret = bar->umap(bar, impl->channels * 0x200, 12, &fifo->user.bar);
        if (ret)
                return ret;
 
+       nvkm_memory_map(fifo->user.mem, &fifo->user.bar, 0);
+
        ret = nvkm_event_init(&gk104_fifo_uevent_func, 1, 1, &fifo->base.uevent);
        if (ret)
                return ret;

diff --git a/drivers/gpu/drm/nouveau/nvkm/engine/fifo/nv50.c b/drivers/gpu/drm/nouveau/nvkm/engine/fifo/nv50.c
index ffa495c1692bb7a470e1627cc91c291487552496..96e5b61daf10c35993a14c101b73423b0b4d3cca 100644 (file)
--- a/drivers/gpu/drm/nouveau/nvkm/engine/fifo/nv50.c
+++ b/drivers/gpu/drm/nouveau/nvkm/engine/fifo/nv50.c
@@ -41,7 +41,7 @@ static void
 nv50_fifo_playlist_update_locked(struct nv50_fifo *fifo)
 {
        struct nvkm_device *device = fifo->base.engine.subdev.device;
-       struct nvkm_gpuobj *cur;
+       struct nvkm_memory *cur;
        int i, p;
 
        cur = fifo->playlist[fifo->cur_playlist];
@@ -54,7 +54,7 @@ nv50_fifo_playlist_update_locked(struct nv50_fifo *fifo)
        }
        nvkm_done(cur);
 
-       nvkm_wr32(device, 0x0032f4, cur->addr >> 12);
+       nvkm_wr32(device, 0x0032f4, nvkm_memory_addr(cur) >> 12);
        nvkm_wr32(device, 0x0032ec, p);
        nvkm_wr32(device, 0x002500, 0x00000101);
 }
@@ -467,6 +467,7 @@ nv50_fifo_ctor(struct nvkm_object *parent, struct nvkm_object *engine,
               struct nvkm_oclass *oclass, void *data, u32 size,
               struct nvkm_object **pobject)
 {
+       struct nvkm_device *device = (void *)parent;
        struct nv50_fifo *fifo;
        int ret;
 
@@ -475,13 +476,13 @@ nv50_fifo_ctor(struct nvkm_object *parent, struct nvkm_object *engine,
        if (ret)
                return ret;
 
-       ret = nvkm_gpuobj_new(nv_object(fifo), NULL, 128 * 4, 0x1000, 0,
-                             &fifo->playlist[0]);
+       ret = nvkm_memory_new(device, NVKM_MEM_TARGET_INST, 128 * 4, 0x1000,
+                             false, &fifo->playlist[0]);
        if (ret)
                return ret;
 
-       ret = nvkm_gpuobj_new(nv_object(fifo), NULL, 128 * 4, 0x1000, 0,
-                             &fifo->playlist[1]);
+       ret = nvkm_memory_new(device, NVKM_MEM_TARGET_INST, 128 * 4, 0x1000,
+                             false, &fifo->playlist[1]);
        if (ret)
                return ret;
 
@@ -499,8 +500,8 @@ nv50_fifo_dtor(struct nvkm_object *object)
 {
        struct nv50_fifo *fifo = (void *)object;
 
-       nvkm_gpuobj_ref(NULL, &fifo->playlist[1]);
-       nvkm_gpuobj_ref(NULL, &fifo->playlist[0]);
+       nvkm_memory_del(&fifo->playlist[1]);
+       nvkm_memory_del(&fifo->playlist[0]);
 
        nvkm_fifo_destroy(&fifo->base);
 }

diff --git a/drivers/gpu/drm/nouveau/nvkm/engine/fifo/nv50.h b/drivers/gpu/drm/nouveau/nvkm/engine/fifo/nv50.h
index 722fcce7070ed4f09fffb014beab2309127b616a..0c826153bac66d1b0b2cdc30c98f8185377a64c0 100644 (file)
--- a/drivers/gpu/drm/nouveau/nvkm/engine/fifo/nv50.h
+++ b/drivers/gpu/drm/nouveau/nvkm/engine/fifo/nv50.h
@@ -4,7 +4,7 @@
 
 struct nv50_fifo {
        struct nvkm_fifo base;
-       struct nvkm_gpuobj *playlist[2];
+       struct nvkm_memory *playlist[2];
        int cur_playlist;
 };