[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <1162222115.30605.52.camel@frg-rhel40-em64t-03>
Date: Mon, 30 Oct 2006 16:28:35 +0100
From: Xavier Bestel <xavier.bestel@...e.fr>
To: Arjan van de Ven <arjan@...radead.org>
Cc: Kyle Moffett <mrmacman_g4@....com>,
Linus Torvalds <torvalds@...l.org>,
"Adam J. Richter" <adam@...drasil.com>, akpm@...l.org,
bunk@...sta.de, greg@...ah.com, linux-kernel@...r.kernel.org,
linux-pci@...ey.karlin.mff.cuni.cz, matthew@....cx, pavel@....cz,
shemminger@...l.org
Subject: Re: [patch] drivers: wait for threaded probes between initcall
levels
On Mon, 2006-10-30 at 16:05 +0100, Arjan van de Ven wrote:
> On Mon, 2006-10-30 at 16:00 +0100, Xavier Bestel wrote:
> > On Mon, 2006-10-30 at 15:38 +0100, Arjan van de Ven wrote:
> >
> > > how much of this complexity goes away if you consider the
> > > scanning/probing as a series of "work elements", and you end up with a
> > > queue of work elements that threads can pull work off one at a time (so
> > > that if one element blocks the others just continue to flow). If you
> > > then find, say, a new PCI bus you just put another work element to
> > > process it at the end of the queue, or you process it synchronously. Etc
> > > etc.
> > >
> > > All you need to scale then is the number of worker threads on the
> > > system, which should be relatively easy to size....
> > > (check every X miliseconds if there are more than X outstanding work
> > > elements, if there are, spawn one new worker thread if the total number
> > > of worker threads is less than the system wide max. Worker threads die
> > > if they have nothing to do for more than Y miliseconds)
> >
> > Instead of checking every X ms, just check at each job insertion.
>
> that would lead to a too eager amount of threads if processing the jobs
> is really really quick ...
Don't you have a "no more than X threads at once" limit ? You just
*check* at job insertion, not unconditionnaly fork.
Xav
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists