gerrit.automotivelinux Code Review - AGL/meta-agl-devel.git/blob

   1 From f583672221b8451d03f739bc96428e97809d021a Mon Sep 17 00:00:00 2001
   2 From: Damian Hobson-Garcia <dhobsong@igel.co.jp>
   3 Date: Tue, 5 Apr 2022 18:09:02 +0900
   4 Subject: [PATCH] Workaround: Disable DRM_AUTH check on wayland protocols
   5
   6 DRM authentication does not work via a DRM Leases because the client
   7 can only authenticate against a lease owner (primary node), since
   8 the lease does not have a corresponding device file in the filesystem
   9 for clients to open.
  10
  11 The wl_drm protocol needs authentication to import/export prime fds
  12 on kernel versions < 5.6.  For newer kernel versions, prime
  13 import/export no longer requires DRM_AUTH, so processing can proceed
  14 even if DRM authentication fails.
  15
  16 A proper implementation would check if the device is actually a
  17 DRM lease and/or whether the import/export can succeed without
  18 authentication.  For now, just try to authenticate and ignore
  19 any failures.
  20 ---
  21  src/egl/drivers/dri2/platform_drm.c | 5 ++++-
  22  1 file changed, 4 insertions(+), 1 deletion(-)
  23
  24 diff --git a/src/egl/drivers/dri2/platform_drm.c b/src/egl/drivers/dri2/platform_drm.c
  25 index d0ab172..260a2c6 100644
  26 --- a/src/egl/drivers/dri2/platform_drm.c
  27 +++ b/src/egl/drivers/dri2/platform_drm.c
  28 @@ -508,7 +508,10 @@ dri2_drm_authenticate(_EGLDisplay *disp, uint32_t id)
  29  {
  30     struct dri2_egl_display *dri2_dpy = dri2_egl_display(disp);
  31
  32 -   return drmAuthMagic(dri2_dpy->fd, id);
  33 +   /* WORKAROUND: Ignore DRM_AUTH failures.  Ideally should only be done for
  34 +      clienits coming through a DRM lease, but just make it global for now */
  35 +   drmAuthMagic(dri2_dpy->fd, id);
  36 +   return 0;
  37  }
  38
  39  static void
  40 --
  41 2.17.1
  42