diff mbox series

[v3,1/3] CHANGELOG: Add missing entries for work during the 4.17 release

Message ID 20221122112623.1441905-2-Henry.Wang@arm.com (mailing list archive)
State Superseded
Headers show
Series Populate entries for 4.17 | expand

Commit Message

Henry Wang Nov. 22, 2022, 11:26 a.m. UTC
Signed-off-by: Henry Wang <Henry.Wang@arm.com>
---
v2 -> v3:
- Move the uncertain release date to patch #3.
v1 -> v2:
- Correct wording for two x86 entries (clock frequency calibration
and SPR & ADL support) following Jan's suggestion.
- Add missing Arm entries from Oleksandr.
- Add missing i.MX entry following Stefano's suggestion.
---
 CHANGELOG.md | 32 ++++++++++++++++++++++++++++++--
 1 file changed, 30 insertions(+), 2 deletions(-)

Comments

Julien Grall Nov. 22, 2022, 12:14 p.m. UTC | #1
(Reducing the CC-list)

Hi,

On 22/11/2022 11:26, Henry Wang wrote:
> Signed-off-by: Henry Wang <Henry.Wang@arm.com>
> ---
> v2 -> v3:
> - Move the uncertain release date to patch #3.
> v1 -> v2:
> - Correct wording for two x86 entries (clock frequency calibration
> and SPR & ADL support) following Jan's suggestion.
> - Add missing Arm entries from Oleksandr.
> - Add missing i.MX entry following Stefano's suggestion.
> ---
>   CHANGELOG.md | 32 ++++++++++++++++++++++++++++++--
>   1 file changed, 30 insertions(+), 2 deletions(-)
> 
> diff --git a/CHANGELOG.md b/CHANGELOG.md
> index adbbb216fa..c593081aaf 100644
> --- a/CHANGELOG.md
> +++ b/CHANGELOG.md
> @@ -4,16 +4,44 @@ Notable changes to Xen will be documented in this file.
>   
>   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
> +## [4.17.0](https://xenbits.xen.org/gitweb/?p=xen.git;a=shortlog;h=staging)
>   
>   ### Changed
>    - On x86 "vga=current" can now be used together with GrUB2's gfxpayload setting. Note that
>      this requires use of "multiboot2" (and "module2") as the GrUB commands loading Xen.
> + - The "gnttab" option now has a new command line sub-option for disabling the
> +   GNTTABOP_transfer functionality.
> + - The x86 MCE command line option info is now updated.
>   
>   ### Added / support upgraded
> + - Out-of-tree builds for the hypervisor now supported.
> + - The project has officially adopted 4 directives and 24 rules of MISRA-C,
> +   added MISRA-C checker build integration, and defined how to document
> +   deviations.
>    - IOMMU superpage support on x86, affecting PV guests as well as HVM/PVH ones
>      when they don't share page tables with the CPU (HAP / EPT / NPT).
> - - Support VIRT_SSBD feature for HVM guests on AMD.
> + - Support VIRT_SSBD feature for HVM guests on AMD and MSR_SPEC_CTRL feature for
> +   SVM guests.

Did you really intend to write "SVM" rather than HVM? If yes, then to me 
"HVM guests on AMD" means the same as "SVM guests".

So I would suggest the following wording:

Support for VIRT_SSBD and MSR_SPEC_CTRL for HVM guests on AMD.


> + - Improved TSC, CPU, and APIC clock frequency calibration on x86.
> + - Add mwait-idle support for SPR and ADL on x86.
> + - Extend security support for hosts to 12 TiB of memory on x86.
> + - Add command line option to set cpuid parameters for dom0 at boot time on x86.
> + - Improved static configuration options on Arm.
> + - cpupools can be specified at boot using device tree on Arm.
> + - It is possible to use PV drivers with dom0less guests, allowing statically
> +   booted dom0less guests with PV devices.
> + - On Arm, p2m structures are now allocated out of a pool of memory set aside at
> +   domain creation.
> + - Improved mitigations against Spectre-BHB on Arm.
> + - Support VirtIO-MMIO devices in toolstack on Arm.

I would clarify that this is *only* creating the device-tree binding. 
IOW, there are no support for ACPI nor device-emulator (still in 
progress from Xen Project PoV).

> + - Allow setting the number of CPUs to activate at runtime from command line
> +   option on Arm.
> + - Grant-table support on Arm was improved and hardened by implementing
> +   "simplified M2P-like approach for the xenheap pages"
> + - Add Renesas R-Car Gen4 IPMMU-VMSA support on Arm.
> + - Add i.MX lpuart and i.MX8QM initial support on Arm.

I was under the impression that the code that was merged is enough to 
support the platform. Do you have any pointer where it says it is not 
sufficient?

> + - Improved toolstack build system.
> + - Add Xue - console over USB 3 Debug Capability.
>   
>   ### Removed / support downgraded
>    - dropped support for the (x86-only) "vesa-mtrr" and "vesa-remap" command line options

Cheers,
Henry Wang Nov. 22, 2022, 12:46 p.m. UTC | #2
Hi Julien,

