drm/vgem: create a render node for vgem
authorEmil Velikov <emil.velikov@collabora.com>
Fri, 26 Oct 2018 12:06:47 +0000 (13:06 +0100)
committerEmil Velikov <emil.l.velikov@gmail.com>
Mon, 5 Nov 2018 13:06:30 +0000 (13:06 +0000)
VGEM doesn't do anything modeset specific, so in a way exposing a
primary node is 'wrong'. At the same time, we extensively use if for
creating dumb buffers, fences, prime fd <> handle imports/exports.

To the point that we explicitly annotate the vgem fence ioctls as
DRM_RENDER_ALLOW and have an IGT test which opens the render node.

close(drm_open_driver_render(DRIVER_VGEM))

Better late than never, let's flip the switch.

Cc: David Airlie <airlied@linux.ie>
Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
Signed-off-by: Emil Velikov <emil.velikov@collabora.com>
Acked-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Link: https://patchwork.freedesktop.org/patch/msgid/20181026120647.7528-1-emil.l.velikov@gmail.com
drivers/gpu/drm/vgem/vgem_drv.c

index f1f7ab9..f1d1d9e 100644 (file)
@@ -431,7 +431,8 @@ static void vgem_release(struct drm_device *dev)
 }
 
 static struct drm_driver vgem_driver = {
-       .driver_features                = DRIVER_GEM | DRIVER_PRIME,
+       .driver_features                = DRIVER_GEM | DRIVER_PRIME |
+                                         DRIVER_RENDER;
        .release                        = vgem_release,
        .open                           = vgem_open,
        .postclose                      = vgem_postclose,