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/compat_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/compat_ioctl.c')
-rw-r--r-- | block/compat_ioctl.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/block/compat_ioctl.c b/block/compat_ioctl.c index 67eb93cff699..f87615dea46b 100644 --- a/block/compat_ioctl.c +++ b/block/compat_ioctl.c | |||
@@ -774,9 +774,7 @@ long compat_blkdev_ioctl(struct file *file, unsigned cmd, unsigned long arg) | |||
774 | bdi = blk_get_backing_dev_info(bdev); | 774 | bdi = blk_get_backing_dev_info(bdev); |
775 | if (bdi == NULL) | 775 | if (bdi == NULL) |
776 | return -ENOTTY; | 776 | return -ENOTTY; |
777 | lock_kernel(); | ||
778 | bdi->ra_pages = (arg * 512) / PAGE_CACHE_SIZE; | 777 | bdi->ra_pages = (arg * 512) / PAGE_CACHE_SIZE; |
779 | unlock_kernel(); | ||
780 | return 0; | 778 | return 0; |
781 | case BLKGETSIZE: | 779 | case BLKGETSIZE: |
782 | size = bdev->bd_inode->i_size; | 780 | size = bdev->bd_inode->i_size; |