Karol Herbst
2018-Sep-08 23:03 UTC
[Nouveau] [PATCH 1/2] nvif: allow userspace to use the notification interfaces
we need those so that userspace can get notified about channel resets. Required to implement GL_ARB_robustness correctly. Currently we advertise it, but we aren't able to handle gpu resets inside mesa without it. Signed-off-by: Karol Herbst <kherbst at redhat.com> --- drm/nouveau/nouveau_abi16.c | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/drm/nouveau/nouveau_abi16.c b/drm/nouveau/nouveau_abi16.c index e67a4713..249b00ae 100644 --- a/drm/nouveau/nouveau_abi16.c +++ b/drm/nouveau/nouveau_abi16.c @@ -383,6 +383,10 @@ nouveau_abi16_usif(struct drm_file *file_priv, void *data, u32 size) case NVIF_IOCTL_V0_NEW: case NVIF_IOCTL_V0_MTHD: case NVIF_IOCTL_V0_SCLASS: + case NVIF_IOCTL_V0_NTFY_NEW: + case NVIF_IOCTL_V0_NTFY_DEL: + case NVIF_IOCTL_V0_NTFY_GET: + case NVIF_IOCTL_V0_NTFY_PUT: break; default: return -EACCES; -- 2.17.1
Karol Herbst
2018-Sep-08 23:03 UTC
[Nouveau] [PATCH 2/2] usif: use correct struct inside usif_notify_get
No functional change, but this is clearly a copy and paste error. Signed-off-by: Karol Herbst <kherbst at redhat.com> --- drm/nouveau/nouveau_usif.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drm/nouveau/nouveau_usif.c b/drm/nouveau/nouveau_usif.c index 9dc10b17..34d48775 100644 --- a/drm/nouveau/nouveau_usif.c +++ b/drm/nouveau/nouveau_usif.c @@ -192,7 +192,7 @@ usif_notify_get(struct drm_file *f, void *data, u32 size, void *argv, u32 argc) struct nouveau_cli *cli = nouveau_cli(f); struct nvif_client *client = &cli->base; union { - struct nvif_ioctl_ntfy_del_v0 v0; + struct nvif_ioctl_ntfy_get_v0 v0; } *args = data; struct usif_notify *ntfy; int ret = -ENOSYS; -- 2.17.1
Possibly Parallel Threads
- [libdrm 09/13] nouveau: import and install a selection of nvif headers from the kernel
- [PATCH] drm/nouveau: fix LEDS_CLASS=m configuration
- [PATCH] drm/nouveau: fixup the uapi header file.
- [PATCH] nouveau/led: introduce CONFIG_DRM_NOUVEAU_LEDS
- [PATCH v2] drm/nouveau: add a LED driver for the NVIDIA logo