Thanks for your review as always!

> -----Original Message-----
> From: Julien Grall <julien@xen.org>
> Subject: Re: [PATCH v3 1/3] CHANGELOG: Add missing entries for work during
> the 4.17 release
> 
> (Reducing the CC-list)

Thanks, I will use this CC-list in v4.

> 
> Hi,
> 
> On 22/11/2022 11:26, Henry Wang wrote:
> > Signed-off-by: Henry Wang <Henry.Wang@arm.com>
> > - - Support VIRT_SSBD feature for HVM guests on AMD.
> > + - Support VIRT_SSBD feature for HVM guests on AMD and
> MSR_SPEC_CTRL feature for
> > +   SVM guests.
> 
> Did you really intend to write "SVM" rather than HVM? If yes, then to me
> "HVM guests on AMD" means the same as "SVM guests".

I am sorry about the confusion. I think the latter part of this sentence is
just a direct copy of title in [1]. But by reading the content of that
cover letter I think...

> 
> So I would suggest the following wording:
> 
> Support for VIRT_SSBD and MSR_SPEC_CTRL for HVM guests on AMD.

...this is correct. So I will use your suggested sentence in v4. Thanks.

> 
> > + - Support VirtIO-MMIO devices in toolstack on Arm.
> 
> I would clarify that this is *only* creating the device-tree binding.
> IOW, there are no support for ACPI nor device-emulator (still in
> progress from Xen Project PoV).

You are correct. I will change to:
"Support VirtIO-MMIO devices device-tree binding creation in toolstack
on Arm"

> 
> > + - Allow setting the number of CPUs to activate at runtime from command
> line
> > +   option on Arm.
> > + - Grant-table support on Arm was improved and hardened by
> implementing
> > +   "simplified M2P-like approach for the xenheap pages"
> > + - Add Renesas R-Car Gen4 IPMMU-VMSA support on Arm.
> > + - Add i.MX lpuart and i.MX8QM initial support on Arm.
> 
> I was under the impression that the code that was merged is enough to
> support the platform. Do you have any pointer where it says it is not
> sufficient?

Ah this is also directly from the title of [2], where it is only a
"xen/arm: add i.MX lpuart and i.MX8QM initial support"

I would be more than happy to remove the "initial" if you think it is
appropriate.

> 
> > + - Improved toolstack build system.
> > + - Add Xue - console over USB 3 Debug Capability.
> >
> >   ### Removed / support downgraded
> >    - dropped support for the (x86-only) "vesa-mtrr" and "vesa-remap"
> command line options

[1] https://lore.kernel.org/xen-devel/20220128132927.14997-1-andrew.cooper3@citrix.com/
[2] https://lore.kernel.org/xen-devel/20220419043927.23542-1-peng.fan@oss.nxp.com/

Kind regards,
Henry

> 
> Cheers,
> 
> --
> Julien Grall
Julien Grall Nov. 23, 2022, 8:45 a.m. UTC | #3
On 22/11/2022 12:46, Henry Wang wrote:
> Hi Julien,

Hi Henry,

> 
> Thanks for your review as always!
> 
>> -----Original Message-----
>> From: Julien Grall <julien@xen.org>
>> Subject: Re: [PATCH v3 1/3] CHANGELOG: Add missing entries for work during
>> the 4.17 release
>>
>> (Reducing the CC-list)
> 
> Thanks, I will use this CC-list in v4.
> 
>>
>> Hi,
>>
>> On 22/11/2022 11:26, Henry Wang wrote:
>>> Signed-off-by: Henry Wang <Henry.Wang@arm.com>
>>> - - Support VIRT_SSBD feature for HVM guests on AMD.
>>> + - Support VIRT_SSBD feature for HVM guests on AMD and
>> MSR_SPEC_CTRL feature for
>>> +   SVM guests.
>>
>> Did you really intend to write "SVM" rather than HVM? If yes, then to me
>> "HVM guests on AMD" means the same as "SVM guests".
> 
> I am sorry about the confusion. I think the latter part of this sentence is
> just a direct copy of title in [1]. But by reading the content of that
> cover letter I think...
> 
>>
>> So I would suggest the following wording:
>>
>> Support for VIRT_SSBD and MSR_SPEC_CTRL for HVM guests on AMD.
> 
> ...this is correct. So I will use your suggested sentence in v4. Thanks.
> 
>>
>>> + - Support VirtIO-MMIO devices in toolstack on Arm.
>>
>> I would clarify that this is *only* creating the device-tree binding.
>> IOW, there are no support for ACPI nor device-emulator (still in
>> progress from Xen Project PoV).
> 
> You are correct. I will change to:
> "Support VirtIO-MMIO devices device-tree binding creation in toolstack
> on Arm"
> 
>>
>>> + - Allow setting the number of CPUs to activate at runtime from command
>> line
>>> +   option on Arm.
>>> + - Grant-table support on Arm was improved and hardened by
>> implementing
>>> +   "simplified M2P-like approach for the xenheap pages"
>>> + - Add Renesas R-Car Gen4 IPMMU-VMSA support on Arm.
>>> + - Add i.MX lpuart and i.MX8QM initial support on Arm.
>>
>> I was under the impression that the code that was merged is enough to
>> support the platform. Do you have any pointer where it says it is not
>> sufficient?
> 
> Ah this is also directly from the title of [2], where it is only a
> "xen/arm: add i.MX lpuart and i.MX8QM initial support"
> 
> I would be more than happy to remove the "initial" if you think it is
> appropriate.

