Linux-ARM-Kernel Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] arm64: Correct value returned by ESR_ELx_FSC_ADDRSZ_nL()
@ 2026-07-03 13:48 Steven Price
  2026-07-13 20:46 ` Marc Zyngier
  0 siblings, 1 reply; 2+ messages in thread
From: Steven Price @ 2026-07-03 13:48 UTC (permalink / raw)
  To: Catalin Marinas, Will Deacon
  Cc: Steven Price, linux-arm-kernel, linux-kernel, Marc Zyngier,
	Oliver Upton

Address size fault, level -1 is encoded as 0b101001 or 0x29 according to
the Arm ARM. Correct the value to match the spec. This also matches the
offset of "level -1 address size fault" in the fault_info array in
fault.c.

Fixes: fb8a3eba9c81 ("KVM: arm64: Only read HPFAR_EL2 when value is architecturally valid")
Signed-off-by: Steven Price <steven.price@arm.com>
---
The following hideous link takes you to the relevant chapter of the Arm
ARM:

https://developer.arm.com/documentation/ddi0487/mc/-Part-D-The-AArch64-System-Level-Architecture/-Chapter-D24-AArch64-System-Register-Descriptions/-D24-2-General-system-control-registers/-D24-2-44-ESR-EL1--Exception-Syndrome-Register--EL1-?lang=en

---
 arch/arm64/include/asm/esr.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm64/include/asm/esr.h b/arch/arm64/include/asm/esr.h
index 81c17320a588..f816f5d77f1a 100644
--- a/arch/arm64/include/asm/esr.h
+++ b/arch/arm64/include/asm/esr.h
@@ -131,7 +131,7 @@
  * Annoyingly, the negative levels for Address size faults aren't laid out
  * contiguously (or in the desired order)
  */
-#define ESR_ELx_FSC_ADDRSZ_nL(n)	((n) == -1 ? 0x25 : 0x2C)
+#define ESR_ELx_FSC_ADDRSZ_nL(n)	((n) == -1 ? 0x29 : 0x2C)
 #define ESR_ELx_FSC_ADDRSZ_L(n)		((n) < 0 ? ESR_ELx_FSC_ADDRSZ_nL(n) : \
 						   (ESR_ELx_FSC_ADDRSZ + (n)))
 
-- 
2.43.0



^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] arm64: Correct value returned by ESR_ELx_FSC_ADDRSZ_nL()
  2026-07-03 13:48 [PATCH] arm64: Correct value returned by ESR_ELx_FSC_ADDRSZ_nL() Steven Price
@ 2026-07-13 20:46 ` Marc Zyngier
  0 siblings, 0 replies; 2+ messages in thread
From: Marc Zyngier @ 2026-07-13 20:46 UTC (permalink / raw)
  To: Steven Price
  Cc: Catalin Marinas, Will Deacon, linux-arm-kernel, linux-kernel,
	Oliver Upton

On Fri, 03 Jul 2026 14:48:35 +0100,
Steven Price <steven.price@arm.com> wrote:
> 
> Address size fault, level -1 is encoded as 0b101001 or 0x29 according to
> the Arm ARM. Correct the value to match the spec. This also matches the
> offset of "level -1 address size fault" in the fault_info array in
> fault.c.
> 
> Fixes: fb8a3eba9c81 ("KVM: arm64: Only read HPFAR_EL2 when value is architecturally valid")
> Signed-off-by: Steven Price <steven.price@arm.com>

Reviewed-by: Marc Zyngier <maz@kernel.org>

	M.

-- 
Jazz isn't dead. It just smells funny.


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2026-07-13 20:45 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-07-03 13:48 [PATCH] arm64: Correct value returned by ESR_ELx_FSC_ADDRSZ_nL() Steven Price
2026-07-13 20:46 ` Marc Zyngier

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox