Re: [PATCH 01/13] arm/arm64: Add new is_kernel_in_hyp_mode predicate

2015-07-16 Thread Will Deacon
On Wed, Jul 08, 2015 at 05:19:04PM +0100, Marc Zyngier wrote: > With ARMv8.1 VHE extension, it will be possible to run the kernel > at EL2 (aka HYP mode). In order for the kernel to easily find out > where it is running, add a new predicate that returns whether or > not the kernel is in HYP mode. >

Re: [PATCH 01/13] arm/arm64: Add new is_kernel_in_hyp_mode predicate

2015-07-09 Thread Mark Rutland
On Thu, Jul 09, 2015 at 11:05:34AM +0100, Marc Zyngier wrote: > On 09/07/15 10:42, Mark Rutland wrote: > > Hi, > > > >> +static inline bool is_kernel_in_hyp_mode(void) > >> +{ > >> + u64 el; > >> + > >> + asm("mrs %0, CurrentEL" : "=r" (el)); > >> + return el == CurrentEL_EL2; > >> +} > > > >

Re: [PATCH 01/13] arm/arm64: Add new is_kernel_in_hyp_mode predicate

2015-07-09 Thread Marc Zyngier
On 09/07/15 10:42, Mark Rutland wrote: > Hi, > >> +static inline bool is_kernel_in_hyp_mode(void) >> +{ >> +u64 el; >> + >> +asm("mrs %0, CurrentEL" : "=r" (el)); >> +return el == CurrentEL_EL2; >> +} > > If you can include cputype.h, I think this can be: > > static inline bool is_ke

Re: [PATCH 01/13] arm/arm64: Add new is_kernel_in_hyp_mode predicate

2015-07-09 Thread Mark Rutland
Hi, > +static inline bool is_kernel_in_hyp_mode(void) > +{ > + u64 el; > + > + asm("mrs %0, CurrentEL" : "=r" (el)); > + return el == CurrentEL_EL2; > +} If you can include cputype.h, I think this can be: static inline bool is_kernel_in_hyp_mode(void) { return read_cpuid(Curr

[PATCH 01/13] arm/arm64: Add new is_kernel_in_hyp_mode predicate

2015-07-08 Thread Marc Zyngier
With ARMv8.1 VHE extension, it will be possible to run the kernel at EL2 (aka HYP mode). In order for the kernel to easily find out where it is running, add a new predicate that returns whether or not the kernel is in HYP mode. For completeness, the 32bit code also get such a predicate (always ret