From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932538AbcH1WiF (ORCPT ); Sun, 28 Aug 2016 18:38:05 -0400 Received: from omzsmtpe03.verizonbusiness.com ([199.249.25.208]:45382 "EHLO omzsmtpe03.verizonbusiness.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932340AbcH1WiE (ORCPT ); Sun, 28 Aug 2016 18:38:04 -0400 X-IronPort-Anti-Spam-Filtered: false From: "Levin, Alexander" Cc: "Levin, Alexander" , Sasha Levin , Greg KH , LKML , "ksummit-discuss@lists.linuxfoundation.org" X-IronPort-AV: E=Sophos;i="5.28,593,1464652800"; d="scan'208";a="767192583" To: Joe Perches Date: Sun, 28 Aug 2016 18:37:59 -0400 Subject: Re: checkkpatch (in)sanity ? Thread-Topic: checkkpatch (in)sanity ? Thread-Index: AdIBfNQ1HjtyqBWiSWiLS0emn87bVA== Message-ID: <20160828223759.GA12993@sasha-lappy> References: <1472330452.26978.23.camel@perches.com> <20160828005636.GB19088@sasha-lappy> <1472348579.26978.47.camel@perches.com> <20160828023807.GC19088@sasha-lappy> <1472404557.26978.84.camel@perches.com> In-Reply-To: <1472404557.26978.84.camel@perches.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: user-agent: Mutt/1.5.24 (2015-08-30) acceptlanguage: en-US Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: 8bit X-MIME-Autoconverted: from quoted-printable to 8bit by mail.home.local id u7SMc9ik019623 On Sun, Aug 28, 2016 at 01:15:57PM -0400, Joe Perches wrote: > On Sat, 2016-08-27 at 22:47 -0400, Levin, Alexander wrote: > > > Would you agree that by default we shouldn't show anything that's > > not an error/defect? > > Not particularly, no. I think that we need to figure out this disagreement first then. My claim is that checkpatch's output isn't useful. Based on your bash snippet, populated with the KS program committee + the first few maintainers I spotted on 'git log': commiter commits issues arnd 858 2155 axboe 53 22 corbet 15 9 davem 55 81 grant.likely 2 0 gregkh 38 46 hch 393 581 James.Bottomley 15 15 martin.petersen 18 20 mchehab 678 1042 mgorman 104 256 mingo 58 192 paulmck 176 68 peterz 226 511 rostedt 123 178 shuahkh 53 6 tglx 200 287 torvalds 64 89 tytso 37 77 viro 350 256 And for the last 10,000 commits in the log, that script has observed 10,783 issues. It'll be interesting to hear from these people about their view of checkpatch, but IMO when on average there are more issues than commits I can suggest two possible causes: 1. People are used to ignore checkpatch warnings. 2. People aren't using checkpatch. Can you really make the claim that this is how checkpatch is supposed to be working? -- Thanks, Sasha