Modify ACPI to only supply _EJ0 methods for PCI slots that support hotplug.
This is done by runtime patching: - Instrument SSDT ASL code with ACPI_EXTRACT directives tagging _EJ0 and _ADR fields. - At compile time, tools/acpi_extract.py looks for these methods in ASL source finds the matching AML, and stores the offsets of these methods in tables named aml_ej0_name and aml_adr_dword. - At run time, go over aml_ej0_name, use aml_adr_dword to get slot information and check which slots support hotplug. If hotplug is disabled, we patch the _EJ0 NameString in ACPI table, replacing _EJ0 with EJ0_. Note that this has the same checksum, but is ignored by OSPM. Note: the method used is robust in that we don't need to change any offsets manually in case of ASL code changes. As all parsing is done at compile time, any unexpected input causes build failure, not a runtime failure. Signed-off-by: Michael S. Tsirkin <m...@redhat.com> --- src/acpi.c | 44 ++++++++++++++++++++++++++++++++++++-------- src/ssdt-pcihp.dsl | 6 ++++++ 2 files changed, 42 insertions(+), 8 deletions(-) diff --git a/src/acpi.c b/src/acpi.c index c5147d9..107469f 100644 --- a/src/acpi.c +++ b/src/acpi.c @@ -484,12 +484,42 @@ build_ssdt(void) return ssdt; } -static void* -copy_table(void *table, int size) +#include "ssdt-pcihp.hex" + +#define PCI_RMV_BASE 0xae0c + +extern void link_time_assertion(void); + +static void* build_pcihp(void) { - u8 *copy = malloc_high(size); - memcpy(copy, table, size); - return copy; + u32 rmvc_pcrm; + int i; + + u8 *ssdt = malloc_high(sizeof ssdp_pcihp_aml); + memcpy(ssdt, ssdp_pcihp_aml, sizeof ssdp_pcihp_aml); + + /* Runtime patching of EJ0: to disable hotplug for a slot, + * replace the method name: _EJ0 by EJ0_. */ + if (ARRAY_SIZE(aml_ej0_name) != ARRAY_SIZE(aml_adr_dword)) { + link_time_assertion(); + } + + rmvc_pcrm = inl(PCI_RMV_BASE); + for (i = 0; i < ARRAY_SIZE(aml_ej0_name); ++i) { + /* Slot is in byte 2 in _ADR */ + u8 slot = ssdp_pcihp_aml[aml_adr_dword[i] + 2] & 0x1F; + /* Sanity check */ + if (memcmp(ssdp_pcihp_aml + aml_ej0_name[i], "_EJ0", 4)) { + warn_internalerror(); + free(ssdt); + return NULL; + } + if (!(rmvc_pcrm & (0x1 << slot))) { + memcpy(ssdt + aml_ej0_name[i], "EJ0_", 4); + } + } + + return ssdt; } #define HPET_SIGNATURE 0x54455048 // HPET @@ -642,8 +672,6 @@ static const struct pci_device_id acpi_find_tbl[] = { struct rsdp_descriptor *RsdpAddr; -#include "ssdt-pcihp.hex" - #define MAX_ACPI_TABLES 20 void acpi_bios_init(void) @@ -675,7 +703,7 @@ acpi_bios_init(void) ACPI_INIT_TABLE(build_madt()); ACPI_INIT_TABLE(build_hpet()); ACPI_INIT_TABLE(build_srat()); - ACPI_INIT_TABLE(copy_table(ssdp_pcihp_aml, sizeof ssdp_pcihp_aml)); + ACPI_INIT_TABLE(build_pcihp()); u16 i, external_tables = qemu_cfg_acpi_additional_tables(); diff --git a/src/ssdt-pcihp.dsl b/src/ssdt-pcihp.dsl index 72d1bb7..cc96ffc 100644 --- a/src/ssdt-pcihp.dsl +++ b/src/ssdt-pcihp.dsl @@ -53,9 +53,15 @@ DefinitionBlock ("ssdt-pcihp.aml", "SSDT", 0x01, "BXPC", "BXSSDTPCIHP", 0x1) gen_pci_device(1f) /* Bulk generated PCI hotplug devices */ + // Method _EJ0 can be patched by BIOS to EJ0_ + // at runtime, if the slot is detected to not support hotplug. + // Extract the offset of the address dword and the + // _EJ0 name to allow this patching. #define hotplug_slot(slot) \ Device (S##slot) { \ + ACPI_EXTRACT_NAME_DWORD_CONST aml_adr_dword \ Name (_ADR, 0x##slot##0000) \ + ACPI_EXTRACT_METHOD_STRING aml_ej0_name \ Method (_EJ0, 1) { Return(PCEJ(0x##slot)) } \ Name (_SUN, 0x##slot) \ } -- 1.7.5.53.gc233e -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html