Re: [PATCH] SUPPORT.md: clarify support of booting 32-bit Xen on ARMv8

[Stefano Stabellini <sstabellini@xxxxxxxxxx>]

On Thu, 29 Feb 2024, Michal Orzel wrote:
> 
> On 29/02/2024 13:40, Julien Grall wrote:
> > 
> > 
> > On 29/02/2024 12:37, Michal Orzel wrote:
> >> Hi Julien,
> > 
> > Hi Michal,
> > 
> >>
> >> On 29/02/2024 13:35, Julien Grall wrote:
> >>> On 29/02/2024 12:13, Michal Orzel wrote:
> >>>> Since commit bd1001db0af1 ("xen/arm: arm32: Allow Xen to boot on
> >>>> unidentified CPUs"), it's been possible to boot 32-bit Xen on ARMv8A CPUs
> >>>> in AArch32 state (assuming HW supports EL2 execution in AArch32). Clarify
> >>>> the support statement and mark it as Tech Preview, as this use case is
> >>>> uncommon and hasn't really been tested/hardened.
> >>>>
> >>>> Signed-off-by: Michal Orzel <michal.orzel@xxxxxxx>
> >>>> ---
> >>>>    SUPPORT.md | 1 +
> >>>>    1 file changed, 1 insertion(+)
> >>>>
> >>>> diff --git a/SUPPORT.md b/SUPPORT.md
> >>>> index a90d1108c9d9..acc61230bb5e 100644
> >>>> --- a/SUPPORT.md
> >>>> +++ b/SUPPORT.md
> >>>> @@ -40,6 +40,7 @@ supported in this document.
> >>>>        Status: Supported
> >>> I would consider to use 'Status, Xen in aarch64 mode: Supported' and 
> >>> then...
> >>>
> >>>>        Status, Cortex A57 r0p0-r1p1: Supported, not security supported
> >>>>        Status, Cortex A77 r0p0-r1p0: Supported, not security supported
> >>>> +    Status, Xen in AArch32 mode: Tech Preview
> >>>
> >>> ... move this line closer. What do you think?
> >> That works for me too (+AArch64 instead of aarch64).
> > 
> > Ah yes. I keep forgetting capitalizing properly :).
> > 
> >> Shall I respin the patch?
> > 
> > Up to you. I am happy to fix it. But I will wait a day or two just to
> > give a chance for the others to comment.
> That works for me, let's wait for Bertrand and Stefano.

I am fine with it