From: "Éric Piel" <Eric.Piel@tremplin-utc.net>
To: Christoph Hellwig <hch@lst.de>
Cc: Sergey Vlasov <vsu@altlinux.ru>,
dsdt@gaugusch.at, len.brown@intel.com,
linux-kernel@vger.kernel.org, Linus Torvalds <torvalds@osdl.org>,
trenn@suse.de
Subject: Re: [PATCH] Allow populate_rootfs() to be called early
Date: Thu, 21 Feb 2008 22:27:09 +0100 [thread overview]
Message-ID: <47BDECAD.3020004@tremplin-utc.net> (raw)
In-Reply-To: <20080221190455.GA7275@lst.de>
21/02/08 20:04, Christoph Hellwig wrote/a écrit:
> On Thu, Feb 21, 2008 at 08:02:36PM +0100, Eric Piel wrote:
>> It's been a week and no one has screamed, so I guess the idea looks fine
>> to everyone :-)
>>
>> Here is a boot tested patch for integration. In addition to the previous
>> version, it removes also rootfs_initcall(), and uses fs_initcall_sync().
>> There should be absolutely no difference in behaviour as there was no
>> user of fs_initcall_sync() in the kernel.
>>
>> I'm not sure what should be the best tree to go through. Maybe the ACPI
>> tree, as all this is needed for CONFIG_ACPI_CUSTOM_DSDT_INITRD...
>
> Actually I've just got a report today where we get an uninitialized
> spinlock due to this in -mm. It's probably possible to work around it,
> though.
What do you mean by "due to this"? Due to userspace helper being called
too early? Then this patch should fix the bug.
Eric
next prev parent reply other threads:[~2008-02-21 21:27 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-02-10 7:12 acpi dsts loading and populate_rootfs Christoph Hellwig
2008-02-10 7:14 ` Christoph Hellwig
2008-02-10 11:58 ` Eric Piel
2008-02-11 13:47 ` Sergey Vlasov
2008-02-11 23:41 ` Éric Piel
2008-02-21 19:02 ` [PATCH] Allow populate_rootfs() to be called early (was: acpi dsts loading and populate_rootfs) Éric Piel
2008-02-21 19:04 ` Christoph Hellwig
2008-02-21 21:27 ` Éric Piel [this message]
2008-02-23 19:40 ` [PATCH] Allow populate_rootfs() to be called early (resent, with sob) Éric Piel
2008-02-12 5:37 ` acpi dsts loading and populate_rootfs Christoph Hellwig
2008-02-21 18:46 ` Éric Piel
2008-02-22 8:51 ` Thomas Renninger
2008-02-22 9:05 ` Thomas Renninger
2008-02-22 9:53 ` Andi Kleen
2008-02-23 19:34 ` [PATCH] Use userland-like functions for reading the ACPI table Éric Piel
2008-02-23 20:45 ` Linus Torvalds
2008-02-24 1:31 ` Christoph Hellwig
2008-02-24 19:02 ` Éric Piel
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=47BDECAD.3020004@tremplin-utc.net \
--to=eric.piel@tremplin-utc.net \
--cc=dsdt@gaugusch.at \
--cc=hch@lst.de \
--cc=len.brown@intel.com \
--cc=linux-kernel@vger.kernel.org \
--cc=torvalds@osdl.org \
--cc=trenn@suse.de \
--cc=vsu@altlinux.ru \
/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.