Re: [PATCH 00/31] Rid W=1 warnings from GFS2 and EncryptFS

2021-04-19 Thread Lee Jones
On Mon, 19 Apr 2021, Tyler Hicks wrote:

> On 2021-03-30 17:44:27, Lee Jones wrote:
> > This set is part of a larger effort attempting to clean-up W=1
> > kernel builds, which are currently overwhelmingly riddled with
> > niggly little warnings.
> > 
> > There is more to do in filesystems.  Another set will follow.
> > 
> > Lee Jones (31):
> ...
> >   fs: ecryptfs: read_write: File headers do not make good candidates for
> > kernel-doc
> >   fs: ecryptfs: debug: Demote a couple of kernel-doc abuses
> >   fs: ecryptfs: dentry: File headers are not good candidates for
> > kernel-doc
> >   fs: ecryptfs: kthread: Demote file header and provide description for
> > 'cred'
> >   fs: ecryptfs: file: Demote kernel-doc abuses
> >   fs: ecryptfs: super: Fix formatting, naming and kernel-doc abuses
> >   fs: ecryptfs: messaging: Add missing param descriptions and demote
> > abuses
> >   fs: ecryptfs: main: Demote a bunch of non-conformant kernel-doc
> > headers
> >   fs: ecryptfs: miscdev: File headers are not good kernel-doc candidates
> >   fs: ecryptfs: crypto: Supply some missing param descriptions and
> > demote abuses
> >   fs: ecryptfs: mmap: Help out one function header and demote other
> > abuses
> >   fs: ecryptfs: inode: Help out nearly-there header and demote
> > non-conformant ones
> >   fs: ecryptfs: keystore: Fix some kernel-doc issues and demote
> > non-conformant headers
> 
> I've applied the eCryptfs fixes to the next branch of
> tyhicks/ecryptfs.git. Thanks for the clean-up!

Thanks Tyler, much appreciated.

-- 
Lee Jones [李琼斯]
Senior Technical Lead - Developer Services
Linaro.org │ Open source software for Arm SoCs
Follow Linaro: Facebook | Twitter | Blog


Re: [PATCH 00/31] Rid W=1 warnings from GFS2 and EncryptFS

2021-04-18 Thread Tyler Hicks
On 2021-03-30 17:44:27, Lee Jones wrote:
> This set is part of a larger effort attempting to clean-up W=1
> kernel builds, which are currently overwhelmingly riddled with
> niggly little warnings.
> 
> There is more to do in filesystems.  Another set will follow.
> 
> Lee Jones (31):
...
>   fs: ecryptfs: read_write: File headers do not make good candidates for
> kernel-doc
>   fs: ecryptfs: debug: Demote a couple of kernel-doc abuses
>   fs: ecryptfs: dentry: File headers are not good candidates for
> kernel-doc
>   fs: ecryptfs: kthread: Demote file header and provide description for
> 'cred'
>   fs: ecryptfs: file: Demote kernel-doc abuses
>   fs: ecryptfs: super: Fix formatting, naming and kernel-doc abuses
>   fs: ecryptfs: messaging: Add missing param descriptions and demote
> abuses
>   fs: ecryptfs: main: Demote a bunch of non-conformant kernel-doc
> headers
>   fs: ecryptfs: miscdev: File headers are not good kernel-doc candidates
>   fs: ecryptfs: crypto: Supply some missing param descriptions and
> demote abuses
>   fs: ecryptfs: mmap: Help out one function header and demote other
> abuses
>   fs: ecryptfs: inode: Help out nearly-there header and demote
> non-conformant ones
>   fs: ecryptfs: keystore: Fix some kernel-doc issues and demote
> non-conformant headers

I've applied the eCryptfs fixes to the next branch of
tyhicks/ecryptfs.git. Thanks for the clean-up!

Tyler


[PATCH 00/31] Rid W=1 warnings from GFS2 and EncryptFS

2021-03-30 Thread Lee Jones
This set is part of a larger effort attempting to clean-up W=1
kernel builds, which are currently overwhelmingly riddled with
niggly little warnings.

There is more to do in filesystems.  Another set will follow.

Lee Jones (31):
  fs: gfs2: dir: Finish off mostly complete headers demote others
  fs: gfs2: xattr: Help out some kernel-doc header demote non-conformant
ones
  fs: gfs2: bmap: Demote half-complete kernel-doc headers fix others
  fs: gfs2: glops: Fix a couple of kernel-doc misdemeanours demote
others
  fs: gfs2: log: Fix a couple of param descriptions and demote
