svn commit: r286524 - in head: . etc sys/dev/drm sys/dev/drm2
Jan Beich
jbeich at FreeBSD.org
Mon Aug 10 00:16:13 UTC 2015
Koop Mast <kwm at 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.
> 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.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 602 bytes
Desc: not available
URL: <http://lists.freebsd.org/pipermail/svn-src-head/attachments/20150810/838f3f9a/attachment.bin>
More information about the svn-src-head
mailing list