mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Nathan Chancellor <nathan@kernel.org>
To: Naresh Kamboju <naresh.kamboju@linaro.org>
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
	clang-built-linux <llvm@lists.linux.dev>,
	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, hargar@microsoft.com, broonie@kernel.org,
	Arnd Bergmann <arnd@arndb.de>,
	Dan Carpenter <dan.carpenter@linaro.org>,
	Anders Roxell <anders.roxell@linaro.org>,
	Ben Copeland <benjamin.copeland@linaro.org>
Subject: Re: [PATCH 6.15 000/187] 6.15.8-rc1 review
Date: Tue, 22 Jul 2025 10:50:28 -0700	[thread overview]
Message-ID: <20250722175028.GA1163440@ax162> (raw)
In-Reply-To: <CA+G9fYs_6KYHSF6eyeFmewDxYGqqD70kO1DuB0_1qwCPV2LjOg@mail.gmail.com>

On Tue, Jul 22, 2025 at 11:06:26PM +0530, Naresh Kamboju wrote:
> Regressions while building allyesconfig build for arm64 and x86 with the
> toolchain clang-nightly version 22.0.0.

This is a "regression" in the sense that the warnings are new but it is
from the toolchain side, not the kernel side.

> ## Build log
> mm/ksm.c:3674:11: error: variable 'output' is used uninitialized
> whenever 'if' condition is false [-Werror,-Wsometimes-uninitialized]
>  3674 |         else if (ksm_advisor == KSM_ADVISOR_SCAN_TIME)
>       |                  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

Fix pending in Andrew's tree:

https://git.kernel.org/akpm/mm/c/153ad566724fe6f57b14f66e9726d295d22e576d

> drivers/media/platform/samsung/s5p-mfc/s5p_mfc_cmd_v6.c:45:7: error:
> variable 'h2r_args' is uninitialized when passed as a const pointer
> argument here [-Werror,-Wuninitialized-const-pointer]
>    45 |                                         &h2r_args);
>       |                                          ^~~~~~~~
> 
> drivers/media/platform/samsung/s5p-mfc/s5p_mfc_cmd_v6.c:133:7: error:
> variable 'h2r_args' is uninitialized when passed as a const pointer
> argument here [-Werror,-Wuninitialized-const-pointer]
>   133 |                                         &h2r_args);
>       |                                          ^~~~~~~~
> 
> drivers/media/platform/samsung/s5p-mfc/s5p_mfc_cmd_v6.c:148:7: error:
> variable 'h2r_args' is uninitialized when passed as a const pointer
> argument here [-Werror,-Wuninitialized-const-pointer]
>   148 |                                         &h2r_args);
>       |                                          ^~~~~~~~

https://lore.kernel.org/all/20250715-media-s5p-mfc-fix-uninit-const-pointer-v1-1-4d52b58cafe9@kernel.org/

> drivers/net/wireless/mediatek/mt76/mt7996/mcu.c:3365:21: error:
> variable 'hdr' is uninitialized when passed as a const pointer
> argument here [-Werror,-Wuninitialized-const-pointer]
>  3365 |         skb_put_data(skb, &hdr, sizeof(hdr));
>       |                            ^~~
> 
> drivers/net/wireless/mediatek/mt76/mt7996/mcu.c:1875:21: error:
> variable 'hdr' is uninitialized when passed as a const pointer
> argument here [-Werror,-Wuninitialized-const-pointer]
>  1875 |         skb_put_data(skb, &hdr, sizeof(hdr));
>       |                            ^~~

https://lore.kernel.org/all/20250715-mt7996-fix-uninit-const-pointer-v1-1-b5d8d11d7b78@kernel.org/

> drivers/net/wireless/broadcom/brcm80211/brcmsmac/phy/phy_lcn.c:2728:13:
> error: variable 'diq_start' is uninitialized when passed as a const
> pointer argument here [-Werror,-Wuninitialized-const-pointer]
>  2728 |
> &diq_start, 1, 16, 69);
>       |                                                       ^~~~~~~~~

