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 <[email protected]>
Cc: Lee Duncan <[email protected]>
Cc: Hannes Reinecke <[email protected]>
Cc: Luis Chamberlain <[email protected]>
Cc: Johannes Thumshirn <[email protected]>
Cc: Christoph Hellwig <[email protected]>
Cc: Greg Kroah-Hartman <[email protected]>
Signed-off-by: Bart Van Assche <[email protected]>
---
 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