drm/nouveau/fifo: expose non-stall intr in host channel event class

Signed-off-by: Ben Skeggs <bskeggs@redhat.com>
Reviewed-by: Lyude Paul <lyude@redhat.com>

diff --git a/drivers/gpu/drm/nouveau/include/nvif/cl826e.h b/drivers/gpu/drm/nouveau/include/nvif/cl826e.h
index 1b6496d3158031d45cd3b7d3d75d2711350c7595..ef3033b836f0e3867e0ba32176cec35699c0231b 100644 (file)
--- a/drivers/gpu/drm/nouveau/include/nvif/cl826e.h
+++ b/drivers/gpu/drm/nouveau/include/nvif/cl826e.h
@@ -10,6 +10,4 @@ struct g82_channel_dma_v0 {
        __u64 pushbuf;
        __u64 offset;
 };
-
-#define NV826E_V0_NTFY_NON_STALL_INTERRUPT                                 0x00
 #endif

diff --git a/drivers/gpu/drm/nouveau/include/nvif/cl826f.h b/drivers/gpu/drm/nouveau/include/nvif/cl826f.h
index 148602264a764b9111b0bae6dc0a0c191624098a..c5d5e56b04cc00b5a718d7f853001c9d3260eedd 100644 (file)
--- a/drivers/gpu/drm/nouveau/include/nvif/cl826f.h
+++ b/drivers/gpu/drm/nouveau/include/nvif/cl826f.h
@@ -11,6 +11,4 @@ struct g82_channel_gpfifo_v0 {
        __u64 pushbuf;
        __u64 vmm;
 };
-
-#define NV826F_V0_NTFY_NON_STALL_INTERRUPT                                 0x00
 #endif

diff --git a/drivers/gpu/drm/nouveau/include/nvif/cl906f.h b/drivers/gpu/drm/nouveau/include/nvif/cl906f.h
index 3823d6891b5560a588f83b70d47d0a65f023b21d..0e17b21e8c6eee531e7ec1416713778dd618a8ee 100644 (file)
--- a/drivers/gpu/drm/nouveau/include/nvif/cl906f.h
+++ b/drivers/gpu/drm/nouveau/include/nvif/cl906f.h
@@ -11,6 +11,5 @@ struct fermi_channel_gpfifo_v0 {
        __u64 vmm;
 };
 
-#define NV906F_V0_NTFY_NON_STALL_INTERRUPT                                 0x00
 #define NV906F_V0_NTFY_KILLED                                              0x01
 #endif

diff --git a/drivers/gpu/drm/nouveau/include/nvif/cla06f.h b/drivers/gpu/drm/nouveau/include/nvif/cla06f.h
index cfa18f1fbf83818afd33cdd3f09b8f9e6a6eb164..00057ddfa41f2a9c29476092f96c8cc154c88afd 100644 (file)
--- a/drivers/gpu/drm/nouveau/include/nvif/cla06f.h
+++ b/drivers/gpu/drm/nouveau/include/nvif/cla06f.h
@@ -13,6 +13,5 @@ struct kepler_channel_gpfifo_a_v0 {
        __u64 inst;
 };
 
-#define NVA06F_V0_NTFY_NON_STALL_INTERRUPT                                 0x00
 #define NVA06F_V0_NTFY_KILLED                                              0x01
 #endif

diff --git a/drivers/gpu/drm/nouveau/include/nvif/class.h b/drivers/gpu/drm/nouveau/include/nvif/class.h
index 29448f77069b7c1a9e19c58a77d00ad59b45ad2a..3e9d181b5dda1e7ddcc5f58a9214503a6892fc4c 100644 (file)
--- a/drivers/gpu/drm/nouveau/include/nvif/class.h
+++ b/drivers/gpu/drm/nouveau/include/nvif/class.h
@@ -40,6 +40,8 @@
 #define NVIF_CLASS_HEAD                              /* if0013.h */  0x80000013
 #define NVIF_CLASS_DISP_CHAN                         /* if0014.h */  0x80000014
 
