diff mbox series

[2/3] CHANGELOG: Add missing entries for work during the 4.17 release

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

Commit Message

Henry Wang Nov. 11, 2022, 11:22 a.m. UTC
Signed-off-by: Henry Wang <Henry.Wang@arm.com>
---
 CHANGELOG.md | 29 +++++++++++++++++++++++++++--
 1 file changed, 27 insertions(+), 2 deletions(-)

Comments

Jan Beulich Nov. 11, 2022, 1:24 p.m. UTC | #1
(reducing the Cc list to just xen-devel@)

On 11.11.2022 12:22, Henry Wang wrote:
>  ### 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 frequency calibration and APIC on x86.

Maybe better "Improved TSC, CPU, and APIC clock frequency calibration on x86"?

> + - Improved support for CET Indirect Branch Tracking on x86.
> + - Improved mwait-idle support for SPR and ADL on x86.

SPR and ADL support is new in the driver, not improved.

Jan
Henry Wang Nov. 11, 2022, 1:58 p.m. UTC | #2
Hi Jan,

> -----Original Message-----
> Subject: Re: [PATCH 2/3] CHANGELOG: Add missing entries for work during
> the 4.17 release
> 
> (reducing the Cc list to just xen-devel@)

Thanks for doing this :) I was just not sure who would be interested in the series.

> 
> On 11.11.2022 12:22, Henry Wang wrote:
> >  ### 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 frequency calibration and APIC on x86.
> 
> Maybe better "Improved TSC, CPU, and APIC clock frequency calibration on
> x86"?

Thanks for your feedback for this and my previous email. They are really
useful.

Yes of course, I guess I misunderstood the original series and thought
there is another improvement for the APIC.

> 
> > + - Improved support for CET Indirect Branch Tracking on x86.
> > + - Improved mwait-idle support for SPR and ADL on x86.
> 
> SPR and ADL support is new in the driver, not improved.

I will correct this in V2.

Another question for adding another entry for x86 that worth to
brag about a little bit: I did another dig into patchwork and found that
this [1] might be a good candidate. IIUC __ro_after_init is a good
feature that both x86 and Arm have leveraged in different level today,
so probably the __ro_after_init might worth an entry in CHANGELOG.

However I saw your comments in [1] so I am not really sure if you are
happy with that, maybe we need to change the wording a little bit to
remove your concern? Would you mind sharing some thoughts?

Thanks!

[1] https://patchwork.kernel.org/project/xen-devel/patch/20220309123936.16991-3-andrew.cooper3@citrix.com/

Kind regards,
Henry
> 
> Jan
Jan Beulich Nov. 11, 2022, 2:08 p.m. UTC | #3
On 11.11.2022 14:58, Henry Wang wrote:
> Another question for adding another entry for x86 that worth to
> brag about a little bit: I did another dig into patchwork and found that
> this [1] might be a good candidate. IIUC __ro_after_init is a good
> feature that both x86 and Arm have leveraged in different level today,
> so probably the __ro_after_init might worth an entry in CHANGELOG.
> 
> However I saw your comments in [1] so I am not really sure if you are
> happy with that, maybe we need to change the wording a little bit to
> remove your concern? Would you mind sharing some thoughts?

My view hasn't changed, but if Andrew and Roger think this should be mentioned,
then I'm not going to stand in the way.

Jan

> [1] https://patchwork.kernel.org/project/xen-devel/patch/20220309123936.16991-3-andrew.cooper3@citrix.com/
> 
> Kind regards,
> Henry
>>
>> Jan
Henry Wang Nov. 11, 2022, 2:12 p.m. UTC | #4
Hi Jan,

> -----Original Message-----
> On 11.11.2022 14:58, Henry Wang wrote:
> > Another question for adding another entry for x86 that worth to
> > brag about a little bit: I did another dig into patchwork and found that
> > this [1] might be a good candidate. IIUC __ro_after_init is a good
> > feature that both x86 and Arm have leveraged in different level today,
> > so probably the __ro_after_init might worth an entry in CHANGELOG.
> >
> > However I saw your comments in [1] so I am not really sure if you are
> > happy with that, maybe we need to change the wording a little bit to
> > remove your concern? Would you mind sharing some thoughts?
> 
> My view hasn't changed, but if Andrew and Roger think this should be
> mentioned,
> then I'm not going to stand in the way.

Thanks for the confirmation. I will wait for Roger's back on next Monday
then.

Kind regards,
Henry

> 
> Jan
> 
> > [1] https://patchwork.kernel.org/project/xen-
> devel/patch/20220309123936.16991-3-andrew.cooper3@citrix.com/
> >
> > Kind regards,
> > Henry
> >>
> >> Jan
Oleksandr Tyshchenko Nov. 11, 2022, 2:48 p.m. UTC | #5
On Fri, Nov 11, 2022 at 11:23 AM Henry Wang <Henry.Wang@arm.com> wrote:

