> On Aug 11, 2015, at 7:53 AM, Koop Mast <k...@freebsd.org> wrote: > > On Mon, 2015-08-10 at 02:15 +0200, Jan Beich wrote: >> Koop Mast <k...@freebsd.org> writes: >> >>> Author: kwm (ports committer) >>> Date: Sun Aug 9 12:58:56 2015 >>> New Revision: 286524 >>> URL: https://svnweb.freebsd.org/changeset/base/286524 >>> >>> Log: >>> Add a new group named 'video' with the id of 44. And make drm >>> create >>> devices in /dev/dri/ with this new group. >> >> Would 'video' group include capture devices as well? Linux seems to >> mix >> /dev/nvidia*, /dev/fb* and /dev/video* all under same group despite >> all of them have different attack vectors. > > We could extend this, but the only example I had where the dri devices. > Also webcamd already chmod's it's devices: > % ll /dev/video0 > crw-rw---- 1 webcamd webcamd 0x9a Aug 10 07:35 /dev/video0 > > I'm unsure if nvidia makes any /dev devices since it been ages since I > had one in a machine of mine. > >>> Modified: head/sys/dev/drm/drmP.h >>> =================================================================== >>> =========== >>> --- head/sys/dev/drm/drmP.h Sun Aug 9 12:20:22 2015 >>> (r286523) >>> +++ head/sys/dev/drm/drmP.h Sun Aug 9 12:58:56 2015 >>> (r286524) >>> @@ -175,7 +175,7 @@ SYSCTL_DECL(_hw_drm); >>> >>> #define DRM_DEV_MODE (S_IRUSR|S_IWUSR|S_IRGRP|S_IWGRP) >>> #define DRM_DEV_UID 0 >>> -#define DRM_DEV_GID 0 >>> +#define DRM_DEV_GID 44 /* "video" group */ >> >> Why hardcode? Linux often uses udev(7) rules to assign a group which >> on >> FreeBSD can easily be translated into devd.conf(5) or devfs.rules(5). >> >> Having 'video' assigned by kernel wouldn't eliminate having to run >> mergemaster/etcupdate + pw groupmod on upgrade. > > I find this way easier to do. However with the devd.conf(5) or > devfs.rules(5) someone still needs to up date the /etc and pw groupmod.
devd.conf isn’t the place for this. This is pure devfs.rules. Warner
signature.asc
Description: Message signed with OpenPGP using GPGMail
