[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 04/23] target-arm: preserve RAO/WI bits of AR
From: |
Aggeler Fabian |
Subject: |
Re: [Qemu-devel] [PATCH v2 04/23] target-arm: preserve RAO/WI bits of ARMv7 SCTLR |
Date: |
Thu, 22 May 2014 08:58:32 +0000 |
On 21 May 2014, at 18:12, Peter Maydell <address@hidden> wrote:
> On 14 May 2014 06:43, Sergey Fedorov <address@hidden> wrote:
>>
>> On 13.05.2014 20:15, Fabian Aggeler wrote:
>>> From: Svetlana Fedoseeva <address@hidden>
>>>
>>> Signed-off-by: Svetlana Fedoseeva <address@hidden>
>>> Signed-off-by: Sergey Fedorov <address@hidden>
>>> Signed-off-by: Fabian Aggeler <address@hidden>
>>> ---
>>> target-arm/helper.c | 5 +++++
>>> 1 file changed, 5 insertions(+)
>>>
>>> diff --git a/target-arm/helper.c b/target-arm/helper.c
>>> index 9c3269f..2b57ad9 100644
>>> --- a/target-arm/helper.c
>>> +++ b/target-arm/helper.c
>>> @@ -2083,6 +2083,11 @@ static void sctlr_write(CPUARMState *env, const
>>> ARMCPRegInfo *ri,
>>> {
>>> ARMCPU *cpu = arm_env_get_cpu(env);
>>>
>>> + if (arm_feature(env, ARM_FEATURE_V7)) {
>>> + value |= SCTLR_XP | SCTLR_U | SCTLR_nTWE | SCTLR_nTWI | SCTLR_L
>>> + | SCTLR_CP15BEN | SCTLR_P; /* These bits are RAO/WI */
>>
>> Actually, some of these bits are RAO/WI since v6. Also, there are some
>> RAZ/WI bits varying over architecture variants. There is some overview
>> at ARM ARM v7-AP section L.7.4. Maybe it is worth to fix more precisely
>> over supported architecture variants? By the way, this patch could be
>> separated from security extensions support patch set.
>
> Agreed. Our compliance for bits that should-be-0/1 is not great,
> but if we don't actually need to do those fixes for TZ support
> then they're probably better separated out (ie drop them from
> this patchset for the moment and submit them separately or
> later...)
>
> Also for v8 many of these RAZ/RAO bits become RES0/RES1 and
> the rules are different...
>
> thanks
> — PMM
Okay, I will separate them and submit them separately.
Thanks,
Fabian
- Re: [Qemu-devel] [PATCH v2 10/23] target-arm: implement CPACR register logic, (continued)
- [Qemu-devel] [PATCH v2 03/23] target-arm: adjust TTBCR for Security Extension feature, Fabian Aggeler, 2014/05/13
- [Qemu-devel] [PATCH v2 12/23] target-arm: add SDER definition, Fabian Aggeler, 2014/05/13
- [Qemu-devel] [PATCH v2 07/23] target-arm: reject switching to monitor mode from non-secure state, Fabian Aggeler, 2014/05/13
- [Qemu-devel] [PATCH v2 08/23] target-arm: adjust arm_current_pl() for Security Extensions, Fabian Aggeler, 2014/05/13
- [Qemu-devel] [PATCH v2 05/23] target-arm: add CPU Monitor mode, Fabian Aggeler, 2014/05/13
- [Qemu-devel] [PATCH v2 04/23] target-arm: preserve RAO/WI bits of ARMv7 SCTLR, Fabian Aggeler, 2014/05/13
- [Qemu-devel] [PATCH v2 22/23] target-arm: implement IRQ/FIQ routing to Monitor mode, Fabian Aggeler, 2014/05/13
- [Qemu-devel] [PATCH v2 11/23] target-arm: add NSACR support, Fabian Aggeler, 2014/05/13
- [Qemu-devel] [PATCH v2 14/23] target-arm: add banked coprocessor register type and macros, Fabian Aggeler, 2014/05/13
- Re: [Qemu-devel] [PATCH v2 14/23] target-arm: add banked coprocessor register type and macros, Edgar E. Iglesias, 2014/05/22