use /dev/dri/ in xenocara

[Jonathan Gray-11]

Index: lib/libdrm/xf86drm.h
===================================================================
RCS file: /cvs/xenocara/lib/libdrm/xf86drm.h,v
retrieving revision 1.21
diff -u -p -r1.21 xf86drm.h
--- lib/libdrm/xf86drm.h 11 Feb 2021 10:27:08 -0000 1.21
+++ lib/libdrm/xf86drm.h 18 Feb 2021 10:02:03 -0000
@@ -76,18 +76,11 @@ extern "C" {
  (S_IRUSR|S_IWUSR|S_IXUSR|S_IRGRP|S_IXGRP|S_IROTH|S_IXOTH)
 #define DRM_DEV_MODE (S_IRUSR|S_IWUSR|S_IRGRP|S_IWGRP)
 
-#ifdef __OpenBSD__
-#define DRM_DIR_NAME  "/dev"
-#define DRM_PRIMARY_MINOR_NAME  "drm"
-#define DRM_CONTROL_MINOR_NAME  "drmC"
-#define DRM_RENDER_MINOR_NAME   "drmR"
-#else
 #define DRM_DIR_NAME  "/dev/dri"
 #define DRM_PRIMARY_MINOR_NAME  "card"
 #define DRM_CONTROL_MINOR_NAME  "controlD"
 #define DRM_RENDER_MINOR_NAME   "renderD"
 #define DRM_PROC_NAME "/proc/dri/" /* For backward Linux compatibility */
-#endif
 
 #define DRM_DEV_NAME          "%s/" DRM_PRIMARY_MINOR_NAME "%d"
 #define DRM_CONTROL_DEV_NAME  "%s/" DRM_CONTROL_MINOR_NAME "%d"
Index: xserver/hw/xfree86/drivers/modesetting/driver.c
===================================================================
RCS file: /cvs/xenocara/xserver/hw/xfree86/drivers/modesetting/driver.c,v
retrieving revision 1.9
diff -u -p -r1.9 driver.c
--- xserver/hw/xfree86/drivers/modesetting/driver.c 12 Dec 2020 09:30:54 -0000 1.9
+++ xserver/hw/xfree86/drivers/modesetting/driver.c 18 Feb 2021 10:02:03 -0000
@@ -226,7 +226,7 @@ open_hw(const char *dev)
     else {
         dev = getenv("KMSDEVICE");
         if ((NULL == dev) || ((fd = priv_open_device(dev)) == -1)) {
-            dev = "/dev/drm0";
+            dev = "/dev/dri/card0";
             fd = priv_open_device(dev);
         }
     }

[Mark Kettenis]

> Date: Thu, 18 Feb 2021 21:18:51 +1100
> From: Jonathan Gray <[hidden email]>

I suspect that there are some ports that need to get their unveils
updated if we do this.

> Index: lib/libdrm/xf86drm.h
> ===================================================================
> RCS file: /cvs/xenocara/lib/libdrm/xf86drm.h,v
> retrieving revision 1.21
> diff -u -p -r1.21 xf86drm.h
> --- lib/libdrm/xf86drm.h 11 Feb 2021 10:27:08 -0000 1.21
> +++ lib/libdrm/xf86drm.h 18 Feb 2021 10:02:03 -0000
> @@ -76,18 +76,11 @@ extern "C" {
>   (S_IRUSR|S_IWUSR|S_IXUSR|S_IRGRP|S_IXGRP|S_IROTH|S_IXOTH)
>  #define DRM_DEV_MODE (S_IRUSR|S_IWUSR|S_IRGRP|S_IWGRP)
>  
> -#ifdef __OpenBSD__
> -#define DRM_DIR_NAME  "/dev"
> -#define DRM_PRIMARY_MINOR_NAME  "drm"
> -#define DRM_CONTROL_MINOR_NAME  "drmC"
> -#define DRM_RENDER_MINOR_NAME   "drmR"
> -#else
>  #define DRM_DIR_NAME  "/dev/dri"
>  #define DRM_PRIMARY_MINOR_NAME  "card"
>  #define DRM_CONTROL_MINOR_NAME  "controlD"
>  #define DRM_RENDER_MINOR_NAME   "renderD"
>  #define DRM_PROC_NAME "/proc/dri/" /* For backward Linux compatibility */
> -#endif
>  
>  #define DRM_DEV_NAME          "%s/" DRM_PRIMARY_MINOR_NAME "%d"
>  #define DRM_CONTROL_DEV_NAME  "%s/" DRM_CONTROL_MINOR_NAME "%d"
> Index: xserver/hw/xfree86/drivers/modesetting/driver.c
> ===================================================================
> RCS file: /cvs/xenocara/xserver/hw/xfree86/drivers/modesetting/driver.c,v
> retrieving revision 1.9
> diff -u -p -r1.9 driver.c
> --- xserver/hw/xfree86/drivers/modesetting/driver.c 12 Dec 2020 09:30:54 -0000 1.9
> +++ xserver/hw/xfree86/drivers/modesetting/driver.c 18 Feb 2021 10:02:03 -0000
> @@ -226,7 +226,7 @@ open_hw(const char *dev)
>      else {
>          dev = getenv("KMSDEVICE");
>          if ((NULL == dev) || ((fd = priv_open_device(dev)) == -1)) {
> -            dev = "/dev/drm0";
> +            dev = "/dev/dri/card0";
>              fd = priv_open_device(dev);
>          }
>      }
>
>

[Jonathan Gray-11]

On Thu, Feb 18, 2021 at 12:01:28PM +0100, Mark Kettenis wrote:
> > Date: Thu, 18 Feb 2021 21:18:51 +1100
> > From: Jonathan Gray <[hidden email]>
>
> I suspect that there are some ports that need to get their unveils
> updated if we do this.

firefox ports were updated.  Not aware of anything else in ports that
unveils /dev/drm.

>
> > Index: lib/libdrm/xf86drm.h
> > ===================================================================
> > RCS file: /cvs/xenocara/lib/libdrm/xf86drm.h,v
> > retrieving revision 1.21
> > diff -u -p -r1.21 xf86drm.h
> > --- lib/libdrm/xf86drm.h 11 Feb 2021 10:27:08 -0000 1.21
> > +++ lib/libdrm/xf86drm.h 18 Feb 2021 10:02:03 -0000
> > @@ -76,18 +76,11 @@ extern "C" {
> >   (S_IRUSR|S_IWUSR|S_IXUSR|S_IRGRP|S_IXGRP|S_IROTH|S_IXOTH)
> >  #define DRM_DEV_MODE (S_IRUSR|S_IWUSR|S_IRGRP|S_IWGRP)
> >  
> > -#ifdef __OpenBSD__
> > -#define DRM_DIR_NAME  "/dev"
> > -#define DRM_PRIMARY_MINOR_NAME  "drm"
> > -#define DRM_CONTROL_MINOR_NAME  "drmC"
> > -#define DRM_RENDER_MINOR_NAME   "drmR"
> > -#else
> >  #define DRM_DIR_NAME  "/dev/dri"
> >  #define DRM_PRIMARY_MINOR_NAME  "card"
> >  #define DRM_CONTROL_MINOR_NAME  "controlD"
> >  #define DRM_RENDER_MINOR_NAME   "renderD"
> >  #define DRM_PROC_NAME "/proc/dri/" /* For backward Linux compatibility */
> > -#endif
> >  
> >  #define DRM_DEV_NAME          "%s/" DRM_PRIMARY_MINOR_NAME "%d"
> >  #define DRM_CONTROL_DEV_NAME  "%s/" DRM_CONTROL_MINOR_NAME "%d"
> > Index: xserver/hw/xfree86/drivers/modesetting/driver.c
> > ===================================================================
> > RCS file: /cvs/xenocara/xserver/hw/xfree86/drivers/modesetting/driver.c,v
> > retrieving revision 1.9
> > diff -u -p -r1.9 driver.c
> > --- xserver/hw/xfree86/drivers/modesetting/driver.c 12 Dec 2020 09:30:54 -0000 1.9
> > +++ xserver/hw/xfree86/drivers/modesetting/driver.c 18 Feb 2021 10:02:03 -0000
> > @@ -226,7 +226,7 @@ open_hw(const char *dev)
> >      else {
> >          dev = getenv("KMSDEVICE");
> >          if ((NULL == dev) || ((fd = priv_open_device(dev)) == -1)) {
> > -            dev = "/dev/drm0";
> > +            dev = "/dev/dri/card0";
> >              fd = priv_open_device(dev);
> >          }
> >      }
> >
> >
>

[Mark Kettenis]

> Date: Thu, 18 Feb 2021 22:24:10 +1100
> From: Jonathan Gray <[hidden email]>
>
> On Thu, Feb 18, 2021 at 12:01:28PM +0100, Mark Kettenis wrote:
> > > Date: Thu, 18 Feb 2021 21:18:51 +1100
> > > From: Jonathan Gray <[hidden email]>
> >
> > I suspect that there are some ports that need to get their unveils
> > updated if we do this.
>
> firefox ports were updated.  Not aware of anything else in ports that
> unveils /dev/drm.

Doesn't chromium use unveil?

Anyway, this should make unveiling easier and if you are coordinating
with the ports folks, this is ok kettenis@

> > > Index: lib/libdrm/xf86drm.h
> > > ===================================================================
> > > RCS file: /cvs/xenocara/lib/libdrm/xf86drm.h,v
> > > retrieving revision 1.21
> > > diff -u -p -r1.21 xf86drm.h
> > > --- lib/libdrm/xf86drm.h 11 Feb 2021 10:27:08 -0000 1.21
> > > +++ lib/libdrm/xf86drm.h 18 Feb 2021 10:02:03 -0000
> > > @@ -76,18 +76,11 @@ extern "C" {
> > >   (S_IRUSR|S_IWUSR|S_IXUSR|S_IRGRP|S_IXGRP|S_IROTH|S_IXOTH)
> > >  #define DRM_DEV_MODE (S_IRUSR|S_IWUSR|S_IRGRP|S_IWGRP)
> > >  
> > > -#ifdef __OpenBSD__
> > > -#define DRM_DIR_NAME  "/dev"
> > > -#define DRM_PRIMARY_MINOR_NAME  "drm"
> > > -#define DRM_CONTROL_MINOR_NAME  "drmC"
> > > -#define DRM_RENDER_MINOR_NAME   "drmR"
> > > -#else
> > >  #define DRM_DIR_NAME  "/dev/dri"
> > >  #define DRM_PRIMARY_MINOR_NAME  "card"
> > >  #define DRM_CONTROL_MINOR_NAME  "controlD"
> > >  #define DRM_RENDER_MINOR_NAME   "renderD"
> > >  #define DRM_PROC_NAME "/proc/dri/" /* For backward Linux compatibility */
> > > -#endif
> > >  
> > >  #define DRM_DEV_NAME          "%s/" DRM_PRIMARY_MINOR_NAME "%d"
> > >  #define DRM_CONTROL_DEV_NAME  "%s/" DRM_CONTROL_MINOR_NAME "%d"
> > > Index: xserver/hw/xfree86/drivers/modesetting/driver.c
> > > ===================================================================
> > > RCS file: /cvs/xenocara/xserver/hw/xfree86/drivers/modesetting/driver.c,v
> > > retrieving revision 1.9
> > > diff -u -p -r1.9 driver.c
> > > --- xserver/hw/xfree86/drivers/modesetting/driver.c 12 Dec 2020 09:30:54 -0000 1.9
> > > +++ xserver/hw/xfree86/drivers/modesetting/driver.c 18 Feb 2021 10:02:03 -0000
> > > @@ -226,7 +226,7 @@ open_hw(const char *dev)
> > >      else {
> > >          dev = getenv("KMSDEVICE");
> > >          if ((NULL == dev) || ((fd = priv_open_device(dev)) == -1)) {
> > > -            dev = "/dev/drm0";
> > > +            dev = "/dev/dri/card0";
> > >              fd = priv_open_device(dev);
> > >          }
> > >      }
> > >
> > >
> >
>

[Stuart Henderson]

On 2021/02/18 22:24, Jonathan Gray wrote:
> On Thu, Feb 18, 2021 at 12:01:28PM +0100, Mark Kettenis wrote:
> > > Date: Thu, 18 Feb 2021 21:18:51 +1100
> > > From: Jonathan Gray <[hidden email]>
> >
> > I suspect that there are some ports that need to get their unveils
> > updated if we do this.
>
> firefox ports were updated.  Not aware of anything else in ports that
> unveils /dev/drm.

unveils: not afaik

others: gdm already handled it, some other ports will need patches changing:

graphics/clutter/cogl/patches/patch-cogl_winsys_cogl-winsys-egl-kms_c
graphics/waffle/patches/patch-src_waffle_gbm_wgbm_display_c
x11/compton/patches/patch-src_compton_c
x11/slim/patches/patch-slim_conf
x11/picom/patches/patch-src_vsync_c

[Jonathan Gray-11]

On Thu, Feb 18, 2021 at 12:29:29PM +0100, Mark Kettenis wrote:

> > Date: Thu, 18 Feb 2021 22:24:10 +1100
> > From: Jonathan Gray <[hidden email]>
> >
> > On Thu, Feb 18, 2021 at 12:01:28PM +0100, Mark Kettenis wrote:
> > > > Date: Thu, 18 Feb 2021 21:18:51 +1100
> > > > From: Jonathan Gray <[hidden email]>
> > >
> > > I suspect that there are some ports that need to get their unveils
> > > updated if we do this.
> >
> > firefox ports were updated.  Not aware of anything else in ports that
> > unveils /dev/drm.
>
> Doesn't chromium use unveil?

chromium opens the device outside of the sandbox

>
> Anyway, this should make unveiling easier and if you are coordinating
> with the ports folks, this is ok kettenis@
>
> > > > Index: lib/libdrm/xf86drm.h
> > > > ===================================================================
> > > > RCS file: /cvs/xenocara/lib/libdrm/xf86drm.h,v
> > > > retrieving revision 1.21
> > > > diff -u -p -r1.21 xf86drm.h
> > > > --- lib/libdrm/xf86drm.h 11 Feb 2021 10:27:08 -0000 1.21
> > > > +++ lib/libdrm/xf86drm.h 18 Feb 2021 10:02:03 -0000
> > > > @@ -76,18 +76,11 @@ extern "C" {
> > > >   (S_IRUSR|S_IWUSR|S_IXUSR|S_IRGRP|S_IXGRP|S_IROTH|S_IXOTH)
> > > >  #define DRM_DEV_MODE (S_IRUSR|S_IWUSR|S_IRGRP|S_IWGRP)
> > > >  
> > > > -#ifdef __OpenBSD__
> > > > -#define DRM_DIR_NAME  "/dev"
> > > > -#define DRM_PRIMARY_MINOR_NAME  "drm"
> > > > -#define DRM_CONTROL_MINOR_NAME  "drmC"
> > > > -#define DRM_RENDER_MINOR_NAME   "drmR"
> > > > -#else
> > > >  #define DRM_DIR_NAME  "/dev/dri"
> > > >  #define DRM_PRIMARY_MINOR_NAME  "card"
> > > >  #define DRM_CONTROL_MINOR_NAME  "controlD"
> > > >  #define DRM_RENDER_MINOR_NAME   "renderD"
> > > >  #define DRM_PROC_NAME "/proc/dri/" /* For backward Linux compatibility */
> > > > -#endif
> > > >  
> > > >  #define DRM_DEV_NAME          "%s/" DRM_PRIMARY_MINOR_NAME "%d"
> > > >  #define DRM_CONTROL_DEV_NAME  "%s/" DRM_CONTROL_MINOR_NAME "%d"
> > > > Index: xserver/hw/xfree86/drivers/modesetting/driver.c
> > > > ===================================================================
> > > > RCS file: /cvs/xenocara/xserver/hw/xfree86/drivers/modesetting/driver.c,v
> > > > retrieving revision 1.9
> > > > diff -u -p -r1.9 driver.c
> > > > --- xserver/hw/xfree86/drivers/modesetting/driver.c 12 Dec 2020 09:30:54 -0000 1.9
> > > > +++ xserver/hw/xfree86/drivers/modesetting/driver.c 18 Feb 2021 10:02:03 -0000
> > > > @@ -226,7 +226,7 @@ open_hw(const char *dev)
> > > >      else {
> > > >          dev = getenv("KMSDEVICE");
> > > >          if ((NULL == dev) || ((fd = priv_open_device(dev)) == -1)) {
> > > > -            dev = "/dev/drm0";
> > > > +            dev = "/dev/dri/card0";
> > > >              fd = priv_open_device(dev);
> > > >          }
> > > >      }
> > > >
> > > >
> > >
> >
>

[Jonathan Gray-11]

On Thu, Feb 18, 2021 at 11:34:19AM +0000, Stuart Henderson wrote:

> On 2021/02/18 22:24, Jonathan Gray wrote:
> > On Thu, Feb 18, 2021 at 12:01:28PM +0100, Mark Kettenis wrote:
> > > > Date: Thu, 18 Feb 2021 21:18:51 +1100
> > > > From: Jonathan Gray <[hidden email]>
> > >
> > > I suspect that there are some ports that need to get their unveils
> > > updated if we do this.
> >
> > firefox ports were updated.  Not aware of anything else in ports that
> > unveils /dev/drm.
>
> unveils: not afaik
>
> others: gdm already handled it, some other ports will need patches changing:
>
> graphics/clutter/cogl/patches/patch-cogl_winsys_cogl-winsys-egl-kms_c
> graphics/waffle/patches/patch-src_waffle_gbm_wgbm_display_c
> x11/compton/patches/patch-src_compton_c
> x11/slim/patches/patch-slim_conf

This is a display manager like xdm/gdm.  The last upstream release was
in 2013.  I can patch it after the xenocara changes go in or perhaps we
remove it as landry suggested in

----------------------------
revision 1.55
date: 2020/07/24 05:41:37;  author: landry;  state: Exp;  lines: +2 -3;  commitid: yb1SZfwKZuXceEq0;
Drop maintainership, i havent used slim in ages.

Anyway it's more or less dead upstream since 7 years, which doesnt look
good for a login manager.. candidate for removal ? xenodm works ootb,
and is customizable once you grok X properties..
----------------------------

> x11/picom/patches/patch-src_vsync_c

/dev/drm will stay for now so the others can be updated later.

[Damien Couderc-4]

Le 18/02/2021 à 13:11, Jonathan Gray a écrit :

> On Thu, Feb 18, 2021 at 11:34:19AM +0000, Stuart Henderson wrote:
>> On 2021/02/18 22:24, Jonathan Gray wrote:
>>> On Thu, Feb 18, 2021 at 12:01:28PM +0100, Mark Kettenis wrote:
>>>>> Date: Thu, 18 Feb 2021 21:18:51 +1100
>>>>> From: Jonathan Gray <[hidden email]>
>>>> I suspect that there are some ports that need to get their unveils
>>>> updated if we do this.
>>> firefox ports were updated.  Not aware of anything else in ports that
>>> unveils /dev/drm.
>> unveils: not afaik
>>
>> others: gdm already handled it, some other ports will need patches changing:
>>
>> graphics/clutter/cogl/patches/patch-cogl_winsys_cogl-winsys-egl-kms_c
>> graphics/waffle/patches/patch-src_waffle_gbm_wgbm_display_c
>> x11/compton/patches/patch-src_compton_c
>> x11/slim/patches/patch-slim_conf
> This is a display manager like xdm/gdm.  The last upstream release was
> in 2013.  I can patch it after the xenocara changes go in or perhaps we
> remove it as landry suggested in

I am using slim on my children computers since many years now and it
works very well. This could explain why there is nothing done upstream:
no bug, no fix and features are already there.

> ----------------------------
> revision 1.55
> date: 2020/07/24 05:41:37;  author: landry;  state: Exp;  lines: +2 -3;  commitid: yb1SZfwKZuXceEq0;
> Drop maintainership, i havent used slim in ages.
>
> Anyway it's more or less dead upstream since 7 years, which doesnt look
> good for a login manager.. candidate for removal ? xenodm works ootb,
> and is customizable once you grok X properties..
> ----------------------------
>
>> x11/picom/patches/patch-src_vsync_c
> /dev/drm will stay for now so the others can be updated later.
>

[Landry Breuil-5]

On Thu, Feb 18, 2021 at 11:11:15PM +1100, Jonathan Gray wrote:

> On Thu, Feb 18, 2021 at 11:34:19AM +0000, Stuart Henderson wrote:
> > On 2021/02/18 22:24, Jonathan Gray wrote:
> > > On Thu, Feb 18, 2021 at 12:01:28PM +0100, Mark Kettenis wrote:
> > > > > Date: Thu, 18 Feb 2021 21:18:51 +1100
> > > > > From: Jonathan Gray <[hidden email]>
> > > >
> > > > I suspect that there are some ports that need to get their unveils
> > > > updated if we do this.
> > >
> > > firefox ports were updated.  Not aware of anything else in ports that
> > > unveils /dev/drm.
> >
> > unveils: not afaik
> >
> > others: gdm already handled it, some other ports will need patches changing:
> >
> > graphics/clutter/cogl/patches/patch-cogl_winsys_cogl-winsys-egl-kms_c
> > graphics/waffle/patches/patch-src_waffle_gbm_wgbm_display_c
> > x11/compton/patches/patch-src_compton_c
> > x11/slim/patches/patch-slim_conf
>
> This is a display manager like xdm/gdm.  The last upstream release was
> in 2013.  I can patch it after the xenocara changes go in or perhaps we
> remove it as landry suggested in

there's a less dead upstream at https://github.com/PeteGozz/slim.
FreeBSD and debian still ship a package for slim :)

I *might* have a look at updating the port to this github fork.

Landry

[Matthieu Herrb-3]

On Thu, Feb 18, 2021 at 09:18:51PM +1100, Jonathan Gray wrote:

Ok matthieu@.

> Index: lib/libdrm/xf86drm.h
> ===================================================================
> RCS file: /cvs/xenocara/lib/libdrm/xf86drm.h,v
> retrieving revision 1.21
> diff -u -p -r1.21 xf86drm.h
> --- lib/libdrm/xf86drm.h 11 Feb 2021 10:27:08 -0000 1.21
> +++ lib/libdrm/xf86drm.h 18 Feb 2021 10:02:03 -0000
> @@ -76,18 +76,11 @@ extern "C" {
>   (S_IRUSR|S_IWUSR|S_IXUSR|S_IRGRP|S_IXGRP|S_IROTH|S_IXOTH)
>  #define DRM_DEV_MODE (S_IRUSR|S_IWUSR|S_IRGRP|S_IWGRP)
>  
> -#ifdef __OpenBSD__
> -#define DRM_DIR_NAME  "/dev"
> -#define DRM_PRIMARY_MINOR_NAME  "drm"
> -#define DRM_CONTROL_MINOR_NAME  "drmC"
> -#define DRM_RENDER_MINOR_NAME   "drmR"
> -#else
>  #define DRM_DIR_NAME  "/dev/dri"
>  #define DRM_PRIMARY_MINOR_NAME  "card"
>  #define DRM_CONTROL_MINOR_NAME  "controlD"
>  #define DRM_RENDER_MINOR_NAME   "renderD"
>  #define DRM_PROC_NAME "/proc/dri/" /* For backward Linux compatibility */
> -#endif
>  
>  #define DRM_DEV_NAME          "%s/" DRM_PRIMARY_MINOR_NAME "%d"
>  #define DRM_CONTROL_DEV_NAME  "%s/" DRM_CONTROL_MINOR_NAME "%d"
> Index: xserver/hw/xfree86/drivers/modesetting/driver.c
> ===================================================================
> RCS file: /cvs/xenocara/xserver/hw/xfree86/drivers/modesetting/driver.c,v
> retrieving revision 1.9
> diff -u -p -r1.9 driver.c
> --- xserver/hw/xfree86/drivers/modesetting/driver.c 12 Dec 2020 09:30:54 -0000 1.9
> +++ xserver/hw/xfree86/drivers/modesetting/driver.c 18 Feb 2021 10:02:03 -0000
> @@ -226,7 +226,7 @@ open_hw(const char *dev)
>      else {
>          dev = getenv("KMSDEVICE");
>          if ((NULL == dev) || ((fd = priv_open_device(dev)) == -1)) {
> -            dev = "/dev/drm0";
> +            dev = "/dev/dri/card0";
>              fd = priv_open_device(dev);
>          }
>      }

--
Matthieu Herrb

[Wolfgang Pfeiffer]

On Thu, Feb 18, 2021 at 11:11:15PM +1100, Jonathan Gray wrote:

>On Thu, Feb 18, 2021 at 11:34:19AM +0000, Stuart Henderson wrote:
>> On 2021/02/18 22:24, Jonathan Gray wrote:
>> > On Thu, Feb 18, 2021 at 12:01:28PM +0100, Mark Kettenis wrote:
>> > > > Date: Thu, 18 Feb 2021 21:18:51 +1100
>> > > > From: Jonathan Gray <[hidden email]>
>> > >
>> > > I suspect that there are some ports that need to get their unveils
>> > > updated if we do this.
>> >
>> > firefox ports were updated.  Not aware of anything else in ports that
>> > unveils /dev/drm.
>>
>> unveils: not afaik
>>
>> others: gdm already handled it, some other ports will need patches changing:
>>
>> graphics/clutter/cogl/patches/patch-cogl_winsys_cogl-winsys-egl-kms_c
>> graphics/waffle/patches/patch-src_waffle_gbm_wgbm_display_c
>> x11/compton/patches/patch-src_compton_c
>> x11/slim/patches/patch-slim_conf
>
>This is a display manager like xdm/gdm.  The last upstream release was
>in 2013.  I can patch it after the xenocara changes go in or perhaps we
>remove it as landry suggested in
>
>----------------------------
>revision 1.55
>date: 2020/07/24 05:41:37;  author: landry;  state: Exp;  lines: +2 -3;  commitid: yb1SZfwKZuXceEq0;
>Drop maintainership, i havent used slim in ages.
>
>Anyway it's more or less dead upstream since 7 years, which doesnt look
>good for a login manager.. candidate for removal ? xenodm works ootb,
>and is customizable once you grok X properties..
>----------------------------

I have slim running on both a 15yrs-old PowerBook5,8 (OBSD 6.7) and,
not that often, on a 19yrs old TitaniumIV (PowerBook3,5, with an older
OBSD version installed) - slim works like a charm on both of them,
with no bugs I'd be aware of: Easy to configure, and at least as
important: doesn't seem to eat too much of resources, which seems to
be really helpful on machines that old. On both machines slim takes
just a little bit more than 6 MB RES memory ...

>
>> x11/picom/patches/patch-src_vsync_c
>
>/dev/drm will stay for now so the others can be updated later.