From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752828AbdEJORX (ORCPT ); Wed, 10 May 2017 10:17:23 -0400 Received: from gate.crashing.org ([63.228.1.57]:55341 "EHLO gate.crashing.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752744AbdEJORW (ORCPT ); Wed, 10 May 2017 10:17:22 -0400 Date: Wed, 10 May 2017 08:40:22 -0500 From: Segher Boessenkool To: Arnd Bergmann Cc: Andre Przywara , LAKML , Tony Breeds , Guenter Roeck , LKML Subject: Re: Updating kernel.org cross compilers? Message-ID: <20170510134019.GB19687@gate.crashing.org> References: <92b5ff36-ad2b-10bc-de4f-4774babd9625@arm.com> <20170430052957.GJ19687@gate.crashing.org> <20170509221843.GZ19687@gate.crashing.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.4.2.3i Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Arnd, long time no see, On Wed, May 10, 2017 at 09:58:13AM +0200, Arnd Bergmann wrote: > >> So in addition to GCC 7.1 I'd like to have at least GCC 6.3 around, > >> which builds kernels without warnings today. > > > > If you don't want warnings, turn off the warnings or just don't look at > > them... or fix the problems? Many of the new warnings point out actual > > problems. > > > > Many of those sprintf problems in the kernel have already been fixed. > > I've been using gcc-7.0 for a long time and fixed a lot of bugs it found, > along with more harmless warnings, but I had disabled a couple of > warning options when I first installed gcc-7 and ended up ignoring > those. > > The exact set of additional options I used is: > > -Wimplicit-fallthrough=0 -Wno-duplicate-decl-specifier > -Wno-int-in-bool-context -Wno-bool-operation -Wno-format-truncation > -Wno-format-overflow > > there were a couple of others that I sent kernel fixes for instead. > I should probably revisit that list and for each of them either > only enable it with "make W=1" or fix all known warnings. > In the long run, I'd actually hope to fix all W=1 warnings too > and enable them by default. Most of those usually point out actual problems (at least code that isn't as clear as it should be). I do hate that first one though. Segher From mboxrd@z Thu Jan 1 00:00:00 1970 From: segher@kernel.crashing.org (Segher Boessenkool) Date: Wed, 10 May 2017 08:40:22 -0500 Subject: Updating kernel.org cross compilers? In-Reply-To: References: <92b5ff36-ad2b-10bc-de4f-4774babd9625@arm.com> <20170430052957.GJ19687@gate.crashing.org> <20170509221843.GZ19687@gate.crashing.org> Message-ID: <20170510134019.GB19687@gate.crashing.org> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hi Arnd, long time no see, On Wed, May 10, 2017 at 09:58:13AM +0200, Arnd Bergmann wrote: > >> So in addition to GCC 7.1 I'd like to have at least GCC 6.3 around, > >> which builds kernels without warnings today. > > > > If you don't want warnings, turn off the warnings or just don't look at > > them... or fix the problems? Many of the new warnings point out actual > > problems. > > > > Many of those sprintf problems in the kernel have already been fixed. > > I've been using gcc-7.0 for a long time and fixed a lot of bugs it found, > along with more harmless warnings, but I had disabled a couple of > warning options when I first installed gcc-7 and ended up ignoring > those. > > The exact set of additional options I used is: > > -Wimplicit-fallthrough=0 -Wno-duplicate-decl-specifier > -Wno-int-in-bool-context -Wno-bool-operation -Wno-format-truncation > -Wno-format-overflow > > there were a couple of others that I sent kernel fixes for instead. > I should probably revisit that list and for each of them either > only enable it with "make W=1" or fix all known warnings. > In the long run, I'd actually hope to fix all W=1 warnings too > and enable them by default. Most of those usually point out actual problems (at least code that isn't as clear as it should be). I do hate that first one though. Segher