summaryrefslogtreecommitdiffstats
path: root/fs/afs/super.c
diff options
context:
space:
mode:
authorArnd Bergmann <arnd@arndb.de>2010-09-16 15:35:54 +0200
committerArnd Bergmann <arnd@arndb.de>2010-10-04 21:10:48 +0200
commit77f2fe036cd51300c80d1aca76bcf23a09977f13 (patch)
tree08598ae2b51e8351cf280fd23e4bd7ef22757ef9 /fs/afs/super.c
parentcdbd2b013d2814ad39e893316943fb58e6d9ec23 (diff)
downloadlinux-stable-77f2fe036cd51300c80d1aca76bcf23a09977f13.tar.gz
linux-stable-77f2fe036cd51300c80d1aca76bcf23a09977f13.tar.bz2
linux-stable-77f2fe036cd51300c80d1aca76bcf23a09977f13.zip
BKL: Remove BKL from afs
The BKL is only used in put_super and fill_super, which are both protected by the superblocks s_umount rw_semaphore. Therefore it is safe to remove the BKL entirely. Signed-off-by: Arnd Bergmann <arnd@arndb.de> Cc: linux-afs@lists.infradead.org Cc: David Howells <dhowells@redhat.com>
Diffstat (limited to 'fs/afs/super.c')
-rw-r--r--fs/afs/super.c10
1 files changed, 0 insertions, 10 deletions
diff --git a/fs/afs/super.c b/fs/afs/super.c
index 6c2fef44d385..eacf76d98ae0 100644
--- a/fs/afs/super.c
+++ b/fs/afs/super.c
@@ -19,7 +19,6 @@
#include <linux/mount.h>
#include <linux/init.h>
#include <linux/slab.h>
-#include <linux/smp_lock.h>
#include <linux/fs.h>
#include <linux/pagemap.h>
#include <linux/parser.h>
@@ -302,15 +301,12 @@ static int afs_fill_super(struct super_block *sb, void *data)
struct inode *inode = NULL;
int ret;
- lock_kernel();
-
_enter("");
/* allocate a superblock info record */
as = kzalloc(sizeof(struct afs_super_info), GFP_KERNEL);
if (!as) {
_leave(" = -ENOMEM");
- unlock_kernel();
return -ENOMEM;
}
@@ -344,7 +340,6 @@ static int afs_fill_super(struct super_block *sb, void *data)
sb->s_root = root;
_leave(" = 0");
- unlock_kernel();
return 0;
error_inode:
@@ -358,7 +353,6 @@ error:
sb->s_fs_info = NULL;
_leave(" = %d", ret);
- unlock_kernel();
return ret;
}
@@ -458,12 +452,8 @@ static void afs_put_super(struct super_block *sb)
_enter("");
- lock_kernel();
-
afs_put_volume(as->volume);
- unlock_kernel();
-
_leave("");
}