hi ----- Original Message ----- > On 11 July 2017 at 18:18, Marc-André Lureau <marcandre.lur...@redhat.com> > 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 <peter.mayd...@linaro.org> > > > > 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 <marcandre.lur...@redhat.com>