From mboxrd@z Thu Jan 1 00:00:00 1970 From: Greg KH Date: Fri, 23 Jan 2004 03:46:47 +0000 Subject: Re: about split the udev Message-Id: <20040123034647.GA17166@kroah.com> List-Id: References: <3ACA40606221794F80A5670F0AF15F8402D4EE96@PDSMSX403.ccr.corp.intel.com> In-Reply-To: <3ACA40606221794F80A5670F0AF15F8402D4EE96@PDSMSX403.ccr.corp.intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: linux-hotplug@vger.kernel.org On Fri, Jan 23, 2004 at 04:37:22AM +0100, Kay Sievers wrote: > On Fri, Jan 23, 2004 at 02:00:29AM +0100, Kay Sievers wrote: > > On Thu, Jan 22, 2004 at 04:34:08PM -0800, Greg KH wrote: > > > On Fri, Jan 23, 2004 at 12:01:29AM +0100, Kay Sievers wrote: > > > > > > > > Here is the next updated updated version to apply to the lastet udev. > > > > I've added infrastructure for getting the state of the IPC queue in the > > > > sender and set the program to exec by the daemon. Also the magic key id > > > > is replaced by the usual key generation by path/nr. > > > > > > Very nice, I applied this. > > > > > > > It looks promising, I use it on my machine and my 4in1 USB-flash-reader > > > > connect/disconnect emits the events "randomly" but udevd is able to > > > > reorder it and calls our normal udev in the right order. > > > > > > Hm, how are you supposed to start up udevd? like: > > > ./udevd ./udev > > > > This should work, I've only tried './udevd /sbin/udev' and './udevd /bin/true' :) > > And '/sbin/udev' is the default. > > > > > > > > > Should we start the daemon if noone listens, when the sender is called? > > > > > > Yes. > > > > I will try it during the weekend. > > > > > > > > Should we convert our udev to a daemon, or stay with the exec? > > > > > > I don't know. I was thinking of turning our udev into the daemon, but > > > with this structure we might not need to. Let's play around with this > > > current structure for a bit now and see how it works out. > > > > Fine, I like it and we can use it for anything that expects a hotplug > > event and if someone don't like the reorder daemon he may use udev > > directly. > > Sorry, some code is missing. > Here is a fix to make the exec functional. > Could you please change 'test/udevd_test.sh' to executable and add > udevd und udevsender to the ignore file. I already made udevd_test.sh executable, and made some whitespace changes to udevd.c which cause this patch to not apply. Care to pull again and send the resulting diff? thanks, greg k-h ------------------------------------------------------- The SF.Net email is sponsored by EclipseCon 2004 Premiere Conference on Open Tools Development and Integration See the breadth of Eclipse activity. February 3-5 in Anaheim, CA. http://www.eclipsecon.org/osdn _______________________________________________ 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