Received: by 2002:a25:c593:0:0:0:0:0 with SMTP id v141csp1719148ybe; Fri, 13 Sep 2019 23:55:24 -0700 (PDT) X-Google-Smtp-Source: APXvYqw8pUZnbNmfMSBYDOESWgmWnoWoBLxqNF/5orFmaP1B+3paiT7xErdU8m8K24wqvr9m43NM X-Received: by 2002:a17:906:eddd:: with SMTP id sb29mr43485364ejb.278.1568444124441; Fri, 13 Sep 2019 23:55:24 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1568444124; cv=none; d=google.com; s=arc-20160816; b=CblaqPbVnajdYffs5eRjeltkMIy56+Qg99qoqzZ4mFcsAm4PKrBap242l4nONk+ofa mKnwcb02eR/j1Qr3H5eLUVlZMaGLxE1WbmVTDzjQTidZ9h78CNU2sPwShNR+XFoW89tn +dqjs0X5TvYdUG8jTYt16+GmeS/LdIqavL9Hng3J8xl35ESCOxr0xNUDCVOSF/XAFscZ 3/CL5USkgBl6MyXDpcCJWaDVFJM9FLatoVki7CzAuNVHjm40GW0BHdBuomy70RCAANLe ply3mAVJqt/i/tPEH9uYrEp1WYM6R3odjMf6j3VFvHjXq6cGQsO59HrGEkl1EkvduwRm oWiA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding:mime-version :references:in-reply-to:message-id:subject:cc:to:from:date :dkim-signature; bh=rlRJRtbWGqF5qJn2FgghKNYZdCECrxLGGuNAvdpBnhE=; b=HI/P26WpoGkXQ+fOoMGVZJRD4FVrf586KxxMTosQjdWdmVNQ+Vbo0jihybIBMBnjXA 9UIC8cSRziIvEFv0WB09SQepa9d49bIubZw7F5cAkEMbsGl38lwEYMA810PvheWH05v5 Kof+KomOadWKsPYMlbXMNJSy1nrcRFLzNEcwWYy0MNKwc/OAOXpEGtUz2ijvMsHqT8X/ VwmFXUdzqPSPLqOOLetyRt9ugnHJ0lmYCMYeUpmCpWiYrcM3Hlh2nWchvB5s1t+f6Nt5 YBmE9BQl4YDf93BO2m7baL5bJX0FePYrUaveuFMaK0W2X9WGtMyv+MerJK41hOa6uyS1 lJ0g== ARC-Authentication-Results: i=1; mx.google.com; dkim=fail header.i=@infradead.org header.s=bombadil.20170209 header.b=soQppz2s; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id f33si19188284edb.281.2019.09.13.23.55.01; Fri, 13 Sep 2019 23:55:24 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; dkim=fail header.i=@infradead.org header.s=bombadil.20170209 header.b=soQppz2s; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S2388683AbfIMTRr (ORCPT + 99 others); Fri, 13 Sep 2019 15:17:47 -0400 Received: from bombadil.infradead.org ([198.137.202.133]:40018 "EHLO bombadil.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726558AbfIMTRr (ORCPT ); Fri, 13 Sep 2019 15:17:47 -0400 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=infradead.org; s=bombadil.20170209; h=Content-Transfer-Encoding: Content-Type:MIME-Version:References:In-Reply-To:Message-ID:Subject:Cc:To: From:Date:Sender:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Id: List-Help:List-Unsubscribe:List-Subscribe:List-Post:List-Owner:List-Archive; bh=rlRJRtbWGqF5qJn2FgghKNYZdCECrxLGGuNAvdpBnhE=; b=soQppz2sjZ542l4x8+d2PrSGd JUy0I/+m3xaGNL+3TaQdmws2k+87EdwqozCJbNQoNkOdYssAzSQft6aYDAKIOS6S+ZIbjO0ujXi4U Z8tWDGTXggK7USMOULInO7NZgGXRQvg83VfEBJiD3nE28UFgdO0JDMb8qcdJwfHetcRB/NBAgJObA Ie+2H2YULowfnE+GM/bw+v48uAfTloz7RO4/9G94UX2N4vuUNNy9xZUD796Bo6JJbJ6iWYK5QILFF 7WuNSj1MHoyTRMQ1c3suXpO0RgUhE+F2F6et9sk6WaTHgecOGqrygNc2HRphWdHBmlKmHZcau7Qu5 slqT0cpBg==; Received: from 177.96.232.144.dynamic.adsl.gvt.net.br ([177.96.232.144] helo=coco.lan) by bombadil.infradead.org with esmtpsa (Exim 4.92.2 #3 (Red Hat Linux)) id 1i8r4b-0000J3-2S; Fri, 13 Sep 2019 19:17:37 +0000 Date: Fri, 13 Sep 2019 16:17:31 -0300 From: Mauro Carvalho Chehab To: Joe Perches Cc: Rob Herring , Bart Van Assche , "Martin K. Petersen" , Dan Williams , Dave Jiang , ksummit , linux-nvdimm , Greg Kroah-Hartman , "linux-kernel@vger.kernel.org" , Steve French , Vishal Verma , Dmitry Vyukov , "Tobin C. Harding" Subject: Re: [Ksummit-discuss] [PATCH v2 0/3] Maintainer Entry Profiles Message-ID: <20190913161731.6e3405a3@coco.lan> In-Reply-To: <78f67ee934f167b433517da81c6a0d3f35c4b123.camel@perches.com> References: <156821692280.2951081.18036584954940423225.stgit@dwillia2-desk3.amr.corp.intel.com> <6fe45562-9493-25cf-afdb-6c0e702a49b4@acm.org> <44c08faf43fa77fb271f8dbb579079fb09007716.camel@perches.com> <74984dc0-d5e4-f272-34b9-9a78619d5a83@acm.org> <4299c79e33f22e237e42515ea436f187d8beb32e.camel@perches.com> <78f67ee934f167b433517da81c6a0d3f35c4b123.camel@perches.com> X-Mailer: Claws Mail 3.17.3 (GTK+ 2.24.32; x86_64-redhat-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Em Fri, 13 Sep 2019 11:42:38 -0700 Joe Perches escreveu: > On Fri, 2019-09-13 at 15:26 +0100, Rob Herring wrote: > > On Fri, Sep 13, 2019 at 3:12 PM Joe Perches wrote: > > > On Thu, 2019-09-12 at 13:01 -0700, Bart Van Assche wrote: > > > > > > > Another argument in favor of W=1 is that the formatting of kernel-doc > > > > headers is checked only if W=1 is passed to make. > > > > > > Actually, but for the fact there are far too many > > > to generally enable that warning right now, > > > (an x86-64 defconfig has more than 1000) > > > that sounds pretty reasonable to me. > > > It's in the 1000s on arm because W=1 turns on more checks in building > > .dts files. There are lots of duplicates as most of the dts content is > > as an include file (e.g. board dts includes soc dts). > > Yeah, dts[i] files in arm compilations are very noisy at W=1 > so moving those message types to W=2 seems sensible. > > $ { opt="ARCH=arm CROSS_COMPILE=arm-unknown-linux-gnueabi-" ; make $opt clean ; make $opt defconfig ; make $opt W=1 -j4 ; } > arm_make.log 2>&1 > > $ grep -i -P 'dtsi?:.*warning' arm_make.log | wc -l > 69164 Yeah, makes sense moving them to W=2, or to make them somehow produce less noise. > Just fyi: for an x86-64 defconfig with gcc 8.3 > > $ { make clean ; make defconfig ; make -j4 W=1 ; } > make.log 2>&1 > > There are ~300 W=1 for non kernel-doc -W warnings. > > $ grep -i -P -oh '\[\-W[\w\-]+\]' make.log |sort | uniq -c | sort -rn > 163 [-Wmissing-prototypes] > 69 [-Wunused-but-set-variable] > 16 [-Wempty-body] > 10 [-Wtype-limits] > 6 [-Woverride-init] > 2 [-Wstringop-truncation] > 2 [-Wcast-function-type] > 1 [-Wunused-but-set-parameter] On my eyes, it doesn't sound too much. I suspect that, with gcc-9, it should produce more warnings, though. Perhaps we could "promote" most of those to W=0. > > And there are ~1000 kernel-doc only messages in various files A significant amount of those warnings appear with "make htmldocs". Not having the kernel-doc warning as part of W=0 helps to make very hard to have them cleared. IMHO, those should be enabled by default with W=0, at least for the files that are included on some kernel-doc tag. I mean, perhaps, when W=0, Kernel build could run: $ ./scripts/kernel-doc -none $(git grep kernel-doc:: Documentation/|cut -d: -f4-|sort|uniq|grep -vE "\bsource\b") That produces 165 warnings (against v5.3-rc4 + media -next patches). Thanks, Mauro