I saw you already remove "initial" in v4 but I will still reply. On an 
earlier revision of the series I asked Peng if this was enough to boot 
Xen upstream on the platform and the answer was yes [3].

So I would assume that the basics work. Not sure about platform device 
passthrough.

> 
>>
>>> + - Improved toolstack build system.
>>> + - Add Xue - console over USB 3 Debug Capability.
>>>
>>>    ### Removed / support downgraded
>>>     - dropped support for the (x86-only) "vesa-mtrr" and "vesa-remap"
>> command line options
> 
> [1] https://lore.kernel.org/xen-devel/20220128132927.14997-1-andrew.cooper3@citrix.com/
> [2] https://lore.kernel.org/xen-devel/20220419043927.23542-1-peng.fan@oss.nxp.com/
[3] 
https://lore.kernel.org/xen-devel/DU0PR04MB9417B86346491B63442D234D88139@DU0PR04MB9417.eurprd04.prod.outlook.com/


Cheers,
Henry Wang Nov. 23, 2022, 8:50 a.m. UTC | #4
Hi Julien,

> -----Original Message-----
> From: Julien Grall <julien@xen.org>
> Subject: Re: [PATCH v3 1/3] CHANGELOG: Add missing entries for work during
> the 4.17 release
> >> I was under the impression that the code that was merged is enough to
> >> support the platform. Do you have any pointer where it says it is not
> >> sufficient?
> >
> > Ah this is also directly from the title of [2], where it is only a
> > "xen/arm: add i.MX lpuart and i.MX8QM initial support"
> >
> > I would be more than happy to remove the "initial" if you think it is
> > appropriate.
> 
> I saw you already remove "initial" in v4 but I will still reply. On an
> earlier revision of the series I asked Peng if this was enough to boot
> Xen upstream on the platform and the answer was yes [3].

Great, then removing the "initial" definitely makes good sense. Thanks
for confirming this.

Kind regards,
Henry
diff mbox series

Patch

diff --git a/CHANGELOG.md b/CHANGELOG.md
index adbbb216fa..c593081aaf 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -4,16 +4,44 @@  Notable changes to Xen will be documented in this file.
 
 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
+## [4.17.0](https://xenbits.xen.org/gitweb/?p=xen.git;a=shortlog;h=staging)
 
 ### Changed
  - On x86 "vga=current" can now be used together with GrUB2's gfxpayload setting. Note that
    this requires use of "multiboot2" (and "module2") as the GrUB commands loading Xen.
+ - The "gnttab" option now has a new command line sub-option for disabling the
+   GNTTABOP_transfer functionality.
+ - The x86 MCE command line option info is now updated.
 
 ### Added / support upgraded
+ - Out-of-tree builds for the hypervisor now supported.
+ - The project has officially adopted 4 directives and 24 rules of MISRA-C,
+   added MISRA-C checker build integration, and defined how to document
+   deviations.
  - IOMMU superpage support on x86, affecting PV guests as well as HVM/PVH ones
    when they don't share page tables with the CPU (HAP / EPT / NPT).
- - Support VIRT_SSBD feature for HVM guests on AMD.
+ - Support VIRT_SSBD feature for HVM guests on AMD and MSR_SPEC_CTRL feature for
+   SVM guests.
+ - Improved TSC, CPU, and APIC clock frequency calibration on x86.
+ - Add mwait-idle support for SPR and ADL on x86.
+ - Extend security support for hosts to 12 TiB of memory on x86.
+ - Add command line option to set cpuid parameters for dom0 at boot time on x86.
+ - Improved static configuration options on Arm.
+ - cpupools can be specified at boot using device tree on Arm.
+ - It is possible to use PV drivers with dom0less guests, allowing statically
+   booted dom0less guests with PV devices.
+ - On Arm, p2m structures are now allocated out of a pool of memory set aside at
+   domain creation.
+ - Improved mitigations against Spectre-BHB on Arm.
+ - Support VirtIO-MMIO devices in toolstack on Arm.
+ - Allow setting the number of CPUs to activate at runtime from command line
+   option on Arm.
+ - Grant-table support on Arm was improved and hardened by implementing
+   "simplified M2P-like approach for the xenheap pages"
+ - Add Renesas R-Car Gen4 IPMMU-VMSA support on Arm.
+ - Add i.MX lpuart and i.MX8QM initial support on Arm.
+ - Improved toolstack build system.
+ - Add Xue - console over USB 3 Debug Capability.
 
 ### Removed / support downgraded
  - dropped support for the (x86-only) "vesa-mtrr" and "vesa-remap" command line options