acpi: strip compiler info in built-in DSDT if any

IASL stores it's revision in each table header it generates.
That's a problem since guests see a change each time
they move between hypervisors.
We generally fill our own info for tables,
but we forgot to do this for the built-in DSDT.

Signed-off-by: Michael S. Tsirkin <mst@redhat.com>

diff --git a/src/fw/acpi.c b/src/fw/acpi.c
index 4504a860eec0024d5f9235c60ad0532b4768e900..042d571ef20b44c1b13385f718d830b38015336f 100644 (file)
--- a/src/fw/acpi.c
+++ b/src/fw/acpi.c
@@ -676,13 +676,16 @@ acpi_setup(void)
 
     if (CONFIG_ACPI_DSDT && fadt && !fadt->dsdt) {
         /* default DSDT */
-        void *dsdt = malloc_high(sizeof(AmlCode));
+        struct acpi_table_header *dsdt = malloc_high(sizeof(AmlCode));
         if (!dsdt) {
             warn_noalloc();
             return;
         }
         memcpy(dsdt, AmlCode, sizeof(AmlCode));
         fill_dsdt(fadt, dsdt);
+        /* Strip out compiler-generated header if any */
+        memset(dsdt, 0, sizeof *dsdt);
+        build_header(dsdt, DSDT_SIGNATURE, sizeof(AmlCode), 1);
     }
 
     // Build final rsdt table