No instance of ->bmap() needs BKL
authorAl Viro <viro@zeniv.linux.org.uk>
Tue, 16 Jun 2009 17:35:01 +0000 (13:35 -0400)
committerAl Viro <viro@zeniv.linux.org.uk>
Wed, 17 Jun 2009 04:36:35 +0000 (00:36 -0400)
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Documentation/filesystems/Locking
fs/ioctl.c

index 3120f8dd2c316c66347051a95d3edce1ad090d44..229d7b7c50a350053170e5fba11ac9a0168c0e70 100644 (file)
@@ -187,7 +187,7 @@ readpages:          no
 write_begin:           no      locks the page          yes
 write_end:             no      yes, unlocks            yes
 perform_write:         no      n/a                     yes
-bmap:                  yes
+bmap:                  no
 invalidatepage:                no      yes
 releasepage:           no      yes
 direct_IO:             no
index 286f38dfc6c0748d90d471c0878e4d08e8f7ece5..001f8d3118f27329d89131feef43c7ba0c0b1a85 100644 (file)
@@ -70,9 +70,7 @@ static int ioctl_fibmap(struct file *filp, int __user *p)
        res = get_user(block, p);
        if (res)
                return res;
-       lock_kernel();
        res = mapping->a_ops->bmap(mapping, block);
-       unlock_kernel();
        return put_user(res, p);
 }