From: Ben Collins <ben.collins@ubuntu.com>
To: arjan@infradead.org
Cc: Alan <alan@lxorguk.ukuu.org.uk>,
linux-kernel@vger.kernel.org, Eric Piel <eric.piel@tremplin-utc>
Subject: Re: [RFC] Include ACPI DSDT from INITRD patch into mainline
Date: Fri, 01 Dec 2006 17:35:38 -0500 [thread overview]
Message-ID: <1165012538.5257.992.camel@gullible> (raw)
In-Reply-To: <1165009747.3233.108.camel@laptopd505.fenrus.org>
On Fri, 2006-12-01 at 22:49 +0100, Arjan van de Ven wrote:
> On Fri, 2006-12-01 at 16:01 -0500, Ben Collins wrote:
> > On Fri, 2006-12-01 at 20:45 +0100, Arjan van de Ven wrote:
> > > On Fri, 2006-12-01 at 14:35 -0500, Ben Collins wrote:
> > > > What about the point that userspace (udev, and such) is not available
> > > > when DSDT loading needs to occur? Init hasn't even started at that
> > > > point.
> > >
> > > that's a moot point; you need to load firmware from the initramfs ANYWAY
> > > for things like qlogic and others...
> >
> > I don't see how that relates. The DSDT needs to be loaded even before
> > driver initialization begins.
>
> in fact it needs to be loaded even before the ACPI engine starts
> executing, otherwise you're hot-replacing code underneath a live
> system... at which point you can do this same feature in another way :)
> there already is a feature that builds a dsdt into the kernel image, all
> a distro would need is a bit of objcopy magic to build the right one
> into the vmlinuz...
Providing object files for on-demand relinking of the kernel just adds a
shit load of overhead. If you're suggesting modifying vmlinuz in place
instead, that just seems really undesirable. Last thing I want is
something mucking with the kernel binary.
It's easier for me to keep this patch in my tree, especially since most
users have come to expect this as the "standard" method for inserting
their DSDT replacement.
The only other thing I can think of is grub providing the DSDT just like
it does for the initrd.
next prev parent reply other threads:[~2006-12-01 22:35 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-12-01 18:36 Ben Collins
2006-12-01 18:56 ` Alan
2006-12-01 19:35 ` Ben Collins
2006-12-01 19:45 ` Arjan van de Ven
2006-12-01 21:01 ` Ben Collins
2006-12-01 21:49 ` Arjan van de Ven
2006-12-01 22:35 ` Ben Collins [this message]
2006-12-02 7:45 ` Arjan van de Ven
2006-12-04 10:58 ` Stefan Seyfried
2006-12-01 19:53 ` Alan
2006-12-01 20:58 ` Ben Collins
2006-12-01 21:55 ` Alan
2006-12-01 22:01 ` Arkadiusz Miskiewicz
2006-12-01 22:17 ` Alan
2006-12-02 10:28 ` Arjan van de Ven
2006-12-02 12:50 ` Acer smart battery (was Re: [RFC] Include ACPI DSDT from INITRD patch into mainline) Pavel Machek
2006-12-02 17:34 ` Arkadiusz Miskiewicz
2006-12-02 18:58 ` Pavel Machek
2006-12-02 0:21 ` [RFC] Include ACPI DSDT from INITRD patch into mainline Dave Jones
2006-12-02 10:25 ` Arjan van de Ven
2006-12-02 10:30 ` Arjan van de Ven
2006-12-02 11:11 ` Jan Engelhardt
2006-12-02 16:15 ` Sergey Vlasov
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=1165012538.5257.992.camel@gullible \
--to=ben.collins@ubuntu.com \
--cc=alan@lxorguk.ukuu.org.uk \
--cc=arjan@infradead.org \
--cc=eric.piel@tremplin-utc \
--cc=linux-kernel@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 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®