From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on gnuweeb.org X-Spam-Level: X-Spam-Status: No, score=-2.2 required=5.0 tests=ALL_TRUSTED,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,NICE_REPLY_A autolearn=ham autolearn_force=no version=3.4.6 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=gnuweeb.org; s=default; t=1682216855; bh=TfWcaPmoDuhSCE6hW+wduHuwMC5aJ3DDU6N4QD3d+Pw=; h=Date:To:Cc:References:From:Subject:In-Reply-To; b=CSFV+zOZPUR4emS5UgsShLmELXOC47132AYTrq5Vpi8NL5qUJ+Ev1Q6o5iNwwnnBp MjK2M6mgd59iO5XcPIJo+P4Ay5baO2e0X3bgcW877mGyVyVw+kzam+cvtdx1vlRU4y NDrn9onMJTPrXXFYAX7I6r5Soqa47Y3xTwP1dsKntc383sLUkGUgqJYCGUF2rjw9li GCagKursDN62VGDh5f+Sm3LYWAtW18eXMfKBy4q5W4Dx1clfbC9us2lD2jVzp0a6wO BsEyTdHN/LUsRLTKTJ36UXIwL03CKPo57hh9NWpRP+Ei9p5hqe5zC5s/B4Owh3Gx1S W9km3qZcp6vMg== Received: from [192.168.238.80] (unknown [182.2.68.80]) by gnuweeb.org (Postfix) with ESMTPSA id 4280B2457F9; Sun, 23 Apr 2023 09:27:33 +0700 (WIB) Message-ID: <4b35f3a4-9972-b7f0-287f-165a817c0f73@gnuweeb.org> Date: Sun, 23 Apr 2023 09:27:30 +0700 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.10.0 To: Linus Torvalds , David Sterba Cc: Qu Wenruo , Josef Bacik , Memet , Linux Btrfs Mailing List , Linux Kernel Mailing List , GNU/Weeb Mailing List References: Content-Language: en-US From: Ammar Faizi Subject: Re: [GIT PULL] Btrfs updates for 6.3 In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit List-Id: On 2/21/23 4:02 AM, Linus Torvalds wrote: > On Mon, Feb 20, 2023 at 11:26 AM David Sterba wrote: >> Other: >> >> - locally enable -Wmaybe-uninitialized after fixing all warnings > > I've pulled this, but I strongly suspect this change will get reverted. > > I bet neither you nor linux-next is testing even _remotely_ a big > chunk of the different compiler versions that are out there, and the > reason flags like '-Wmaybe-uninitialized' get undone is because some > random compiler version on some random config and target archiecture > gives completely nonsensical warnings for odd reasons. > > But hey, maybe the btrfs code is special. Maybe it's too late for 6.3. So please fix this in 6.4 and backport it to 6.3 stable. If someone fixes it, kindly add: Reported-by: Memet Compiling with GCC 9.4: gcc (Ubuntu 9.4.0-1ubuntu1~20.04.1) 9.4.0 Copyright (C) 2019 Free Software Foundation, Inc. This is free software; see the source for copying conditions. There is NO warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. Linux version 6.3-rc7: CC [M] fs/btrfs/volumes.o fs/btrfs/volumes.c: In function ‘btrfs_init_new_device’: fs/btrfs/volumes.c:2524:2: error: ‘seed_devices’ may be used uninitialized in this function [-Werror=maybe-uninitialized] 2524 | list_add(&seed_devices->seed_list, &fs_devices->seed_list); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ fs/btrfs/volumes.c:2621:27: note: ‘seed_devices’ was declared here 2621 | struct btrfs_fs_devices *seed_devices; | ^~~~~~~~~~~~ cc1: all warnings being treated as errors make[7]: *** [scripts/Makefile.build:252: fs/btrfs/volumes.o] Error 1 make[6]: *** [scripts/Makefile.build:494: fs/btrfs] Error 2 make[5]: *** [scripts/Makefile.build:494: fs] Error 2 make[4]: *** [Makefile:2025: .] Error 2 make[3]: *** [Makefile:357: __build_one_by_one] Error 2 make[2]: *** [debian/rules:8: build-arch] Error 2 dpkg-buildpackage: error: debian/rules binary subprocess returned exit status 2 make[1]: *** [scripts/Makefile.package:120: bindeb-pkg] Error 2 make: *** [Makefile:1656: bindeb-pkg] Error 2 fs/btrfs/send.c: In function ‘get_cur_inode_state’: fs/btrfs/send.c:1909:13: error: ‘right_gen’ may be used uninitialized in this function [-Werror=maybe-uninitialized] 1909 | } else if (right_gen == gen) { | ^ -- Ammar Faizi