mbox series

[0/3] nfs_common: misc patchs

Message ID 20170926071407.31719-1-clabbe.montjoie@gmail.com
Headers show
Series nfs_common: misc patchs | expand

Message

Corentin Labbe Sept. 26, 2017, 7:14 a.m. UTC
Hello

The origin of this series is a build warning fix.
But reading the code show me two other possible patch.

Regards

Corentin Labbe (3):
  nfs_common: fix build warning in grace.c
  nfs_common: move locks_in_grace comment at the right place
  nfs_common: convert int to bool

 fs/nfs_common/grace.c | 24 ++++++++++++------------
 include/linux/fs.h    |  4 ++--
 2 files changed, 14 insertions(+), 14 deletions(-)

-- 
2.13.5

Comments

Jeff Layton Sept. 26, 2017, 11:15 a.m. UTC | #1
On Tue, 2017-09-26 at 09:14 +0200, Corentin Labbe wrote:
> Hello

> 

> The origin of this series is a build warning fix.

> But reading the code show me two other possible patch.

> 

> Regards

> 

> Corentin Labbe (3):

>   nfs_common: fix build warning in grace.c

>   nfs_common: move locks_in_grace comment at the right place

>   nfs_common: convert int to bool

> 

>  fs/nfs_common/grace.c | 24 ++++++++++++------------

>  include/linux/fs.h    |  4 ++--

>  2 files changed, 14 insertions(+), 14 deletions(-)

> 


Looks like a reasonable set of cleanups. Probably fine for v4.15?

Reviewed-by: Jeff Layton <jlayton@redhat.com>
J. Bruce Fields Sept. 26, 2017, 1:25 p.m. UTC | #2
On Tue, Sep 26, 2017 at 07:15:09AM -0400, Jeff Layton wrote:
> On Tue, 2017-09-26 at 09:14 +0200, Corentin Labbe wrote:

> > Hello

> > 

> > The origin of this series is a build warning fix.

> > But reading the code show me two other possible patch.

> > 

> > Regards

> > 

> > Corentin Labbe (3):

> >   nfs_common: fix build warning in grace.c

> >   nfs_common: move locks_in_grace comment at the right place

> >   nfs_common: convert int to bool

> > 

> >  fs/nfs_common/grace.c | 24 ++++++++++++------------

> >  include/linux/fs.h    |  4 ++--

> >  2 files changed, 14 insertions(+), 14 deletions(-)

> > 

> 

> Looks like a reasonable set of cleanups. Probably fine for v4.15?

> 

> Reviewed-by: Jeff Layton <jlayton@redhat.com>


Yep, applying, thanks.

--b.