diff options
| author | Al Viro <viro@zeniv.linux.org.uk> | 2009-06-16 23:35:46 -0400 |
|---|---|---|
| committer | Al Viro <viro@zeniv.linux.org.uk> | 2009-06-17 00:36:36 -0400 |
| commit | e7ec952f6aa6ac1649ac49eb5e4de5b92c829d1e (patch) | |
| tree | cfe58a2f9b82ab94bd5ddbbaefb38b4790bdc65c /fs/efs/symlink.c | |
| parent | 536c94901eb8f2eb6fccf81ae6be814899a9f6e8 (diff) | |
| download | op-kernel-dev-e7ec952f6aa6ac1649ac49eb5e4de5b92c829d1e.zip op-kernel-dev-e7ec952f6aa6ac1649ac49eb5e4de5b92c829d1e.tar.gz | |
get rid of BKL in fs/efs
Only readdir() really needed it, and that's easily fixable by switch to
generic_file_llseek()
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Diffstat (limited to 'fs/efs/symlink.c')
| -rw-r--r-- | fs/efs/symlink.c | 7 |
1 files changed, 1 insertions, 6 deletions
diff --git a/fs/efs/symlink.c b/fs/efs/symlink.c index 41911ec..75117d0 100644 --- a/fs/efs/symlink.c +++ b/fs/efs/symlink.c @@ -9,7 +9,6 @@ #include <linux/string.h> #include <linux/pagemap.h> #include <linux/buffer_head.h> -#include <linux/smp_lock.h> #include "efs.h" static int efs_symlink_readpage(struct file *file, struct page *page) @@ -22,9 +21,8 @@ static int efs_symlink_readpage(struct file *file, struct page *page) err = -ENAMETOOLONG; if (size > 2 * EFS_BLOCKSIZE) - goto fail_notlocked; + goto fail; - lock_kernel(); /* read first 512 bytes of link target */ err = -EIO; bh = sb_bread(inode->i_sb, efs_bmap(inode, 0)); @@ -40,14 +38,11 @@ static int efs_symlink_readpage(struct file *file, struct page *page) brelse(bh); } link[size] = '\0'; - unlock_kernel(); SetPageUptodate(page); kunmap(page); unlock_page(page); return 0; fail: - unlock_kernel(); -fail_notlocked: SetPageError(page); kunmap(page); unlock_page(page); |
