| From 176a88d79d6b5aebabaff16734e8b3107efcaaad Mon Sep 17 00:00:00 2001 |
| From: "Rafael J. Wysocki" <rafael.j.wysocki@intel.com> |
| Date: Thu, 7 Nov 2013 01:42:17 +0100 |
| Subject: ACPI / hotplug: Do not execute "insert in progress" _OST |
| |
| From: "Rafael J. Wysocki" <rafael.j.wysocki@intel.com> |
| |
| commit 176a88d79d6b5aebabaff16734e8b3107efcaaad upstream. |
| |
| According to the ACPI spec (5.0, Section 6.3.5), the "Device |
| insertion in progress (pending)" (0x80) _OST status code is |
| reserved for the "Insertion Processing" (0x200) source event |
| which is "a result of an OSPM action". Specifically, it is not |
| a notification, so that status code should not be used during |
| notification processing, which unfortunately is done by |
| acpi_scan_bus_device_check(). |
| |
| For this reason, drop the ACPI_OST_SC_INSERT_IN_PROGRESS _OST |
| status evaluation from there (it was a mistake to put it in there |
| in the first place). |
| |
| Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com> |
| Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> |
| |
| --- |
| drivers/acpi/scan.c | 2 -- |
| 1 file changed, 2 deletions(-) |
| |
| --- a/drivers/acpi/scan.c |
| +++ b/drivers/acpi/scan.c |
| @@ -359,8 +359,6 @@ static void acpi_scan_bus_device_check(a |
| goto out; |
| } |
| } |
| - acpi_evaluate_hotplug_ost(handle, ost_source, |
| - ACPI_OST_SC_INSERT_IN_PROGRESS, NULL); |
| error = acpi_bus_scan(handle); |
| if (error) { |
| acpi_handle_warn(handle, "Namespace scan failure\n"); |
