Re: PATCH : new key MODEFILE

[Jean Tourrilhes <jt@hpl.hp.com>]

On Fri, Mar 16, 2007 at 01:55:40AM +0100, Kay Sievers wrote:
> On Tue, 2007-02-27 at 09:19 -0800, Jean Tourrilhes wrote:
> > 
> > 	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.

	Ok, I admit that my rule was contrieved. And most likely useless.

> 
> 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"

	It seems to me that accepting the letters would be a lot of
work, because of the variety of combinations, unless we can grab that
from a library. But we don't want to depend on libraries.
	As the setting of mode with MODE="" seems to accept only octal
bitmask, I was trying to match both with the same functionality. In
any case, if you implement letters, MODE="" would be the first place
where I would use it ;-)

> 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

	Thinking over it, we may want to think a bit more about
it. Just drop it in your todo list and in a few months check if it
still make sense. I use that method to great effect ;-)

	Thanks, and have fun...

	Jean

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