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: 44+ 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 14:29 ` 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: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-23 20:54 ` Yinghai Lu
2012-03-24 0:15 ` 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 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.