Re: PATCH : new key MODEFILE

[Kay Sievers <kay.sievers@vrfy.org>]

On Tue, 2007-02-27 at 09:19 -0800, Jean Tourrilhes wrote:
> On Tue, Feb 27, 2007 at 01:47:27PM +0100, Kay Sievers wrote:
> > On Mon, 2007-02-26 at 09:42 -0800, Jean Tourrilhes wrote:
> > 
> > > I designed it so that it's as
> > > generic as possible and can be used for many other things.
> > > 	For example, you can do :
> > > ------------------------------------------
> > > KERNEL="sdc1", MODEMASK{/mnt/media}="0100", RUN="mount /dev/%k /mnt/media"
> > > ------------------------------------------

> > > 	You could also do :
> > > ------------------------------------------
> > > KERNEL="sdb", MODEMASK{sda}="0001", MODEMASK{sda}="0002", MODE="0666"
> > > ------------------------------------------
> > 
> > The node usually doesn't exist at this time. :)
> 
> 	Check again the rule, it's more subtle than that.

We don't know, if sda is created before sdb. I don't think we should
match on other device properties, that just asks for timing trouble,
especially during coldplug where everything (that hasn't a parent-child
relation) runs in parallel.


How about a TEST{<mask>}=<file> key that accepts the usual:
  e, f, d, r, w, x, or an octal bitmask?
It would look like:
  TEST{r}="/etc/foo"
  TEST{w}="/tmp/foo"
  TEST{0002}="/tmp/foo"

I don't think, we should default to /dev, reading from there is nothing
we want to encourage anybody to do. And I think a single key per rule
would be fine for a start.

Thanks,
Kay


-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CIDÞVDEV
_______________________________________________
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