Pending in the wireless tree:

https://git.kernel.org/wireless/wireless-next/c/81284e86bf8849f8e98e8ead3ff5811926b2107f

> drivers/usb/atm/cxacru.c:1104:6: error: variable 'bp' is used
> uninitialized whenever 'if' condition is false
> [-Werror,-Wsometimes-uninitialized]
>  1104 |         if (instance->modem_type->boot_rom_patch) {
>       |             ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

I need to send a v2 for this based on Greg's feedback:

https://lore.kernel.org/all/20250716172903.GA4010969@ax162/

> drivers/gpu/drm/msm/disp/dpu1/dpu_plane.c:1066:6: error: variable
> 'crtc_state' is used uninitialized whenever 'if' condition is false
> [-Werror,-Wsometimes-uninitialized]
>  1066 |         if (plane_state->crtc)
>       |             ^~~~~~~~~~~~~~~~~

https://lore.kernel.org/all/20250715-drm-msm-fix-const-uninit-warning-v1-1-d6a366fd9a32@kernel.org/

> drivers/gpu/drm/amd/amdgpu/imu_v12_0.c:374:30: error: variable 'data'
> is uninitialized when used here [-Werror,-Wuninitialized]
>   374 |                         program_imu_rlc_ram(adev, data, (const
> u32)size);
>       |                                                   ^~~~

https://lore.kernel.org/all/20250715-drm-amdgpu-fix-const-uninit-warning-v1-1-9683661f3197@kernel.org/

Alex said he picked it up but I have not seen it appear in his tree yet:

https://lore.kernel.org/all/BL1PR12MB5144E40FC1D87BABD459A0D6F750A@BL1PR12MB5144.namprd12.prod.outlook.com/

Cheers,
Nathan

  reply	other threads:[~2025-07-22 17:50 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-07-22 13:42 Greg Kroah-Hartman
2025-07-22 16:13 ` Ronald Warsow
2025-07-22 17:36 ` Naresh Kamboju
2025-07-22 17:50   ` Nathan Chancellor [this message]
2025-07-22 19:16 ` Florian Fainelli
2025-07-22 20:57 ` Christian Heusel
2025-07-22 21:19 ` Shuah Khan
2025-07-22 21:42 ` Miguel Ojeda
2025-07-23  3:17 ` Takeshi Ogasawara
2025-07-23  7:18 ` Pascal Ernster
2025-07-23  8:44 ` Peter Schneider
2025-07-23 10:54 ` Mark Brown
2025-07-23 11:34 ` Jon Hunter
2025-07-23 16:27 ` Justin Forbes
2025-07-24  3:34 ` Ron Economos
2025-07-24  3:52 ` Hardik Garg
2025-07-24  8:01 ` Naresh Kamboju
2025-07-26 18:04 ` Pavel Machek

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20250722175028.GA1163440@ax162 \
    --to=nathan@kernel.org \
    --cc=akpm@linux-foundation.org \
    --cc=anders.roxell@linaro.org \
    --cc=arnd@arndb.de \
    --cc=benjamin.copeland@linaro.org \
    --cc=broonie@kernel.org \
    --cc=conor@kernel.org \
    --cc=dan.carpenter@linaro.org \
    --cc=f.fainelli@gmail.com \
    --cc=gregkh@linuxfoundation.org \
    --cc=hargar@microsoft.com \
    --cc=jonathanh@nvidia.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux@roeck-us.net \
    --cc=lkft-triage@lists.linaro.org \
    --cc=llvm@lists.linux.dev \
    --cc=naresh.kamboju@linaro.org \
    --cc=patches@kernelci.org \
    --cc=patches@lists.linux.dev \
    --cc=pavel@denx.de \
    --cc=rwarsow@gmx.de \
    --cc=shuah@kernel.org \
    --cc=srw@sladewatkins.net \
    --cc=stable@vger.kernel.org \
    --cc=sudipm.mukherjee@gmail.com \
    --cc=torvalds@linux-foundation.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

Powered by JetHome