diff options
| author | Al Viro <al@aretha.pdmi.ras.ru> | 2008-09-19 09:08:13 +0200 |
|---|---|---|
| committer | Al Viro <viro@zeniv.linux.org.uk> | 2008-10-21 13:49:12 +0200 |
| commit | 6af3a56e1dd4d95836a47214e5c60d5b749a5501 (patch) | |
| tree | 757b01fe8668783fcf52a4e8fe82c5d8ebbc6b72 /block | |
| parent | [PATCH] get rid of blkdev_locked_ioctl() (diff) | |
| download | linux-6af3a56e1dd4d95836a47214e5c60d5b749a5501.tar.xz linux-6af3a56e1dd4d95836a47214e5c60d5b749a5501.zip | |
[PATCH] get rid of struct file use in blkdev_ioctl() BLKBSZSET
We need to do bd_claim() only if file hadn't been opened with O_EXCL
and then we have no need to use file itself as owner.
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Diffstat (limited to 'block')
| -rw-r--r-- | block/ioctl.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/block/ioctl.c b/block/ioctl.c index bd214cb37f2b..14b7f2c10662 100644 --- a/block/ioctl.c +++ b/block/ioctl.c @@ -343,10 +343,11 @@ int blkdev_ioctl(struct inode *inode, struct file *file, unsigned cmd, return -EINVAL; if (get_user(n, (int __user *) arg)) return -EFAULT; - if (bd_claim(bdev, file) < 0) + if (!(mode & FMODE_EXCL) && bd_claim(bdev, &bdev) < 0) return -EBUSY; ret = set_blocksize(bdev, n); - bd_release(bdev); + if (!(mode & FMODE_EXCL)) + bd_release(bdev); return ret; case BLKPG: lock_kernel(); |