linux-kernel - Re: [PATCH AUTOSEL 6.13 32/32] irqchip/qcom-pdc: Workaround hardware register bug on X1E80100

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: <Z7yGLDkI1T4laWBd@hovoldconsulting.com>
Date: Mon, 24 Feb 2025 15:46:04 +0100
From: Johan Hovold <johan@...nel.org>
To: Sasha Levin <sashal@...nel.org>
Cc: linux-kernel@...r.kernel.org, stable@...r.kernel.org,
	Stephan Gerhold <stephan.gerhold@...aro.org>,
	Thomas Gleixner <tglx@...utronix.de>,
	Johan Hovold <johan+linaro@...nel.org>,
	linux-arm-msm@...r.kernel.org
Subject: Re: [PATCH AUTOSEL 6.13 32/32] irqchip/qcom-pdc: Workaround hardware
 register bug on X1E80100

Hi Sasha,

On Mon, Feb 24, 2025 at 06:16:38AM -0500, Sasha Levin wrote:
> From: Stephan Gerhold <stephan.gerhold@...aro.org>
> 
> [ Upstream commit e9a48ea4d90be251e0d057d41665745caccb0351 ]
> 
> On X1E80100, there is a hardware bug in the register logic of the
> IRQ_ENABLE_BANK register: While read accesses work on the normal address,
> all write accesses must be made to a shifted address. Without a workaround
> for this, the wrong interrupt gets enabled in the PDC and it is impossible
> to wakeup from deep suspend (CX collapse). This has not caused problems so
> far, because the deep suspend state was not enabled. A workaround is
> required now since work is ongoing to fix this.

> Signed-off-by: Stephan Gerhold <stephan.gerhold@...aro.org>
> Signed-off-by: Thomas Gleixner <tglx@...utronix.de>
> Tested-by: Johan Hovold <johan+linaro@...nel.org>
> Link: https://lore.kernel.org/all/20250218-x1e80100-pdc-hw-wa-v2-1-29be4c98e355@linaro.org
> Signed-off-by: Sasha Levin <sashal@...nel.org>

This one was not marked for backporting on purpose and is not needed in
older kernels, please drop from all autosel queues.

Johan