Re: [PATCH for-4.17] xen/arm: Support properly __ro_after_init on Arm

[Julien Grall <julien@xxxxxxx>]

On 18/08/2022 15:06, Bertrand Marquis wrote:
> Hi Julien,
Hi Bertrand,

> > On 16 Aug 2022, at 19:59, Julien Grall <julien@xxxxxxx> wrote:
> >
> > From: Julien Grall <jgrall@xxxxxxxxxx>
> >
> > __ro_after_init was introduced recently to prevent modifying
> > some variables after init.
> >
> > At the moment, on Arm, the variables will still be accessible
> > because the region permission is not updated.
> >
> > Address that, but moving the sections .data.ro_after_init
> Typo here s/but/by/ and remove ,
I updated it.

> > out of .data and then mark the region read-only once we finish
> > to boot.
> I would s/mark/map/
Ok.

> > Signed-off-by: Julien Grall <jgrall@xxxxxxxxxx>
> Reviewed-by: Bertrand Marquis <bertrand.marquis@xxxxxxx>
Thanks!

> (Commit message can be fixed on commit)

I have fixed the commit message, addressed the typo from Penny and 
committed the patch.
Cheers,




--
Julien Grall