Skip to content

Commit e11daaf

Browse files
committed
Revert "driver core: fix async device shutdown hang"
This reverts commit 4f2c346. The series is being reverted before -rc1 as there are still reports of lockups on shutdown, so it's not quite ready for "prime time." Reported-by: Andrey Skvortsov <[email protected]> Link: https://lore.kernel.org/r/[email protected] Cc: Christoph Hellwig <[email protected]> Cc: David Jeffery <[email protected]> Cc: Keith Busch <[email protected]> Cc: Laurence Oberman <[email protected]> Cc: Nathan Chancellor <[email protected]> Cc: Sagi Grimberg <[email protected]> Cc: Stuart Hayes <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
1 parent 4f2c346 commit e11daaf

File tree

1 file changed

+1
-9
lines changed

1 file changed

+1
-9
lines changed

drivers/base/core.c

Lines changed: 1 addition & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -4898,16 +4898,8 @@ void device_shutdown(void)
48984898

48994899
idx = device_links_read_lock();
49004900
list_for_each_entry_rcu(link, &dev->links.suppliers, c_node,
4901-
device_links_read_lock_held()) {
4902-
/*
4903-
* sync_state_only suppliers don't need to wait,
4904-
* aren't reordered on devices_kset, so making them
4905-
* wait could result in a hang
4906-
*/
4907-
if (device_link_flag_is_sync_state_only(link->flags))
4908-
continue;
4901+
device_links_read_lock_held())
49094902
link->supplier->p->shutdown_after = cookie;
4910-
}
49114903
device_links_read_unlock(idx);
49124904
put_device(dev);
49134905

0 commit comments

Comments
 (0)