On Thu, Oct 08, Boaz Harrosh wrote:
> Jan Blunck <jblunck@xxxxxxx> wrote
> > diff --git a/fs/exofs/super.c b/fs/exofs/super.c
> > index 9f500de..ea045b8 100644
> > --- a/fs/exofs/super.c
> > +++ b/fs/exofs/super.c
> > @@ -297,9 +297,13 @@ static int exofs_fill_super(struct super_block *sb, void *data, int silent)
> > struct osd_obj_id obj;
> > int ret;
> >
> > + lock_kernel();
> > +
> > sbi = kzalloc(sizeof(*sbi), GFP_KERNEL);
> > - if (!sbi)
> > + if (!sbi) {
> > + unlock_kernel();
> > return -ENOMEM;
> > + }
> > sb->s_fs_info = sbi;
> >
> > /* use mount options to fill superblock */
> > @@ -399,6 +403,7 @@ static int exofs_fill_super(struct super_block *sb, void *data, int silent)
> > out:
> > if (or)
> > osd_end_request(or);
> > + unlock_kernel();
> > return ret;
> >
> > free_sbi:
>
> You can add the revert of this patch to your series when submitted.
> We don't need the BKL in exofs.
>
> Signed-off-by: Boaz Harrosh <bharrosh@xxxxxxxxxxx>
>
Thanks Boaz. I've added it to my series.
Cheers,
Jan
> @@ -297,13 +297,9 @@ static int exofs_fill_super(struct super_block *sb, void *data, int silent)
> struct osd_obj_id obj;
> int ret;
>
> - lock_kernel();
> -
> sbi = kzalloc(sizeof(*sbi), GFP_KERNEL);
> - if (!sbi) {
> - unlock_kernel();
> + if (!sbi)
> return -ENOMEM;
> - }
> sb->s_fs_info = sbi;
>
> /* use mount options to fill superblock */
> @@ -399,7 +403,6 @@ static int exofs_fill_super(struct super_block *sb, void *data, int silent)
> out:
> if (or)
> osd_end_request(or);
> - unlock_kernel();
> return ret;
>
> free_sbi:
--
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
