mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Stefan Richter <stefanr@s5r6.in-berlin.de>
To: "Huang, Ying" <ying.huang@intel.com>
Cc: Greg K-H <greg@kroah.com>,
	Cornelia Huck <cornelia.huck@de.ibm.com>,
	Adrian Bunk <bunk@stusta.de>,
	david@lang.hm, David Miller <davem@davemloft.net>,
	Duncan Sands <duncan.sands@math.u-psud.fr>,
	Phillip Susi <psusi@cfl.rr.com>,
	linux-kernel <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH] driver core: multithreaded probing - more parallelism control
Date: Thu, 21 Jun 2007 18:21:42 +0200	[thread overview]
Message-ID: <467AA596.5030509@s5r6.in-berlin.de> (raw)
In-Reply-To: <9D7649D18729DE4BB2BD7B494F7FEDC218022B@pdsmsx415.ccr.corp.intel.com>

Huang, Ying wrote:
> I think the queue IDs of different subsystem need not to be exclusive.
> The subsystem can allocate queue IDs arbitrarily. If one queue ID is
> shared between several subsystems, corresponding probing will be
> serialized. This will slow down the probing unnecessarily, but there
> will be no race condition.

Parallelism between subsystems may be interesting during boot ==
"coldplug", /if/ the machine has time-consuming devices to probe on
/different/ types of buses.  Of course some machines do the really
time-consuming stuff on only one type of bus.  Granted, parallelism
betwen subsystems is not very interesting anymore later after boot ==
"hotplug".

[...]
> Whether is it possible that a new unit is inserted during probing
> kthread is running.

Nodes and units on nodes may came and go at arbitrary points in time,
and I'm sure similar things can be said about the majority of other bus
architectures or network architectures.  We take this into account.

(The old FireWire stack will re-enter the main loop of the bus scanning
thread sometime after a bus reset event signaled that nodes or units may
have appeared or disappeared.  The new FireWire stack will schedule
respective scanning workqueue jobs after such an event.)
-- 
Stefan Richter
-=====-=-=== -==- =-=-=
http://arcgraph.de/sr/

  reply	other threads:[~2007-06-21 16:21 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <1182373258.30574.30.camel@caritas-dev.intel.com>
2007-06-20 15:09 ` Stefan Richter
2007-06-20 15:14   ` Stefan Richter
2007-06-21  9:38   ` Huang, Ying
2007-06-21  8:49     ` Stefan Richter
2007-06-21 13:51       ` Huang, Ying
2007-06-21 16:21         ` Stefan Richter [this message]
2007-06-22  9:52           ` Huang, Ying
2007-07-03 15:04             ` Cornelia Huck
2007-06-24  7:06 ` Greg KH
2007-06-24  9:38   ` Stefan Richter
2007-06-24 15:04   ` [PATCH] driver core: multithreaded probing - more parallelismcontrol Huang, Ying
2007-06-25  8:16     ` Greg KH
2007-07-03  9:33       ` Cornelia Huck
2007-06-21 10:17 [PATCH] driver core: multithreaded probing - more parallelism control Huang, Ying

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=467AA596.5030509@s5r6.in-berlin.de \
    --to=stefanr@s5r6.in-berlin.de \
    --cc=bunk@stusta.de \
    --cc=cornelia.huck@de.ibm.com \
    --cc=davem@davemloft.net \
    --cc=david@lang.hm \
    --cc=duncan.sands@math.u-psud.fr \
    --cc=greg@kroah.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=psusi@cfl.rr.com \
    --cc=ying.huang@intel.com \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®