+#define NVIF_CLASS_CHAN                              /* if0020.h */  0x80000020
+
 /* the below match nvidia-assigned (either in hw, or sw) class numbers */
 #define NV_NULL_CLASS                                                0x00000030
 

diff --git a/drivers/gpu/drm/nouveau/include/nvif/clc36f.h b/drivers/gpu/drm/nouveau/include/nvif/clc36f.h
index f66885891238ebb3a70a6425c0f20936340b0244..b36624760dea3a3e9ab2b22aab67dbd64dbe2e3f 100644 (file)
--- a/drivers/gpu/drm/nouveau/include/nvif/clc36f.h
+++ b/drivers/gpu/drm/nouveau/include/nvif/clc36f.h
@@ -14,6 +14,5 @@ struct volta_channel_gpfifo_a_v0 {
        __u32 token;
 };
 
-#define NVC36F_V0_NTFY_NON_STALL_INTERRUPT                                 0x00
 #define NVC36F_V0_NTFY_KILLED                                              0x01
 #endif

diff --git a/drivers/gpu/drm/nouveau/include/nvif/if0020.h b/drivers/gpu/drm/nouveau/include/nvif/if0020.h
new file mode 100644 (file)
index 0000000..29a9fe2
--- /dev/null
+++ b/drivers/gpu/drm/nouveau/include/nvif/if0020.h
@@ -0,0 +1,12 @@
+/* SPDX-License-Identifier: MIT */
+#ifndef __NVIF_IF0020_H__
+#define __NVIF_IF0020_H__
+
+union nvif_chan_event_args {
+       struct nvif_chan_event_v0 {
+               __u8 version;
+#define NVIF_CHAN_EVENT_V0_NON_STALL_INTR 0x00
+               __u8 type;
+       } v0;
+};
+#endif

diff --git a/drivers/gpu/drm/nouveau/include/nvkm/engine/fifo.h b/drivers/gpu/drm/nouveau/include/nvkm/engine/fifo.h
index 15099913504d550bfc2a4e75393e5a723df2e8ec..ebfca3ffe9ae4670e019dd9c6035efff1e16b881 100644 (file)
--- a/drivers/gpu/drm/nouveau/include/nvkm/engine/fifo.h
+++ b/drivers/gpu/drm/nouveau/include/nvkm/engine/fifo.h
@@ -42,6 +42,7 @@ struct nvkm_fifo {
        spinlock_t lock;
        struct mutex mutex;
 
+#define NVKM_FIFO_EVENT_NON_STALL_INTR BIT(0)
        struct nvkm_event uevent; /* async user trigger */
        struct nvkm_event kevent; /* channel killed */
 };

diff --git a/drivers/gpu/drm/nouveau/nouveau_fence.c b/drivers/gpu/drm/nouveau/nouveau_fence.c
index abcac7db434738c8bb6c637f8479d103506c59c3..62560e30febb4b0dd43c1678dcb406567c35d748 100644 (file)
--- a/drivers/gpu/drm/nouveau/nouveau_fence.c
+++ b/drivers/gpu/drm/nouveau/nouveau_fence.c
@@ -29,9 +29,7 @@
 #include <linux/sched/signal.h>
 #include <trace/events/dma_fence.h>
 
-#include <nvif/cl826e.h>
-#include <nvif/notify.h>
-#include <nvif/event.h>
+#include <nvif/if0020.h>
 
 #include "nouveau_drv.h"
 #include "nouveau_dma.h"
@@ -99,7 +97,7 @@ nouveau_fence_context_kill(struct nouveau_fence_chan *fctx, int error)
                        dma_fence_set_error(&fence->base, error);
 
                if (nouveau_fence_signal(fence))
-                       nvif_notify_put(&fctx->notify);
+                       nvif_event_block(&fctx->event);
        }
        spin_unlock_irq(&fctx->lock);
 }
