linux-hotplug.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* Re: Call for Assistance - udev
@ 2007-08-24 18:14 Alexander E. Patrakov
  2007-08-24 18:17 ` Alexander E. Patrakov
                   ` (3 more replies)
  0 siblings, 4 replies; 5+ messages in thread
From: Alexander E. Patrakov @ 2007-08-24 18:14 UTC (permalink / raw)
  To: linux-hotplug

Greg Schafer wrote:
> Anyhoo, it seems the latest release (udev-115) finally installs something
> that looks like a sane default config.

Do you really think this is sane?

1) Strange symlinks that never existed before:
KERNEL="null", SYMLINK+="XOR"
KERNEL="ram0", SYMLINK+="ramdisk"
KERNEL="ram1", SYMLINK+="ram"
KERNEL="lp[0-9]*", GROUP="lp" SYMLINK+="par%n"

2) Mismatch between /dev/random and /dev/urandom permissions:
KERNEL="...|random", MODE="0666"
KERNEL="urandom", MODE="0644"

3) Syntax errors (missing comma, extra quote)
KERNEL="lp[0-9]*", GROUP="lp" SYMLINK+="par%n"
KERNEL="ht[0-9]*|nht[0-9]*",   GROUP="disk""

4) persistent CD symlinks and network interface names are provided only 
by distro-specific rules

-- 
Alexander E. Patrakov

-------------------------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc.
Still grepping through log files to find problems?  Stop.
Now Search log events and configuration files using AJAX and a browser.
Download your FREE copy of Splunk now >>  http://get.splunk.com/
_______________________________________________
Linux-hotplug-devel mailing list  http://linux-hotplug.sourceforge.net
Linux-hotplug-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linux-hotplug-devel

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: Call for Assistance - udev
  2007-08-24 18:14 Call for Assistance - udev Alexander E. Patrakov
@ 2007-08-24 18:17 ` Alexander E. Patrakov
  2007-08-24 18:28 ` Kay Sievers
                   ` (2 subsequent siblings)
  3 siblings, 0 replies; 5+ messages in thread
From: Alexander E. Patrakov @ 2007-08-24 18:17 UTC (permalink / raw)
  To: linux-hotplug

I wrote:
> 4) persistent CD symlinks and network interface names are provided only 
> by distro-specific rules
>   

Scratch that - I should not be awake at night. The other objections 
still apply.

-- 
Alexander E. Patrakov

-------------------------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc.
Still grepping through log files to find problems?  Stop.
Now Search log events and configuration files using AJAX and a browser.
Download your FREE copy of Splunk now >>  http://get.splunk.com/
_______________________________________________
Linux-hotplug-devel mailing list  http://linux-hotplug.sourceforge.net
Linux-hotplug-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linux-hotplug-devel

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: Call for Assistance - udev
  2007-08-24 18:14 Call for Assistance - udev Alexander E. Patrakov
  2007-08-24 18:17 ` Alexander E. Patrakov
