Commit 37f90877 authored by Rafael J. Wysocki's avatar Rafael J. Wysocki
Browse files

ACPI / dock: Walk list in reverse order during removal of devices 



If there are indirect dependencies between devices in a dock
station's dependent devices list, they may be broken if the devices
are removed in the same order in which they have been added.

For this reason, make the code in handle_eject_request() walk the
list of dependent devices in reverse order.

Signed-off-by: default avatarRafael J. Wysocki <rafael.j.wysocki@intel.com>
Acked-by: default avatarYinghai Lu <yinghai@kernel.org>
parent 96c0a4d4

drivers/acpi/dock.c

+29 −16
Original line number Diff line number Diff line
@@ -337,9 +337,29 @@ static void dock_remove_acpi_device(acpi_handle handle) 
}



/**

 * hotplug_dock_devices - insert or remove devices on the dock station

 * hot_remove_dock_devices - Remove dock station devices.

 * @ds: Dock station.

 */

static void hot_remove_dock_devices(struct dock_station *ds)

{

	struct dock_dependent_device *dd;



	/*

	 * Walk the list in reverse order so that devices that have been added

	 * last are removed first (in case there are some indirect dependencies

	 * between them).

	 */

	list_for_each_entry_reverse(dd, &ds->dependent_devices, list)

		dock_hotplug_event(dd, ACPI_NOTIFY_EJECT_REQUEST, false);



	list_for_each_entry_reverse(dd, &ds->dependent_devices, list)

		dock_remove_acpi_device(dd->handle);

}



/**

 * hotplug_dock_devices - Insert devices on a dock station.

 * @ds: the dock station

 * @event: either bus check or eject request

 * @event: either bus check or device check request

 *

 * Some devices on the dock station need to have drivers called

 * to perform hotplug operations after a dock event has occurred.
@@ -350,25 +370,18 @@ static void hotplug_dock_devices(struct dock_station *ds, u32 event) 
{

	struct dock_dependent_device *dd;



	/*

	 * First call driver specific hotplug functions

	 */

	/* Call driver specific hotplug functions. */

	list_for_each_entry(dd, &ds->dependent_devices, list)

		dock_hotplug_event(dd, event, false);



	/*

	 * Now make sure that an acpi_device is created for each

	 * dependent device, or removed if this is an eject request.

	 * This will cause acpi_drivers to be stopped/started if they

	 * exist

	 * Now make sure that an acpi_device is created for each dependent

	 * device.  That will cause scan handlers to be attached to device

	 * objects or acpi_drivers to be stopped/started if they are present.

	 */

	list_for_each_entry(dd, &ds->dependent_devices, list) {

		if (event == ACPI_NOTIFY_EJECT_REQUEST)

			dock_remove_acpi_device(dd->handle);

		else

	list_for_each_entry(dd, &ds->dependent_devices, list)

		dock_create_acpi_device(dd->handle);

}

}



static void dock_event(struct dock_station *ds, u32 event, int num)

{
@@ -588,7 +601,7 @@ static int handle_eject_request(struct dock_station *ds, u32 event) 
	 */

	dock_event(ds, event, UNDOCK_EVENT);



	hotplug_dock_devices(ds, ACPI_NOTIFY_EJECT_REQUEST);

	hot_remove_dock_devices(ds);

	undock(ds);

	acpi_evaluate_lck(ds->handle, 0);

	acpi_evaluate_ej0(ds->handle);

CRA Git | Maintained and supported by SUSTech CRA and CCSE