[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [Xen-devel] [PATCH v3 4/5] tools/libacpi: update FADT layout to support version 5
Update the structure of the FADT table to version 5, and use that version for PVHv2 guests. Note that HVM guests will continue to use FADT 4. Signed-off-by: Roger Pau Monné <roger.pau@xxxxxxxxxx> --- Cc: Jan Beulich <jbeulich@xxxxxxxx> Cc: Andrew Cooper <andrew.cooper3@xxxxxxxxxx> Cc: Ian Jackson <ian.jackson@xxxxxxxxxxxxx> Cc: Wei Liu <wei.liu2@xxxxxxxxxx> Cc: boris.ostrovsky@xxxxxxxxxx Cc: konrad.wilk@xxxxxxxxxx --- tools/firmware/hvmloader/util.c | 2 +- tools/libacpi/acpi2_0.h | 9 ++++++--- tools/libacpi/build.c | 24 ++++++++++++++++++------ tools/libacpi/libacpi.h | 1 + tools/libacpi/static_tables.c | 8 ++++---- 5 files changed, 30 insertions(+), 14 deletions(-) diff --git a/tools/firmware/hvmloader/util.c b/tools/firmware/hvmloader/util.c index 3f86c88..5dbb448 100644 --- a/tools/firmware/hvmloader/util.c +++ b/tools/firmware/hvmloader/util.c @@ -949,7 +949,7 @@ void hvmloader_acpi_build_tables(struct acpi_config *config, config->table_flags |= ACPI_HAS_SSDT_S4; config->table_flags |= (ACPI_HAS_TCPA | ACPI_HAS_IOAPIC | ACPI_HAS_WAET | - ACPI_HAS_VGA | ACPI_HAS_8042); + ACPI_HAS_VGA | ACPI_HAS_8042 | ACPI_FADT_4); config->tis_hdr = (uint16_t *)ACPI_TIS_HDR_ADDRESS; diff --git a/tools/libacpi/acpi2_0.h b/tools/libacpi/acpi2_0.h index 500f95e..b8ef777 100644 --- a/tools/libacpi/acpi2_0.h +++ b/tools/libacpi/acpi2_0.h @@ -169,7 +169,7 @@ struct acpi_10_fadt { /* * Fixed ACPI Description Table Structure (FADT). */ -struct acpi_20_fadt { +struct acpi_50_fadt { struct acpi_header header; uint32_t firmware_ctrl; uint32_t dsdt; @@ -222,6 +222,8 @@ struct acpi_20_fadt { struct acpi_20_generic_address x_pm_tmr_blk; struct acpi_20_generic_address x_gpe0_blk; struct acpi_20_generic_address x_gpe1_blk; + struct acpi_20_generic_address sleep_control; + struct acpi_20_generic_address sleep_status; }; /* @@ -422,7 +424,7 @@ struct acpi_20_slit { */ #define ACPI_2_0_RSDP_SIGNATURE ASCII64('R','S','D',' ','P','T','R',' ') #define ACPI_2_0_FACS_SIGNATURE ASCII32('F','A','C','S') -#define ACPI_2_0_FADT_SIGNATURE ASCII32('F','A','C','P') +#define ACPI_FADT_SIGNATURE ASCII32('F','A','C','P') #define ACPI_2_0_MADT_SIGNATURE ASCII32('A','P','I','C') #define ACPI_2_0_RSDT_SIGNATURE ASCII32('R','S','D','T') #define ACPI_2_0_XSDT_SIGNATURE ASCII32('X','S','D','T') @@ -436,7 +438,8 @@ struct acpi_20_slit { * Table revision numbers. */ #define ACPI_2_0_RSDP_REVISION 0x02 -#define ACPI_2_0_FADT_REVISION 0x04 +#define ACPI_4_0_FADT_REVISION 0x04 +#define ACPI_5_0_FADT_REVISION 0x05 #define ACPI_2_0_MADT_REVISION 0x02 #define ACPI_2_0_RSDT_REVISION 0x01 #define ACPI_2_0_XSDT_REVISION 0x01 diff --git a/tools/libacpi/build.c b/tools/libacpi/build.c index 844dea1..eb40989 100644 --- a/tools/libacpi/build.c +++ b/tools/libacpi/build.c @@ -503,12 +503,13 @@ int acpi_build_tables(struct acpi_ctxt *ctxt, struct acpi_config *config) struct acpi_20_rsdp *rsdp; struct acpi_20_rsdt *rsdt; struct acpi_20_xsdt *xsdt; - struct acpi_20_fadt *fadt; + struct acpi_50_fadt *fadt; struct acpi_10_fadt *fadt_10; struct acpi_20_facs *facs; unsigned char *dsdt; unsigned long secondary_tables[ACPI_MAX_SECONDARY_TABLES]; int nr_secondaries, i; + unsigned long fadt_size; acpi_info = (struct acpi_info *)config->infop; memset(acpi_info, 0, sizeof(*acpi_info)); @@ -572,9 +573,16 @@ int acpi_build_tables(struct acpi_ctxt *ctxt, struct acpi_config *config) offsetof(struct acpi_header, checksum), sizeof(struct acpi_10_fadt)); - fadt = ctxt->mem_ops.alloc(ctxt, sizeof(struct acpi_20_fadt), 16); + /* + * NB: we can use offsetof because there's no padding between + * x_gpe1_blk and sleep_control. + */ + fadt_size = config->table_flags & ACPI_FADT_4 ? + offsetof(struct acpi_50_fadt, sleep_control) : + sizeof(struct acpi_50_fadt); + fadt = ctxt->mem_ops.alloc(ctxt, fadt_size, 16); if (!fadt) goto oom; - memcpy(fadt, &Fadt, sizeof(struct acpi_20_fadt)); + memcpy(fadt, &Fadt, fadt_size); fadt->dsdt = ctxt->mem_ops.v2p(ctxt, dsdt); fadt->x_dsdt = ctxt->mem_ops.v2p(ctxt, dsdt); fadt->firmware_ctrl = ctxt->mem_ops.v2p(ctxt, facs); @@ -583,9 +591,13 @@ int acpi_build_tables(struct acpi_ctxt *ctxt, struct acpi_config *config) fadt->iapc_boot_arch |= ACPI_FADT_NO_VGA; if ( config->table_flags & ACPI_HAS_8042 ) fadt->iapc_boot_arch |= ACPI_FADT_8042; - set_checksum(fadt, - offsetof(struct acpi_header, checksum), - sizeof(struct acpi_20_fadt)); + + if ( config->table_flags & ACPI_FADT_4 ) + { + fadt->header.revision = ACPI_4_0_FADT_REVISION; + fadt->header.length = fadt_size; + } + set_checksum(fadt, offsetof(struct acpi_header, checksum), fadt_size); nr_secondaries = construct_secondary_tables(ctxt, secondary_tables, config, acpi_info); diff --git a/tools/libacpi/libacpi.h b/tools/libacpi/libacpi.h index e0f1537..89bfcc6 100644 --- a/tools/libacpi/libacpi.h +++ b/tools/libacpi/libacpi.h @@ -32,6 +32,7 @@ #define ACPI_HAS_WAET (1<<9) #define ACPI_HAS_VGA (1<<10) #define ACPI_HAS_8042 (1<<11) +#define ACPI_FADT_4 (1<<12) struct xen_vmemrange; struct acpi_numa { diff --git a/tools/libacpi/static_tables.c b/tools/libacpi/static_tables.c index 1f6247d..ab7fa3b 100644 --- a/tools/libacpi/static_tables.c +++ b/tools/libacpi/static_tables.c @@ -38,11 +38,11 @@ struct acpi_20_facs Facs = { #define ACPI_PM_TMR_BLK_BIT_WIDTH 0x20 #define ACPI_PM_TMR_BLK_BIT_OFFSET 0x00 -struct acpi_20_fadt Fadt = { +struct acpi_50_fadt Fadt = { .header = { - .signature = ACPI_2_0_FADT_SIGNATURE, - .length = sizeof(struct acpi_20_fadt), - .revision = ACPI_2_0_FADT_REVISION, + .signature = ACPI_FADT_SIGNATURE, + .length = sizeof(struct acpi_50_fadt), + .revision = ACPI_5_0_FADT_REVISION, .oem_id = ACPI_OEM_ID, .oem_table_id = ACPI_OEM_TABLE_ID, .oem_revision = ACPI_OEM_REVISION, -- 2.9.3 (Apple Git-75) _______________________________________________ Xen-devel mailing list Xen-devel@xxxxxxxxxxxxx https://lists.xen.org/xen-devel
|
![]() |
Lists.xenproject.org is hosted with RackSpace, monitoring our |