Re: [PATCH v2] xen/arm32: head: Widen the use of the temporary mapping

[Julien Grall <julien@xxxxxxx>]

Hi,

On 24/07/2023 18:15, Luca Fancellu wrote:
> > On 24 Jul 2023, at 11:14, Julien Grall <julien@xxxxxxx> wrote:
> >
> > From: Julien Grall <jgrall@xxxxxxxxxx>
> >
> > At the moment, the temporary mapping is only used when the virtual
> > runtime region of Xen is clashing with the physical region.
> >
> > In follow-up patches, we will rework how secondary CPU bring-up works
> > and it will be convenient to use the fixmap area for accessing
> > the root page-table (it is per-cpu).
> >
> > Rework the code to use temporary mapping when the Xen physical address
> > is not overlapping with the temporary mapping.
> >
> > This also has the advantage to simplify the logic to identity map
> > Xen.
> >
> > Signed-off-by: Julien Grall <jgrall@xxxxxxxxxx>
> Hi Julien,
>
> Seems good to me, I’ve also tested on qemu and FVP, creating/destroying guests 
> and no issues so far.
>
> Reviewed-by: Luca Fancellu <luca.fancellu@xxxxxxx>
> Tested-by: Luca Fancellu <luca.fancellu@xxxxxxx>
Thanks. I have committed with Henry's tag. But I also realized that I 
forgot to strip the changelog :/.
Cheers,

--
Julien Grall