2016-03-01 09:00:57

by Chao Yu

[permalink] [raw]
Subject: RE: [PATCH] f2fs: mutex can't be used by down_write_nest_lock()

Hi Yang,

> -----Original Message-----
> From: Yang Shi [mailto:[email protected]]
> Sent: Saturday, February 27, 2016 8:25 AM
> To: [email protected]; [email protected]; [email protected]; [email protected];
> [email protected]; [email protected]
> Cc: [email protected]; [email protected];
> [email protected]; [email protected];
> [email protected]; [email protected]
> Subject: [PATCH] f2fs: mutex can't be used by down_write_nest_lock()
>
> fsf2_lock_all() calls down_write_nest_lock() to acquire a rw_sem and check

Nitpick, f2fs_lock_all().

Anyway, this patch looks good to me.

Reviewed-by: Chao Yu <[email protected]>


2016-03-02 18:22:52

by Jaegeuk Kim

[permalink] [raw]
Subject: Re: [PATCH] f2fs: mutex can't be used by down_write_nest_lock()

On Tue, Mar 01, 2016 at 04:59:39PM +0800, Chao Yu wrote:
> Hi Yang,
>
> > -----Original Message-----
> > From: Yang Shi [mailto:[email protected]]
> > Sent: Saturday, February 27, 2016 8:25 AM
> > To: [email protected]; [email protected]; [email protected]; [email protected];
> > [email protected]; [email protected]
> > Cc: [email protected]; [email protected];
> > [email protected]; [email protected];
> > [email protected]; [email protected]
> > Subject: [PATCH] f2fs: mutex can't be used by down_write_nest_lock()
> >
> > fsf2_lock_all() calls down_write_nest_lock() to acquire a rw_sem and check

Fixed, and merged. :)

>
> Nitpick, f2fs_lock_all().
>
> Anyway, this patch looks good to me.
>
> Reviewed-by: Chao Yu <[email protected]>