Re: [PATCH v3 2/6] xen/arm: mpu: Introduce choice between MMU and MPU

[Julien Grall <julien@xxxxxxx>]

Hi Ayan,

On 21/10/2024 12:25, Ayan Kumar Halder wrote:
> On 18/10/2024 15:08, Julien Grall wrote:
> > Hi,
> Hi,
> > On 10/10/2024 15:03, Ayan Kumar Halder wrote:
> > > There are features in the forthcoming patches which are dependent on
> > > MPU. For eg fixed start address.
> > > Also, some of the Xen features (eg STATIC_MEMORY) will be selected
> > > by the MPU configuration.
> > >
> > > Thus, this patch introduces a choice between MMU and MPU for the type
> > > of memory management system. By default, MMU is selected.
> > > MPU is now gated by UNSUPPORTED.
> > >
> > > Update SUPPORT.md to state that the support for MPU is experimental.
> > >
> > > Signed-off-by: Ayan Kumar Halder <ayan.kumar.halder@xxxxxxx>
> > With one remark below:
> >
> > Acked-by: Julien Grall <jgrall@xxxxxxxxxx>
> Thanks.
> > > ---
> > > Changes from :-
> > >
> > > v1 - 1. Reworded the help messages.
> > > 2. Updated Support.md.
> > >
> > > v2 - 1. Reworded the help message.
> > >
> > >   SUPPORT.md                     |  1 +
> > >   xen/arch/arm/Kconfig           | 17 ++++++++++++++++-
> > >   xen/arch/arm/platforms/Kconfig |  2 +-
> > >   3 files changed, 18 insertions(+), 2 deletions(-)
> > >
> > > diff --git a/SUPPORT.md b/SUPPORT.md
> > > index 23dd7e6424..3f6d788a43 100644
> > > --- a/SUPPORT.md
> > > +++ b/SUPPORT.md
> > > @@ -40,6 +40,7 @@ supported in this document.
> > >         Status, Xen in AArch64 mode: Supported
> > >       Status, Xen in AArch32 mode: Tech Preview
> > > +    Status, Xen with MPU: Experimental
> > Maybe mention Armv8-R?
> Yes.
>
> I propose this change be done on commit as well.
Unless Stefano wants to do, I would rather not as you are going to send 
a new version of the series. Same applies for patch #1.
Cheers,

--
Julien Grall