Re: [PATCH v5 12/12] xen/changelog: Add SVE and "dom0" options to the changelog for Arm

[Bertrand Marquis <Bertrand.Marquis@xxxxxxx>]

Hi Luca,

> On 19 Apr 2023, at 09:11, Luca Fancellu <Luca.Fancellu@xxxxxxx> wrote:
> 
> 
> 
>> On 18 Apr 2023, at 13:56, Bertrand Marquis <Bertrand.Marquis@xxxxxxx> wrote:
>> 
>> Hi Luca,
>> 
>>> On 12 Apr 2023, at 11:49, Luca Fancellu <Luca.Fancellu@xxxxxxx> wrote:
>>> 
>>> Arm now can use the "dom0=" Xen command line option and the support
>>> for guests running SVE instructions is added, put entries in the
>>> changelog.
>>> 
>>> Signed-off-by: Luca Fancellu <luca.fancellu@xxxxxxx>
>>> ---
>>> Changes from v4:
>>> - No changes
>>> Change from v3:
>>> - new patch
>>> ---
>>> CHANGELOG.md | 5 +++++
>>> 1 file changed, 5 insertions(+)
>>> 
>>> diff --git a/CHANGELOG.md b/CHANGELOG.md
>>> index c978cfd9b68f..a24951603359 100644
>>> --- a/CHANGELOG.md
>>> +++ b/CHANGELOG.md
>>> @@ -6,6 +6,10 @@ The format is based on [Keep a 
>>> Changelog](https://keepachangelog.com/en/1.0.0/)
>>> 
>>> ## [unstable 
>>> UNRELEASED](https://xenbits.xen.org/gitweb/?p=xen.git;a=shortlog;h=staging) 
>>> - TBD
>>> 
>>> +### Changed
>>> +- The "dom0" option is now supported on Arm and "sve=" sub-option can be 
>>> used
>>> +  to enable dom0 guest to use SVE/SVE2 instructions.
>>> +
>>> ### Added
>>> - On x86, support for features new in Intel Sapphire Rapids CPUs:
>>>  - PKS (Protection Key Supervisor) available to HVM/PVH guests.
>>> @@ -14,6 +18,7 @@ The format is based on [Keep a 
>>> Changelog](https://keepachangelog.com/en/1.0.0/)
>>>  - Bus-lock detection, used by Xen to mitigate (by rate-limiting) the system
>>>    wide impact of a guest misusing atomic instructions.
>>> - xl/libxl can customize SMBIOS strings for HVM guests.
>>> + - On Arm, Xen supports guests running SVE/SVE2 instructions.
>> 
>> Might be a good idea to mention that this is a tech preview ?
> 
> Sure I’ll mention that, is there something in particular that needs to be 
> done or is it enough to say that here?

It is clearer i think if you mention here that it is a tech preview.

For the rest it must be listed as UNSUPPORTED in Kconfig and Tech Preview state 
must be also mentioned in SUPPORT.md.

Cheers
Bertrand

> 
>> 
>> Cheers
>> Bertrand
>> 
>>> 
>>> ## 
>>> [4.17.0](https://xenbits.xen.org/gitweb/?p=xen.git;a=shortlog;h=RELEASE-4.17.0)
>>>  - 2022-12-12
>>> 
>>> --
>>> 2.34.1