linux-hotplug.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Scott James Remnant <scott@canonical.com>
To: linux-hotplug@vger.kernel.org
Subject: Re: Moving Ubuntu to upstream udev rules
Date: Mon, 22 Dec 2008 09:39:42 +0000	[thread overview]
Message-ID: <1229938782.6944.12.camel@quest> (raw)
In-Reply-To: <1229669228.14012.30.camel@quest>

[-- Attachment #1: Type: text/plain, Size: 4433 bytes --]

On Fri, 2008-12-19 at 17:03 +0100, Kay Sievers wrote:

> > scd/sr
> > 
> >   while the kernel still calls them sr, that prefix is deprecated.  You
> >   symlink the new name to the old one, we do it the other way around to
> >   make it clearer:
> > 
> >     SUBSYSTEM=="block", KERNEL=="sr[0-9]*", NAME="scd%n", SYMLINK+="sr%
> 
> We do not want to change kernel names unless absolutely needed,
> especially for block devices we don't want to do that.
> 
> Where does the "deprecated prefix" information comes from? We should
> check with the kernel SCSI maintainers, and then do this, if it's the
> right thing.
> 
We set this up as devices.txt says to, with the name as the
non-deprecated one.

> > rtc
> > 
> >   our rtc rules are more careful about only symlinking the CMOS
> >   to /dev/rtc:
> > 
> >     SUBSYSTEM=="rtc", DRIVERS=="rtc_cmos", SYMLINK+="rtc"
> 
> Sounds fine if we do not want that for other drivers? It's only
> compatibility for old x86 behavior?
> 
As far as I know, I talked with the upstream about it for a while before
adding the rule.

> >   additionally, we place these in the "audio" group
> > 
> >     SUBSYSTEM=="rtc", GROUP="audio"
> 
> Hmm, that sounds like a really weird workaround of some audio problem.
> Is this really what we want. :)
> 
Actually, drop this one.  We don't want it anymore.

> > raw1394
> > 
> >   This should be in the "disk" group, not the "video" group; it can be
> >   used to execute code as root
> 
> It may not belong into be in the "video" group, but you definitely also
> don't want to put anybody in the "disk" group to access it, it's almost
> the same as being root. We can leave it as root, if we need to change it
> for security measures.
> 
> You can not execute code on the host, only on a device, which may be
> another box connected over firewire, right?
> 
Loopback cable?  This is something our security guy and kernel
maintainer (who is the upstream maintainer of the old firewire system)
felt strongly about -- obviously new firewire stack is much better
anyway.

> >   you also don't rename some devices, and thus disagree
> > 
> >     KERNEL=="controlC[0-9]*", NAME="snd/%k"
> >     KERNEL=="hwC[D0-9]*", NAME="snd/%k"
> >     KERNEL=="midiC[D0-9]*", NAME="snd/%k"
> >     KERNEL=="pcmC[D0-9cp]*", NAME="snd/%k"
> >     KERNEL=="seq", NAME="snd/%k"
> >     KERNEL=="timer", NAME="snd/%k"
> 
> That's in packages/40-alsa.rules. We do not carry it in the default udev
> rules, but in the alsa package.
> 
These aren't installed by default, right?  If we're all using the same
rules, is there any reason not to?

> > symlinks
> > 
> >   you have lots of extra symlinks, any particular reason?  are you paid
> >   by the symlink? :p
> > 
> >     X0R -> null		??!! WTF
> 
> It's in devices.txt. :) Harald wanted it. :)
> 
I'm ok with it if it's in devices.txt :p

> >     vbi -> vbi0
> >     radio -> radio0
> >     video -> video0
> 
> That is needed for some old tools. People have complained.
> 
We've not had these, and nobody complains too loudly ;)  I want to avoid
adding legacy symlinks to our config if I can help it, because people
then will complain if they go away again later :p

> >   no /dev/pilot? :)
> 
> No, that doesn't work reliably, you never know which port is the right
> one. It can not be done with udev today, only with a specific
> vendor/device list match.
> 
See vbi, radio, video, etc. above :p  you can't have them and complain
about this ;)  I'd rather have none of them, and leave legacy/deprecated
symlinks to distro-specific rules.

> >   tty		620	instead of 600? your ttys are writable?
> 
> No idea. Just change it?
> 
wall/write related.  I'm happy to change to your default.

> > groups
> > 
> >   no cdrom group?
> >   no scanner group?
> >   no tape group?
> >   no dialout group?	- this may be Debianish, but we rely on it
> 
> We do not want to put users in any such groups, but use dynamically
> assigned and managed ACLs for it. We use groups only for easy setups of
> system daemons like "video", for a streaming server and such. These
> groups do not make much sense for Fedora or SUSE, and you may want
> to put them in your own rules, in cases we don't find a common solution.
> 
Agree, we can handle these in our own rules.

Scott
-- 
Scott James Remnant
scott@canonical.com

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 197 bytes --]

  parent reply	other threads:[~2008-12-22  9:39 UTC|newest]

Thread overview: 29+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-12-19  6:47 Moving Ubuntu to upstream udev rules Scott James Remnant
2008-12-19 16:03 ` Kay Sievers
2008-12-19 16:10 ` Marco d'Itri
2008-12-20  8:12 ` Piter PUNK
2008-12-20  8:41 ` Robby Workman
2008-12-20 11:30 ` Marco d'Itri
2008-12-20 18:33 ` Dan Nicholson
2008-12-21 12:39 ` Kay Sievers
2008-12-21 12:42 ` Kay Sievers
2008-12-21 12:53 ` Kay Sievers
2008-12-21 13:07 ` Marco d'Itri
2008-12-21 14:29 ` Kay Sievers
2008-12-21 21:52 ` Piter PUNK
2008-12-22  2:00 ` Kay Sievers
2008-12-22  8:41 ` Harald Hoyer
2008-12-22  8:53 ` Harald Hoyer
2008-12-22  9:39 ` Scott James Remnant [this message]
2008-12-22 10:00 ` Bernhard Walle
2008-12-22 11:57 ` Marco d'Itri
2008-12-22 12:37 ` Kay Sievers
2008-12-22 12:39 ` Scott James Remnant
2008-12-22 12:40 ` Kay Sievers
2008-12-22 12:42 ` Kay Sievers
2008-12-22 12:43 ` Kay Sievers
2008-12-22 13:57 ` Kay Sievers
2008-12-25 23:26 ` Karel Zak
2008-12-25 23:39 ` Kay Sievers
2008-12-26  0:26 ` Karel Zak
2008-12-26  0:48 ` Kay Sievers

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1229938782.6944.12.camel@quest \
    --to=scott@canonical.com \
    --cc=linux-hotplug@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).