@ 2007-08-24 18:28 ` Kay Sievers
  2007-08-25  4:36 ` Alexander E. Patrakov
  2007-08-25 12:08 ` Kay Sievers
  3 siblings, 0 replies; 5+ messages in thread
From: Kay Sievers @ 2007-08-24 18:28 UTC (permalink / raw)
  To: linux-hotplug

On 8/24/07, Alexander E. Patrakov <patrakov@ums.usu.ru> wrote:
> Greg Schafer wrote:
> > Anyhoo, it seems the latest release (udev-115) finally installs something
> > that looks like a sane default config.
>
> Do you really think this is sane?
>
> 1) Strange symlinks that never existed before:
> KERNEL="null", SYMLINK+="XOR"

Old ancient stuff. Mentioned in linux/Documentation/devices.txt

> KERNEL="ram0", SYMLINK+="ramdisk"

Recommended in devices.txt.

> KERNEL="ram1", SYMLINK+="ram"

Don't know.

> KERNEL="lp[0-9]*", GROUP="lp" SYMLINK+="par%n"

Don't know.

> 2) Mismatch between /dev/random and /dev/urandom permissions:
> KERNEL="...|random", MODE="0666"
> KERNEL="urandom", MODE="0644"

You can write to random to add entropy to the pool. I guess, you can't
write to urandom.

> 3) Syntax errors (missing comma, extra quote)
> KERNEL="lp[0-9]*", GROUP="lp" SYMLINK+="par%n"
> KERNEL="ht[0-9]*|nht[0-9]*",   GROUP="disk""

I'll fix that now. Will probably just work though. :)

Kay

-------------------------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc.
Still grepping through log files to find problems?  Stop.
Now Search log events and configuration files using AJAX and a browser.
Download your FREE copy of Splunk now >>  http://get.splunk.com/
_______________________________________________
Linux-hotplug-devel mailing list  http://linux-hotplug.sourceforge.net
Linux-hotplug-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linux-hotplug-devel

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: Call for Assistance - udev
  2007-08-24 18:14 Call for Assistance - udev Alexander E. Patrakov
  2007-08-24 18:17 ` Alexander E. Patrakov
  2007-08-24 18:28 ` Kay Sievers
@ 2007-08-25  4:36 ` Alexander E. Patrakov
  2007-08-25 12:08 ` Kay Sievers
  3 siblings, 0 replies; 5+ messages in thread
From: Alexander E. Patrakov @ 2007-08-25  4:36 UTC (permalink / raw)
  To: linux-hotplug

Kay Sievers wrote:
> On 8/24/07, Alexander E. Patrakov <patrakov@ums.usu.ru> wrote:
>   
>> 2) Mismatch between /dev/random and /dev/urandom permissions:
>> KERNEL="...|random", MODE="0666"
>> KERNEL="urandom", MODE="0644"
>>     
>
> You can write to random to add entropy to the pool. I guess, you can't
> write to urandom.
>   

The drivers/char/random.c file says:

const struct file_operations random_fops = {
        .read  = random_read,
        .write = random_write,
        .poll  = random_poll,
        .ioctl = random_ioctl,
};

const struct file_operations urandom_fops = {
        .read  = urandom_read,
        .write = random_write,
        .ioctl = random_ioctl,
};

So there is no valid reason for this mismatch.

-- 
Alexander E. Patrakov

-------------------------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc.
Still grepping through log files to find problems?  Stop.
Now Search log events and configuration files using AJAX and a browser.
Download your FREE copy of Splunk now >>  http://get.splunk.com/
_______________________________________________
Linux-hotplug-devel mailing list  http://linux-hotplug.sourceforge.net
Linux-hotplug-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linux-hotplug-devel

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: Call for Assistance - udev
  2007-08-24 18:14 Call for Assistance - udev Alexander E. Patrakov
                   ` (2 preceding siblings ...)
  2007-08-25  4:36 ` Alexander E. Patrakov
@ 2007-08-25 12:08 ` Kay Sievers
  3 siblings, 0 replies; 5+ messages in thread
From: Kay Sievers @ 2007-08-25 12:08 UTC (permalink / raw)
  To: linux-hotplug

On Sat, 2007-08-25 at 10:36 +0600, Alexander E. Patrakov wrote:
> Kay Sievers wrote:
> > On 8/24/07, Alexander E. Patrakov <patrakov@ums.usu.ru> wrote:
> >   
> >> 2) Mismatch between /dev/random and /dev/urandom permissions:
> >> KERNEL="...|random", MODE="0666"
> >> KERNEL="urandom", MODE="0644"
> >>     
> >
> > You can write to random to add entropy to the pool. I guess, you can't
> > write to urandom.
> >   
> 
> The drivers/char/random.c file says:
> 
> const struct file_operations random_fops = {
>         .read  = random_read,
>         .write = random_write,
>         .poll  = random_poll,
>         .ioctl = random_ioctl,
> };
> 
> const struct file_operations urandom_fops = {
>         .read  = urandom_read,
>         .write = random_write,
>         .ioctl = random_ioctl,
> };
> 
> So there is no valid reason for this mismatch.

Fine. I updated the rules.

Thanks,
Kay


-------------------------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc.
Still grepping through log files to find problems?  Stop.
Now Search log events and configuration files using AJAX and a browser.
Download your FREE copy of Splunk now >>  http://get.splunk.com/
_______________________________________________
Linux-hotplug-devel mailing list  http://linux-hotplug.sourceforge.net
Linux-hotplug-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linux-hotplug-devel

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2007-08-25 12:08 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2007-08-24 18:14 Call for Assistance - udev Alexander E. Patrakov
2007-08-24 18:17 ` Alexander E. Patrakov
2007-08-24 18:28 ` Kay Sievers
2007-08-25  4:36 ` Alexander E. Patrakov
2007-08-25 12:08 ` Kay Sievers

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).