Commit e11daafd authored by Greg Kroah-Hartman's avatar Greg Kroah-Hartman

Revert "driver core: fix async device shutdown hang"

This reverts commit 4f2c346e.

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: default avatarAndrey Skvortsov <andrej.skvortzov@gmail.com>
Link: https://lore.kernel.org/r/ZvMkkhyJrohaajuk@skv.local
Cc: Christoph Hellwig <hch@lst.de>
Cc: David Jeffery <djeffery@redhat.com>
Cc: Keith Busch <kbusch@kernel.org>
Cc: Laurence Oberman <loberman@redhat.com>
Cc: Nathan Chancellor <nathan@kernel.org>
Cc: Sagi Grimberg <sagi@grimberg.me>
Cc: Stuart Hayes <stuart.w.hayes@gmail.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 4f2c346e
...@@ -4898,16 +4898,8 @@ void device_shutdown(void) ...@@ -4898,16 +4898,8 @@ void device_shutdown(void)
idx = device_links_read_lock(); idx = device_links_read_lock();
list_for_each_entry_rcu(link, &dev->links.suppliers, c_node, list_for_each_entry_rcu(link, &dev->links.suppliers, c_node,
device_links_read_lock_held()) { device_links_read_lock_held())
/*
* sync_state_only suppliers don't need to wait,
* aren't reordered on devices_kset, so making them
* wait could result in a hang
*/
if (device_link_flag_is_sync_state_only(link->flags))
continue;
link->supplier->p->shutdown_after = cookie; link->supplier->p->shutdown_after = cookie;
}
device_links_read_unlock(idx); device_links_read_unlock(idx);
put_device(dev); put_device(dev);
......
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