Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1759105AbcJRKFE (ORCPT ); Tue, 18 Oct 2016 06:05:04 -0400 Received: from mout.kundenserver.de ([212.227.126.131]:64414 "EHLO mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752418AbcJRKE4 (ORCPT ); Tue, 18 Oct 2016 06:04:56 -0400 From: Arnd Bergmann To: Ilya Dryomov Cc: Sage Weil , Linus Torvalds , "linux-kernel@vger.kernel.org" , Alex Elder , Mike Christie , Ceph Development Subject: Re: [PATCH 11/28] block: rdb: false-postive gcc-4.9 -Wmaybe-uninitialized Date: Tue, 18 Oct 2016 12:04:37 +0200 Message-ID: <11351333.Y86c2RWQE1@wuerfel> User-Agent: KMail/5.1.3 (Linux/4.4.0-34-generic; KDE/5.18.0; x86_64; ; ) In-Reply-To: References: <20161017220342.1627073-1-arnd@arndb.de> <20161017221037.1781185-2-arnd@arndb.de> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" X-Provags-ID: V03:K0:Ij3TStQ70GL9xoY+5UYoPyGHQwt9fF7iaq2ilf3IqEvZASge0N5 YmXbeZntAnDW1d7jD+hKXSIkgIaWQPSC/rYsbZ5Rqjmg32g1TWlBIiDCKe2QPeemFlXexFj moTUf8As2et4ao1JSmq6arkwE0alTHJOm+izJ4/kilgHWxeXlf5sgp/nKnx5M9pvC8o5EG6 6SmD5GSBmM5flJzofSJqw== X-UI-Out-Filterresults: notjunk:1;V01:K0:lUMB6cztlHs=:Cg28rDTeQn2hSqU0qsy2YB kmj2XN79p0BD/Y7/To8wTVCbH9nA7GsKtbMMAAj0FZD6+pcds/QxurSi6DmmjrPSD3gEaMg/k fcm5+Nxau/wm7JnOzchER6FHV3U/S6TAS8pQaoK7x/6RNPM67+CSshekFDCivOAFc1yeyyenb TiU3vJxOOdztsWTHQo/D63jGF02IztJUBKBAq7Fb8e2bBNfHhbWdt9a5iUGCdLR7wp+GvsxQ0 qrIO1GNYPLpsBT2PgoY33VM5mgB+Z6mL7qLOd1+C4nS+gbDOziS2sdmxVJtCZlX+sgRhvIYD5 ypTblG2DaZNw6KgZbMS5hzpIH9W7FOHGav1yVI6lEaLf446XeRmE6bbbgyaL/fDL6S5zax4QK 2fOEma3fUc5u45WM+wdJrDylQpSX2FFSnwPYa8xpozsQEEf2umLohOKm5WM7LBmH5Ok3iiIzi wbAam32I3Ac35sbZ3gJ+E3Giv/P6rd3ZqSld0iaAjT/Wfr/a9sAp1LKCKTSxzotoQTjHMIoyr m+PR8VCHs3xFGoMXMGP6zl/6EyFIgdp7Aye2XpwxAQKOomuFDsnTdm76mLuqLek6xntQCUh/T zUe2BIEuKcR94m2RSOj9Zxgv4uv9qYHMLDsWbrFK5s01FMjknr1UBe4UDwAcytYU16cs0gnCW G8kfcDSHNURaUy3YdDcBQhz5vAyN/CRfjbnLFTyjJT+061zQx9hNdfC+bWz97w9EUZUSEvVtc 4f6tqNJNZPnjhyFl Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 841 Lines: 28 On Tuesday, October 18, 2016 11:57:33 AM CEST Ilya Dryomov wrote: > It already got silenced by initializing at declaration in one of the > downstream trees, so I'd rather we do > > @@ -3756,7 +3819,7 @@ static void rbd_watch_cb(void *arg, u64 > notify_id, u64 cookie, > struct rbd_device *rbd_dev = arg; > void *p = data; > void *const end = p + data_len; > - u8 struct_v; > + u8 struct_v = 0; > u32 len; > u32 notify_op; > int ret; > > to reduce the churn. Fair enough. I try to avoid adding extraneous initializations like this, but my suggested change is not all that different here, except if ceph_start_decoding() got changed in a way that could lead to another uninitialized use. > The "block" prefix is redundant and "rdb" should be "rbd" in the subject. Oops. Arnd