No instance of ->bmap() needs BKL
Al Viro [Tue, 16 Jun 2009 17:35:01 +0000 (13:35 -0400)]
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>

Documentation/filesystems/Locking
fs/ioctl.c

index 3120f8d..229d7b7 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 286f38d..001f8d3 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);
 }