On 3 December 2013 12:20, Peter Crosthwaite <peter.crosthwa...@xilinx.com> wrote: > On Tue, Dec 3, 2013 at 6:48 PM, Sergey Fedorov <s.fedo...@samsung.com> wrote: >> From: Svetlana Fedoseeva <s.fedose...@samsung.com> >> >> Define CPU monitor mode. Adjust core registers banking. Adjust CPU VM >> state info. Provide CPU mode name for monitor mode. >> >> Signed-off-by: Svetlana Fedoseeva <s.fedose...@samsung.com> >> Signed-off-by: Sergey Fedorov <s.fedo...@samsung.com> >> --- >> target-arm/cpu.h | 7 ++++--- >> target-arm/helper.c | 3 +++ >> target-arm/machine.c | 12 ++++++------ >> target-arm/translate.c | 2 +- >> 4 files changed, 14 insertions(+), 10 deletions(-) >> >> diff --git a/target-arm/cpu.h b/target-arm/cpu.h >> index 0b93e39..94d8bd1 100644 >> --- a/target-arm/cpu.h >> +++ b/target-arm/cpu.h >> @@ -124,9 +124,9 @@ typedef struct CPUARMState { >> uint32_t spsr; >> >> /* Banked registers. */ >> - uint32_t banked_spsr[6]; >> - uint32_t banked_r13[6]; >> - uint32_t banked_r14[6]; >> + uint32_t banked_spsr[7]; >> + uint32_t banked_r13[7]; >> + uint32_t banked_r14[7]; >> > > Are there any more modes yet to be implemented? It might save on > future VMSD version bumps if we just pad this out to its ultimate > value now.
The remaining mode defined for AArch32 which we don't implement yet is Hyp mode, which has a banked R13 and SPSR, but not a banked LR. -- PMM