Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S965350AbWAFXMW (ORCPT ); Fri, 6 Jan 2006 18:12:22 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S965352AbWAFXMW (ORCPT ); Fri, 6 Jan 2006 18:12:22 -0500 Received: from iabervon.org ([66.92.72.58]:59146 "EHLO iabervon.org") by vger.kernel.org with ESMTP id S965350AbWAFXMU (ORCPT ); Fri, 6 Jan 2006 18:12:20 -0500 Date: Fri, 6 Jan 2006 18:13:55 -0500 (EST) From: Daniel Barkalow To: Jesper Juhl cc: Al Viro , Luiz Fernando Capitulino , akpm , axboe@suse.de, lkml Subject: Re: [PATCH] bio: gcc warning fix. In-Reply-To: <9a8748490601061452h4b61161cie41ca9605ebef00@mail.gmail.com> Message-ID: References: <20060106130729.31561730.lcapitulino@mandriva.com.br> <20060106153950.GV27946@ftp.linux.org.uk> <9a8748490601061452h4b61161cie41ca9605ebef00@mail.gmail.com> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1823 Lines: 41 On Fri, 6 Jan 2006, Jesper Juhl wrote: > On 1/6/06, Daniel Barkalow wrote: > > On Fri, 6 Jan 2006, Al Viro wrote: > > > > > On Fri, Jan 06, 2006 at 01:07:29PM -0200, Luiz Fernando Capitulino wrote: > > > > > > > > Fixes the following gcc 4.0.2 warning: > > > > > > > > fs/bio.c: In function 'bio_alloc_bioset': > > > > fs/bio.c:167: warning: 'idx' may be used uninitialized in this function > > > > > > NAK. There is nothing to fix except for broken logics in gcc. > > > Please, do not obfuscate correct code just to make gcc to shut up. > > > Especially for this call of warnings; gcc4 *blows* in that area. > > > > Wouldn't it be worthwhile to add -Wno-uninitialized for gcc4, since those > > warnings mostly have to be ignored (and are therefore not useful for > > finding actual uninitialized variables) and make it harder to see other > > types of warnings which might be informative? > > > > This would also reduce the number of patches submitted for correct code, > > since people wouldn't keep having to be told to ignore those warnings. > > > > Hmm, the uninitialized warnings do find real bugs now and then. > Generate some noice as well, true, but won't we rather tollerate a > little noice rather than miss bugs? Uninitialized warnings from some gcc versions do find real bugs, but I bet that randomly choosing variable declarations to inspect would be more likely to find bugs than looking at gcc4 warnings. If we tollerate noise, we're likely to ignore the noise that identifies a bug. -Daniel *This .sig left intentionally blank* - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/