Commit 89ec2f2e authored by Rafael J. Wysocki's avatar Rafael J. Wysocki

ACPI / hotplug / PCI: Don't trim devices before scanning the namespace

In acpiphp_bus_add() we first remove device objects corresponding to
the given handle and the ACPI namespace branch below it, which are
then re-created by acpi_bus_scan().  This used to be done to clean
up after surprise removals, but now we do the cleanup through
trim_stale_devices() which checks if the devices in question are
actually gone before removing them, so the device hierarchy trimming
in acpiphp_bus_add() is not necessary any more and, moreover, it may
lead to problems if it removes device objects corresponding to
devices that are actually present.

For this reason, remove the leftover acpiphp_bus_trim() from
acpiphp_bus_add().
Reported-by: default avatarAlex Williamson <alex.williamson@redhat.com>
Signed-off-by: default avatarRafael J. Wysocki <rafael.j.wysocki@intel.com>
parent f41b8312
...@@ -487,7 +487,6 @@ static void acpiphp_bus_add(acpi_handle handle) ...@@ -487,7 +487,6 @@ static void acpiphp_bus_add(acpi_handle handle)
{ {
struct acpi_device *adev = NULL; struct acpi_device *adev = NULL;
acpiphp_bus_trim(handle);
acpi_bus_scan(handle); acpi_bus_scan(handle);
acpi_bus_get_device(handle, &adev); acpi_bus_get_device(handle, &adev);
if (adev) if (adev)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment