Acked-by: Coly Li <coly.li@xxxxxxx>
On 2009年11月02日 18:04, Jan Blunck Wrote:
> BKL is only used in fill_super. It is safe to remove it.
>
> Signed-off-by: Jan Blunck <jblunck@xxxxxxx>
> ---
> fs/efs/super.c | 10 ++--------
> 1 files changed, 2 insertions(+), 8 deletions(-)
>
> diff --git a/fs/efs/super.c b/fs/efs/super.c
> index 0981141..f049428 100644
> --- a/fs/efs/super.c
> +++ b/fs/efs/super.c
> @@ -249,13 +249,9 @@ static int efs_fill_super(struct super_block *s, void *d, int silent)
> struct inode *root;
> int ret = -EINVAL;
>
> - lock_kernel();
> -
> - sb = kzalloc(sizeof(struct efs_sb_info), GFP_KERNEL);
> - if (!sb) {
> - unlock_kernel();
> + sb = kzalloc(sizeof(struct efs_sb_info), GFP_KERNEL);
> + if (!sb)
> return -ENOMEM;
> - }
> s->s_fs_info = sb;
>
> s->s_magic = EFS_SUPER_MAGIC;
> @@ -323,14 +319,12 @@ static int efs_fill_super(struct super_block *s, void *d, int silent)
> goto out_no_fs;
> }
>
> - unlock_kernel();
> return 0;
>
> out_no_fs_ul:
> out_no_fs:
> s->s_fs_info = NULL;
> kfree(sb);
> - unlock_kernel();
> return ret;
> }
>
--
Coly Li
SuSE Labs
--
To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
