Re: Hotunplug

[David Brownell <david-b@pacbell.net>]

> +    # Create a directory to hold remover programs. Close it up so
> +    # that only hotplug scripts can write to this directory.
> +    mkdir /var/run/usb
> +    chmod 0700 /var/run/usb
> +
> 
> Things like this should be done by the installer of the package, so as
> to allow different distros handle this properly.
>
> Also, the creation of directories should be done only once, not every
> time.

Yes.  Though ... shouldn't the directory be configurable?
A default of /var/run/usb would be fine; the existing
machinery in /etc/sysconfig/usb should be able to change
such defaults.

And there are bootstrap issues too ... if this is run through
the /etc/rc.d/init.d/hotplug script, "/var/run/usb" won't be
readonly, but perhaps existing scripts should be run and then
removed (for crash recovery, with some appropriate ACTION
set in the environment during hotplug startup).


> If David doesn't object, I'll add this patch to cvs (without the above
> change, but with the .spec file changed.)

I'll not object, though as Miles noted there may be interesting
races on unplugging that don't get seen very often ... also, the
README directions for manual install should get updated.

- Dave




_______________________________________________
Linux-hotplug-devel mailing list  http://linux-hotplug.sourceforge.net
Linux-hotplug-devel@lists.sourceforge.net
http://lists.sourceforge.net/lists/listinfo/linux-hotplug-devel