linux-kernel - Re: [PATCH v2 0/5] Support for generalized use of make C={1,2} via a wrapper program

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Mon, 18 Dec 2017 19:39:50 +0100
From:   Knut Omang <knut.omang@...cle.com>
To:     Bart Van Assche <Bart.VanAssche@....com>,
        "joe@...ches.com" <joe@...ches.com>, "jgg@...pe.ca" <jgg@...pe.ca>
Cc:     "corbet@....net" <corbet@....net>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        "keescook@...omium.org" <keescook@...omium.org>,
        "linux-rdma@...r.kernel.org" <linux-rdma@...r.kernel.org>,
        "linux-doc@...r.kernel.org" <linux-doc@...r.kernel.org>,
        "willy@...radead.org" <willy@...radead.org>,
        "nicolas.palix@...g.fr" <nicolas.palix@...g.fr>,
        "asmund.ostvold@...cle.com" <asmund.ostvold@...cle.com>,
        "john.haxby@...cle.com" <john.haxby@...cle.com>,
        "alexander.levin@...izon.com" <alexander.levin@...izon.com>,
        "mchehab@...nel.org" <mchehab@...nel.org>,
        "haakon.bugge@...cle.com" <haakon.bugge@...cle.com>,
        "michal.lkml@...kovi.net" <michal.lkml@...kovi.net>,
        "Gilles.Muller@...6.fr" <Gilles.Muller@...6.fr>,
        "yamada.masahiro@...ionext.com" <yamada.masahiro@...ionext.com>,
        "mic@...ikod.net" <mic@...ikod.net>,
        "shuah@...nel.org" <shuah@...nel.org>,
        "paulmck@...ux.vnet.ibm.com" <paulmck@...ux.vnet.ibm.com>,
        "linux-kbuild@...r.kernel.org" <linux-kbuild@...r.kernel.org>,
        "rds-devel@....oracle.com" <rds-devel@....oracle.com>,
        "Julia.Lawall@...6.fr" <Julia.Lawall@...6.fr>,
        "netdev@...r.kernel.org" <netdev@...r.kernel.org>,
        "davem@...emloft.net" <davem@...emloft.net>,
        "stephen@...workplumber.org" <stephen@...workplumber.org>,
        "cocci@...teme.lip6.fr" <cocci@...teme.lip6.fr>,
        "dledford@...hat.com" <dledford@...hat.com>,
        "santosh.shilimkar@...cle.com" <santosh.shilimkar@...cle.com>,
        "apw@...onical.com" <apw@...onical.com>
Subject: Re: [PATCH v2 0/5] Support for generalized use of make C={1,2} via
 a wrapper program

On Mon, 2017-12-18 at 17:56 +0000, Bart Van Assche wrote:
> On Mon, 2017-12-18 at 10:46 -0700, Jason Gunthorpe wrote:
> > 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?
> 
> In my opinion patches that only change the coding style and do not change any
> functionality are annoying. Before posting a patch that fixes a bug the change
> history (git log -p) has to be cheched to figure out which patch introduced
> the bug. Patches that only change coding style pollute the change history.

I agree with you - the problem is that style issues should not have existed. 
But when they do it becomes a problem to remove them and a problem to 
keep them - for instance us who try to be compliant by having style helpers 
in our editor, we end up having to manually revert old style mistakes back in
to avoid making unrelated whitespace changes or similar.

I think what we get with this patch set is that there's a way to rein in the
issues and to enable some regression testing without enforcing a lot of work 
or annoying history issues *right away*. That way this problem can be tackled when
appropriate for the subsystem in question, and the reason for holding back on 
some changes can be documented in the local runchecks.cfg file, focusing
willing helping hands on the issues considered most important for that 
subsystem.

Thanks,
Knut