rbd: turn off interrupts for open/remove locking
authorAlex Elder <elder@inktank.com>
Tue, 5 Feb 2013 19:23:12 +0000 (13:23 -0600)
committerSage Weil <sage@inktank.com>
Thu, 14 Feb 2013 02:29:11 +0000 (18:29 -0800)
This commit:
    bc7a62ee5 rbd: prevent open for image being removed
added checking for removing rbd before allowing an open, and used
the same request spinlock for protecting that and updating the open
count as is used for the request queue.

However it used the non-irq protected version of the spinlocks.
Fix that.

Signed-off-by: Alex Elder <elder@inktank.com>
Reviewed-by: Josh Durgin <josh.durgin@inktank.com>
drivers/block/rbd.c

index 14a6967291d34e37ec9b1ed12e552b6c18e3c28b..91983a60487b88ebb87dd9a843813633780e7676 100644 (file)
@@ -394,12 +394,12 @@ static int rbd_open(struct block_device *bdev, fmode_t mode)
        if ((mode & FMODE_WRITE) && rbd_dev->mapping.read_only)
                return -EROFS;
 
-       spin_lock(&rbd_dev->lock);
+       spin_lock_irq(&rbd_dev->lock);
        if (test_bit(RBD_DEV_FLAG_REMOVING, &rbd_dev->flags))
                removing = true;
        else
                rbd_dev->open_count++;
-       spin_unlock(&rbd_dev->lock);
+       spin_unlock_irq(&rbd_dev->lock);
        if (removing)
                return -ENOENT;
 
@@ -416,9 +416,9 @@ static int rbd_release(struct gendisk *disk, fmode_t mode)
        struct rbd_device *rbd_dev = disk->private_data;
        unsigned long open_count_before;
 
-       spin_lock(&rbd_dev->lock);
+       spin_lock_irq(&rbd_dev->lock);
        open_count_before = rbd_dev->open_count--;
-       spin_unlock(&rbd_dev->lock);
+       spin_unlock_irq(&rbd_dev->lock);
        rbd_assert(open_count_before > 0);
 
        mutex_lock_nested(&ctl_mutex, SINGLE_DEPTH_NESTING);
@@ -4099,12 +4099,12 @@ static ssize_t rbd_remove(struct bus_type *bus,
                goto done;
        }
 
-       spin_lock(&rbd_dev->lock);
+       spin_lock_irq(&rbd_dev->lock);
        if (rbd_dev->open_count)
                ret = -EBUSY;
        else
                set_bit(RBD_DEV_FLAG_REMOVING, &rbd_dev->flags);
-       spin_unlock(&rbd_dev->lock);
+       spin_unlock_irq(&rbd_dev->lock);
        if (ret < 0)
                goto done;