svn commit: r286524 - in head: . etc sys/dev/drm sys/dev/drm2
Warner Losh
imp at bsdimp.com
Tue Aug 11 15:10:57 UTC 2015
> On Aug 11, 2015, at 7:53 AM, Koop Mast <kwm at FreeBSD.org> wrote:
>
> On Mon, 2015-08-10 at 02:15 +0200, Jan Beich wrote:
>> 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.
>
> 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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 842 bytes
Desc: Message signed with OpenPGP using GPGMail
URL: <http://lists.freebsd.org/pipermail/svn-src-all/attachments/20150811/a2e029c1/attachment.bin>
More information about the svn-src-all
mailing list