From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754136AbdBISsW (ORCPT ); Thu, 9 Feb 2017 13:48:22 -0500 Received: from one.firstfloor.org ([193.170.194.197]:46110 "EHLO one.firstfloor.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753303AbdBISrx (ORCPT ); Thu, 9 Feb 2017 13:47:53 -0500 Date: Thu, 9 Feb 2017 10:25:19 -0800 From: Andi Kleen To: Arnaldo Carvalho de Melo Cc: Adrian Hunter , Alexander Shishkin , Andi Kleen , Jiri Olsa , Linux Kernel Mailing List Subject: Re: Intel PT decoder switch case fallthrough cases reported by gcc 7 Message-ID: <20170209182519.GR26852@two.firstfloor.org> References: <20170209165039.GA2488@kernel.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20170209165039.GA2488@kernel.org> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, Feb 09, 2017 at 01:50:39PM -0300, Arnaldo Carvalho de Melo wrote: > Hi, > > I've updated the container with Fedora Rawhide I use to build > tools/perf/ and samples/bcc/ and it now comes with gcc 7, where I get > things like: FWIW, but it just shows that you should never ship software with -Werror enabled. New compiler releases add new warnings, and you just randomly break the build for users who use newer compilers. It's ok to use in your own builds, but should never be default. The kernel gets it right, but perf is wrong here. -Andi