Re: [PATCH] add USB coldplay support into udevstart

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

On Mon, Aug 08, 2005 at 06:14:19PM +0200, Thierry Vignaud wrote:
> Kay Sievers <kay.sievers@vrfy.org> writes:
> 
> > > > the following patch from Olivier Blin adds USB coldplay support
> > > > into udevstart (patch is against udev-065):
> > > 
> > > Care to just change this to work for all busses that might have
> > > "modalias" files in them?  USB isn't the only one...
> > 
> > I don't think, that we want to do this in udevstart, at least at
> > it's current state. It will break to many setups, cause all the
> > distros have their own way to handle the coldplug case.
> > 
> > Everything in udevstart is serialized, which may not be optimal for
> > all the things to do during a coldplug session, cause every sleeping
> > script will delay the whole process.
> 
> udev'll serialize event processing anyway...

udev is a event process! udevd will not serialize the most cases!

> > In general I like the idea and we should experiment what the best
> > solution would be: event-replay, coldplug-binary, udevstart +
> > coldplug, synthesizing events and pass them to udevd for parallel
> > execution...
> > But this should happen in a separate binary, so we don't break too many
> > things at once. :)
> 
> udevstart already performs coldplug regarding classes & block devices.
> it's exactly what udevstart was designed for IMHO.

Yeah, great, tell me what it is designed for.

Thanks,
Kay


-------------------------------------------------------
SF.Net email is Sponsored by the Better Software Conference & EXPO
September 19-22, 2005 * San Francisco, CA * Development Lifecycle Practices
Agile & Plan-Driven Development * Managing Projects & Teams * Testing & QA
Security * Process Improvement & Measurement * http://www.sqe.com/bsce5sf
_______________________________________________
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