diff mbox series

[v2] arm64: Correct wrong label in macro __init_el2_gicv3

Message ID 20220214175643.21931-1-joakim.tjernlund@infinera.com (mailing list archive)
State New, archived
Headers show
Series [v2] arm64: Correct wrong label in macro __init_el2_gicv3 | expand

Commit Message

Joakim Tjernlund Feb. 14, 2022, 5:56 p.m. UTC 
In commit:

  114945d84a30a5fe ("arm64: Fix labels in el2_setup macros")

We renamed a label from '1' to '.Lskip_gicv3_\@', but failed to update
a branch to it, which now targets a later label also called '1'.

The branch is taken rarely, when GICv3 is present but SRE is disabled
at EL3, causing a boot-time crash.
Update the caller to the new label name.

Fixes: 114945d84a30a5fe ("arm64: Fix labels in el2_setup macros")
Cc: stable@vger.kernel.org
Signed-off-by: Joakim Tjernlund <joakim.tjernlund@infinera.com>
---
 - v2: fix commit msg
 
 arch/arm64/include/asm/el2_setup.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Catalin Marinas Feb. 15, 2022, 2:46 p.m. UTC | #1 
On Mon, 14 Feb 2022 18:56:43 +0100, Joakim Tjernlund wrote:
> In commit:
> 
>   114945d84a30a5fe ("arm64: Fix labels in el2_setup macros")
> 
> We renamed a label from '1' to '.Lskip_gicv3_\@', but failed to update
> a branch to it, which now targets a later label also called '1'.
> 
> [...]

Applied to arm64 (for-next/fixes), thanks!

[1/1] arm64: Correct wrong label in macro __init_el2_gicv3
      https://git.kernel.org/arm64/c/4f6de676d94e
diff mbox series

Patch

diff --git a/arch/arm64/include/asm/el2_setup.h b/arch/arm64/include/asm/el2_setup.h
index 3198acb2aad8..7f3c87f7a0ce 100644
--- a/arch/arm64/include/asm/el2_setup.h
+++ b/arch/arm64/include/asm/el2_setup.h
@@ -106,7 +106,7 @@ 
 	msr_s	SYS_ICC_SRE_EL2, x0
 	isb					// Make sure SRE is now set
 	mrs_s	x0, SYS_ICC_SRE_EL2		// Read SRE back,
-	tbz	x0, #0, 1f			// and check that it sticks
+	tbz	x0, #0, .Lskip_gicv3_\@		// and check that it sticks
 	msr_s	SYS_ICH_HCR_EL2, xzr		// Reset ICC_HCR_EL2 to defaults
 .Lskip_gicv3_\@:
 .endm