diff options
author | Wu Fengguang <fengguang.wu@intel.com> | 2008-11-25 03:08:39 -0500 |
---|---|---|
committer | Jens Axboe <jens.axboe@oracle.com> | 2008-12-29 02:28:43 -0500 |
commit | 7c239517d9f18427fc2e7ed259fb3b866595f5af (patch) | |
tree | cd2149cc2f2eb0faa83d38fe64e1228f7c703a85 /block/ioctl.c | |
parent | 42364690992e592c05f85c76fda4055820b48c1b (diff) |
block: don't take lock on changing ra_pages
There's no need to take queue_lock or kernel_lock when modifying
bdi->ra_pages. So remove them. Also remove out of date comment for
queue_max_sectors_store().
Signed-off-by: Wu Fengguang <wfg@linux.intel.com>
Signed-off-by: Jens Axboe <jens.axboe@oracle.com>
Diffstat (limited to 'block/ioctl.c')
-rw-r--r-- | block/ioctl.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/block/ioctl.c b/block/ioctl.c index d03985b04d67..0f22e629b13c 100644 --- a/block/ioctl.c +++ b/block/ioctl.c | |||
@@ -323,9 +323,7 @@ int blkdev_ioctl(struct block_device *bdev, fmode_t mode, unsigned cmd, | |||
323 | bdi = blk_get_backing_dev_info(bdev); | 323 | bdi = blk_get_backing_dev_info(bdev); |
324 | if (bdi == NULL) | 324 | if (bdi == NULL) |
325 | return -ENOTTY; | 325 | return -ENOTTY; |
326 | lock_kernel(); | ||
327 | bdi->ra_pages = (arg * 512) / PAGE_CACHE_SIZE; | 326 | bdi->ra_pages = (arg * 512) / PAGE_CACHE_SIZE; |
328 | unlock_kernel(); | ||
329 | return 0; | 327 | return 0; |
330 | case BLKBSZSET: | 328 | case BLKBSZSET: |
331 | /* set the logical block size */ | 329 | /* set the logical block size */ |