non-conformant headers
  fs: gfs2: lops: Help out worthy kernel-doc headers and demote others
  fs: gfs2: glock: Fix some deficient kernel-doc headers and demote
non-conformant ones
  fs: gfs2: aops: Fix a little potential doc-rot
  fs: gfs2: meta_io: Add missing description for 'rahead' param
  fs: gfs2: inode: Fix worthy function headers demote others
  fs: gfs2: file: Strip out superfluous param description and demote
kernel-doc abuse
  fs: gfs2: ops_fstype: Fix a little doc-rot
  fs: gfs2: quota: Fix incorrect param name in 'gfs2_quotad'
  fs: gfs2: rgrp: Fix a few kernel-doc misdemeanours
  fs: gfs2: recovery: Provide missing param descriptions and remove one
other
  fs: gfs2: super: Fix kernel-doc issues where feasible demote one other
  fs: gfs2: util: Fix one conformant kernel-doc header and demote a
bunch of abuses
  fs: gfs2: lock_dlm: Demote incomplete kernel-doc header
  fs: ecryptfs: read_write: File headers do not make good candidates for
kernel-doc
  fs: ecryptfs: debug: Demote a couple of kernel-doc abuses
  fs: ecryptfs: dentry: File headers are not good candidates for
kernel-doc
  fs: ecryptfs: kthread: Demote file header and provide description for
'cred'
  fs: ecryptfs: file: Demote kernel-doc abuses
  fs: ecryptfs: super: Fix formatting, naming and kernel-doc abuses
  fs: ecryptfs: messaging: Add missing param descriptions and demote
abuses
  fs: ecryptfs: main: Demote a bunch of non-conformant kernel-doc
headers
  fs: ecryptfs: miscdev: File headers are not good kernel-doc candidates
  fs: ecryptfs: crypto: Supply some missing param descriptions and
demote abuses
  fs: ecryptfs: mmap: Help out one function header and demote other
abuses
  fs: ecryptfs: inode: Help out nearly-there header and demote
non-conformant ones
  fs: ecryptfs: keystore: Fix some kernel-doc issues and demote
non-conformant headers

 fs/ecryptfs/crypto.c | 20 ++--
 fs/ecryptfs/debug.c  |  4 ++--
 fs/ecryptfs/dentry.c |  2 +-
 fs/ecryptfs/file.c   |  4 ++--
 fs/ecryptfs/inode.c  | 11 +--
 fs/ecryptfs/keystore.c   | 13 +++--
 fs/ecryptfs/kthread.c|  3 ++-
 fs/ecryptfs/main.c   | 20 +---
 fs/ecryptfs/messaging.c  |  8 +---
 fs/ecryptfs/miscdev.c|  3 ++-
 fs/ecryptfs/mmap.c   | 11 ++-
 fs/ecryptfs/read_write.c |  4 +++-
 fs/ecryptfs/super.c  |  8 
 fs/gfs2/aops.c   |  5 +
 fs/gfs2/bmap.c   |  4 ++--
 fs/gfs2/dir.c| 35 +--
 fs/gfs2/file.c   |  3 +--
 fs/gfs2/glock.c  |  9 -
 fs/gfs2/glops.c  | 18 ++
 fs/gfs2/inode.c  | 22 --
 fs/gfs2/lock_dlm.c   |  3 +--
 fs/gfs2/log.c| 11 ---
 fs/gfs2/lops.c   | 13 +++--
 fs/gfs2/meta_io.c|  1 +
 fs/gfs2/ops_fstype.c |  7 ++-
 fs/gfs2/quota.c  |  2 +-
 fs/gfs2/recovery.c   |  3 ++-
 fs/gfs2/rgrp.c   |  4 +++-
 fs/gfs2/super.c  | 13 ++---
 fs/gfs2/util.c   | 19 ++-
 fs/gfs2/xattr.c  | 25 +++--
 31 files changed, 141 insertions(+), 167 deletions(-)

Cc: Andreas Gruenbacher 
Cc: Andrew Morton 
Cc: Bob Peterson 
Cc: Christian Brauner 
Cc: cluster-de...@redhat.com
Cc: David Howells 
Cc: ecryp...@vger.kernel.org
Cc: Eric Biggers 
Cc: James Morris 
Cc: Johannes Weiner 
Cc: "Michael A. Halcrow" 
Cc: "Michael A. Halcrow" 
Cc: "Michael C. Thompsion" 
Cc: "Michael C. Thompson" 
Cc: Michal Hocko 
Cc: "Trevor S. Highland" 
Cc: Tycho Andersen 
Cc: Tyler Hicks 
Cc: Waiman Long 
-- 
2.27.0