[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [Qemu-arm] [PATCH for-2.10 3/5] target/arm: Rename cp15
From: |
Peter Maydell |
Subject: |
Re: [Qemu-devel] [Qemu-arm] [PATCH for-2.10 3/5] target/arm: Rename cp15.c6_rgnr to pmsav7.rnr |
Date: |
Fri, 28 Jul 2017 09:42:12 +0100 |
On 27 July 2017 at 23:58, Philippe Mathieu-Daudé <address@hidden> wrote:
> On 07/27/2017 07:43 PM, Philippe Mathieu-Daudé wrote:
>>
>> On 07/27/2017 07:59 AM, Peter Maydell wrote:
>
> [...]
>>>
>>> - u32p += env->cp15.c6_rgnr;
>>> + u32p += env->pmsav7.rnr;
>>> tlb_flush(CPU(cpu)); /* Mappings may have changed - purge! */
>>> *u32p = value;
>>> }
>>> @@ -2447,7 +2447,7 @@ static const ARMCPRegInfo pmsav7_cp_reginfo[] = {
>>> .readfn = pmsav7_read, .writefn = pmsav7_write, .resetfn =
>>> pmsav7_reset },
>>> { .name = "RGNR", .cp = 15, .crn = 6, .opc1 = 0, .crm = 2, .opc2 =
>>> 0,
>>
>>
>> "RGNR" -> "RNR"
>
>
> Ah "RGNR" for -R and "RNR" for -M hmmm... still better keep the name
> matching the field, "rnr".
It's a bit awkward, yes -- we're going to get a mismatch one way
or the other.
In this patch I wanted only to change the field name, not
anything else (it's already a bit borderline for 2.10).
thanks
-- PMM
[Qemu-devel] [PATCH for-2.10 2/5] target/arm: Don't allow guest to make System space executable for M profile, Peter Maydell, 2017/07/27
[Qemu-devel] [PATCH for-2.10 1/5] target/arm: Don't do MPU lookups for addresses in M profile PPB region, Peter Maydell, 2017/07/27
Re: [Qemu-devel] [Qemu-arm] [PATCH for-2.10 0/5] M profile MPU bugfixes, Peter Maydell, 2017/07/31