From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id E4123CA0EC3 for ; Tue, 12 Sep 2023 10:39:08 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S233999AbjILKjL (ORCPT ); Tue, 12 Sep 2023 06:39:11 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:60454 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S234224AbjILKjG (ORCPT ); Tue, 12 Sep 2023 06:39:06 -0400 Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 6CC921706; Tue, 12 Sep 2023 03:32:14 -0700 (PDT) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 58D3AC433CA; Tue, 12 Sep 2023 10:32:13 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=linuxfoundation.org; s=korg; t=1694514733; bh=F+3htaiK86iu0hovTxNnP1IUyQ40FjMrFsKbNvgy7vE=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=CsLrUrInURChNjSy8XKJOvskoV0IXEGbUr+VcrEoje70b+iqLqquOwaWv4NyhGBJw feCI6nlIHu1MdTWtgq6hfgJMjoxGUsZbG+gTXY0xZ+bY4Fee7RviKTB1v49VwaeugE rTCqJfjeiFe3b07maPXGKH22lWzvBFxwgpOu70yw= Date: Tue, 12 Sep 2023 12:32:10 +0200 From: Greg Kroah-Hartman To: Naresh Kamboju Cc: jack@suse.cz, stable@vger.kernel.org, patches@lists.linux.dev, linux-kernel@vger.kernel.org, torvalds@linux-foundation.org, akpm@linux-foundation.org, linux@roeck-us.net, shuah@kernel.org, patches@kernelci.org, lkft-triage@lists.linaro.org, pavel@denx.de, jonathanh@nvidia.com, f.fainelli@gmail.com, sudipm.mukherjee@gmail.com, srw@sladewatkins.net, rwarsow@gmx.de, conor@kernel.org, Daniel =?iso-8859-1?Q?D=EDaz?= , Tom Rix , Nathan Chancellor Subject: Re: [PATCH 6.1 000/600] 6.1.53-rc1 review Message-ID: <2023091233-boots-line-a3d4@gregkh> References: <20230911134633.619970489@linuxfoundation.org> <1ffe4f64-f238-859a-ab14-7559d03c4671@linaro.org> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Sep 12, 2023 at 02:19:34PM +0530, Naresh Kamboju wrote: > On Tue, 12 Sept 2023 at 07:55, Daniel Díaz wrote: > > > > Hello! > > > > On Mon, 11 Sept 2023 at 14:58, Daniel Díaz wrote: > > > On 11/09/23 7:40 a. m., Greg Kroah-Hartman wrote: > > > > This is the start of the stable review cycle for the 6.1.53 release. > > > > There are 600 patches in this series, all will be posted as a response > > > > to this one. If anyone has any issues with these being applied, please > > > > let me know. > > > > > > > > Responses should be made by Wed, 13 Sep 2023 13:44:56 +0000. > > > > Anything received after that time might be too late. > > > > > > > > The whole patch series can be found in one patch at: > > > > https://www.kernel.org/pub/linux/kernel/v6.x/stable-review/patch-6.1.53-rc1.gz > > > > or in the git tree and branch at: > > > > git://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git linux-6.1.y > > > > and the diffstat can be found below. > > > > > > > > thanks, > > > > > > > > greg k-h > > > > > > We're seeing this new warning: > > > -----8<----- > > > /builds/linux/fs/udf/inode.c:892:6: warning: variable 'newblock' is used uninitialized whenever 'if' condition is true [-Wsometimes-uninitialized] > > > 892 | if (*err < 0) > > > | ^~~~~~~~ > > > /builds/linux/fs/udf/inode.c:914:9: note: uninitialized use occurs here > > > 914 | return newblock; > > > | ^~~~~~~~ > > > /builds/linux/fs/udf/inode.c:892:2: note: remove the 'if' if its condition is always false > > > 892 | if (*err < 0) > > > | ^~~~~~~~~~~~~ > > > 893 | goto out_free; > > > | ~~~~~~~~~~~~~ > > > /builds/linux/fs/udf/inode.c:699:34: note: initialize the variable 'newblock' to silence this warning > > > 699 | udf_pblk_t newblocknum, newblock; > > > | ^ > > > | = 0 > > > 1 warning generated. > > > ----->8----- > > > > > > That's with Clang 17 (and nightly) on: > > > * arm > > > * powerpc > > > * s390 > > > > For what it's worth, bisection points to 903b487b5ba6 ("udf: Handle > > error when adding extent to a file"). > > I see the following commit is fixing the reported problem. > > commit 6d5ab7c2f7cf90877dab8f2bb06eb5ca8edc73ef > Author: Tom Rix > Date: Fri Dec 30 12:53:41 2022 -0500 > > udf: initialize newblock to 0 > > The clang build reports this error > fs/udf/inode.c:805:6: error: variable 'newblock' is used > uninitialized whenever 'if' condition is true > [-Werror,-Wsometimes-uninitialized] > if (*err < 0) > ^~~~~~~~ > newblock is never set before error handling jump. > Initialize newblock to 0 and remove redundant settings. > > Fixes: d8b39db5fab8 ("udf: Handle error when adding extent to a file") > Reported-by: Nathan Chancellor > Signed-off-by: Tom Rix > Signed-off-by: Jan Kara > Message-Id: <20221230175341.1629734-1-trix@redhat.com> Wait, where is this commit? I don't see it in Linus's tree either, nor in linux-next. Where did you find it? confused, greg k-h