ACPI: configfs: Resolve objects on host-directed table loads
authorNikolaus Voss <nikolaus.voss@loewensteinmedical.de>
Wed, 12 Jun 2019 08:36:06 +0000 (10:36 +0200)
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>
Fri, 14 Jun 2019 09:39:42 +0000 (11:39 +0200)
If an ACPI SSDT overlay is loaded after built-in tables
have been loaded e.g. via configfs or efivar_ssdt_load()
it is necessary to rewalk the namespace to resolve
references. Without this, relative and absolute paths
like ^PCI0.SBUS or \_SB.PCI0.SBUS are not resolved
correctly.

Make configfs loads use the same method as efivar_ssdt_load().

Signed-off-by: Nikolaus Voss <nikolaus.voss@loewensteinmedical.de>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
drivers/acpi/acpi_configfs.c
drivers/acpi/acpica/tbxfload.c

index f92033661239bea858d2a7288db062c2538f91a0..663f0d88f9128403d759a2e966711e969ce161e1 100644 (file)
@@ -56,11 +56,7 @@ static ssize_t acpi_table_aml_write(struct config_item *cfg,
        if (!table->header)
                return -ENOMEM;
 
-       ACPI_INFO(("Host-directed Dynamic ACPI Table Load:"));
-       ret = acpi_tb_install_and_load_table(
-                       ACPI_PTR_TO_PHYSADDR(table->header),
-                       ACPI_TABLE_ORIGIN_EXTERNAL_VIRTUAL, FALSE,
-                       &table->index);
+       ret = acpi_load_table(table->header);
        if (ret) {
                kfree(table->header);
                table->header = NULL;
index 4f30f06a6f7874c43339b913ea4942a37c6b8a95..ef8f8a9f3c9c598251db7ce59985c316dfefce7f 100644 (file)
@@ -297,6 +297,17 @@ acpi_status acpi_load_table(struct acpi_table_header *table)
        status = acpi_tb_install_and_load_table(ACPI_PTR_TO_PHYSADDR(table),
                                                ACPI_TABLE_ORIGIN_EXTERNAL_VIRTUAL,
                                                FALSE, &table_index);
+
+       if (ACPI_SUCCESS(status)) {
+               /* Complete the initialization/resolution of package objects */
+
+               status = acpi_ns_walk_namespace(ACPI_TYPE_PACKAGE,
+                                               ACPI_ROOT_OBJECT,
+                                               ACPI_UINT32_MAX, 0,
+                                               acpi_ns_init_one_package,
+                                               NULL, NULL, NULL);
+       }
+
        return_ACPI_STATUS(status);
 }