Skip to contentSkip to conversation

reduce BKL contention in block device ->open/->release

RFC New Christoph Hellwig 1 participant 1 message Feb 4, 2003

Subsystems (2)
lore
Share & tools
lore
#0 lore raw

RFC 1/1 reduce BKL contention in block device ->open/->release

I looked through the callchain carefully, but it can't find anything that needs BKL protection but the actual LLDD methods.

===========================================================================
Index: linux/fs/block_dev.c
===========================================================================

fs/block_dev.c renamed from usr/tmp/TmpDir.1943-0/linux/fs/block_dev.c_1.35 +8 -6
@@ -560,15 +560,17 @@
 	kdev_t dev = to_kdev_t(bdev->bd_dev);
 
 	down(&bdev->bd_sem);
-	lock_kernel();
 	if (!bdev->bd_op)
 		bdev->bd_op = get_blkfops(MAJOR(dev));
 	if (bdev->bd_op) {
 		ret = 0;
 		if (bdev->bd_op->owner)
 			__MOD_INC_USE_COUNT(bdev->bd_op->owner);
-		if (bdev->bd_op->open)
+		if (bdev->bd_op->open) {
+			lock_kernel();
 			ret = bdev->bd_op->open(inode, file);
+			unlock_kernel();
+		}
 		if (!ret) {
 			bdev->bd_openers++;
 			bdev->bd_inode->i_size = blkdev_size(dev);
@@ -580,7 +582,6 @@
 				bdev->bd_op = NULL;
 		}
 	}
-	unlock_kernel();
 	up(&bdev->bd_sem);
 	if (ret)
 		bdput(bdev);
@@ -630,20 +631,21 @@
 	struct inode *bd_inode = bdev->bd_inode;
 
 	down(&bdev->bd_sem);
-	lock_kernel();
 	if (kind == BDEV_FILE && bdev->bd_openers == 1)
 		__block_fsync(bd_inode);
 	else if (kind == BDEV_FS)
 		fsync_no_super(rdev);
 	if (!--bdev->bd_openers)
 		kill_bdev(bdev);
-	if (bdev->bd_op->release)
+	if (bdev->bd_op->release) {
+		lock_kernel();
 		ret = bdev->bd_op->release(bd_inode, NULL);
+		unlock_kernel();
+	}
 	if (bdev->bd_op->owner)
 		__MOD_DEC_USE_COUNT(bdev->bd_op->owner);
 	if (!bdev->bd_openers)
 		bdev->bd_op = NULL;
-	unlock_kernel();
 	up(&bdev->bd_sem);
 	bdput(bdev);
 	return ret;