All of lore.kernel.org
 help / color / mirror / Atom feed
From: Thierry Vignaud <tvignaud@mandriva.com>
To: linux-hotplug@vger.kernel.org
Subject: Re: [PATCH] add USB coldplay support into udevstart
Date: Mon, 08 Aug 2005 16:14:19 +0000	[thread overview]
Message-ID: <m2irygquck.fsf@vador.mandriva.com> (raw)
In-Reply-To: <m2ll3dxya3.fsf@vador.mandriva.com>

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



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

  parent reply	other threads:[~2005-08-08 16:14 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-08-07 20:55 [PATCH] add USB coldplay support into udevstart Thierry Vignaud
2005-08-08 13:12 ` Greg KH
2005-08-08 13:49 ` Greg KH
2005-08-08 14:21 ` Greg KH
2005-08-08 16:09 ` Kay Sievers
2005-08-08 16:14 ` Thierry Vignaud [this message]
2005-08-08 16:17 ` Kay Sievers
2005-08-08 16:30 ` Olaf Hering
2005-08-08 19:40 ` Greg KH
2005-08-10 10:59 ` Olivier Blin
2005-08-10 14:47 ` Kay Sievers
2005-08-10 15:06 ` Marco d'Itri

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=m2irygquck.fsf@vador.mandriva.com \
    --to=tvignaud@mandriva.com \
    --cc=linux-hotplug@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.