drm/nouveau/fb: remove dependence on namedb/engctx lookup
authorBen Skeggs <bskeggs@redhat.com>
Thu, 20 Aug 2015 04:54:18 +0000 (14:54 +1000)
committerBen Skeggs <bskeggs@redhat.com>
Fri, 28 Aug 2015 02:40:38 +0000 (12:40 +1000)
Signed-off-by: Ben Skeggs <bskeggs@redhat.com>
drivers/gpu/drm/nouveau/include/nvkm/engine/fifo.h
drivers/gpu/drm/nouveau/nvkm/engine/fifo/base.c
drivers/gpu/drm/nouveau/nvkm/engine/fifo/g84.c
drivers/gpu/drm/nouveau/nvkm/engine/fifo/gf100.c
drivers/gpu/drm/nouveau/nvkm/engine/fifo/gk104.c
drivers/gpu/drm/nouveau/nvkm/engine/fifo/nv50.c
drivers/gpu/drm/nouveau/nvkm/subdev/fb/nv50.c

index 1efcf0fb18e3afb6b023b098c0e3d204e1c39f04..d70fe7f9a44d934e3817708573efb2752053e4fa 100644 (file)
@@ -9,6 +9,7 @@ struct nvkm_fifo_chan {
        u64 addr;
        u32 size;
        u16 chid;
+       u64 inst;
 };
 
 static inline struct nvkm_fifo_chan *
@@ -81,6 +82,13 @@ struct nvkm_fifo {
        void (*start)(struct nvkm_fifo *, unsigned long *);
 };
 
