From 3b3593b567761d2568ea70c117c7111fd85e79b9 Mon Sep 17 00:00:00 2001 From: Ard Biesheuvel Date: Fri, 1 Jul 2016 12:49:12 +0200 Subject: [PATCH] ArmPkg/CpuDxe: unmask SErrors in DEBUG builds SErrors (formerly called asynchronous aborts) are a distinct class of exceptions that are not closely tied to the currently executing instruction. Since execution may be able to proceed in such a condition, this class of exception is masked by default, and software needs to unmask it explicitly if it is prepared to handle such exceptions. On DEBUG builds, we are well equipped to report the CPU context to the user and it makes sense to report an SError as soon as it occurs rather than to wait for the OS to take it when it unmasks them, especially since the current arm64/Linux implementation simply panics in that case. So unmask them when ArmCpuDxe loads. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel Acked-by: Mark Rutland Reviewed-by: Leif Lindholm --- ArmPkg/Drivers/CpuDxe/Exception.c | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/ArmPkg/Drivers/CpuDxe/Exception.c b/ArmPkg/Drivers/CpuDxe/Exception.c index c3107cd4a6..d806a5fdf9 100644 --- a/ArmPkg/Drivers/CpuDxe/Exception.c +++ b/ArmPkg/Drivers/CpuDxe/Exception.c @@ -62,6 +62,15 @@ InitializeExceptions ( Status = Cpu->EnableInterrupt (Cpu); } + // + // On a DEBUG build, unmask SErrors so they are delivered right away rather + // than when the OS unmasks them. This gives us a better chance of figuring + // out the cause. + // + DEBUG_CODE ( + ArmEnableAsynchronousAbort (); + ); + return Status; } -- 2.39.5