loop: Get rid of 'nested' acquisition of loop_ctl_mutex
authorJan Kara <jack@suse.cz>
Thu, 8 Nov 2018 13:01:16 +0000 (14:01 +0100)
committerJens Axboe <axboe@kernel.dk>
Thu, 8 Nov 2018 13:30:37 +0000 (06:30 -0700)
The nested acquisition of loop_ctl_mutex (->lo_ctl_mutex back then) has
been introduced by commit f028f3b2f987e "loop: fix circular locking in
loop_clr_fd()" to fix lockdep complains about bd_mutex being acquired
after lo_ctl_mutex during partition rereading. Now that these are
properly fixed, let's stop fooling lockdep.

Signed-off-by: Jan Kara <jack@suse.cz>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
drivers/block/loop.c

index 112afc9bc6040afc9a0a437dee59dd54fe3bdcad..bf6bc35aaf8818c3e384f743c9bc200e4891357d 100644 (file)
@@ -681,7 +681,7 @@ static int loop_change_fd(struct loop_device *lo, struct block_device *bdev,
        int             error;
        bool            partscan;
 
-       error = mutex_lock_killable_nested(&loop_ctl_mutex, 1);
+       error = mutex_lock_killable(&loop_ctl_mutex);
        if (error)
                return error;
        error = -ENXIO;
@@ -919,7 +919,7 @@ static int loop_set_fd(struct loop_device *lo, fmode_t mode,
        if (!file)
                goto out;
 
-       error = mutex_lock_killable_nested(&loop_ctl_mutex, 1);
+       error = mutex_lock_killable(&loop_ctl_mutex);
        if (error)
                goto out_putf;
 
@@ -1135,7 +1135,7 @@ static int loop_clr_fd(struct loop_device *lo)
 {
        int err;
 
-       err = mutex_lock_killable_nested(&loop_ctl_mutex, 1);
+       err = mutex_lock_killable(&loop_ctl_mutex);
        if (err)
                return err;
        if (lo->lo_state != Lo_bound) {
@@ -1172,7 +1172,7 @@ loop_set_status(struct loop_device *lo, const struct loop_info64 *info)
        struct block_device *bdev;
        bool partscan = false;
 
-       err = mutex_lock_killable_nested(&loop_ctl_mutex, 1);
+       err = mutex_lock_killable(&loop_ctl_mutex);
        if (err)
                return err;
        if (lo->lo_encrypt_key_size &&
@@ -1277,7 +1277,7 @@ loop_get_status(struct loop_device *lo, struct loop_info64 *info)
        struct kstat stat;
        int ret;
 
-       ret = mutex_lock_killable_nested(&loop_ctl_mutex, 1);
+       ret = mutex_lock_killable(&loop_ctl_mutex);
        if (ret)
                return ret;
        if (lo->lo_state != Lo_bound) {
@@ -1466,7 +1466,7 @@ static int lo_simple_ioctl(struct loop_device *lo, unsigned int cmd,
 {
        int err;
 
-       err = mutex_lock_killable_nested(&loop_ctl_mutex, 1);
+       err = mutex_lock_killable(&loop_ctl_mutex);
        if (err)
                return err;
        switch (cmd) {