Re: [PATCH 01/14] target/arm: add arm_is_el2_enabled() helper

2020-11-03 Thread Richard Henderson
On 11/2/20 2:57 AM, remi.denis.courm...@huawei.com wrote: > From: Rémi Denis-Courmont > > This checks if EL2 is enabled (meaning EL2 registers take effects) in > the current security context. > > Signed-off-by: Rémi Denis-Courmont > --- > target/arm/cpu.h | 14 ++ > 1 file changed,

RE: [PATCH 01/14] target/arm: add arm_is_el2_enabled() helper

2020-11-02 Thread Remi Denis Courmont
Hi, > I replied to this before the cover letter explaining what the series > is doing arrived in my inbox, so the second part of my suggestion > would be a bit pointless. The first part (mentioning the name of > the pseudocode function) still seems worth doing. Yeah well, some spam filter seems t

Re: [PATCH 01/14] target/arm: add arm_is_el2_enabled() helper

2020-11-02 Thread Peter Maydell
On Mon, 2 Nov 2020 at 11:06, Peter Maydell wrote: > > On Mon, 2 Nov 2020 at 10:58, wrote: > > > > From: Rémi Denis-Courmont > > > > This checks if EL2 is enabled (meaning EL2 registers take effects) in > > the current security context. > > > > Signed-off-by: Rémi Denis-Courmont > > --- > > tar

Re: [PATCH 01/14] target/arm: add arm_is_el2_enabled() helper

2020-11-02 Thread Peter Maydell
On Mon, 2 Nov 2020 at 10:58, wrote: > > From: Rémi Denis-Courmont > > This checks if EL2 is enabled (meaning EL2 registers take effects) in > the current security context. > > Signed-off-by: Rémi Denis-Courmont > --- > target/arm/cpu.h | 14 ++ > 1 file changed, 14 insertions(+) > >