From: Lee Jones <lee.jones@linaro.org>
To: Bob Peterson <rpeterso@redhat.com>
Cc: linux-kernel@vger.kernel.org,
Andreas Gruenbacher <agruenba@redhat.com>,
cluster-devel@redhat.com
Subject: Re: [PATCH 00/18] Rid W=1 warnings from GFS2
Date: Tue, 6 Apr 2021 12:54:34 +0100 [thread overview]
Message-ID: <20210406115434.GU2916463@dell> (raw)
In-Reply-To: <468723920.4255981.1617709446972.JavaMail.zimbra@redhat.com>
On Tue, 06 Apr 2021, Bob Peterson wrote:
> ----- Original Message -----
> > On Fri, 26 Mar 2021, 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.
> > >
> > > Lee Jones (18):
> > > 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 superflouous 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
> >
> > These have been on the list for a couple of weeks now.
> >
> > Is there anything I can do to help expedite their merge?
> >
> > I'm concerned since -rc6 has just been released.
>
> Hi Lee,
>
> Thanks for your patches. Andreas and I have a slight disagreement about them.
> I wanted to merge them, but Andreas is concerned that there may be some
> cases in which we should get rid of parameter descriptions altogether.
> So I've been waiting for Andreas to review them all before we push them.
> Also, the dir.c patch might be out of date because of a patch I recently did that
> was recently added to for-next. I'll work with Andreas to expedite them so
> they can go into the next merge window.
Thanks for the update Bob.
My initial response to that would be; this patch-set ensures all of
the warnings are taken care of. Removal of superfluous documentation
could be easily undertaken as part of a subsequent effort.
It would be good to have these warnings quashed for v5.13.
--
Lee Jones [李琼斯]
Senior Technical Lead - Developer Services
Linaro.org │ Open source software for Arm SoCs
Follow Linaro: Facebook | Twitter | Blog
next prev parent reply other threads:[~2021-04-06 11:54 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-03-26 9:11 [PATCH 00/18] Rid W=1 warnings from GFS2 Lee Jones
2021-03-26 9:11 ` [PATCH 01/18] fs: gfs2: dir: Finish off mostly complete headers demote others Lee Jones
2021-03-26 9:11 ` [PATCH 02/18] fs: gfs2: xattr: Help out some kernel-doc header demote non-conformant ones Lee Jones
2021-03-26 9:11 ` [PATCH 03/18] fs: gfs2: bmap: Demote half-complete kernel-doc headers fix others Lee Jones
2021-03-26 9:11 ` [PATCH 04/18] fs: gfs2: glops: Fix a couple of kernel-doc misdemeanours demote others Lee Jones
2021-03-26 9:11 ` [PATCH 05/18] fs: gfs2: log: Fix a couple of param descriptions and demote non-conformant headers Lee Jones
2021-03-26 9:11 ` [PATCH 06/18] fs: gfs2: lops: Help out worthy kernel-doc headers and demote others Lee Jones
2021-03-26 9:11 ` [PATCH 07/18] fs: gfs2: glock: Fix some deficient kernel-doc headers and demote non-conformant ones Lee Jones
2021-03-26 9:11 ` [PATCH 08/18] fs: gfs2: aops: Fix a little potential doc-rot Lee Jones
2021-03-26 9:11 ` [PATCH 09/18] fs: gfs2: meta_io: Add missing description for 'rahead' param Lee Jones
2021-03-26 9:11 ` [PATCH 10/18] fs: gfs2: inode: Fix worthy function headers demote others Lee Jones
2021-03-26 9:11 ` [PATCH 11/18] fs: gfs2: file: Strip out superflouous param description and demote kernel-doc abuse Lee Jones
2021-03-26 9:11 ` [PATCH 12/18] fs: gfs2: ops_fstype: Fix a little doc-rot Lee Jones
2021-03-26 9:11 ` [PATCH 13/18] fs: gfs2: quota: Fix incorrect param name in 'gfs2_quotad' Lee Jones
2021-03-26 9:11 ` [PATCH 14/18] fs: gfs2: rgrp: Fix a few kernel-doc misdemeanours Lee Jones
2021-03-26 9:11 ` [PATCH 15/18] fs: gfs2: recovery: Provide missing param descriptions and remove one other Lee Jones
2021-03-26 9:11 ` [PATCH 16/18] fs: gfs2: super: Fix kernel-doc issues where feasible demote " Lee Jones
2021-03-26 9:11 ` [PATCH 17/18] fs: gfs2: util: Fix one conformant kernel-doc header and demote a bunch of abuses Lee Jones
2021-03-26 9:11 ` [PATCH 18/18] fs: gfs2: lock_dlm: Demote incomplete kernel-doc header Lee Jones
2021-04-06 9:11 ` [PATCH 00/18] Rid W=1 warnings from GFS2 Lee Jones
2021-04-06 11:44 ` Bob Peterson
2021-04-06 11:54 ` Lee Jones [this message]
2021-04-09 20:35 ` Andreas Gruenbacher
2021-04-12 7:37 ` Lee Jones
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20210406115434.GU2916463@dell \
--to=lee.jones@linaro.org \
--cc=agruenba@redhat.com \
--cc=cluster-devel@redhat.com \
--cc=linux-kernel@vger.kernel.org \
--cc=rpeterso@redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).