[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <20080720083530.1bc60ead@infradead.org>
Date: Sun, 20 Jul 2008 08:35:30 -0700
From: Arjan van de Ven <arjan@...radead.org>
To: Rene Herman <rene.herman@...access.nl>
Cc: linux-kernel@...r.kernel.org, mingo@...e.hu,
Pavel Machek <pavel@....cz>, "Rafael J. Wysocki" <rjw@...k.pl>,
Paul Diefenbaugh <paul.s.diefenbaugh@...el.com>,
Andy Grover <andrew.grover@...el.com>,
Len Brown <lenb@...nel.org>
Subject: Re: [patch 1/3] fastboot: Create a "asynchronous" initlevel
On Sun, 20 Jul 2008 16:20:06 +0200
Rene Herman <rene.herman@...access.nl> wrote:
> On 20-07-08 13:10, Arjan van de Ven wrote:
>
> > On Sun, 20 Jul 2008 09:23:31 +0200
> > Rene Herman <rene.herman@...access.nl> wrote:
> >
> >> Yes, I see. Unfortunately, WITH your patches, driver_probe_done()
> >> would also no longer be safe when run from a late_initcall() it
> >> would appear.
> >
> > true for now (but see below)
> >
> >> I have the sneaking suspicion that this is a bit of a fundamental
> >> issue. Turning some of the driver level (6) async basicaly removes
> >> the ordering between drivers and late_initcall (level 7).
> >
> > I was hoping to not need this ordering.
>
> May have found an issue with 3/3 for this same reason.
> > Thankfully it doesn't destroy it, the reason for this is that level
> > 6 itself tends to take long enough to get benefits. It's just that
> > if we can get both 6 and 7 it's nicer. But if we end up needing to
> > sync, so be it.
>
> I worry...
ok you convinced me... I'll post a followon patch that will have this
shortly once it finishes compiling/booting.
--
If you want to reach me at my work email, use arjan@...ux.intel.com
For development, discussion and tips for power savings,
visit http://www.lesswatts.org
--
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