From: Thomas Renninger <trenn@suse.de>
To: Yinghai Lu <yinghai@kernel.org>
Cc: eric.piel@tremplin-utc.net, vojcek@tlen.pl, dsdt@gaugusch.at,
linux-acpi@vger.kernel.org, linux-kernel@vger.kernel.org,
x86@kernel.org, Lin Ming <ming.m.lin@intel.com>,
lenb@kernel.org, robert.moore@intel.com, hpa@zytor.com
Subject: Re: [PATCH] ACPI: Implement overriding of arbitrary ACPI tables via initrd
Date: Sat, 24 Mar 2012 02:22:26 +0100 [thread overview]
Message-ID: <201203240222.27165.trenn@suse.de> (raw)
In-Reply-To: <CAE9FiQX8hfD6o_UZxR-pLEZ631G+D5odP3MmVOZ=MN2AFPJ+Lw@mail.gmail.com>
On Saturday 24 March 2012 02:05:18 Yinghai Lu wrote:
> On Fri, Mar 23, 2012 at 1:54 PM, Yinghai Lu <yinghai@kernel.org> wrote:
> > On Fri, Mar 23, 2012 at 7:29 AM, Thomas Renninger <trenn@suse.de> wrote:
> >> Details can be found in:
> >> Documentation/acpi/initrd_table_override.txt
> >>
> >> Additional dmesg output of a booted system with
> >> FACP (FADT), DSDT and SSDT (the 9th dynamically loaded one)
> >> tables overridden (with ### marked comments):
> >>
> >> ### ACPI tables found glued to initrd
> >> DSDT ACPI table found in initrd - size: 16234
> >> FACP ACPI table found in initrd - size: 116
> >> SSDT ACPI table found in initrd - size: 334
> >> ### Re-printed e820 map via e820_update() with additionally created
> >> ### ACPI data section at 0xcff55000 where the ACPI tables passed via
> >> ### initrd where copied to
> >> modified physical RAM map:
> >> ...
> >> ### New ACPI data section:
> >> modified: 00000000cff55000 - 00000000cff5912c (ACPI data)
> >> ### BIOS e820 provided ACPI data section:
> >> modified: 00000000cff60000 - 00000000cff69000 (ACPI data)
> >> ...
> >> ### Total size of all ACPI tables glued to initrd
> >> ### The address is initrd_start which gets updated to
> >> ### initrd_start = initrd_start + "size of all ACPI tables glued to initrd"
> >> Found acpi tables of size: 16684 at 0xffff8800374c4000
> >>
> >> Disabling lock debugging due to kernel taint
> >> ### initrd provided FACP and DSDT tables are used instead of BIOS provided ones
> >> ACPI: FACP @ 0x00000000cff68dd8 Phys table override, replaced with:
> >> ACPI: FACP 00000000cff58f6a 00074 (v01 INTEL TUMWATER 06040000 PTL 00000003)
> >> ACPI: DSDT @ 0x00000000cff649d4 Phys table override, replaced with:
> >> ACPI: DSDT 00000000cff55000 04404 (v01 Intel BLAKFORD 06040000 MSFT 0100000E)
> >> ...
> >> ### Much later, the 9th (/sys/firmware/acpi/table/dynamic/SSDT9) dynamically
> >> ### loaded ACPI table matches and gets overridden:
> >> ACPI: SSDT @ 0x00000000cff64824 Phys table override, replaced with:
> >> ACPI: SSDT 00000000cff58fde 0014E (v01 PmRef Cpu7Ist 00003000 INTL 20110316)
> >> ACPI: Dynamic OEM Table Load:
> >> ACPI: SSDT (null) 0014E (v01 PmRef Cpu7Ist 00003000 INTL 20110316)
> >> ...
> >>
> >> If the initrd does not start with a valid ACPI table signature or the ACPI
> >> table's checksum is wrong, there is no functional change.
>
> updated it a little bit to remove 2 #ifdef. please check attached.
Plus getting rid of the e820 stuff in drivers/acpi/osl.c
Thanks, nice cleanups.
Thomas
next prev parent reply other threads:[~2012-03-24 1:22 UTC|newest]
Thread overview: 40+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-03-23 14:29 [PATCH] ACPI: Implement overriding of arbitrary ACPI tables via initrd Thomas Renninger
2012-03-23 15:51 ` Thomas Renninger
2012-03-23 20:05 ` H. Peter Anvin
2012-03-24 1:42 ` Thomas Renninger
2012-03-24 2:01 ` H. Peter Anvin
2012-03-24 3:02 ` Thomas Renninger
2012-03-24 4:40 ` H. Peter Anvin
2012-03-24 4:43 ` H. Peter Anvin
2012-03-24 4:50 ` Yinghai Lu
2012-03-24 4:58 ` H. Peter Anvin
2012-03-24 9:24 ` Borislav Petkov
2012-03-24 18:49 ` H. Peter Anvin
2012-03-25 8:54 ` Borislav Petkov
2012-03-26 1:36 ` H. Peter Anvin
2012-03-26 14:21 ` Konrad Rzeszutek Wilk
2012-03-26 0:45 ` Thomas Renninger
2012-03-26 1:25 ` H. Peter Anvin
2012-03-26 14:19 ` Thomas Renninger
2012-03-26 14:46 ` H. Peter Anvin
2012-03-26 14:51 ` Thomas Renninger
2012-03-27 4:15 ` H. Peter Anvin
2012-03-27 4:46 ` Peter Stuge
2012-03-27 6:18 ` H. Peter Anvin
2012-03-24 18:42 ` Konrad Rzeszutek Wilk
2012-03-24 19:15 ` H. Peter Anvin
2012-03-24 19:17 ` Konrad Rzeszutek Wilk
2012-03-24 19:44 ` H. Peter Anvin
2012-03-24 22:21 ` H. Peter Anvin
2012-03-24 22:44 ` H. Peter Anvin
2012-03-25 9:25 ` Borislav Petkov
2012-03-25 23:29 ` H. Peter Anvin
2012-03-25 4:17 ` H. Peter Anvin
2012-03-25 9:07 ` Borislav Petkov
2012-03-23 20:54 ` Yinghai Lu
2012-03-24 0:15 ` Yinghai Lu
2012-03-24 1:05 ` Yinghai Lu
2012-03-24 1:22 ` Thomas Renninger [this message]
2012-03-24 1:26 ` Thomas Renninger
2012-03-24 4:41 ` Yinghai Lu
2012-03-26 0:45 ` Thomas Renninger
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=201203240222.27165.trenn@suse.de \
--to=trenn@suse.de \
--cc=dsdt@gaugusch.at \
--cc=eric.piel@tremplin-utc.net \
--cc=hpa@zytor.com \
--cc=lenb@kernel.org \
--cc=linux-acpi@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=ming.m.lin@intel.com \
--cc=robert.moore@intel.com \
--cc=vojcek@tlen.pl \
--cc=x86@kernel.org \
--cc=yinghai@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