linux-kernel - Re: mm: huge_memory.c:2736:31: error: variable 'page' is uninitialized when used here [-Werror,-Wuninitialized]

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <20240626170023.GA65296@fedora-macbook-air-m2>
Date: Wed, 26 Jun 2024 10:00:23 -0700
From: Nathan Chancellor <nathan@...nel.org>
To: Naresh Kamboju <naresh.kamboju@...aro.org>
Cc: open list <linux-kernel@...r.kernel.org>, linux-mm <linux-mm@...ck.org>,
	lkft-triage@...ts.linaro.org,
	clang-built-linux <llvm@...ts.linux.dev>,
	Andrew Morton <akpm@...ux-foundation.org>,
	Arnd Bergmann <arnd@...db.de>, Lance Yang <ioworker0@...il.com>
Subject: Re: mm: huge_memory.c:2736:31: error: variable 'page' is
 uninitialized when used here [-Werror,-Wuninitialized]

Hi Naresh,

On Wed, Jun 26, 2024 at 10:14:56PM +0530, Naresh Kamboju wrote:
> The x86_64 clang builds failed on Linux next due to these warnings / errors.
> 
> Reported-by: Linux Kernel Functional Testing <lkft@...aro.org>
> 
> Build error:
> ------
> mm/huge_memory.c:2736:31: error: variable 'page' is uninitialized when
> used here [-Werror,-Wuninitialized]
>  2736 |         folio_remove_rmap_pmd(folio, page, vma);
>       |                                      ^~~~
> /builds/linux/mm/huge_memory.c:2700:19: note: initialize the variable
> 'page' to silence this warning
>  2700 |         struct page *page;
>       |                          ^
>       |                           = NULL
> 1 error generated.
> 
> patch that is causing this build failures,
>   mm/vmscan: avoid split lazyfree THP during shrink_folio_list()

Thanks for the report. It looks like Andrew addressed this in
mm-unstable based on an earlier report (although that change might need
some more polish ;)

https://lore.kernel.org/202406260514.SLhNM9kQ-lkp@intel.com/

https://git.kernel.org/akpm/mm/c/199e6c3bd568657e85ead22d6f41c46e74ad9a75

Cheers,
Nathan