Re: [Qemu-devel] [PATCH v1 04/10] target-arm: Add VTCR_EL2

2015-09-11 Thread Edgar E. Iglesias
On Fri, Sep 11, 2015 at 03:43:48PM +0100, Peter Maydell wrote: > On 11 September 2015 at 15:40, Edgar E. Iglesias > wrote: > > In the interest avoiding duplication, do you think the following makes > > sense for regs with the el3_aa32ns_aa64any access checks? > > > > 1. Use STATE_BOTH for "low-act

Re: [Qemu-devel] [PATCH v1 04/10] target-arm: Add VTCR_EL2

2015-09-11 Thread Peter Maydell
On 11 September 2015 at 15:40, Edgar E. Iglesias wrote: > In the interest avoiding duplication, do you think the following makes > sense for regs with the el3_aa32ns_aa64any access checks? > > 1. Use STATE_BOTH for "low-activity" registers (e.g the EL3 view when EL2 > does not exist). > 2. Use ST

Re: [Qemu-devel] [PATCH v1 04/10] target-arm: Add VTCR_EL2

2015-09-11 Thread Edgar E. Iglesias
On Tue, Sep 08, 2015 at 03:19:37PM +0100, Peter Maydell wrote: > On 3 September 2015 at 21:14, Edgar E. Iglesias > wrote: > > From: "Edgar E. Iglesias" > > > > Signed-off-by: Edgar E. Iglesias > > --- > > target-arm/cpu.h| 1 + > > target-arm/helper.c | 28 ++-- > >

Re: [Qemu-devel] [PATCH v1 04/10] target-arm: Add VTCR_EL2

2015-09-08 Thread Edgar E. Iglesias
On Tue, Sep 08, 2015 at 03:19:37PM +0100, Peter Maydell wrote: > On 3 September 2015 at 21:14, Edgar E. Iglesias > wrote: > > From: "Edgar E. Iglesias" > > > > Signed-off-by: Edgar E. Iglesias > > --- > > target-arm/cpu.h| 1 + > > target-arm/helper.c | 28 ++-- > >

Re: [Qemu-devel] [PATCH v1 04/10] target-arm: Add VTCR_EL2

2015-09-08 Thread Peter Maydell
On 3 September 2015 at 21:14, Edgar E. Iglesias wrote: > From: "Edgar E. Iglesias" > > Signed-off-by: Edgar E. Iglesias > --- > target-arm/cpu.h| 1 + > target-arm/helper.c | 28 ++-- > 2 files changed, 27 insertions(+), 2 deletions(-) > > diff --git a/target-arm/cp

[Qemu-devel] [PATCH v1 04/10] target-arm: Add VTCR_EL2

2015-09-03 Thread Edgar E. Iglesias
From: "Edgar E. Iglesias" Signed-off-by: Edgar E. Iglesias --- target-arm/cpu.h| 1 + target-arm/helper.c | 28 ++-- 2 files changed, 27 insertions(+), 2 deletions(-) diff --git a/target-arm/cpu.h b/target-arm/cpu.h index 31825d3..ba22e12 100644 --- a/target-arm/cp