Hello Henry

[leave only xen-devel@lists.xenproject.org in CC]
[sorry for the possible format issues]

Signed-off-by: Henry Wang <Henry.Wang@arm.com>
> ---
>  CHANGELOG.md | 29 +++++++++++++++++++++++++++--
>  1 file changed, 27 insertions(+), 2 deletions(-)
>
> diff --git a/CHANGELOG.md b/CHANGELOG.md
> index adbbb216fa..fa8cc476b3 100644
> --- a/CHANGELOG.md
> +++ b/CHANGELOG.md
> @@ -4,16 +4,41 @@ 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) -
> 2022-11-??
>
>  ### 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 frequency calibration and APIC on x86.
> + - Improved support for CET Indirect Branch Tracking on x86.
> + - Improved 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.
> + - Add support for VirtIO toolstack on Arm.
>

I would clarify that only virtio-mmio is supported on Arm.


> + - Allow setting the number of CPUs to activate at runtime from command
> line
> +   option on Arm.
> + - Improved toolstack build system.
> + - Add Xue - console over USB 3 Debug Capability.
>

I would probably also add the following:

- Add Renesas R-Car Gen4 IPMMU-VMSA support (Arm)
- grant-table support on Arm was improved and hardened by implementing
“simplified M2P-like
approach for the xenheap pages”



>
>  ### Removed / support downgraded
>   - dropped support for the (x86-only) "vesa-mtrr" and "vesa-remap"
> command line options
> --
> 2.25.1
>
>
>
Henry Wang Nov. 11, 2022, 3:10 p.m. UTC | #6
Hi Oleksandr,

(+ Arm maintainers for a question)

Please see below inline comments (I somehow cannot reply as plain text for this email).

+ - Add support for VirtIO toolstack on Arm.

I would clarify that only virtio-mmio is supported on Arm.

(Henry): Sure, will clarify that.

+ - Allow setting the number of CPUs to activate at runtime from command line
+   option on Arm.
+ - Improved toolstack build system.
+ - Add Xue - console over USB 3 Debug Capability.

I would probably also add the following:

- Add Renesas R-Car Gen4 IPMMU-VMSA support (Arm)

(Henry): If something from a specific vendor needs to be included, I wonder if “add i.MX lpuart and i.MX8QM initial support” also qualify. I will let Arm maintainers to do the final call here. Hope you would be fine with that.

- grant-table support on Arm was improved and hardened by implementing “simplified M2P-like
approach for the xenheap pages”

(Henry): Sure, I will add this.

Kind regards,
Henry
Stefano Stabellini Nov. 11, 2022, 8:35 p.m. UTC | #7
On Fri, 11 Nov 2022, Henry Wang wrote:
> - Add Renesas R-Car Gen4 IPMMU-VMSA support (Arm)
> 
>  
> 
> (Henry): If something from a specific vendor needs to be included, I wonder if “add i.MX lpuart and i.MX8QM initial support” also qualify.
> I will let Arm maintainers to do the final call here. Hope you would be fine with that.
 
Yes I think that's fine to add things from a specific vendor, including
i.MX related stuff
Henry Wang Nov. 15, 2022, 12:40 a.m. UTC | #8
Hi Roger,

> -----Original Message-----
> Subject: RE: [PATCH 2/3] CHANGELOG: Add missing entries for work during
> the 4.17 release
> 
> Hi Jan,
> 
> > -----Original Message-----
> > On 11.11.2022 14:58, Henry Wang wrote:
> > > Another question for adding another entry for x86 that worth to
> > > brag about a little bit: I did another dig into patchwork and found that
> > > this [1] might be a good candidate. IIUC __ro_after_init is a good
> > > feature that both x86 and Arm have leveraged in different level today,
> > > so probably the __ro_after_init might worth an entry in CHANGELOG.
> > >
> > > However I saw your comments in [1] so I am not really sure if you are
> > > happy with that, maybe we need to change the wording a little bit to
> > > remove your concern? Would you mind sharing some thoughts?
> >
> > My view hasn't changed, but if Andrew and Roger think this should be
> > mentioned,
> > then I'm not going to stand in the way.
> 
> Thanks for the confirmation. I will wait for Roger's back on next Monday
> then.

Sorry I forgot to add you to the thread in last email, could you please share
Your opinion about this one? Thanks.

Kind regards,
Henry
diff mbox series

Patch

diff --git a/CHANGELOG.md b/CHANGELOG.md
index adbbb216fa..fa8cc476b3 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -4,16 +4,41 @@  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) - 2022-11-??
 
 ### 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 frequency calibration and APIC on x86.
+ - Improved support for CET Indirect Branch Tracking on x86.
+ - Improved 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.
+ - Add support for VirtIO toolstack on Arm.
+ - Allow setting the number of CPUs to activate at runtime from command line
+   option 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