qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH V4 2/8] Provide ACPI SSDT table for TPM device +


From: Stefan Berger
Subject: Re: [Qemu-devel] [PATCH V4 2/8] Provide ACPI SSDT table for TPM device + S3 resume support
Date: Wed, 18 May 2011 21:18:16 -0400
User-agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.2.17) Gecko/20110428 Fedora/3.1.10-1.fc14 Lightning/1.0b3pre Thunderbird/3.1.10

On 05/18/2011 07:44 PM, Kevin O'Connor wrote:
On Wed, May 18, 2011 at 03:49:25PM -0400, Stefan Berger wrote:
On 04/12/2011 09:32 AM, Stefan Berger wrote:
+
+static u32 add_tpm_device(void **tpm_addr, void **tcpa_addr)
+{
+    struct tcpa_descriptor_rev2 *tcpa;
+
+    *tpm_addr = NULL;
+    *tcpa_addr = NULL;
+
+    if (has_working_tpm()) {
+        u32 laml = 64 * 1024;
Kevin,

   the above line prepares 64 kb to be allocated for an ACPI table.
It works fine if booting a VM from an image. However, when passing
the kernel, initrd and command line parameters to Qemu directly
(-kernel, -append, ..), the 64kb above seem to be too much -- I see
errors in the Linux kernel's dmesg when Linux tries to access the
ACPI tables. Lowering the above to 48kb (happens to) makes it work.
I am wondering whether something is copying into the ACPI area or
what else may be the reason - do you know?
I don't know of any reason why that would fail.  You can print out the
address and compare it to the e820 map that SeaBIOS outputs and verify
it is properly reserved.

Does increasing the amount of guest system ram help?
It helped. At some point the initrd and ACPI tables weren't overlapping anymore... I forgot to increase the size reserved for ACPI tables inside the Qemu code...

BTW, is this just a buffer or could the size be dynamically
calculated?

Following specs it's supposed to be 64kb. The BIOS writes logs into this area.

   Stefan
-Kevin

Attachment: dmsg.png
Description: PNG image


reply via email to

[Prev in Thread] Current Thread [Next in Thread]