From: Jason Gunthorpe <jgg@ziepe.ca>
To: Joe Perches <joe@perches.com>
Cc: "Knut Omang" <knut.omang@oracle.com>,
"Stephen Hemminger" <stephen@networkplumber.org>,
linux-kernel@vger.kernel.org,
"Mauro Carvalho Chehab" <mchehab@kernel.org>,
"Nicolas Palix" <nicolas.palix@imag.fr>,
"Jonathan Corbet" <corbet@lwn.net>,
"Santosh Shilimkar" <santosh.shilimkar@oracle.com>,
"Matthew Wilcox" <willy@infradead.org>,
cocci@systeme.lip6.fr, rds-devel@oss.oracle.com,
linux-rdma@vger.kernel.org, linux-doc@vger.kernel.org,
"Doug Ledford" <dledford@redhat.com>,
"Mickaël Salaün" <mic@digikod.net>,
"Shuah Khan" <shuah@kernel.org>,
linux-kbuild@vger.kernel.org,
"Michal Marek" <michal.lkml@markovi.net>,
"Julia Lawall" <Julia.Lawall@lip6.fr>,
"John Haxby" <john.haxby@oracle.com>,
"Åsmund Østvold" <asmund.ostvold@oracle.com>,
"Masahiro Yamada" <yamada.masahiro@socionext.com>,
"Kees Cook" <keescook@chromium.org>,
netdev@vger.kernel.org, "Gilles Muller" <Gilles.Muller@lip6.fr>,
"David S. Miller" <davem@davemloft.net>,
"Paul E. McKenney" <paulmck@linux.vnet.ibm.com>,
"Håkon Bugge" <haakon.bugge@oracle.com>,
"Andy Whitcroft" <apw@canonical.com>,
"Levin, Alexander (Sasha Levin)" <alexander.levin@verizon.com>
Subject: Re: [PATCH v2 0/5] Support for generalized use of make C={1,2} via a wrapper program
Date: Mon, 18 Dec 2017 10:46:04 -0700 [thread overview]
Message-ID: <20171218174604.GA19056@ziepe.ca> (raw)
In-Reply-To: <1513576817.31581.58.camel@perches.com>
On Sun, Dec 17, 2017 at 10:00:17PM -0800, Joe Perches wrote:
> > Today when we run checkers we get so many warnings it is too hard to
> > make any sense of it.
>
> Here is a list of the checkpatch messages for drivers/infiniband
> sorted by type.
>
> Many of these might be corrected by using
>
> $ ./scripts/checkpatch.pl -f --fix-inplace --types=<TYPE> \
> $(git ls-files drivers/infiniband/)
How many of these do you think it is worth to fix?
We do get a steady trickle of changes in this topic every cycle.
Is it better to just do a big number of them all at once? Do you have
an idea how disruptive this kind of work is to the whole patch flow
eg new patches no longer applying to for-next, backports no longer
applying, merge conflicts?
Jason
next prev parent reply other threads:[~2017-12-18 17:46 UTC|newest]
Thread overview: 42+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-12-16 14:42 [PATCH v2 0/5] Support for generalized use of make C={1,2} via a wrapper program Knut Omang
2017-12-16 14:42 ` [PATCH v2 1/5] runchecks: Generalize make C={1,2} to support multiple checkers Knut Omang
2017-12-16 14:42 ` [PATCH v2 2/5] Documentation: Add doc for runchecks, a checker runner Knut Omang
2017-12-16 15:08 ` Julia Lawall
2017-12-16 15:52 ` Knut Omang
2017-12-16 14:42 ` [PATCH v2 3/5] checkpatch: Improve --fix-inplace for TABSTOP Knut Omang
2017-12-16 15:13 ` Joe Perches
2017-12-16 15:55 ` Knut Omang
2017-12-16 14:42 ` [PATCH v2 4/5] rds: Add runchecks.cfg for net/rds Knut Omang
2017-12-16 17:45 ` Stephen Hemminger
2017-12-16 18:24 ` Joe Perches
2017-12-16 20:00 ` santosh.shilimkar
2017-12-17 2:02 ` Knut Omang
2017-12-18 19:28 ` Santosh Shilimkar
2017-12-17 1:57 ` Knut Omang
2017-12-17 1:46 ` Knut Omang
2017-12-16 14:42 ` [PATCH v2 5/5] RDMA/core: Add runchecks.cfg for drivers/infiniband/core Knut Omang
2017-12-18 8:02 ` Leon Romanovsky
2017-12-18 12:36 ` Knut Omang
2017-12-18 14:04 ` Leon Romanovsky
2017-12-18 15:23 ` Knut Omang
2017-12-18 19:03 ` Joe Perches
2017-12-18 19:18 ` Leon Romanovsky
2017-12-16 15:21 ` [PATCH v2 0/5] Support for generalized use of make C={1,2} via a wrapper program Joe Perches
2017-12-16 16:27 ` Knut Omang
2017-12-16 17:00 ` Joe Perches
2017-12-16 17:11 ` Knut Omang
2017-12-16 17:47 ` Stephen Hemminger
2017-12-16 18:02 ` Joe Perches
2017-12-17 2:14 ` Knut Omang
2017-12-18 5:00 ` Jason Gunthorpe
2017-12-18 6:00 ` Joe Perches
2017-12-18 13:05 ` Knut Omang
2017-12-18 15:30 ` Joe Perches
2017-12-18 16:41 ` Knut Omang
2017-12-18 17:49 ` Jason Gunthorpe
2017-12-18 17:46 ` Jason Gunthorpe [this message]
2017-12-18 17:53 ` Joe Perches
2017-12-18 17:56 ` Bart Van Assche
2017-12-18 18:39 ` Knut Omang
2017-12-18 19:24 ` Leon Romanovsky
2017-12-18 13:41 ` Knut Omang
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=20171218174604.GA19056@ziepe.ca \
--to=jgg@ziepe.ca \
--cc=Gilles.Muller@lip6.fr \
--cc=Julia.Lawall@lip6.fr \
--cc=alexander.levin@verizon.com \
--cc=apw@canonical.com \
--cc=asmund.ostvold@oracle.com \
--cc=cocci@systeme.lip6.fr \
--cc=corbet@lwn.net \
--cc=davem@davemloft.net \
--cc=dledford@redhat.com \
--cc=haakon.bugge@oracle.com \
--cc=joe@perches.com \
--cc=john.haxby@oracle.com \
--cc=keescook@chromium.org \
--cc=knut.omang@oracle.com \
--cc=linux-doc@vger.kernel.org \
--cc=linux-kbuild@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-rdma@vger.kernel.org \
--cc=mchehab@kernel.org \
--cc=mic@digikod.net \
--cc=michal.lkml@markovi.net \
--cc=netdev@vger.kernel.org \
--cc=nicolas.palix@imag.fr \
--cc=paulmck@linux.vnet.ibm.com \
--cc=rds-devel@oss.oracle.com \
--cc=santosh.shilimkar@oracle.com \
--cc=shuah@kernel.org \
--cc=stephen@networkplumber.org \
--cc=willy@infradead.org \
--cc=yamada.masahiro@socionext.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