+void nvkm_fifo_chan_put(struct nvkm_fifo *, unsigned long flags,
+                       struct nvkm_fifo_chan **);
+struct nvkm_fifo_chan *
+nvkm_fifo_chan_inst(struct nvkm_fifo *, u64 inst, unsigned long *flags);
+struct nvkm_fifo_chan *
+nvkm_fifo_chan_chid(struct nvkm_fifo *, int chid, unsigned long *flags);
+
 static inline struct nvkm_fifo *
 nvkm_fifo(void *obj)
 {
index 1934dfb5a434a7cb33e07175c7fba2e674498666..60e5fab4c80a3a16edaa20f69739336ac52ac449 100644 (file)
 #include <nvif/event.h>
 #include <nvif/unpack.h>
 
+void
+nvkm_fifo_chan_put(struct nvkm_fifo *fifo, unsigned long flags,
+                  struct nvkm_fifo_chan **pchan)
+{
+       struct nvkm_fifo_chan *chan = *pchan;
+       if (likely(chan)) {
+               *pchan = NULL;
+               spin_unlock_irqrestore(&fifo->lock, flags);
+       }
+}
+
+struct nvkm_fifo_chan *
+nvkm_fifo_chan_inst(struct nvkm_fifo *fifo, u64 inst, unsigned long *rflags)
+{
+       unsigned long flags;
+       int i;
+       spin_lock_irqsave(&fifo->lock, flags);
+       for (i = fifo->min; i < fifo->max; i++) {
+               struct nvkm_fifo_chan *chan = (void *)fifo->channel[i];
+               if (chan && chan->inst == inst) {
+                       *rflags = flags;
+                       return chan;
+               }
+       }
+       spin_unlock_irqrestore(&fifo->lock, flags);
+       return NULL;
+}
+
+struct nvkm_fifo_chan *
+nvkm_fifo_chan_chid(struct nvkm_fifo *fifo, int chid, unsigned long *rflags)
+{
+       unsigned long flags;
+       spin_lock_irqsave(&fifo->lock, flags);
+       if (fifo->channel[chid]) {
+               *rflags = flags;
+               return (void *)fifo->channel[chid];
+       }
+       spin_unlock_irqrestore(&fifo->lock, flags);
+       return NULL;
+}
+
 static int
 nvkm_fifo_event_ctor(struct nvkm_object *object, void *data, u32 size,
                     struct nvkm_notify *notify)
index 2997193697c0eb6fe49eda6f29353bae062652f1..c2ce3fa31f10aa4dd30946a014d63bcf38a007dd 100644 (file)
@@ -207,6 +207,7 @@ g84_fifo_chan_ctor_dma(struct nvkm_object *parent, struct nvkm_object *engine,
        if (ret)
                return ret;
 
+       chan->base.inst = base->base.gpuobj.addr;
        args->v0.chid = chan->base.chid;
 
        ret = nvkm_ramht_new(device, 0x8000, 16, &base->base.gpuobj,
@@ -285,6 +286,7 @@ g84_fifo_chan_ctor_ind(struct nvkm_object *parent, struct nvkm_object *engine,
        if (ret)
                return ret;
 
+       chan->base.inst = base->base.gpuobj.addr;
        args->v0.chid = chan->base.chid;
 
        ret = nvkm_ramht_new(device, 0x8000, 16, &base->base.gpuobj,
index 77b8df1f57fbab4d7f27b2ba908f25d9a3378cb8..9053730aafca358a397f0203769092618c2af4d3 100644 (file)
@@ -226,6 +226,7 @@ gf100_fifo_chan_ctor(struct nvkm_object *parent, struct nvkm_object *engine,
        if (ret)
                return ret;
 
+       chan->base.inst = base->base.gpuobj.addr;
        args->v0.chid = chan->base.chid;
 
        nv_parent(chan)->context_attach = gf100_fifo_context_attach;
index 39dae1a28dd1e1fe4fe034d181c9cb09270a4660..4407f6bbf94fae53b87b402ccc8dda4bd6c023fb 100644 (file)
@@ -283,6 +283,7 @@ gk104_fifo_chan_ctor(struct nvkm_object *parent, struct nvkm_object *engine,
        if (ret)
                return ret;
 
+       chan->base.inst = base->base.gpuobj.addr;
        args->v0.chid = chan->base.chid;
 
        nv_parent(chan)->context_attach = gk104_fifo_context_attach;
index 2b37fda876f4355f4380ee28f8e298781515cb75..23d5ee2fed853b6ad7b8b886f03579cd8ae5a9c3 100644 (file)
@@ -225,6 +225,7 @@ nv50_fifo_chan_ctor_dma(struct nvkm_object *parent, struct nvkm_object *engine,
        if (ret)
                return ret;
 
+       chan->base.inst = base->base.gpuobj.addr;
        args->v0.chid = chan->base.chid;
 
        nv_parent(chan)->context_attach = nv50_fifo_context_attach;
@@ -291,6 +292,7 @@ nv50_fifo_chan_ctor_ind(struct nvkm_object *parent, struct nvkm_object *engine,
        if (ret)
                return ret;
 
+       chan->base.inst = base->base.gpuobj.addr;
        args->v0.chid = chan->base.chid;
 
        nv_parent(chan)->context_attach = nv50_fifo_context_attach;
index 072b0c3881df5c8a8f99b81a2dd4d0270cf6092d..ea83f7d9ddc81dfc6b65faca099889d70623e01d 100644 (file)
@@ -25,8 +25,8 @@
 #include "ram.h"
 
 #include <core/client.h>
-#include <core/engctx.h>
 #include <core/enum.h>
+#include <engine/fifo.h>
 
 int
 nv50_fb_memtype[0x80] = {
@@ -47,98 +47,98 @@ nv50_fb_memtype_valid(struct nvkm_fb *fb, u32 memtype)
 }
 
 static const struct nvkm_enum vm_dispatch_subclients[] = {
-       { 0x00000000, "GRCTX", NULL },
-       { 0x00000001, "NOTIFY", NULL },
-       { 0x00000002, "QUERY", NULL },
-       { 0x00000003, "COND", NULL },
-       { 0x00000004, "M2M_IN", NULL },
-       { 0x00000005, "M2M_OUT", NULL },
-       { 0x00000006, "M2M_NOTIFY", NULL },
+       { 0x00000000, "GRCTX" },
+       { 0x00000001, "NOTIFY" },
+       { 0x00000002, "QUERY" },
+       { 0x00000003, "COND" },
+       { 0x00000004, "M2M_IN" },
+       { 0x00000005, "M2M_OUT" },
+       { 0x00000006, "M2M_NOTIFY" },
        {}
 };
 
 static const struct nvkm_enum vm_ccache_subclients[] = {
-       { 0x00000000, "CB", NULL },
-       { 0x00000001, "TIC", NULL },
-       { 0x00000002, "TSC", NULL },
+       { 0x00000000, "CB" },
+       { 0x00000001, "TIC" },
+       { 0x00000002, "TSC" },
        {}
 };
 
 static const struct nvkm_enum vm_prop_subclients[] = {
-       { 0x00000000, "RT0", NULL },
-       { 0x00000001, "RT1", NULL },
-       { 0x00000002, "RT2", NULL },
-       { 0x00000003, "RT3", NULL },
-       { 0x00000004, "RT4", NULL },
-       { 0x00000005, "RT5", NULL },
-       { 0x00000006, "RT6", NULL },
-       { 0x00000007, "RT7", NULL },
-       { 0x00000008, "ZETA", NULL },
-       { 0x00000009, "LOCAL", NULL },
-       { 0x0000000a, "GLOBAL", NULL },
-       { 0x0000000b, "STACK", NULL },
-       { 0x0000000c, "DST2D", NULL },
+       { 0x00000000, "RT0" },
+       { 0x00000001, "RT1" },
+       { 0x00000002, "RT2" },
+       { 0x00000003, "RT3" },
+       { 0x00000004, "RT4" },
+       { 0x00000005, "RT5" },
+       { 0x00000006, "RT6" },
+       { 0x00000007, "RT7" },
+       { 0x00000008, "ZETA" },
+       { 0x00000009, "LOCAL" },
+       { 0x0000000a, "GLOBAL" },
+       { 0x0000000b, "STACK" },
+       { 0x0000000c, "DST2D" },
        {}
 };
 
 static const struct nvkm_enum vm_pfifo_subclients[] = {
-       { 0x00000000, "PUSHBUF", NULL },
-       { 0x00000001, "SEMAPHORE", NULL },
+       { 0x00000000, "PUSHBUF" },
+       { 0x00000001, "SEMAPHORE" },
        {}
 };
 
 static const struct nvkm_enum vm_bar_subclients[] = {
-       { 0x00000000, "FB", NULL },
-       { 0x00000001, "IN", NULL },
+       { 0x00000000, "FB" },
+       { 0x00000001, "IN" },
        {}
 };
 
 static const struct nvkm_enum vm_client[] = {
-       { 0x00000000, "STRMOUT", NULL },
+       { 0x00000000, "STRMOUT" },
        { 0x00000003, "DISPATCH", vm_dispatch_subclients },
-       { 0x00000004, "PFIFO_WRITE", NULL },
+       { 0x00000004, "PFIFO_WRITE" },
        { 0x00000005, "CCACHE", vm_ccache_subclients },
-       { 0x00000006, "PMSPPP", NULL },
-       { 0x00000007, "CLIPID", NULL },
-       { 0x00000008, "PFIFO_READ", NULL },
-       { 0x00000009, "VFETCH", NULL },
-       { 0x0000000a, "TEXTURE", NULL },
+       { 0x00000006, "PMSPPP" },
+       { 0x00000007, "CLIPID" },
+       { 0x00000008, "PFIFO_READ" },
+       { 0x00000009, "VFETCH" },
+       { 0x0000000a, "TEXTURE" },
        { 0x0000000b, "PROP", vm_prop_subclients },
-       { 0x0000000c, "PVP", NULL },
-       { 0x0000000d, "PBSP", NULL },
-       { 0x0000000e, "PCRYPT", NULL },
-       { 0x0000000f, "PCOUNTER", NULL },
-       { 0x00000011, "PDAEMON", NULL },
+       { 0x0000000c, "PVP" },
+       { 0x0000000d, "PBSP" },
+       { 0x0000000e, "PCRYPT" },
+       { 0x0000000f, "PCOUNTER" },
+       { 0x00000011, "PDAEMON" },
        {}
 };
 
 static const struct nvkm_enum vm_engine[] = {
-       { 0x00000000, "PGRAPH", NULL, NVDEV_ENGINE_GR },
-       { 0x00000001, "PVP", NULL, NVDEV_ENGINE_VP },
-       { 0x00000004, "PEEPHOLE", NULL },
-       { 0x00000005, "PFIFO", vm_pfifo_subclients, NVDEV_ENGINE_FIFO },
+       { 0x00000000, "PGRAPH" },
+       { 0x00000001, "PVP" },
+       { 0x00000004, "PEEPHOLE" },
+       { 0x00000005, "PFIFO", vm_pfifo_subclients },
        { 0x00000006, "BAR", vm_bar_subclients },
-       { 0x00000008, "PMSPPP", NULL, NVDEV_ENGINE_MSPPP },
-       { 0x00000008, "PMPEG", NULL, NVDEV_ENGINE_MPEG },
-       { 0x00000009, "PBSP", NULL, NVDEV_ENGINE_BSP },
-       { 0x0000000a, "PCRYPT", NULL, NVDEV_ENGINE_CIPHER },
-       { 0x0000000b, "PCOUNTER", NULL },
-       { 0x0000000c, "SEMAPHORE_BG", NULL },
-       { 0x0000000d, "PCE0", NULL, NVDEV_ENGINE_CE0 },
-       { 0x0000000e, "PDAEMON", NULL },
+       { 0x00000008, "PMSPPP" },
+       { 0x00000008, "PMPEG" },
+       { 0x00000009, "PBSP" },
+       { 0x0000000a, "PCRYPT" },
+       { 0x0000000b, "PCOUNTER" },
+       { 0x0000000c, "SEMAPHORE_BG" },
+       { 0x0000000d, "PCE0" },
+       { 0x0000000e, "PDAEMON" },
        {}
 };
 
 static const struct nvkm_enum vm_fault[] = {
-       { 0x00000000, "PT_NOT_PRESENT", NULL },
-       { 0x00000001, "PT_TOO_SHORT", NULL },
-       { 0x00000002, "PAGE_NOT_PRESENT", NULL },
-       { 0x00000003, "PAGE_SYSTEM_ONLY", NULL },
-       { 0x00000004, "PAGE_READ_ONLY", NULL },
-       { 0x00000006, "NULL_DMAOBJ", NULL },
-       { 0x00000007, "WRONG_MEMTYPE", NULL },
-       { 0x0000000b, "VRAM_LIMIT", NULL },
-       { 0x0000000f, "DMAOBJ_LIMIT", NULL },
+       { 0x00000000, "PT_NOT_PRESENT" },
+       { 0x00000001, "PT_TOO_SHORT" },
+       { 0x00000002, "PAGE_NOT_PRESENT" },
+       { 0x00000003, "PAGE_SYSTEM_ONLY" },
+       { 0x00000004, "PAGE_READ_ONLY" },
+       { 0x00000006, "NULL_DMAOBJ" },
+       { 0x00000007, "WRONG_MEMTYPE" },
+       { 0x0000000b, "VRAM_LIMIT" },
+       { 0x0000000f, "DMAOBJ_LIMIT" },
        {}
 };
 
@@ -147,11 +147,12 @@ nv50_fb_intr(struct nvkm_subdev *subdev)
 {
        struct nv50_fb *fb = (void *)subdev;
        struct nvkm_device *device = fb->base.subdev.device;
-       struct nvkm_engine *engine;
+       struct nvkm_fifo *fifo = device->fifo;
+       struct nvkm_fifo_chan *chan;
        const struct nvkm_enum *en, *re, *cl, *sc;
-       struct nvkm_object *engctx = NULL;
-       u32 trap[6], idx, chan;
+       u32 trap[6], idx, inst;
        u8 st0, st1, st2, st3;
+       unsigned long flags;
        int i;
 
        idx = nvkm_rd32(device, 0x100c90);
@@ -178,48 +179,25 @@ nv50_fb_intr(struct nvkm_subdev *subdev)
                st2 = (trap[0] & 0x00ff0000) >> 16;
                st3 = (trap[0] & 0xff000000) >> 24;
        }
-       chan = (trap[2] << 16) | trap[1];
+       inst = ((trap[2] << 16) | trap[1]) << 12;
 
        en = nvkm_enum_find(vm_engine, st0);
-
-       if (en && en->data2) {
-               const struct nvkm_enum *orig_en = en;
-               while (en->name && en->value == st0 && en->data2) {
-                       engine = nvkm_engine(subdev, en->data2);
-                       /*XXX: clean this up */
-                       if (!engine && en->data2 == NVDEV_ENGINE_BSP)
-                               engine = nvkm_engine(subdev, NVDEV_ENGINE_MSVLD);
-                       if (!engine && en->data2 == NVDEV_ENGINE_CIPHER)
-                               engine = nvkm_engine(subdev, NVDEV_ENGINE_SEC);
-                       if (!engine && en->data2 == NVDEV_ENGINE_VP)
-                               engine = nvkm_engine(subdev, NVDEV_ENGINE_MSPDEC);
-                       if (engine) {
-                               engctx = nvkm_engctx_get(engine, chan);
-                               if (engctx)
-                                       break;
-                       }
-                       en++;
-               }
-               if (!engctx)
-                       en = orig_en;
-       }
-
        re = nvkm_enum_find(vm_fault , st1);
        cl = nvkm_enum_find(vm_client, st2);
        if      (cl && cl->data) sc = nvkm_enum_find(cl->data, st3);
        else if (en && en->data) sc = nvkm_enum_find(en->data, st3);
        else                     sc = NULL;
 
+       chan = nvkm_fifo_chan_inst(fifo, inst, &flags);
        nvkm_error(subdev, "trapped %s at %02x%04x%04x on channel "
                           "%08x [%s] engine %02x [%s] client %02x [%s] "
                           "subclient %02x [%s] reason %08x [%s]\n",
                   (trap[5] & 0x00000100) ? "read" : "write",
-                  trap[5] & 0xff, trap[4] & 0xffff, trap[3] & 0xffff, chan,
-                  nvkm_client_name(engctx), st0, en ? en->name : "",
+                  trap[5] & 0xff, trap[4] & 0xffff, trap[3] & 0xffff, inst,
+                  nvkm_client_name(chan), st0, en ? en->name : "",
                   st2, cl ? cl->name : "", st3, sc ? sc->name : "",
                   st1, re ? re->name : "");
-
-       nvkm_engctx_put(engctx);
+       nvkm_fifo_chan_put(fifo, flags, &chan);
 }
 
 int