Re: [edk2] [PATCH] ArmPkg/Library: Add ArmReadSctlr for aarch64

[Supreeth Venkatesh]

Thanks Ard.

-Original Message-
From: Ard Biesheuvel [mailto:ard.biesheu...@linaro.org]
Sent: Monday, August 01, 2016 7:11 AM
To: Supreeth Venkatesh
Cc: edk2-devel-01; Leif Lindholm; John Powell
Subject: Re: [PATCH] ArmPkg/Library: Add ArmReadSctlr for aarch64

On 30 July 2016 at 01:06, Supreeth Venkatesh  wrote:
> One of the UEFI Self Certification tests (UEFI-SCT) need to read the
> current exception level SCTLR Register.
>
> Contributed-under: TianoCore Contribution Agreement 1.0
> Signed-off-by: John Powell 
> Signed-off-by: Supreeth Venkatesh 
> ---

Thanks for the patch. I don't think mentioning the UEFI-SCT here makes any 
sense, since the function is defined in ArmLIb.h, and simply missing from the 
AARCH64 implementation.

I fixed this up when applying (07783fdd67e4)

Thanks,
Ard.




>  ArmPkg/Library/ArmLib/Common/AArch64/ArmLibSupport.S | 12
> +++-
>  1 file changed, 11 insertions(+), 1 deletion(-)
>
> diff --git a/ArmPkg/Library/ArmLib/Common/AArch64/ArmLibSupport.S
> b/ArmPkg/Library/ArmLib/Common/AArch64/ArmLibSupport.S
> index a6fd5e3..c9f3bd1 100644
> --- a/ArmPkg/Library/ArmLib/Common/AArch64/ArmLibSupport.S
> +++ b/ArmPkg/Library/ArmLib/Common/AArch64/ArmLibSupport.S
> @@ -1,7 +1,7 @@
>
> #-
> -
>  #
>  # Copyright (c) 2008 - 2009, Apple Inc. All rights reserved. -#
> Copyright (c) 2011 - 2014, ARM Limited. All rights reserved.
> +# Copyright (c) 2011 - 2016, ARM Limited. All rights reserved.
>  #
>  # This program and the accompanying materials  # are licensed and
> made available under the terms and conditions of the BSD License @@
> -39,6 +39,7 @@ GCC_ASM_EXPORT (ArmCallWFE)  GCC_ASM_EXPORT
> (ArmCallSEV)  GCC_ASM_EXPORT (ArmReadCpuActlr)  GCC_ASM_EXPORT
> (ArmWriteCpuActlr)
> +GCC_ASM_EXPORT (ArmReadSctlr)
>
>
> #-
> -
>
> @@ -205,4 +206,13 @@ ASM_PFX(ArmWriteCpuActlr):
>isb
>ret
>
> +ASM_PFX(ArmReadSctlr):
> +  EL1_OR_EL2_OR_EL3(x1)
> +1:mrs   x0, sctlr_el1
> +  ret
> +2:mrs   x0, sctlr_el2
> +  ret
> +3:mrs   x0, sctlr_el3
> +4:ret
> +
>  ASM_FUNCTION_REMOVE_IF_UNREFERENCED
> --
> 2.8.0
>

IMPORTANT NOTICE: The contents of this email and any attachments are 
confidential and may also be privileged. If you are not the intended recipient, 
please notify the sender immediately and do not disclose the contents to any 
other person, use it for any purpose, or store or copy the information in any 
medium. Thank you.
___
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel

Re: [edk2] [PATCH] ArmPkg/Library: Add ArmReadSctlr for aarch64

[Ard Biesheuvel]

On 30 July 2016 at 01:06, Supreeth Venkatesh  wrote:
> One of the UEFI Self Certification tests (UEFI-SCT) need to read the
> current exception level SCTLR Register.
>
> Contributed-under: TianoCore Contribution Agreement 1.0
> Signed-off-by: John Powell 
> Signed-off-by: Supreeth Venkatesh 
> ---

Thanks for the patch. I don't think mentioning the UEFI-SCT here makes
any sense, since the function is defined in ArmLIb.h, and simply
missing from the AARCH64 implementation.

I fixed this up when applying (07783fdd67e4)

Thanks,
Ard.




>  ArmPkg/Library/ArmLib/Common/AArch64/ArmLibSupport.S | 12 +++-
>  1 file changed, 11 insertions(+), 1 deletion(-)
>
> diff --git a/ArmPkg/Library/ArmLib/Common/AArch64/ArmLibSupport.S 
> b/ArmPkg/Library/ArmLib/Common/AArch64/ArmLibSupport.S
> index a6fd5e3..c9f3bd1 100644
> --- a/ArmPkg/Library/ArmLib/Common/AArch64/ArmLibSupport.S
> +++ b/ArmPkg/Library/ArmLib/Common/AArch64/ArmLibSupport.S
> @@ -1,7 +1,7 @@
>  
> #--
>  #
>  # Copyright (c) 2008 - 2009, Apple Inc. All rights reserved.
> -# Copyright (c) 2011 - 2014, ARM Limited. All rights reserved.
> +# Copyright (c) 2011 - 2016, ARM Limited. All rights reserved.
>  #
>  # This program and the accompanying materials
>  # are licensed and made available under the terms and conditions of the BSD 
> License
> @@ -39,6 +39,7 @@ GCC_ASM_EXPORT (ArmCallWFE)
>  GCC_ASM_EXPORT (ArmCallSEV)
>  GCC_ASM_EXPORT (ArmReadCpuActlr)
>  GCC_ASM_EXPORT (ArmWriteCpuActlr)
> +GCC_ASM_EXPORT (ArmReadSctlr)
>
>  
> #--
>
> @@ -205,4 +206,13 @@ ASM_PFX(ArmWriteCpuActlr):
>isb
>ret
>
> +ASM_PFX(ArmReadSctlr):
> +  EL1_OR_EL2_OR_EL3(x1)
> +1:mrs   x0, sctlr_el1
> +  ret
> +2:mrs   x0, sctlr_el2
> +  ret
> +3:mrs   x0, sctlr_el3
> +4:ret
> +
>  ASM_FUNCTION_REMOVE_IF_UNREFERENCED
> --
> 2.8.0
>
___
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel