These changes are the raw update to linux-4.4.6-rt14. Kernel sources
[kvmfornfv.git] / kernel / drivers / gpu / drm / nouveau / nvkm / engine / fifo / gf100.c
index b745252..ff6fcbd 100644 (file)
  *
  * Authors: Ben Skeggs
  */
-#include <engine/fifo.h>
+#include "gf100.h"
+#include "changf100.h"
 
 #include <core/client.h>
-#include <core/engctx.h>
 #include <core/enum.h>
-#include <core/handle.h>
+#include <core/gpuobj.h>
 #include <subdev/bar.h>
-#include <subdev/fb.h>
-#include <subdev/mmu.h>
-#include <subdev/timer.h>
+#include <engine/sw.h>
 
 #include <nvif/class.h>
-#include <nvif/unpack.h>
-
-struct gf100_fifo_priv {
-       struct nvkm_fifo base;
-
-       struct work_struct fault;
-       u64 mask;
-
-       struct {
-               struct nvkm_gpuobj *mem[2];
-               int active;
-               wait_queue_head_t wait;
-       } runlist;
-
-       struct {
-               struct nvkm_gpuobj *mem;
-               struct nvkm_vma bar;
-       } user;
-       int spoon_nr;
-};
-
-struct gf100_fifo_base {
-       struct nvkm_fifo_base base;
-       struct nvkm_gpuobj *pgd;
-       struct nvkm_vm *vm;
-};
-
-struct gf100_fifo_chan {
-       struct nvkm_fifo_chan base;
-       enum {
-               STOPPED,
-               RUNNING,
-               KILLED
-       } state;
-};
-
-/*******************************************************************************
- * FIFO channel objects
- ******************************************************************************/
 
 static void
-gf100_fifo_runlist_update(struct gf100_fifo_priv *priv)
+gf100_fifo_uevent_init(struct nvkm_fifo *fifo)
 {
-       struct nvkm_bar *bar = nvkm_bar(priv);
-       struct nvkm_gpuobj *cur;
-       int i, p;
-
-       mutex_lock(&nv_subdev(priv)->mutex);
-       cur = priv->runlist.mem[priv->runlist.active];
-       priv->runlist.active = !priv->runlist.active;
-
-       for (i = 0, p = 0; i < 128; i++) {
-               struct gf100_fifo_chan *chan = (void *)priv->base.channel[i];
-               if (chan && chan->state == RUNNING) {
-                       nv_wo32(cur, p + 0, i);
-                       nv_wo32(cur, p + 4, 0x00000004);
-                       p += 8;
-               }
-       }
-       bar->flush(bar);
-
-       nv_wr32(priv, 0x002270, cur->addr >> 12);
-       nv_wr32(priv, 0x002274, 0x01f00000 | (p >> 3));
-
-       if (wait_event_timeout(priv->runlist.wait,
-                              !(nv_rd32(priv, 0x00227c) & 0x00100000),
-                              msecs_to_jiffies(2000)) == 0)
-               nv_error(priv, "runlist update timeout\n");
-       mutex_unlock(&nv_subdev(priv)->mutex);
+       struct nvkm_device *device = fifo->engine.subdev.device;
+       nvkm_mask(device, 0x002140, 0x80000000, 0x80000000);
 }
 
-static int
-gf100_fifo_context_attach(struct nvkm_object *parent,
-                         struct nvkm_object *object)
-{
-       struct nvkm_bar *bar = nvkm_bar(parent);
-       struct gf100_fifo_base *base = (void *)parent->parent;
-       struct nvkm_engctx *ectx = (void *)object;
-       u32 addr;
-       int ret;
-
-       switch (nv_engidx(object->engine)) {
-       case NVDEV_ENGINE_SW    : return 0;
-       case NVDEV_ENGINE_GR    : addr = 0x0210; break;
-       case NVDEV_ENGINE_CE0   : addr = 0x0230; break;
-       case NVDEV_ENGINE_CE1   : addr = 0x0240; break;
-       case NVDEV_ENGINE_MSVLD : addr = 0x0270; break;
-       case NVDEV_ENGINE_MSPDEC: addr = 0x0250; break;
-       case NVDEV_ENGINE_MSPPP : addr = 0x0260; break;
-       default:
-               return -EINVAL;
-       }
-
-       if (!ectx->vma.node) {
-               ret = nvkm_gpuobj_map_vm(nv_gpuobj(ectx), base->vm,
-                                        NV_MEM_ACCESS_RW, &ectx->vma);
-               if (ret)
-                       return ret;
-
-               nv_engctx(ectx)->addr = nv_gpuobj(base)->addr >> 12;
-       }
-
-       nv_wo32(base, addr + 0x00, lower_32_bits(ectx->vma.offset) | 4);
-       nv_wo32(base, addr + 0x04, upper_32_bits(ectx->vma.offset));
-       bar->flush(bar);
-       return 0;
-}
-
-static int
-gf100_fifo_context_detach(struct nvkm_object *parent, bool suspend,
-                         struct nvkm_object *object)
+static void
+gf100_fifo_uevent_fini(struct nvkm_fifo *fifo)
 {
-       struct nvkm_bar *bar = nvkm_bar(parent);
-       struct gf100_fifo_priv *priv = (void *)parent->engine;
-       struct gf100_fifo_base *base = (void *)parent->parent;
-       struct gf100_fifo_chan *chan = (void *)parent;
-       u32 addr;
-
-       switch (nv_engidx(object->engine)) {
-       case NVDEV_ENGINE_SW    : return 0;
-       case NVDEV_ENGINE_GR    : addr = 0x0210; break;
-       case NVDEV_ENGINE_CE0   : addr = 0x0230; break;
-       case NVDEV_ENGINE_CE1   : addr = 0x0240; break;
-       case NVDEV_ENGINE_MSVLD : addr = 0x0270; break;
-       case NVDEV_ENGINE_MSPDEC: addr = 0x0250; break;
-       case NVDEV_ENGINE_MSPPP : addr = 0x0260; break;
-       default:
-               return -EINVAL;
-       }
-
-       nv_wr32(priv, 0x002634, chan->base.chid);
-       if (!nv_wait(priv, 0x002634, 0xffffffff, chan->base.chid)) {
-               nv_error(priv, "channel %d [%s] kick timeout\n",
-                        chan->base.chid, nvkm_client_name(chan));
-               if (suspend)
-                       return -EBUSY;
-       }
-
-       nv_wo32(base, addr + 0x00, 0x00000000);
-       nv_wo32(base, addr + 0x04, 0x00000000);
-       bar->flush(bar);
-       return 0;
+       struct nvkm_device *device = fifo->engine.subdev.device;
+       nvkm_mask(device, 0x002140, 0x80000000, 0x00000000);
 }
 
