Re: [Qemu-devel] [PATCH v2 02/12] ARM: Prepare translation for AArch64 code

2013-05-02 Thread John Rigby
On Tue, Apr 30, 2013 at 6:36 AM, Peter Maydell wrote: > > On 30 April 2013 10:36, Alexander Graf wrote: > > > > On 30.04.2013, at 08:36, John Rigby wrote: > >> - remove many uses of is_a64 as that is a indicator the target arch is > >> aarch64 not what mode it is running in. > > > > It indicates

Re: [Qemu-devel] [PATCH v2 02/12] ARM: Prepare translation for AArch64 code

2013-05-02 Thread John Rigby
On Wed, May 1, 2013 at 4:33 AM, Peter Maydell wrote: > On 1 May 2013 11:19, Laurent Desnogues > wrote: > > On Wednesday, May 1, 2013, Richard Henderson wrote: > >> On 2013-04-30 07:36, John Rigby wrote: > >>> > >>> uint32_t regs[16]; > >>> + > >>> +/* Regs for A64 mode. */ > >>> +

Re: [Qemu-devel] [PATCH v2 02/12] ARM: Prepare translation for AArch64 code

2013-05-01 Thread Peter Maydell
On 1 May 2013 11:19, Laurent Desnogues wrote: > On Wednesday, May 1, 2013, Richard Henderson wrote: >> On 2013-04-30 07:36, John Rigby wrote: >>> >>> uint32_t regs[16]; >>> + >>> +/* Regs for A64 mode. */ >>> +uint64_t xregs[31]; >>> +uint64_t pc; >>> +uint64_t sp; >>> +

Re: [Qemu-devel] [PATCH v2 02/12] ARM: Prepare translation for AArch64 code

2013-05-01 Thread Laurent Desnogues
On Wednesday, May 1, 2013, Richard Henderson wrote: > On 2013-04-30 07:36, John Rigby wrote: >> >> uint32_t regs[16]; >> + >> +/* Regs for A64 mode. */ >> +uint64_t xregs[31]; >> +uint64_t pc; >> +uint64_t sp; >> +uint32_t pstate; >> +uint32_t aarch64_state; /* 1 if

Re: [Qemu-devel] [PATCH v2 02/12] ARM: Prepare translation for AArch64 code

2013-05-01 Thread Richard Henderson
On 2013-04-30 07:36, John Rigby wrote: uint32_t regs[16]; + +/* Regs for A64 mode. */ +uint64_t xregs[31]; +uint64_t pc; +uint64_t sp; +uint32_t pstate; +uint32_t aarch64_state; /* 1 if CPU is in aarch64 state */ + How do these registers overlap (or not) in real h

Re: [Qemu-devel] [PATCH v2 02/12] ARM: Prepare translation for AArch64 code

2013-04-30 Thread Peter Maydell
On 30 April 2013 10:36, Alexander Graf wrote: > > On 30.04.2013, at 08:36, John Rigby wrote: >> - remove many uses of is_a64 as that is a indicator the target arch is >> aarch64 not what mode it is running in. > > It indicates what mode the CPU is running on. The only reason it > was defined stati

Re: [Qemu-devel] [PATCH v2 02/12] ARM: Prepare translation for AArch64 code

2013-04-30 Thread Alexander Graf
On 30.04.2013, at 08:36, John Rigby wrote: > From: Alexander Graf > > This patch adds all the prerequisites for AArch64 support that didn't > fit into split up patches. It extends important bits in the core cpu > headers to also take AArch64 mode into account. > > Add new ARM_TBFLAG_AARCH64_ST

[Qemu-devel] [PATCH v2 02/12] ARM: Prepare translation for AArch64 code

2013-04-29 Thread John Rigby
From: Alexander Graf This patch adds all the prerequisites for AArch64 support that didn't fit into split up patches. It extends important bits in the core cpu headers to also take AArch64 mode into account. Add new ARM_TBFLAG_AARCH64_STATE flag that indicates an ARMv8 cpu running in aarch64 mod