linux-kernel - Re: [PATCH] EDAC/ie31200: work around false positive build warning

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 for Android: free password hash cracker in your pocket

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

Message-ID: <Z7fTLWH5G_UgVZIQ@agluck-desk3>
Date: Thu, 20 Feb 2025 17:13:17 -0800
From: "Luck, Tony" <tony.luck@...el.com>
To: Jason Baron <jbaron@...mai.com>
Cc: "Zhuo, Qiuxu" <qiuxu.zhuo@...el.com>, Arnd Bergmann <arnd@...nel.org>,
	Borislav Petkov <bp@...en8.de>, Arnd Bergmann <arnd@...db.de>,
	James Morse <james.morse@....com>,
	Mauro Carvalho Chehab <mchehab@...nel.org>,
	Robert Richter <rric@...nel.org>, James Ye <jye836@...il.com>,
	"linux-edac@...r.kernel.org" <linux-edac@...r.kernel.org>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] EDAC/ie31200: work around false positive build warning

> > It looks like gcc-14 isn't smart enough to recognize that this is a false positive build warning :-).
> > Actually, the dim_info[][] array is always initialized by populate_dimm_info().
> > 
> > > see why the compiler gets confused by the two loops.
> > > 
> > > Instead, rework the two nested loops to only read the addr_decode registers
> > > and then keep only one instance of the dimm info structure.
> > > 
> > > Signed-off-by: Arnd Bergmann <arnd@...db.de>
> 
> Seems fine with the comment adjustment.
> 
> Thanks.
> 
> Acked-by: Jason Baron <jbaron@...mai.com>

Applied (with comment adjustment) to RAS tree edac-drivers branch

Thanks

-Tony