[PATCH 4/7] arm64: Disable TTBR0_EL1 during normal kernel execution

Mark Rutland mark.rutland at arm.com
Mon Aug 15 04:18:58 PDT 2016


On Fri, Aug 12, 2016 at 04:27:43PM +0100, Catalin Marinas wrote:
> diff --git a/arch/arm64/include/uapi/asm/ptrace.h b/arch/arm64/include/uapi/asm/ptrace.h
> index b5c3933ed441..9283e6b247f9 100644
> --- a/arch/arm64/include/uapi/asm/ptrace.h
> +++ b/arch/arm64/include/uapi/asm/ptrace.h
> @@ -52,6 +52,8 @@
>  #define PSR_Z_BIT	0x40000000
>  #define PSR_N_BIT	0x80000000
>  
> +#define _PSR_PAN_BIT	22

Given this is under uapi/, shouldn't we lose the leading underscore to align
with other PSR_* definitions?

Or should we not have this under uapi/?

[...]

> +	mrs	lr, ttbr0_el1
> +	tst	lr, #0xffff << 48		// Check for the reserved ASID

Did we not have a regular register spare here? Not a problem, but using the lr
here stands out as unusual.

Thanks,
Mark.



More information about the linux-arm-kernel mailing list