Since __device_release_driver() is called with the device lock held
and since the same device lock is obtained by the functions that
perform asynchronous probing (driver_attach_async() and
__device_attach_async_helper()), asynchronous probing is already
serialized against __device_release_driver(). Remove the
async_synchronize_full() call from __device_release_driver() to
avoid that a deadlock is triggered.

Fixes: 765230b5f084 ("driver-core: add asynchronous probing support for 
drivers").
Acked-by: Dmitry Torokhov <dmitry.torok...@gmail.com>
Cc: Lee Duncan <ldun...@suse.com>
Cc: Hannes Reinecke <h...@suse.com>
Cc: Luis Chamberlain <mcg...@kernel.org>
Cc: Johannes Thumshirn <jthumsh...@suse.de>
Cc: Christoph Hellwig <h...@lst.de>
Cc: Greg Kroah-Hartman <gre...@linuxfoundation.org>
Signed-off-by: Bart Van Assche <bvanass...@acm.org>
---
 drivers/base/dd.c | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/drivers/base/dd.c b/drivers/base/dd.c
index f8d645aa09be..0a3c9f8702de 100644
--- a/drivers/base/dd.c
+++ b/drivers/base/dd.c
@@ -999,9 +999,6 @@ static void __device_release_driver(struct device *dev, 
struct device *parent)
 
        drv = dev->driver;
        if (drv) {
-               if (driver_allows_async_probing(drv))
-                       async_synchronize_full();
-
                while (device_links_busy(dev)) {
                        device_unlock(dev);
                        if (parent)
-- 
2.19.1.568.g152ad8e336-goog

Reply via email to