ACPI: Update PNPID match handling for notify
When installing/removing a notify handler to/from an ACPI device
object, ACPI core tries to match its associated scan handler to
see if it supports hotplug. However, the matching logic of the
notify handler is different from the matching logic of attaching
a scan handler to an ACPI device object. This patch updates the
matching logic of the notify handlers to be consistent with the
attach handling.
Signed-off-by: Toshi Kani <toshi.kani@hp.com>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
diff --git a/drivers/acpi/scan.c b/drivers/acpi/scan.c
index e9a71ed..3f6ec0d 100644
--- a/drivers/acpi/scan.c
+++ b/drivers/acpi/scan.c
@@ -1703,25 +1703,48 @@
return false;
}
+static struct acpi_scan_handler *acpi_scan_match_handler(char *idstr,
+ const struct acpi_device_id **matchid)
+{
+ struct acpi_scan_handler *handler;
+
+ list_for_each_entry(handler, &acpi_scan_handlers_list, list_node)
+ if (acpi_scan_handler_matching(handler, idstr, matchid))
+ return handler;
+
+ return NULL;
+}
+
static acpi_status acpi_scan_hotplug_modify(acpi_handle handle,
u32 lvl_not_used, void *data,
void **ret_not_used)
{
- struct acpi_scan_handler *handler = data;
- struct acpi_device_info *info;
+ struct acpi_device_pnp pnp = {};
+ struct acpi_scan_handler *tgt_handler = data, *handler;
+ struct acpi_hardware_id *hwid;
+ unsigned long long sta_not_used;
+ int type;
bool match = false;
- if (ACPI_FAILURE(acpi_get_object_info(handle, &info)))
+ if (acpi_bus_type_and_status(handle, &type, &sta_not_used))
return AE_OK;
- if (info->valid & ACPI_VALID_HID) {
- char *idstr = info->hardware_id.string;
- match = acpi_scan_handler_matching(handler, idstr, NULL);
- }
- kfree(info);
- if (!match)
+ INIT_LIST_HEAD(&pnp.ids);
+ acpi_set_pnp_ids(handle, &pnp, type);
+
+ if (!pnp.type.hardware_id)
return AE_OK;
+ list_for_each_entry(hwid, &pnp.ids, list) {
+ handler = acpi_scan_match_handler(hwid->id, NULL);
+ if (handler && handler == tgt_handler) {
+ match = true;
+ break;
+ }
+ }
+ if (!match)
+ goto out;
+
if (handler->hotplug.enabled)
acpi_install_notify_handler(handle, ACPI_SYSTEM_NOTIFY,
acpi_hotplug_notify_cb, NULL);
@@ -1729,6 +1752,8 @@
acpi_remove_notify_handler(handle, ACPI_SYSTEM_NOTIFY,
acpi_hotplug_notify_cb);
+out:
+ acpi_free_pnp_ids(&pnp);
return AE_OK;
}
@@ -1749,40 +1774,34 @@
mutex_unlock(&acpi_scan_lock);
}
-static struct acpi_scan_handler *acpi_scan_match_handler(char *idstr,
- const struct acpi_device_id **matchid)
+static void acpi_scan_init_hotplug(acpi_handle handle, int type)
{
+ struct acpi_device_pnp pnp = {};
+ struct acpi_hardware_id *hwid;
struct acpi_scan_handler *handler;
- list_for_each_entry(handler, &acpi_scan_handlers_list, list_node)
- if (acpi_scan_handler_matching(handler, idstr, matchid))
- return handler;
+ INIT_LIST_HEAD(&pnp.ids);
+ acpi_set_pnp_ids(handle, &pnp, type);
- return NULL;
-}
-
-static void acpi_scan_init_hotplug(acpi_handle handle)
-{
- struct acpi_device_info *info;
- struct acpi_scan_handler *handler;
-
- if (ACPI_FAILURE(acpi_get_object_info(handle, &info)))
+ if (!pnp.type.hardware_id)
return;
- if (!(info->valid & ACPI_VALID_HID)) {
- kfree(info);
- return;
- }
-
/*
* This relies on the fact that acpi_install_notify_handler() will not
* install the same notify handler routine twice for the same handle.
*/
- handler = acpi_scan_match_handler(info->hardware_id.string, NULL);
- kfree(info);
- if (handler && handler->hotplug.enabled)
- acpi_install_notify_handler(handle, ACPI_SYSTEM_NOTIFY,
- acpi_hotplug_notify_cb, NULL);
+ list_for_each_entry(hwid, &pnp.ids, list) {
+ handler = acpi_scan_match_handler(hwid->id, NULL);
+ if (handler) {
+ if (handler->hotplug.enabled)
+ acpi_install_notify_handler(handle,
+ ACPI_SYSTEM_NOTIFY,
+ acpi_hotplug_notify_cb, NULL);
+ break;
+ }
+ }
+
+ acpi_free_pnp_ids(&pnp);
}
static acpi_status acpi_bus_check_add(acpi_handle handle, u32 lvl_not_used,
@@ -1807,7 +1826,7 @@
return AE_OK;
}
- acpi_scan_init_hotplug(handle);
+ acpi_scan_init_hotplug(handle, type);
if (!(sta & ACPI_STA_DEVICE_PRESENT) &&
!(sta & ACPI_STA_DEVICE_FUNCTIONING)) {