[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-arm] [PATCH 3/3] target/arm: Make Cortex-M3 and M4 default to
From: |
Marc-André Lureau |
Subject: |
Re: [Qemu-arm] [PATCH 3/3] target/arm: Make Cortex-M3 and M4 default to 8 PMSA regions |
Date: |
Tue, 11 Jul 2017 13:30:06 -0400 (EDT) |
hi
----- Original Message -----
> On 11 July 2017 at 18:18, Marc-André Lureau <address@hidden>
> wrote:
> > ----- Original Message -----
> >> The Cortex-M3 and M4 CPUs always have 8 PMSA MPU regions (this isn't
> >> a configurable option for the hardware). Make the default value of
> >> the pmsav7-dregion property be set per-cpu, so we don't need to have
> >> every user of these CPUs set it manually. (The existing default of
> >> 16 is correct for the other PMSAv7 core, the Cortex-R5.)
> >>
> >> Signed-off-by: Peter Maydell <address@hidden>
> >
> > So until now that value was wrong for m3/m4 if I understand correctly.
>
> Correct. (It was too high, so didn't cause a problem for
> guests typically -- a guest assuming 8 memory regions would
> just not use the registers associated with the extra
> unexpected ones.)
Would be nice to add that to the commit message
in any case:
Reviewed-by: Marc-André Lureau <address@hidden>
[Qemu-arm] [PATCH 2/3] qdev: support properties which don't set a default value, Peter Maydell, 2017/07/11
- Re: [Qemu-arm] [PATCH 2/3] qdev: support properties which don't set a default value, Marc-André Lureau, 2017/07/12
- Re: [Qemu-arm] [Qemu-devel] [PATCH 2/3] qdev: support properties which don't set a default value, Markus Armbruster, 2017/07/12
- Re: [Qemu-arm] [Qemu-devel] [PATCH 2/3] qdev: support properties which don't set a default value, Peter Maydell, 2017/07/12
- Re: [Qemu-arm] [Qemu-devel] [PATCH 2/3] qdev: support properties which don't set a default value, Peter Maydell, 2017/07/13
- Re: [Qemu-arm] [Qemu-devel] [PATCH 2/3] qdev: support properties which don't set a default value, Markus Armbruster, 2017/07/13
- Re: [Qemu-arm] [Qemu-devel] [PATCH 2/3] qdev: support properties which don't set a default value, Peter Maydell, 2017/07/13
- Re: [Qemu-arm] [Qemu-devel] [PATCH 2/3] qdev: support properties which don't set a default value, Markus Armbruster, 2017/07/13