diff options
| author | Arnd Bergmann <arnd@arndb.de> | 2010-09-14 23:07:27 +0200 |
|---|---|---|
| committer | Arnd Bergmann <arnd@arndb.de> | 2010-10-04 21:10:45 +0200 |
| commit | 3768744cfea7b995dce27f02341161fbfdfee80c (patch) | |
| tree | 29655ece93aa751f2f9ff98e21b840b5b80b91c0 /fs/fat/inode.c | |
| parent | 3e44f9f1dc19e2b8d4f7ca3b3c63e976b22ad372 (diff) | |
| download | op-kernel-dev-3768744cfea7b995dce27f02341161fbfdfee80c.zip op-kernel-dev-3768744cfea7b995dce27f02341161fbfdfee80c.tar.gz | |
BKL: Remove BKL from fat
The lock_kernel in fat_put_super is not needed because
it only protects the super block itself and we know that
no other thread can reach it because we are about to
kfree the object.
In the two fill_super functions, this converts the locking
to use lock_super like elsewhere in the fat code. This
is probably not needed either, but is consistent and puts
us on the safe side.
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Cc: OGAWA Hirofumi <hirofumi@mail.parknet.co.jp>
Cc: Jan Blunck <jblunck@infradead.org>
Diffstat (limited to 'fs/fat/inode.c')
| -rw-r--r-- | fs/fat/inode.c | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/fs/fat/inode.c b/fs/fat/inode.c index 8300580..ad6998a 100644 --- a/fs/fat/inode.c +++ b/fs/fat/inode.c @@ -14,7 +14,6 @@ #include <linux/init.h> #include <linux/time.h> #include <linux/slab.h> -#include <linux/smp_lock.h> #include <linux/seq_file.h> #include <linux/pagemap.h> #include <linux/mpage.h> @@ -489,8 +488,6 @@ static void fat_put_super(struct super_block *sb) { struct msdos_sb_info *sbi = MSDOS_SB(sb); - lock_kernel(); - if (sb->s_dirt) fat_write_super(sb); @@ -504,8 +501,6 @@ static void fat_put_super(struct super_block *sb) sb->s_fs_info = NULL; kfree(sbi); - - unlock_kernel(); } static struct kmem_cache *fat_inode_cachep; |