-static int
-gf100_fifo_chan_ctor(struct nvkm_object *parent, struct nvkm_object *engine,
-                    struct nvkm_oclass *oclass, void *data, u32 size,
-                    struct nvkm_object **pobject)
+void
+gf100_fifo_runlist_update(struct gf100_fifo *fifo)
 {
-       union {
-               struct nv50_channel_gpfifo_v0 v0;
-       } *args = data;
-       struct nvkm_bar *bar = nvkm_bar(parent);
-       struct gf100_fifo_priv *priv = (void *)engine;
-       struct gf100_fifo_base *base = (void *)parent;
        struct gf100_fifo_chan *chan;
-       u64 usermem, ioffset, ilength;
-       int ret, i;
-
-       nv_ioctl(parent, "create channel gpfifo size %d\n", size);
-       if (nvif_unpack(args->v0, 0, 0, false)) {
-               nv_ioctl(parent, "create channel gpfifo vers %d pushbuf %08x "
-                                "ioffset %016llx ilength %08x\n",
-                        args->v0.version, args->v0.pushbuf, args->v0.ioffset,
-                        args->v0.ilength);
-       } else
-               return ret;
+       struct nvkm_subdev *subdev = &fifo->base.engine.subdev;
+       struct nvkm_device *device = subdev->device;
+       struct nvkm_memory *cur;
+       int nr = 0;
 
-       ret = nvkm_fifo_channel_create(parent, engine, oclass, 1,
-                                      priv->user.bar.offset, 0x1000,
-                                      args->v0.pushbuf,
-                                      (1ULL << NVDEV_ENGINE_SW) |
-                                      (1ULL << NVDEV_ENGINE_GR) |
-                                      (1ULL << NVDEV_ENGINE_CE0) |
-                                      (1ULL << NVDEV_ENGINE_CE1) |
-                                      (1ULL << NVDEV_ENGINE_MSVLD) |
-                                      (1ULL << NVDEV_ENGINE_MSPDEC) |
-                                      (1ULL << NVDEV_ENGINE_MSPPP), &chan);
-       *pobject = nv_object(chan);
-       if (ret)
-               return ret;
+       mutex_lock(&subdev->mutex);
+       cur = fifo->runlist.mem[fifo->runlist.active];
+       fifo->runlist.active = !fifo->runlist.active;
 
-       args->v0.chid = chan->base.chid;
-
-       nv_parent(chan)->context_attach = gf100_fifo_context_attach;
-       nv_parent(chan)->context_detach = gf100_fifo_context_detach;
-
-       usermem = chan->base.chid * 0x1000;
-       ioffset = args->v0.ioffset;
-       ilength = order_base_2(args->v0.ilength / 8);
-
-       for (i = 0; i < 0x1000; i += 4)
-               nv_wo32(priv->user.mem, usermem + i, 0x00000000);
-
-       nv_wo32(base, 0x08, lower_32_bits(priv->user.mem->addr + usermem));
-       nv_wo32(base, 0x0c, upper_32_bits(priv->user.mem->addr + usermem));
-       nv_wo32(base, 0x10, 0x0000face);
-       nv_wo32(base, 0x30, 0xfffff902);
-       nv_wo32(base, 0x48, lower_32_bits(ioffset));
-       nv_wo32(base, 0x4c, upper_32_bits(ioffset) | (ilength << 16));
-       nv_wo32(base, 0x54, 0x00000002);
-       nv_wo32(base, 0x84, 0x20400000);
-       nv_wo32(base, 0x94, 0x30000001);
-       nv_wo32(base, 0x9c, 0x00000100);
-       nv_wo32(base, 0xa4, 0x1f1f1f1f);
-       nv_wo32(base, 0xa8, 0x1f1f1f1f);
-       nv_wo32(base, 0xac, 0x0000001f);
-       nv_wo32(base, 0xb8, 0xf8000000);
-       nv_wo32(base, 0xf8, 0x10003080); /* 0x002310 */
-       nv_wo32(base, 0xfc, 0x10000010); /* 0x002350 */
-       bar->flush(bar);
-       return 0;
-}
-
-static int
-gf100_fifo_chan_init(struct nvkm_object *object)
-{
-       struct nvkm_gpuobj *base = nv_gpuobj(object->parent);
-       struct gf100_fifo_priv *priv = (void *)object->engine;
-       struct gf100_fifo_chan *chan = (void *)object;
-       u32 chid = chan->base.chid;
-       int ret;
-
-       ret = nvkm_fifo_channel_init(&chan->base);
-       if (ret)
-               return ret;
-
-       nv_wr32(priv, 0x003000 + (chid * 8), 0xc0000000 | base->addr >> 12);
-
-       if (chan->state == STOPPED && (chan->state = RUNNING) == RUNNING) {
-               nv_wr32(priv, 0x003004 + (chid * 8), 0x001f0001);
-               gf100_fifo_runlist_update(priv);
-       }
-
-       return 0;
-}
-
-static void gf100_fifo_intr_engine(struct gf100_fifo_priv *priv);
-
-static int
-gf100_fifo_chan_fini(struct nvkm_object *object, bool suspend)
-{
-       struct gf100_fifo_priv *priv = (void *)object->engine;
-       struct gf100_fifo_chan *chan = (void *)object;
-       u32 chid = chan->base.chid;
-
-       if (chan->state == RUNNING && (chan->state = STOPPED) == STOPPED) {
-               nv_mask(priv, 0x003004 + (chid * 8), 0x00000001, 0x00000000);
-               gf100_fifo_runlist_update(priv);
+       nvkm_kmap(cur);
+       list_for_each_entry(chan, &fifo->chan, head) {
+               nvkm_wo32(cur, (nr * 8) + 0, chan->base.chid);
+               nvkm_wo32(cur, (nr * 8) + 4, 0x00000004);
+               nr++;
        }
+       nvkm_done(cur);
 
-       gf100_fifo_intr_engine(priv);
-
-       nv_wr32(priv, 0x003000 + (chid * 8), 0x00000000);
-       return nvkm_fifo_channel_fini(&chan->base, suspend);
-}
-
-static struct nvkm_ofuncs
-gf100_fifo_ofuncs = {
-       .ctor = gf100_fifo_chan_ctor,
-       .dtor = _nvkm_fifo_channel_dtor,
-       .init = gf100_fifo_chan_init,
-       .fini = gf100_fifo_chan_fini,
-       .map  = _nvkm_fifo_channel_map,
-       .rd32 = _nvkm_fifo_channel_rd32,
-       .wr32 = _nvkm_fifo_channel_wr32,
-       .ntfy = _nvkm_fifo_channel_ntfy
-};
-
-static struct nvkm_oclass
-gf100_fifo_sclass[] = {
-       { FERMI_CHANNEL_GPFIFO, &gf100_fifo_ofuncs },
-       {}
-};
-
-/*******************************************************************************
- * FIFO context - instmem heap and vm setup
- ******************************************************************************/
-
-static int
-gf100_fifo_context_ctor(struct nvkm_object *parent, struct nvkm_object *engine,
-                       struct nvkm_oclass *oclass, void *data, u32 size,
-                       struct nvkm_object **pobject)
-{
-       struct gf100_fifo_base *base;
-       int ret;
-
-       ret = nvkm_fifo_context_create(parent, engine, oclass, NULL, 0x1000,
-                                      0x1000, NVOBJ_FLAG_ZERO_ALLOC |
-                                      NVOBJ_FLAG_HEAP, &base);
-       *pobject = nv_object(base);
-       if (ret)
-               return ret;
-
-       ret = nvkm_gpuobj_new(nv_object(base), NULL, 0x10000, 0x1000, 0,
-                             &base->pgd);
-       if (ret)
-               return ret;
-
-       nv_wo32(base, 0x0200, lower_32_bits(base->pgd->addr));
-       nv_wo32(base, 0x0204, upper_32_bits(base->pgd->addr));
-       nv_wo32(base, 0x0208, 0xffffffff);
-       nv_wo32(base, 0x020c, 0x000000ff);
+       nvkm_wr32(device, 0x002270, nvkm_memory_addr(cur) >> 12);
+       nvkm_wr32(device, 0x002274, 0x01f00000 | nr);
 
-       ret = nvkm_vm_ref(nvkm_client(parent)->vm, &base->vm, base->pgd);
-       if (ret)
-               return ret;
-
-       return 0;
-}
-
-static void
-gf100_fifo_context_dtor(struct nvkm_object *object)
-{
-       struct gf100_fifo_base *base = (void *)object;
-       nvkm_vm_ref(NULL, &base->vm, base->pgd);
-       nvkm_gpuobj_ref(NULL, &base->pgd);
-       nvkm_fifo_context_destroy(&base->base);
+       if (wait_event_timeout(fifo->runlist.wait,
+                              !(nvkm_rd32(device, 0x00227c) & 0x00100000),
+                              msecs_to_jiffies(2000)) == 0)
+               nvkm_error(subdev, "runlist update timeout\n");
+       mutex_unlock(&subdev->mutex);
 }
 
-static struct nvkm_oclass
-gf100_fifo_cclass = {
-       .handle = NV_ENGCTX(FIFO, 0xc0),
-       .ofuncs = &(struct nvkm_ofuncs) {
-               .ctor = gf100_fifo_context_ctor,
-               .dtor = gf100_fifo_context_dtor,
-               .init = _nvkm_fifo_context_init,
-               .fini = _nvkm_fifo_context_fini,
-               .rd32 = _nvkm_fifo_context_rd32,
-               .wr32 = _nvkm_fifo_context_wr32,
-       },
-};
-
-/*******************************************************************************
- * PFIFO engine
- ******************************************************************************/
-
 static inline int
-gf100_fifo_engidx(struct gf100_fifo_priv *priv, u32 engn)
+gf100_fifo_engidx(struct gf100_fifo *fifo, u32 engn)
 {
        switch (engn) {
-       case NVDEV_ENGINE_GR    : engn = 0; break;
-       case NVDEV_ENGINE_MSVLD : engn = 1; break;
-       case NVDEV_ENGINE_MSPPP : engn = 2; break;
-       case NVDEV_ENGINE_MSPDEC: engn = 3; break;
-       case NVDEV_ENGINE_CE0   : engn = 4; break;
-       case NVDEV_ENGINE_CE1   : engn = 5; break;
+       case NVKM_ENGINE_GR    : engn = 0; break;
+       case NVKM_ENGINE_MSVLD : engn = 1; break;
+       case NVKM_ENGINE_MSPPP : engn = 2; break;
+       case NVKM_ENGINE_MSPDEC: engn = 3; break;
+       case NVKM_ENGINE_CE0   : engn = 4; break;
+       case NVKM_ENGINE_CE1   : engn = 5; break;
        default:
                return -1;
        }
@@ -388,95 +95,73 @@ gf100_fifo_engidx(struct gf100_fifo_priv *priv, u32 engn)
 }
 
 static inline struct nvkm_engine *
-gf100_fifo_engine(struct gf100_fifo_priv *priv, u32 engn)
+gf100_fifo_engine(struct gf100_fifo *fifo, u32 engn)
 {
+       struct nvkm_device *device = fifo->base.engine.subdev.device;
+
        switch (engn) {
-       case 0: engn = NVDEV_ENGINE_GR; break;
-       case 1: engn = NVDEV_ENGINE_MSVLD; break;
-       case 2: engn = NVDEV_ENGINE_MSPPP; break;
-       case 3: engn = NVDEV_ENGINE_MSPDEC; break;
-       case 4: engn = NVDEV_ENGINE_CE0; break;
-       case 5: engn = NVDEV_ENGINE_CE1; break;
+       case 0: engn = NVKM_ENGINE_GR; break;
+       case 1: engn = NVKM_ENGINE_MSVLD; break;
+       case 2: engn = NVKM_ENGINE_MSPPP; break;
+       case 3: engn = NVKM_ENGINE_MSPDEC; break;
+       case 4: engn = NVKM_ENGINE_CE0; break;
+       case 5: engn = NVKM_ENGINE_CE1; break;
        default:
                return NULL;
        }
 
-       return nvkm_engine(priv, engn);
+       return nvkm_device_engine(device, engn);
 }
 
 static void
 gf100_fifo_recover_work(struct work_struct *work)
 {
-       struct gf100_fifo_priv *priv = container_of(work, typeof(*priv), fault);
-       struct nvkm_object *engine;
+       struct gf100_fifo *fifo = container_of(work, typeof(*fifo), fault);
+       struct nvkm_device *device = fifo->base.engine.subdev.device;
+       struct nvkm_engine *engine;
        unsigned long flags;
        u32 engn, engm = 0;
        u64 mask, todo;
 
-       spin_lock_irqsave(&priv->base.lock, flags);
-       mask = priv->mask;
-       priv->mask = 0ULL;
-       spin_unlock_irqrestore(&priv->base.lock, flags);
+       spin_lock_irqsave(&fifo->base.lock, flags);
+       mask = fifo->mask;
+       fifo->mask = 0ULL;
+       spin_unlock_irqrestore(&fifo->base.lock, flags);
 
        for (todo = mask; engn = __ffs64(todo), todo; todo &= ~(1 << engn))
-               engm |= 1 << gf100_fifo_engidx(priv, engn);
-       nv_mask(priv, 0x002630, engm, engm);
+               engm |= 1 << gf100_fifo_engidx(fifo, engn);
+       nvkm_mask(device, 0x002630, engm, engm);
 
        for (todo = mask; engn = __ffs64(todo), todo; todo &= ~(1 << engn)) {
-               if ((engine = (void *)nvkm_engine(priv, engn))) {
-                       nv_ofuncs(engine)->fini(engine, false);
-                       WARN_ON(nv_ofuncs(engine)->init(engine));
+               if ((engine = nvkm_device_engine(device, engn))) {
+                       nvkm_subdev_fini(&engine->subdev, false);
+                       WARN_ON(nvkm_subdev_init(&engine->subdev));
                }
        }
 
-       gf100_fifo_runlist_update(priv);
-       nv_wr32(priv, 0x00262c, engm);
-       nv_mask(priv, 0x002630, engm, 0x00000000);
+       gf100_fifo_runlist_update(fifo);
+       nvkm_wr32(device, 0x00262c, engm);
+       nvkm_mask(device, 0x002630, engm, 0x00000000);
 }
 
 static void
-gf100_fifo_recover(struct gf100_fifo_priv *priv, struct nvkm_engine *engine,
+gf100_fifo_recover(struct gf100_fifo *fifo, struct nvkm_engine *engine,
                   struct gf100_fifo_chan *chan)
 {
+       struct nvkm_subdev *subdev = &fifo->base.engine.subdev;
+       struct nvkm_device *device = subdev->device;
        u32 chid = chan->base.chid;
-       unsigned long flags;
 
-       nv_error(priv, "%s engine fault on channel %d, recovering...\n",
-                      nv_subdev(engine)->name, chid);
+       nvkm_error(subdev, "%s engine fault on channel %d, recovering...\n",
+                  nvkm_subdev_name[engine->subdev.index], chid);
+       assert_spin_locked(&fifo->base.lock);
 
-       nv_mask(priv, 0x003004 + (chid * 0x08), 0x00000001, 0x00000000);
-       chan->state = KILLED;
+       nvkm_mask(device, 0x003004 + (chid * 0x08), 0x00000001, 0x00000000);
+       list_del_init(&chan->head);
+       chan->killed = true;
 
-       spin_lock_irqsave(&priv->base.lock, flags);
-       priv->mask |= 1ULL << nv_engidx(engine);
-       spin_unlock_irqrestore(&priv->base.lock, flags);
-       schedule_work(&priv->fault);
-}
-
-static int
-gf100_fifo_swmthd(struct gf100_fifo_priv *priv, u32 chid, u32 mthd, u32 data)
-{
-       struct gf100_fifo_chan *chan = NULL;
-       struct nvkm_handle *bind;
-       unsigned long flags;
-       int ret = -EINVAL;
-
-       spin_lock_irqsave(&priv->base.lock, flags);
-       if (likely(chid >= priv->base.min && chid <= priv->base.max))
-               chan = (void *)priv->base.channel[chid];
-       if (unlikely(!chan))
-               goto out;
-
-       bind = nvkm_namedb_get_class(nv_namedb(chan), 0x906e);
-       if (likely(bind)) {
-               if (!mthd || !nv_call(bind->object, mthd, data))
-                       ret = 0;
-               nvkm_namedb_put(bind);
-       }
-
-out:
-       spin_unlock_irqrestore(&priv->base.lock, flags);
-       return ret;
+       fifo->mask |= 1ULL << engine->subdev.index;
+       schedule_work(&fifo->fault);
 }
 
 static const struct nvkm_enum
@@ -486,14 +171,17 @@ gf100_fifo_sched_reason[] = {
 };
 
 static void
-gf100_fifo_intr_sched_ctxsw(struct gf100_fifo_priv *priv)
+gf100_fifo_intr_sched_ctxsw(struct gf100_fifo *fifo)
 {
+       struct nvkm_device *device = fifo->base.engine.subdev.device;
        struct nvkm_engine *engine;
        struct gf100_fifo_chan *chan;
+       unsigned long flags;
        u32 engn;
 
+       spin_lock_irqsave(&fifo->base.lock, flags);
        for (engn = 0; engn < 6; engn++) {
-               u32 stat = nv_rd32(priv, 0x002640 + (engn * 0x04));
+               u32 stat = nvkm_rd32(device, 0x002640 + (engn * 0x04));
                u32 busy = (stat & 0x80000000);
                u32 save = (stat & 0x00100000); /* maybe? */
                u32 unk0 = (stat & 0x00040000);
@@ -502,32 +190,36 @@ gf100_fifo_intr_sched_ctxsw(struct gf100_fifo_priv *priv)
                (void)save;
 
                if (busy && unk0 && unk1) {
-                       if (!(chan = (void *)priv->base.channel[chid]))
-                               continue;
-                       if (!(engine = gf100_fifo_engine(priv, engn)))
-                               continue;
-                       gf100_fifo_recover(priv, engine, chan);
+                       list_for_each_entry(chan, &fifo->chan, head) {
+                               if (chan->base.chid == chid) {
+                                       engine = gf100_fifo_engine(fifo, engn);
+                                       if (!engine)
+                                               break;
+                                       gf100_fifo_recover(fifo, engine, chan);
+                                       break;
+                               }
+                       }
                }
        }
+       spin_unlock_irqrestore(&fifo->base.lock, flags);
 }
 
 static void
-gf100_fifo_intr_sched(struct gf100_fifo_priv *priv)
+gf100_fifo_intr_sched(struct gf100_fifo *fifo)
 {
-       u32 intr = nv_rd32(priv, 0x00254c);
+       struct nvkm_subdev *subdev = &fifo->base.engine.subdev;
+       struct nvkm_device *device = subdev->device;
+       u32 intr = nvkm_rd32(device, 0x00254c);
        u32 code = intr & 0x000000ff;
        const struct nvkm_enum *en;
-       char enunk[6] = "";
 
        en = nvkm_enum_find(gf100_fifo_sched_reason, code);
-       if (!en)
-               snprintf(enunk, sizeof(enunk), "UNK%02x", code);
 
-       nv_error(priv, "SCHED_ERROR [ %s ]\n", en ? en->name : enunk);
+       nvkm_error(subdev, "SCHED_ERROR %02x [%s]\n", code, en ? en->name : "");
 
        switch (code) {
        case 0x0a:
-               gf100_fifo_intr_sched_ctxsw(priv);
+               gf100_fifo_intr_sched_ctxsw(fifo);
                break;
        default:
                break;
@@ -536,17 +228,17 @@ gf100_fifo_intr_sched(struct gf100_fifo_priv *priv)
 
 static const struct nvkm_enum
 gf100_fifo_fault_engine[] = {
-       { 0x00, "PGRAPH", NULL, NVDEV_ENGINE_GR },
-       { 0x03, "PEEPHOLE", NULL, NVDEV_ENGINE_IFB },
-       { 0x04, "BAR1", NULL, NVDEV_SUBDEV_BAR },
-       { 0x05, "BAR3", NULL, NVDEV_SUBDEV_INSTMEM },
-       { 0x07, "PFIFO", NULL, NVDEV_ENGINE_FIFO },
-       { 0x10, "PMSVLD", NULL, NVDEV_ENGINE_MSVLD },
-       { 0x11, "PMSPPP", NULL, NVDEV_ENGINE_MSPPP },
+       { 0x00, "PGRAPH", NULL, NVKM_ENGINE_GR },
+       { 0x03, "PEEPHOLE", NULL, NVKM_ENGINE_IFB },
+       { 0x04, "BAR1", NULL, NVKM_SUBDEV_BAR },
+       { 0x05, "BAR3", NULL, NVKM_SUBDEV_INSTMEM },
+       { 0x07, "PFIFO", NULL, NVKM_ENGINE_FIFO },
+       { 0x10, "PMSVLD", NULL, NVKM_ENGINE_MSVLD },
+       { 0x11, "PMSPPP", NULL, NVKM_ENGINE_MSPPP },
        { 0x13, "PCOUNTER" },
-       { 0x14, "PMSPDEC", NULL, NVDEV_ENGINE_MSPDEC },
-       { 0x15, "PCE0", NULL, NVDEV_ENGINE_CE0 },
-       { 0x16, "PCE1", NULL, NVDEV_ENGINE_CE1 },
+       { 0x14, "PMSPDEC", NULL, NVKM_ENGINE_MSPDEC },
+       { 0x15, "PCE0", NULL, NVKM_ENGINE_CE0 },
+       { 0x16, "PCE1", NULL, NVKM_ENGINE_CE1 },
        { 0x17, "PDAEMON" },
        {}
 };
@@ -594,79 +286,65 @@ gf100_fifo_fault_gpcclient[] = {
 };
 
 static void
-gf100_fifo_intr_fault(struct gf100_fifo_priv *priv, int unit)
+gf100_fifo_intr_fault(struct gf100_fifo *fifo, int unit)
 {
-       u32 inst = nv_rd32(priv, 0x002800 + (unit * 0x10));
-       u32 valo = nv_rd32(priv, 0x002804 + (unit * 0x10));
-       u32 vahi = nv_rd32(priv, 0x002808 + (unit * 0x10));
-       u32 stat = nv_rd32(priv, 0x00280c + (unit * 0x10));
+       struct nvkm_subdev *subdev = &fifo->base.engine.subdev;
+       struct nvkm_device *device = subdev->device;
+       u32 inst = nvkm_rd32(device, 0x002800 + (unit * 0x10));
+       u32 valo = nvkm_rd32(device, 0x002804 + (unit * 0x10));
+       u32 vahi = nvkm_rd32(device, 0x002808 + (unit * 0x10));
+       u32 stat = nvkm_rd32(device, 0x00280c + (unit * 0x10));
        u32 gpc    = (stat & 0x1f000000) >> 24;
        u32 client = (stat & 0x00001f00) >> 8;
        u32 write  = (stat & 0x00000080);
        u32 hub    = (stat & 0x00000040);
        u32 reason = (stat & 0x0000000f);
-       struct nvkm_object *engctx = NULL, *object;
-       struct nvkm_engine *engine = NULL;
        const struct nvkm_enum *er, *eu, *ec;
-       char erunk[6] = "";
-       char euunk[6] = "";
-       char ecunk[6] = "";
-       char gpcid[3] = "";
+       struct nvkm_engine *engine = NULL;
+       struct nvkm_fifo_chan *chan;
+       unsigned long flags;
+       char gpcid[8] = "";
 
        er = nvkm_enum_find(gf100_fifo_fault_reason, reason);
-       if (!er)
-               snprintf(erunk, sizeof(erunk), "UNK%02X", reason);
-
        eu = nvkm_enum_find(gf100_fifo_fault_engine, unit);
+       if (hub) {
+               ec = nvkm_enum_find(gf100_fifo_fault_hubclient, client);
+       } else {
+               ec = nvkm_enum_find(gf100_fifo_fault_gpcclient, client);
+               snprintf(gpcid, sizeof(gpcid), "GPC%d/", gpc);
+       }
+
        if (eu) {
                switch (eu->data2) {
-               case NVDEV_SUBDEV_BAR:
-                       nv_mask(priv, 0x001704, 0x00000000, 0x00000000);
+               case NVKM_SUBDEV_BAR:
+                       nvkm_mask(device, 0x001704, 0x00000000, 0x00000000);
                        break;
-               case NVDEV_SUBDEV_INSTMEM:
-                       nv_mask(priv, 0x001714, 0x00000000, 0x00000000);
+               case NVKM_SUBDEV_INSTMEM:
+                       nvkm_mask(device, 0x001714, 0x00000000, 0x00000000);
                        break;
-               case NVDEV_ENGINE_IFB:
-                       nv_mask(priv, 0x001718, 0x00000000, 0x00000000);
+               case NVKM_ENGINE_IFB:
+                       nvkm_mask(device, 0x001718, 0x00000000, 0x00000000);
                        break;
                default:
-                       engine = nvkm_engine(priv, eu->data2);
-                       if (engine)
-                               engctx = nvkm_engctx_get(engine, inst);
+                       engine = nvkm_device_engine(device, eu->data2);
                        break;
                }
-       } else {
-               snprintf(euunk, sizeof(euunk), "UNK%02x", unit);
        }
 
-       if (hub) {
-               ec = nvkm_enum_find(gf100_fifo_fault_hubclient, client);
-       } else {
-               ec = nvkm_enum_find(gf100_fifo_fault_gpcclient, client);
-               snprintf(gpcid, sizeof(gpcid), "%d", gpc);
-       }
+       chan = nvkm_fifo_chan_inst(&fifo->base, (u64)inst << 12, &flags);
 
-       if (!ec)
-               snprintf(ecunk, sizeof(ecunk), "UNK%02x", client);
-
-       nv_error(priv, "%s fault at 0x%010llx [%s] from %s/%s%s%s%s on "
-                      "channel 0x%010llx [%s]\n", write ? "write" : "read",
-                (u64)vahi << 32 | valo, er ? er->name : erunk,
-                eu ? eu->name : euunk, hub ? "" : "GPC", gpcid, hub ? "" : "/",
-                ec ? ec->name : ecunk, (u64)inst << 12,
-                nvkm_client_name(engctx));
-
-       object = engctx;
-       while (object) {
-               switch (nv_mclass(object)) {
-               case FERMI_CHANNEL_GPFIFO:
-                       gf100_fifo_recover(priv, engine, (void *)object);
-                       break;
-               }
-               object = object->parent;
-       }
+       nvkm_error(subdev,
+                  "%s fault at %010llx engine %02x [%s] client %02x [%s%s] "
+                  "reason %02x [%s] on channel %d [%010llx %s]\n",
+                  write ? "write" : "read", (u64)vahi << 32 | valo,
+                  unit, eu ? eu->name : "", client, gpcid, ec ? ec->name : "",
+                  reason, er ? er->name : "", chan ? chan->chid : -1,
+                  (u64)inst << 12,
+                  chan ? chan->object.client->name : "unknown");
 
-       nvkm_engctx_put(engctx);
+       if (engine && chan)
+               gf100_fifo_recover(fifo, engine, (void *)chan);
+       nvkm_fifo_chan_put(&fifo->base, flags, &chan);
 }
 
 static const struct nvkm_bitfield
@@ -678,290 +356,288 @@ gf100_fifo_pbdma_intr[] = {
 };
 
 static void
-gf100_fifo_intr_pbdma(struct gf100_fifo_priv *priv, int unit)
+gf100_fifo_intr_pbdma(struct gf100_fifo *fifo, int unit)
 {
-       u32 stat = nv_rd32(priv, 0x040108 + (unit * 0x2000));
-       u32 addr = nv_rd32(priv, 0x0400c0 + (unit * 0x2000));
-       u32 data = nv_rd32(priv, 0x0400c4 + (unit * 0x2000));
-       u32 chid = nv_rd32(priv, 0x040120 + (unit * 0x2000)) & 0x7f;
+       struct nvkm_subdev *subdev = &fifo->base.engine.subdev;
+       struct nvkm_device *device = subdev->device;
+       u32 stat = nvkm_rd32(device, 0x040108 + (unit * 0x2000));
+       u32 addr = nvkm_rd32(device, 0x0400c0 + (unit * 0x2000));
+       u32 data = nvkm_rd32(device, 0x0400c4 + (unit * 0x2000));
+       u32 chid = nvkm_rd32(device, 0x040120 + (unit * 0x2000)) & 0x7f;
        u32 subc = (addr & 0x00070000) >> 16;
        u32 mthd = (addr & 0x00003ffc);
-       u32 show = stat;
+       struct nvkm_fifo_chan *chan;
+       unsigned long flags;
+       u32 show= stat;
+       char msg[128];
 
        if (stat & 0x00800000) {
-               if (!gf100_fifo_swmthd(priv, chid, mthd, data))
-                       show &= ~0x00800000;
+               if (device->sw) {
+                       if (nvkm_sw_mthd(device->sw, chid, subc, mthd, data))
+                               show &= ~0x00800000;
+               }
        }
 
        if (show) {
-               nv_error(priv, "PBDMA%d:", unit);
-               nvkm_bitfield_print(gf100_fifo_pbdma_intr, show);
-               pr_cont("\n");
-               nv_error(priv,
-                        "PBDMA%d: ch %d [%s] subc %d mthd 0x%04x data 0x%08x\n",
-                        unit, chid,
-                        nvkm_client_name_for_fifo_chid(&priv->base, chid),
-                        subc, mthd, data);
-       }
-
-       nv_wr32(priv, 0x0400c0 + (unit * 0x2000), 0x80600008);
-       nv_wr32(priv, 0x040108 + (unit * 0x2000), stat);
+               nvkm_snprintbf(msg, sizeof(msg), gf100_fifo_pbdma_intr, show);
+               chan = nvkm_fifo_chan_chid(&fifo->base, chid, &flags);
+               nvkm_error(subdev, "PBDMA%d: %08x [%s] ch %d [%010llx %s] "
+                                  "subc %d mthd %04x data %08x\n",
+                          unit, show, msg, chid, chan ? chan->inst->addr : 0,
+                          chan ? chan->object.client->name : "unknown",
+                          subc, mthd, data);
+               nvkm_fifo_chan_put(&fifo->base, flags, &chan);
+       }
+
+       nvkm_wr32(device, 0x0400c0 + (unit * 0x2000), 0x80600008);
+       nvkm_wr32(device, 0x040108 + (unit * 0x2000), stat);
 }
 
 static void
-gf100_fifo_intr_runlist(struct gf100_fifo_priv *priv)
+gf100_fifo_intr_runlist(struct gf100_fifo *fifo)
 {
-       u32 intr = nv_rd32(priv, 0x002a00);
+       struct nvkm_subdev *subdev = &fifo->base.engine.subdev;
+       struct nvkm_device *device = subdev->device;
+       u32 intr = nvkm_rd32(device, 0x002a00);
 
        if (intr & 0x10000000) {
-               wake_up(&priv->runlist.wait);
-               nv_wr32(priv, 0x002a00, 0x10000000);
+               wake_up(&fifo->runlist.wait);
+               nvkm_wr32(device, 0x002a00, 0x10000000);
                intr &= ~0x10000000;
        }
 
        if (intr) {
-               nv_error(priv, "RUNLIST 0x%08x\n", intr);
-               nv_wr32(priv, 0x002a00, intr);
+               nvkm_error(subdev, "RUNLIST %08x\n", intr);
+               nvkm_wr32(device, 0x002a00, intr);
        }
 }
 
 static void
-gf100_fifo_intr_engine_unit(struct gf100_fifo_priv *priv, int engn)
+gf100_fifo_intr_engine_unit(struct gf100_fifo *fifo, int engn)
 {
-       u32 intr = nv_rd32(priv, 0x0025a8 + (engn * 0x04));
-       u32 inte = nv_rd32(priv, 0x002628);
+       struct nvkm_subdev *subdev = &fifo->base.engine.subdev;
+       struct nvkm_device *device = subdev->device;
+       u32 intr = nvkm_rd32(device, 0x0025a8 + (engn * 0x04));
+       u32 inte = nvkm_rd32(device, 0x002628);
        u32 unkn;
 
-       nv_wr32(priv, 0x0025a8 + (engn * 0x04), intr);
+       nvkm_wr32(device, 0x0025a8 + (engn * 0x04), intr);
 
        for (unkn = 0; unkn < 8; unkn++) {
                u32 ints = (intr >> (unkn * 0x04)) & inte;
                if (ints & 0x1) {
-                       nvkm_fifo_uevent(&priv->base);
+                       nvkm_fifo_uevent(&fifo->base);
                        ints &= ~1;
                }
                if (ints) {
-                       nv_error(priv, "ENGINE %d %d %01x", engn, unkn, ints);
-                       nv_mask(priv, 0x002628, ints, 0);
+                       nvkm_error(subdev, "ENGINE %d %d %01x",
+                                  engn, unkn, ints);
+                       nvkm_mask(device, 0x002628, ints, 0);
                }
        }
 }
 
-static void
-gf100_fifo_intr_engine(struct gf100_fifo_priv *priv)
+void
+gf100_fifo_intr_engine(struct gf100_fifo *fifo)
 {
-       u32 mask = nv_rd32(priv, 0x0025a4);
+       struct nvkm_device *device = fifo->base.engine.subdev.device;
+       u32 mask = nvkm_rd32(device, 0x0025a4);
        while (mask) {
                u32 unit = __ffs(mask);
-               gf100_fifo_intr_engine_unit(priv, unit);
+               gf100_fifo_intr_engine_unit(fifo, unit);
                mask &= ~(1 << unit);
        }
 }
 
 static void
-gf100_fifo_intr(struct nvkm_subdev *subdev)
+gf100_fifo_intr(struct nvkm_fifo *base)
 {
-       struct gf100_fifo_priv *priv = (void *)subdev;
-       u32 mask = nv_rd32(priv, 0x002140);
-       u32 stat = nv_rd32(priv, 0x002100) & mask;
+       struct gf100_fifo *fifo = gf100_fifo(base);
+       struct nvkm_subdev *subdev = &fifo->base.engine.subdev;
+       struct nvkm_device *device = subdev->device;
+       u32 mask = nvkm_rd32(device, 0x002140);
+       u32 stat = nvkm_rd32(device, 0x002100) & mask;
 
        if (stat & 0x00000001) {
-               u32 intr = nv_rd32(priv, 0x00252c);
-               nv_warn(priv, "INTR 0x00000001: 0x%08x\n", intr);
-               nv_wr32(priv, 0x002100, 0x00000001);
+               u32 intr = nvkm_rd32(device, 0x00252c);
+               nvkm_warn(subdev, "INTR 00000001: %08x\n", intr);
+               nvkm_wr32(device, 0x002100, 0x00000001);
                stat &= ~0x00000001;
        }
 
        if (stat & 0x00000100) {
-               gf100_fifo_intr_sched(priv);
-               nv_wr32(priv, 0x002100, 0x00000100);
+               gf100_fifo_intr_sched(fifo);
+               nvkm_wr32(device, 0x002100, 0x00000100);
                stat &= ~0x00000100;
        }
 
        if (stat & 0x00010000) {
-               u32 intr = nv_rd32(priv, 0x00256c);
-               nv_warn(priv, "INTR 0x00010000: 0x%08x\n", intr);
-               nv_wr32(priv, 0x002100, 0x00010000);
+               u32 intr = nvkm_rd32(device, 0x00256c);
+               nvkm_warn(subdev, "INTR 00010000: %08x\n", intr);
+               nvkm_wr32(device, 0x002100, 0x00010000);
                stat &= ~0x00010000;
        }
 
        if (stat & 0x01000000) {
-               u32 intr = nv_rd32(priv, 0x00258c);
-               nv_warn(priv, "INTR 0x01000000: 0x%08x\n", intr);
-               nv_wr32(priv, 0x002100, 0x01000000);
+               u32 intr = nvkm_rd32(device, 0x00258c);
+               nvkm_warn(subdev, "INTR 01000000: %08x\n", intr);
+               nvkm_wr32(device, 0x002100, 0x01000000);
                stat &= ~0x01000000;
        }
 
        if (stat & 0x10000000) {
-               u32 mask = nv_rd32(priv, 0x00259c);
+               u32 mask = nvkm_rd32(device, 0x00259c);
                while (mask) {
                        u32 unit = __ffs(mask);
-                       gf100_fifo_intr_fault(priv, unit);
-                       nv_wr32(priv, 0x00259c, (1 << unit));
+                       gf100_fifo_intr_fault(fifo, unit);
+                       nvkm_wr32(device, 0x00259c, (1 << unit));
                        mask &= ~(1 << unit);
                }
                stat &= ~0x10000000;
        }
 
        if (stat & 0x20000000) {
-               u32 mask = nv_rd32(priv, 0x0025a0);
+               u32 mask = nvkm_rd32(device, 0x0025a0);
                while (mask) {
                        u32 unit = __ffs(mask);
-                       gf100_fifo_intr_pbdma(priv, unit);
-                       nv_wr32(priv, 0x0025a0, (1 << unit));
+                       gf100_fifo_intr_pbdma(fifo, unit);
+                       nvkm_wr32(device, 0x0025a0, (1 << unit));
                        mask &= ~(1 << unit);
                }
                stat &= ~0x20000000;
        }
 
        if (stat & 0x40000000) {
-               gf100_fifo_intr_runlist(priv);
+               gf100_fifo_intr_runlist(fifo);
                stat &= ~0x40000000;
        }
 
        if (stat & 0x80000000) {
-               gf100_fifo_intr_engine(priv);
+               gf100_fifo_intr_engine(fifo);
                stat &= ~0x80000000;
        }
 
        if (stat) {
-               nv_error(priv, "INTR 0x%08x\n", stat);
-               nv_mask(priv, 0x002140, stat, 0x00000000);
-               nv_wr32(priv, 0x002100, stat);
+               nvkm_error(subdev, "INTR %08x\n", stat);
+               nvkm_mask(device, 0x002140, stat, 0x00000000);
+               nvkm_wr32(device, 0x002100, stat);
        }
 }
 
-static void
-gf100_fifo_uevent_init(struct nvkm_event *event, int type, int index)
-{
-       struct nvkm_fifo *fifo = container_of(event, typeof(*fifo), uevent);
-       nv_mask(fifo, 0x002140, 0x80000000, 0x80000000);
-}
-
-static void
-gf100_fifo_uevent_fini(struct nvkm_event *event, int type, int index)
-{
-       struct nvkm_fifo *fifo = container_of(event, typeof(*fifo), uevent);
-       nv_mask(fifo, 0x002140, 0x80000000, 0x00000000);
-}
-
-static const struct nvkm_event_func
-gf100_fifo_uevent_func = {
-       .ctor = nvkm_fifo_uevent_ctor,
-       .init = gf100_fifo_uevent_init,
-       .fini = gf100_fifo_uevent_fini,
-};
-
 static int
-gf100_fifo_ctor(struct nvkm_object *parent, struct nvkm_object *engine,
-               struct nvkm_oclass *oclass, void *data, u32 size,
-               struct nvkm_object **pobject)
+gf100_fifo_oneinit(struct nvkm_fifo *base)
 {
-       struct gf100_fifo_priv *priv;
+       struct gf100_fifo *fifo = gf100_fifo(base);
+       struct nvkm_device *device = fifo->base.engine.subdev.device;
        int ret;
 
-       ret = nvkm_fifo_create(parent, engine, oclass, 0, 127, &priv);
-       *pobject = nv_object(priv);
-       if (ret)
-               return ret;
-
-       INIT_WORK(&priv->fault, gf100_fifo_recover_work);
-
-       ret = nvkm_gpuobj_new(nv_object(priv), NULL, 0x1000, 0x1000, 0,
-                             &priv->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(priv), NULL, 0x1000, 0x1000, 0,
-                             &priv->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(&priv->runlist.wait);
+       init_waitqueue_head(&fifo->runlist.wait);
 
-       ret = nvkm_gpuobj_new(nv_object(priv), NULL, 128 * 0x1000, 0x1000, 0,
-                             &priv->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(priv->user.mem, NV_MEM_ACCESS_RW,
-                             &priv->user.bar);
+       ret = nvkm_bar_umap(device->bar, 128 * 0x1000, 12, &fifo->user.bar);
        if (ret)
                return ret;
 
-       ret = nvkm_event_init(&gf100_fifo_uevent_func, 1, 1, &priv->base.uevent);
-       if (ret)
-               return ret;
-
-       nv_subdev(priv)->unit = 0x00000100;
-       nv_subdev(priv)->intr = gf100_fifo_intr;
-       nv_engine(priv)->cclass = &gf100_fifo_cclass;
-       nv_engine(priv)->sclass = gf100_fifo_sclass;
+       nvkm_memory_map(fifo->user.mem, &fifo->user.bar, 0);
        return 0;
 }
 
 static void
-gf100_fifo_dtor(struct nvkm_object *object)
+gf100_fifo_fini(struct nvkm_fifo *base)
 {
-       struct gf100_fifo_priv *priv = (void *)object;
-
-       nvkm_gpuobj_unmap(&priv->user.bar);
-       nvkm_gpuobj_ref(NULL, &priv->user.mem);
-       nvkm_gpuobj_ref(NULL, &priv->runlist.mem[0]);
-       nvkm_gpuobj_ref(NULL, &priv->runlist.mem[1]);
-
-       nvkm_fifo_destroy(&priv->base);
+       struct gf100_fifo *fifo = gf100_fifo(base);
+       flush_work(&fifo->fault);
 }
 
-static int
-gf100_fifo_init(struct nvkm_object *object)
+static void
+gf100_fifo_init(struct nvkm_fifo *base)
 {
-       struct gf100_fifo_priv *priv = (void *)object;
-       int ret, i;
-
-       ret = nvkm_fifo_init(&priv->base);
-       if (ret)
-               return ret;
+       struct gf100_fifo *fifo = gf100_fifo(base);
+       struct nvkm_subdev *subdev = &fifo->base.engine.subdev;
+       struct nvkm_device *device = subdev->device;
+       int i;
 
-       nv_wr32(priv, 0x000204, 0xffffffff);
-       nv_wr32(priv, 0x002204, 0xffffffff);
+       nvkm_wr32(device, 0x000204, 0xffffffff);
+       nvkm_wr32(device, 0x002204, 0xffffffff);
 
-       priv->spoon_nr = hweight32(nv_rd32(priv, 0x002204));
-       nv_debug(priv, "%d PBDMA unit(s)\n", priv->spoon_nr);
+       fifo->spoon_nr = hweight32(nvkm_rd32(device, 0x002204));
+       nvkm_debug(subdev, "%d PBDMA unit(s)\n", fifo->spoon_nr);
 
        /* assign engines to PBDMAs */
-       if (priv->spoon_nr >= 3) {
-               nv_wr32(priv, 0x002208, ~(1 << 0)); /* PGRAPH */
-               nv_wr32(priv, 0x00220c, ~(1 << 1)); /* PVP */
-               nv_wr32(priv, 0x002210, ~(1 << 1)); /* PMSPP */
-               nv_wr32(priv, 0x002214, ~(1 << 1)); /* PMSVLD */
-               nv_wr32(priv, 0x002218, ~(1 << 2)); /* PCE0 */
-               nv_wr32(priv, 0x00221c, ~(1 << 1)); /* PCE1 */
+       if (fifo->spoon_nr >= 3) {
+               nvkm_wr32(device, 0x002208, ~(1 << 0)); /* PGRAPH */
+               nvkm_wr32(device, 0x00220c, ~(1 << 1)); /* PVP */
+               nvkm_wr32(device, 0x002210, ~(1 << 1)); /* PMSPP */
+               nvkm_wr32(device, 0x002214, ~(1 << 1)); /* PMSVLD */
+               nvkm_wr32(device, 0x002218, ~(1 << 2)); /* PCE0 */
+               nvkm_wr32(device, 0x00221c, ~(1 << 1)); /* PCE1 */
        }
 
        /* PBDMA[n] */
-       for (i = 0; i < priv->spoon_nr; i++) {
-               nv_mask(priv, 0x04013c + (i * 0x2000), 0x10000100, 0x00000000);
-               nv_wr32(priv, 0x040108 + (i * 0x2000), 0xffffffff); /* INTR */
-               nv_wr32(priv, 0x04010c + (i * 0x2000), 0xfffffeff); /* INTREN */
+       for (i = 0; i < fifo->spoon_nr; i++) {
+               nvkm_mask(device, 0x04013c + (i * 0x2000), 0x10000100, 0x00000000);
+               nvkm_wr32(device, 0x040108 + (i * 0x2000), 0xffffffff); /* INTR */
+               nvkm_wr32(device, 0x04010c + (i * 0x2000), 0xfffffeff); /* INTREN */
        }
 
-       nv_mask(priv, 0x002200, 0x00000001, 0x00000001);
-       nv_wr32(priv, 0x002254, 0x10000000 | priv->user.bar.offset >> 12);
+       nvkm_mask(device, 0x002200, 0x00000001, 0x00000001);
+       nvkm_wr32(device, 0x002254, 0x10000000 | fifo->user.bar.offset >> 12);
 
-       nv_wr32(priv, 0x002100, 0xffffffff);
-       nv_wr32(priv, 0x002140, 0x7fffffff);
-       nv_wr32(priv, 0x002628, 0x00000001); /* ENGINE_INTR_EN */
-       return 0;
+       nvkm_wr32(device, 0x002100, 0xffffffff);
+       nvkm_wr32(device, 0x002140, 0x7fffffff);
+       nvkm_wr32(device, 0x002628, 0x00000001); /* ENGINE_INTR_EN */
+}
+
+static void *
+gf100_fifo_dtor(struct nvkm_fifo *base)
+{
+       struct gf100_fifo *fifo = gf100_fifo(base);
+       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]);
+       return fifo;
 }
 
-struct nvkm_oclass *
-gf100_fifo_oclass = &(struct nvkm_oclass) {
-       .handle = NV_ENGINE(FIFO, 0xc0),
-       .ofuncs = &(struct nvkm_ofuncs) {
-               .ctor = gf100_fifo_ctor,
-               .dtor = gf100_fifo_dtor,
-               .init = gf100_fifo_init,
-               .fini = _nvkm_fifo_fini,
+static const struct nvkm_fifo_func
+gf100_fifo = {
+       .dtor = gf100_fifo_dtor,
+       .oneinit = gf100_fifo_oneinit,
+       .init = gf100_fifo_init,
+       .fini = gf100_fifo_fini,
+       .intr = gf100_fifo_intr,
+       .uevent_init = gf100_fifo_uevent_init,
+       .uevent_fini = gf100_fifo_uevent_fini,
+       .chan = {
+               &gf100_fifo_gpfifo_oclass,
+               NULL
        },
 };
+
+int
+gf100_fifo_new(struct nvkm_device *device, int index, struct nvkm_fifo **pfifo)
+{
+       struct gf100_fifo *fifo;
+
+       if (!(fifo = kzalloc(sizeof(*fifo), GFP_KERNEL)))
+               return -ENOMEM;
+       INIT_LIST_HEAD(&fifo->chan);
+       INIT_WORK(&fifo->fault, gf100_fifo_recover_work);
+       *pfifo = &fifo->base;
+
+       return nvkm_fifo_ctor(&gf100_fifo, device, index, 128, &fifo->base);
+}