@@ -108,7 +106,7 @@ void
 nouveau_fence_context_del(struct nouveau_fence_chan *fctx)
 {
        nouveau_fence_context_kill(fctx, 0);
-       nvif_notify_dtor(&fctx->notify);
+       nvif_event_dtor(&fctx->event);
        fctx->dead = 1;
 
        /*
@@ -150,12 +148,11 @@ nouveau_fence_update(struct nouveau_channel *chan, struct nouveau_fence_chan *fc
 }
 
 static int
-nouveau_fence_wait_uevent_handler(struct nvif_notify *notify)
+nouveau_fence_wait_uevent_handler(struct nvif_event *event, void *repv, u32 repc)
 {
-       struct nouveau_fence_chan *fctx =
-               container_of(notify, typeof(*fctx), notify);
+       struct nouveau_fence_chan *fctx = container_of(event, typeof(*fctx), event);
        unsigned long flags;
-       int ret = NVIF_NOTIFY_KEEP;
+       int ret = NVIF_EVENT_KEEP;
 
        spin_lock_irqsave(&fctx->lock, flags);
        if (!list_empty(&fctx->pending)) {
@@ -165,7 +162,7 @@ nouveau_fence_wait_uevent_handler(struct nvif_notify *notify)
                fence = list_entry(fctx->pending.next, typeof(*fence), head);
                chan = rcu_dereference_protected(fence->channel, lockdep_is_held(&fctx->lock));
                if (nouveau_fence_update(chan, fctx))
-                       ret = NVIF_NOTIFY_DROP;
+                       ret = NVIF_EVENT_DROP;
        }
        spin_unlock_irqrestore(&fctx->lock, flags);
 
@@ -177,6 +174,10 @@ nouveau_fence_context_new(struct nouveau_channel *chan, struct nouveau_fence_cha
 {
        struct nouveau_fence_priv *priv = (void*)chan->drm->fence;
        struct nouveau_cli *cli = (void *)chan->user.client;
+       struct {
+               struct nvif_event_v0 base;
+               struct nvif_chan_event_v0 host;
+       } args;
        int ret;
 
        INIT_LIST_HEAD(&fctx->flip);
@@ -195,13 +196,12 @@ nouveau_fence_context_new(struct nouveau_channel *chan, struct nouveau_fence_cha
        if (!priv->uevent)
                return;
 
-       ret = nvif_notify_ctor(&chan->user, "fenceNonStallIntr",
-                              nouveau_fence_wait_uevent_handler,
-                              false, NV826E_V0_NTFY_NON_STALL_INTERRUPT,
-                              &(struct nvif_notify_uevent_req) { },
-                              sizeof(struct nvif_notify_uevent_req),
-                              sizeof(struct nvif_notify_uevent_rep),
-                              &fctx->notify);
+       args.host.version = 0;
+       args.host.type = NVIF_CHAN_EVENT_V0_NON_STALL_INTR;
+
+       ret = nvif_event_ctor(&chan->user, "fenceNonStallIntr", chan->chid,
+                             nouveau_fence_wait_uevent_handler, false,
+                             &args.base, sizeof(args), &fctx->event);
 
        WARN_ON(ret);
 }
@@ -230,7 +230,7 @@ nouveau_fence_emit(struct nouveau_fence *fence, struct nouveau_channel *chan)
                spin_lock_irq(&fctx->lock);
 
                if (nouveau_fence_update(chan, fctx))
-                       nvif_notify_put(&fctx->notify);
+                       nvif_event_block(&fctx->event);
 
                list_add_tail(&fence->head, &fctx->pending);
                spin_unlock_irq(&fctx->lock);
@@ -254,7 +254,7 @@ nouveau_fence_done(struct nouveau_fence *fence)
                spin_lock_irqsave(&fctx->lock, flags);
                chan = rcu_dereference_protected(fence->channel, lockdep_is_held(&fctx->lock));
                if (chan && nouveau_fence_update(chan, fctx))
-                       nvif_notify_put(&fctx->notify);
+                       nvif_event_block(&fctx->event);
                spin_unlock_irqrestore(&fctx->lock, flags);
        }
        return dma_fence_is_signaled(&fence->base);
@@ -505,13 +505,13 @@ static bool nouveau_fence_enable_signaling(struct dma_fence *f)
        bool ret;
 
        if (!fctx->notify_ref++)
-               nvif_notify_get(&fctx->notify);
+               nvif_event_allow(&fctx->event);
 
        ret = nouveau_fence_no_signaling(f);
        if (ret)
                set_bit(DMA_FENCE_FLAG_USER_BITS, &fence->base.flags);
        else if (!--fctx->notify_ref)
-               nvif_notify_put(&fctx->notify);
+               nvif_event_block(&fctx->event);
 
        return ret;
 }

diff --git a/drivers/gpu/drm/nouveau/nouveau_fence.h b/drivers/gpu/drm/nouveau/nouveau_fence.h
index 4887caa69c6577256a188e34450690c5a759686d..0ca2bc85adf617a9c428fcac614148d2b8f712f1 100644 (file)
--- a/drivers/gpu/drm/nouveau/nouveau_fence.h
+++ b/drivers/gpu/drm/nouveau/nouveau_fence.h
@@ -3,7 +3,7 @@
 #define __NOUVEAU_FENCE_H__
 
 #include <linux/dma-fence.h>
-#include <nvif/notify.h>
+#include <nvif/event.h>
 
 struct nouveau_drm;
 struct nouveau_bo;
@@ -44,7 +44,7 @@ struct nouveau_fence_chan {
        u32 context;
        char name[32];
 
-       struct nvif_notify notify;
+       struct nvif_event event;
        int notify_ref, dead;
 };
 

diff --git a/drivers/gpu/drm/nouveau/nvkm/engine/fifo/base.c b/drivers/gpu/drm/nouveau/nvkm/engine/fifo/base.c
index c94b2b9b93294c5853b3ec4ce4ed9e72478d812f..ec790af9b24f0a87361b5da60297827a2072d6cf 100644 (file)
--- a/drivers/gpu/drm/nouveau/nvkm/engine/fifo/base.c
+++ b/drivers/gpu/drm/nouveau/nvkm/engine/fifo/base.c
@@ -158,27 +158,8 @@ nvkm_fifo_uevent_init(struct nvkm_event *event, int type, int index)
        fifo->func->uevent_init(fifo);
 }
 
-static int
-nvkm_fifo_uevent_ctor(struct nvkm_object *object, void *data, u32 size,
-                     struct nvkm_notify *notify)
-{
-       union {
-               struct nvif_notify_uevent_req none;
-       } *req = data;
-       int ret = -ENOSYS;
-
-       if (!(ret = nvif_unvers(ret, &data, &size, req->none))) {
-               notify->size  = sizeof(struct nvif_notify_uevent_rep);
-               notify->types = 1;
-               notify->index = 0;
-       }
-
-       return ret;
-}
-
 static const struct nvkm_event_func
 nvkm_fifo_uevent_func = {
-       .ctor = nvkm_fifo_uevent_ctor,
        .init = nvkm_fifo_uevent_init,
        .fini = nvkm_fifo_uevent_fini,
 };
@@ -186,9 +167,7 @@ nvkm_fifo_uevent_func = {
 void
 nvkm_fifo_uevent(struct nvkm_fifo *fifo)
 {
-       struct nvif_notify_uevent_rep rep = {
-       };
-       nvkm_event_send(&fifo->uevent, 1, 0, &rep, sizeof(rep));
+       nvkm_event_send(&fifo->uevent, NVKM_FIFO_EVENT_NON_STALL_INTR, 0, NULL, 0);
 }
 
 static int

diff --git a/drivers/gpu/drm/nouveau/nvkm/engine/fifo/chan.c b/drivers/gpu/drm/nouveau/nvkm/engine/fifo/chan.c
index 2e7f32cebf2a1b1773387930e8dcd196c2112e5e..08f09f40defb398e4df0375bed6ad356e4b7b3e4 100644 (file)
--- a/drivers/gpu/drm/nouveau/nvkm/engine/fifo/chan.c
+++ b/drivers/gpu/drm/nouveau/nvkm/engine/fifo/chan.c
@@ -29,6 +29,8 @@
 #include <subdev/mmu.h>
 #include <engine/dma.h>
 
+#include <nvif/if0020.h>
+
 struct nvkm_fifo_chan_object {
        struct nvkm_oproxy oproxy;
        struct nvkm_fifo_chan *chan;
@@ -250,6 +252,28 @@ nvkm_fifo_chan_child_get(struct nvkm_object *object, int index,
        return -EINVAL;
 }
 
+static int
+nvkm_fifo_chan_uevent(struct nvkm_object *object, void *argv, u32 argc, struct nvkm_uevent *uevent)
+{
+       struct nvkm_fifo_chan *chan = nvkm_fifo_chan(object);
+       union nvif_chan_event_args *args = argv;
+
+       if (!uevent)
+               return 0;
+       if (argc != sizeof(args->v0) || args->v0.version != 0)
+               return -ENOSYS;
+
+       switch (args->v0.type) {
+       case NVIF_CHAN_EVENT_V0_NON_STALL_INTR:
+               return nvkm_uevent_add(uevent, &chan->fifo->uevent, 0,
+                                      NVKM_FIFO_EVENT_NON_STALL_INTR, NULL);
+       default:
+               break;
+       }
+
+       return -ENOSYS;
+}
+
 static int
 nvkm_fifo_chan_ntfy(struct nvkm_object *object, u32 type,
                    struct nvkm_event **pevent)
@@ -320,6 +344,7 @@ nvkm_fifo_chan_func = {
        .ntfy = nvkm_fifo_chan_ntfy,
        .map = nvkm_fifo_chan_map,
        .sclass = nvkm_fifo_chan_child_get,
+       .uevent = nvkm_fifo_chan_uevent,
 };
 
 int

diff --git a/drivers/gpu/drm/nouveau/nvkm/engine/fifo/chang84.c b/drivers/gpu/drm/nouveau/nvkm/engine/fifo/chang84.c
index 3492c561f2cfc858c4f776e1fd3d754d0136d544..903b54c07750e27ffaf8cd659a30c922dd762fd6 100644 (file)
--- a/drivers/gpu/drm/nouveau/nvkm/engine/fifo/chang84.c
+++ b/drivers/gpu/drm/nouveau/nvkm/engine/fifo/chang84.c
@@ -30,20 +30,6 @@
 
 #include <nvif/cl826e.h>
 
-static int
-g84_fifo_chan_ntfy(struct nvkm_fifo_chan *chan, u32 type,
-                  struct nvkm_event **pevent)
-{
-       switch (type) {
-       case NV826E_V0_NTFY_NON_STALL_INTERRUPT:
-               *pevent = &chan->fifo->uevent;
-               return 0;
-       default:
-               break;
-       }
-       return -EINVAL;
-}
-
 static int
 g84_fifo_chan_engine_addr(struct nvkm_engine *engine)
 {
@@ -198,7 +184,6 @@ g84_fifo_chan_func = {
        .dtor = nv50_fifo_chan_dtor,
        .init = g84_fifo_chan_init,
        .fini = nv50_fifo_chan_fini,
-       .ntfy = g84_fifo_chan_ntfy,
        .engine_ctor = g84_fifo_chan_engine_ctor,
        .engine_dtor = nv50_fifo_chan_engine_dtor,
        .engine_init = g84_fifo_chan_engine_init,

diff --git a/drivers/gpu/drm/nouveau/nvkm/engine/fifo/gpfifogf100.c b/drivers/gpu/drm/nouveau/nvkm/engine/fifo/gpfifogf100.c
index 4e78bbe3b94bb29f17715a6e321f21de44752cc4..569b5eedb84d6c54d77c1bea13793192a64b1228 100644 (file)
--- a/drivers/gpu/drm/nouveau/nvkm/engine/fifo/gpfifogf100.c
+++ b/drivers/gpu/drm/nouveau/nvkm/engine/fifo/gpfifogf100.c
@@ -37,9 +37,6 @@ gf100_fifo_chan_ntfy(struct nvkm_fifo_chan *chan, u32 type,
                     struct nvkm_event **pevent)
 {
        switch (type) {
-       case NV906F_V0_NTFY_NON_STALL_INTERRUPT:
-               *pevent = &chan->fifo->uevent;
-               return 0;
        case NV906F_V0_NTFY_KILLED:
                *pevent = &chan->fifo->kevent;
                return 0;