From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Brownell Date: Fri, 25 May 2001 14:19:11 +0000 Subject: Re: Hotunplug Message-Id: List-Id: References: In-Reply-To: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: linux-hotplug@vger.kernel.org > + # 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