[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-arm] [Qemu-devel] [PATCH 1/7] target/arm: Use M_REG_NUM_BANKS
From: |
Alistair Francis |
Subject: |
Re: [Qemu-arm] [Qemu-devel] [PATCH 1/7] target/arm: Use M_REG_NUM_BANKS rather than hardcoding 2 |
Date: |
Mon, 11 Sep 2017 10:32:38 -0700 |
On Mon, Sep 11, 2017 at 7:59 AM, Philippe Mathieu-Daudé <address@hidden> wrote:
> On 09/11/2017 10:52 AM, Peter Maydell wrote:
>>
>> Use a symbolic constant M_REG_NUM_BANKS for the array size for
>> registers which are banked by M profile security state, rather
>> than hardcoding lots of 2s.
>>
>> Suggested-by: Philippe Mathieu-Daudé <address@hidden>
>> Signed-off-by: Peter Maydell <address@hidden>
>
>
> Reviewed-by: Philippe Mathieu-Daudé <address@hidden>
Reviewed-by: Alistair Francis <address@hidden>
Thanks,
Alistair
>
>> ---
>> Philippe suggested this in review on the last round of
>> patches but I forgot about it :-(
>
>
> Thank you, it eases review and looks cleaner :)
>
>
>> ---
>> target/arm/cpu.h | 35 +++++++++++++++++++----------------
>> 1 file changed, 19 insertions(+), 16 deletions(-)
>>
>> diff --git a/target/arm/cpu.h b/target/arm/cpu.h
>> index 98b9b26..5a1f957 100644
>> --- a/target/arm/cpu.h
>> +++ b/target/arm/cpu.h
>> @@ -81,8 +81,11 @@
>> * accessed via env->registerfield[env->v7m.secure] (whether the
>> security
>> * extension is implemented or not).
>> */
>> -#define M_REG_NS 0
>> -#define M_REG_S 1
>> +enum {
>> + M_REG_NS = 0,
>> + M_REG_S = 1,
>> + M_REG_NUM_BANKS = 2,
>> +};
>> /* ARM-specific interrupt pending bits. */
>> #define CPU_INTERRUPT_FIQ CPU_INTERRUPT_TGT_EXT_1
>> @@ -433,19 +436,19 @@ typedef struct CPUARMState {
>> uint32_t other_sp;
>> uint32_t other_ss_msp;
>> uint32_t other_ss_psp;
>> - uint32_t vecbase[2];
>> - uint32_t basepri[2];
>> - uint32_t control[2];
>> - uint32_t ccr[2]; /* Configuration and Control */
>> - uint32_t cfsr[2]; /* Configurable Fault Status */
>> + uint32_t vecbase[M_REG_NUM_BANKS];
>> + uint32_t basepri[M_REG_NUM_BANKS];
>> + uint32_t control[M_REG_NUM_BANKS];
>> + uint32_t ccr[M_REG_NUM_BANKS]; /* Configuration and Control */
>> + uint32_t cfsr[M_REG_NUM_BANKS]; /* Configurable Fault Status */
>> uint32_t hfsr; /* HardFault Status */
>> uint32_t dfsr; /* Debug Fault Status Register */
>> - uint32_t mmfar[2]; /* MemManage Fault Address */
>> + uint32_t mmfar[M_REG_NUM_BANKS]; /* MemManage Fault Address */
>> uint32_t bfar; /* BusFault Address */
>> - unsigned mpu_ctrl[2]; /* MPU_CTRL */
>> + unsigned mpu_ctrl[M_REG_NUM_BANKS]; /* MPU_CTRL */
>> int exception;
>> - uint32_t primask[2];
>> - uint32_t faultmask[2];
>> + uint32_t primask[M_REG_NUM_BANKS];
>> + uint32_t faultmask[M_REG_NUM_BANKS];
>> uint32_t secure; /* Is CPU in Secure state? (not guest visible)
>> */
>> } v7m;
>> @@ -546,7 +549,7 @@ typedef struct CPUARMState {
>> uint32_t *drbar;
>> uint32_t *drsr;
>> uint32_t *dracr;
>> - uint32_t rnr[2];
>> + uint32_t rnr[M_REG_NUM_BANKS];
>> } pmsav7;
>> /* PMSAv8 MPU */
>> @@ -556,10 +559,10 @@ typedef struct CPUARMState {
>> * pmsav7.rnr (region number register)
>> * pmsav7_dregion (number of configured regions)
>> */
>> - uint32_t *rbar[2];
>> - uint32_t *rlar[2];
>> - uint32_t mair0[2];
>> - uint32_t mair1[2];
>> + uint32_t *rbar[M_REG_NUM_BANKS];
>> + uint32_t *rlar[M_REG_NUM_BANKS];
>> + uint32_t mair0[M_REG_NUM_BANKS];
>> + uint32_t mair1[M_REG_NUM_BANKS];
>> } pmsav8;
>> void *nvic;
>>
>
- [Qemu-arm] [PATCH 3/7] target/arm: Get PRECISERR and IBUSERR the right way round, (continued)
- [Qemu-arm] [PATCH 3/7] target/arm: Get PRECISERR and IBUSERR the right way round, Peter Maydell, 2017/09/11
- [Qemu-arm] [PATCH 2/7] target/arm: Clear exclusive monitor on v7M reset, exception entry/exit, Peter Maydell, 2017/09/11
- [Qemu-arm] [PATCH 6/7] target/arm: Add and use defines for EXCRET constants, Peter Maydell, 2017/09/11
- [Qemu-arm] [PATCH 1/7] target/arm: Use M_REG_NUM_BANKS rather than hardcoding 2, Peter Maydell, 2017/09/11
- [Qemu-arm] [PATCH 5/7] target/arm: Remove unnecessary '| 0xf0000000' from do_v7m_exception_exit(), Peter Maydell, 2017/09/11