From owner-svn-src-head@freebsd.org Thu May 24 05:33:15 2018 Return-Path: Delivered-To: svn-src-head@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id 23D96EFEFF8; Thu, 24 May 2018 05:33:15 +0000 (UTC) (envelope-from mmacy@freebsd.org) Received: from smtp.freebsd.org (smtp.freebsd.org [96.47.72.83]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client CN "smtp.freebsd.org", Issuer "Let's Encrypt Authority X3" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id E0285755D6; Thu, 24 May 2018 05:33:13 +0000 (UTC) (envelope-from mmacy@freebsd.org) Received: from mail-it0-f42.google.com (mail-it0-f42.google.com [209.85.214.42]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (Client CN "smtp.gmail.com", Issuer "Google Internet Authority G2" (verified OK)) (Authenticated sender: mmacy) by smtp.freebsd.org (Postfix) with ESMTPSA id F0624E1C7; Thu, 24 May 2018 05:33:11 +0000 (UTC) (envelope-from mmacy@freebsd.org) Received: by mail-it0-f42.google.com with SMTP id d10-v6so862272itj.1; Wed, 23 May 2018 22:33:11 -0700 (PDT) X-Gm-Message-State: ALKqPwdDpLqNY18i+xdGm7i6OygGz5dKG/jH8pvylFXVpAS7G/CSxGDp yWR8lbIIskEdAn97ywQz/mh+nrSWlpHLmNZkM0A= X-Google-Smtp-Source: AB8JxZryRZI7Ypmst9S3NJ5a/fiHwOjMj4sCarH5dzKQnZSTU7pROgq2dHV67z5eW4g9AUAl5P6j6Kg2a0BHmwkOdrg= X-Received: by 2002:a24:ddcc:: with SMTP id t195-v6mr7841947itf.77.1527139990828; Wed, 23 May 2018 22:33:10 -0700 (PDT) MIME-Version: 1.0 Received: by 2002:a02:85ca:0:0:0:0:0 with HTTP; Wed, 23 May 2018 22:33:10 -0700 (PDT) In-Reply-To: <20180524052520.GZ71675@FreeBSD.org> References: <201805190510.w4J5AqfS054367@repo.freebsd.org> <20180523222743.GU71675@FreeBSD.org> <20180523225729.GV71675@FreeBSD.org> <20180524044252.GW71675@FreeBSD.org> <20180524050717.GY71675@FreeBSD.org> <20180524052520.GZ71675@FreeBSD.org> From: Matthew Macy Date: Wed, 23 May 2018 22:33:10 -0700 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: svn commit: r333860 - head/sys/kern To: Gleb Smirnoff Cc: src-committers , svn-src-all@freebsd.org, svn-src-head@freebsd.org Content-Type: text/plain; charset="UTF-8" X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.26 Precedence: list List-Id: SVN commit messages for the src tree for head/-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 24 May 2018 05:33:15 -0000 On Wed, May 23, 2018 at 10:25 PM, Gleb Smirnoff wrote: > On Wed, May 23, 2018 at 10:13:25PM -0700, Matthew Macy wrote: > M> On Wed, May 23, 2018 at 10:07 PM, Gleb Smirnoff wrote: > M> > Can you please explain the bug supposed to be fixed by r333860 QUESTION MARK > M> > M> Did I say it fixed a bug? Or are you saying we should just turn off > M> compiler warnings because Gleb Smirnoff knows better? > > You just did say "Warnings find bugs PERIOD." This implicitly claimed > that shutting this one fixed a bug. > > Indeed, I am saying that in this particular case Gleb Smirnoff knows > better than gcc8 does. However, I'm not saying to turn off compiler warnings. > Never said that. I am saying (several times already) not to insert redundant > code to shut up a warning that is false. I'm not saying you're right or wrong. On initial inspection it appeared uninitialized. I may be wrong. I never asserted that you were wrong. What I'm trying to understand is what your desired policy with respect to warnings is. If the author knows that a warning is false he should a) Disable that warning for the entire build because it is wrong in one case? b) Edit the Makefile to disable the warning for that file? c) Leave the warning enabled and just permit a noisy build where the warning potentially drowns out real issues? Right now you're discussing a single line in a single file when in fact this is a kernel wide policy issue. All three of these choices seem less desirable than selectively placating the compiler as I have en masse for much of the tree. -M