[PATCH for-4.16 0/2] CHANGELOG: start to populate entries for 4.16

[PATCH for-4.16 0/2] CHANGELOG: start to populate entries for 4.16

[Roger Pau Monne]

Hello,

The following changes are preparation work for the 4.16 release. This
is my first pass at the log for the release, it's likely missing more
entries.

I'm Ccing all maintainers in this cover letter and also in patch #2 for
further feedback on missing items.

Thanks, Roger.

Roger Pau Monne (2):
  CHANGELOG: set Xen 4.15 release date
  CHANGELOG: add missing entries for work during the 4.16 release cycle

 CHANGELOG.md | 12 +++++++++++-
 1 file changed, 11 insertions(+), 1 deletion(-)

-- 
2.33.0

[PATCH for-4.16 1/2] CHANGELOG: set Xen 4.15 release date

[Roger Pau Monne]

Signed-off-by: Roger Pau Monné <roger.pau@citrix.com>
---
 CHANGELOG.md | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index e5ab49e779..ad1a8c2bc2 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -22,7 +22,7 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/)
    no longer be built per default. In order to be able to use those, configure needs to
    be called with "--enable-qemu-traditional" as parameter.
 
-## [4.15.0 UNRELEASED](https://xenbits.xen.org/gitweb/?p=xen.git;a=shortlog;h=RELEASE-4.15.0) - TBD
+## [4.15.0](https://xenbits.xen.org/gitweb/?p=xen.git;a=shortlog;h=RELEASE-4.15.0) - 2021-04-08
 
 ### Added / support upgraded
  - ARM IOREQ servers (device emulation etc.) (Tech Preview)
-- 
2.33.0

[PATCH for-4.16 2/2] CHANGELOG: add missing entries for work during the 4.16 release cycle

[Roger Pau Monne]

Document some of the relevant changes during the 4.16 release cycle,
likely more entries are missing.

Signed-off-by: Roger Pau Monné <roger.pau@citrix.com>
---
 CHANGELOG.md | 11 +++++++++++
 1 file changed, 11 insertions(+)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index ad1a8c2bc2..8b0bdd9cf0 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -21,6 +21,17 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/)
  - qemu-traditional based device models (both, qemu-traditional and ioemu-stubdom) will
    no longer be built per default. In order to be able to use those, configure needs to
    be called with "--enable-qemu-traditional" as parameter.
+ - Fixes for credit2 scheduler stability in corner case conditions.
+ - Ongoing improvements in the hypervisor build system.
+ - vtpmmgr miscellaneous fixes in preparation for TPM 2.0 support.
+ - 32bit PV guests only supported in shim mode.
+ - Improved PVH dom0 debug key handling.
+ - Fix booting on some Intel systems without a PIT (i8254).
+
+### Added
+ - 32bit Arm builds to the automated tests.
+ - New x86 pagetable APIs.
+ - Arm vPMU support.
 
 ## [4.15.0](https://xenbits.xen.org/gitweb/?p=xen.git;a=shortlog;h=RELEASE-4.15.0) - 2021-04-08
 
-- 
2.33.0

Re: [PATCH for-4.16 2/2] CHANGELOG: add missing entries for work during the 4.16 release cycle

[Julien Grall]

Hi,

On 17/11/2021 09:53, Roger Pau Monne wrote:
> Document some of the relevant changes during the 4.16 release cycle,
> likely more entries are missing.
> 
> Signed-off-by: Roger Pau Monné <roger.pau@citrix.com>
> ---
>   CHANGELOG.md | 11 +++++++++++
>   1 file changed, 11 insertions(+)
> 
> diff --git a/CHANGELOG.md b/CHANGELOG.md
> index ad1a8c2bc2..8b0bdd9cf0 100644
> --- a/CHANGELOG.md
> +++ b/CHANGELOG.md
> @@ -21,6 +21,17 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/)
>    - qemu-traditional based device models (both, qemu-traditional and ioemu-stubdom) will
>      no longer be built per default. In order to be able to use those, configure needs to
>      be called with "--enable-qemu-traditional" as parameter.
> + - Fixes for credit2 scheduler stability in corner case conditions.
> + - Ongoing improvements in the hypervisor build system.
> + - vtpmmgr miscellaneous fixes in preparation for TPM 2.0 support.
> + - 32bit PV guests only supported in shim mode.
> + - Improved PVH dom0 debug key handling.
> + - Fix booting on some Intel systems without a PIT (i8254).
> +
> +### Added
> + - 32bit Arm builds to the automated tests.
> + - New x86 pagetable APIs.
> + - Arm vPMU support.

I would add "Early" or something similar because this can only work in a 
constrained setup.

Cheers,

-- 
Julien Grall

Re: [PATCH for-4.16 2/2] CHANGELOG: add missing entries for work during the 4.16 release cycle

[Juergen Gross]

[-- Attachment #1.1.1: Type: text/plain, Size: 1488 bytes --]

On 17.11.21 10:53, Roger Pau Monne wrote:
> Document some of the relevant changes during the 4.16 release cycle,
> likely more entries are missing.
> 
> Signed-off-by: Roger Pau Monné <roger.pau@citrix.com>
> ---
>   CHANGELOG.md | 11 +++++++++++
>   1 file changed, 11 insertions(+)
> 
> diff --git a/CHANGELOG.md b/CHANGELOG.md
> index ad1a8c2bc2..8b0bdd9cf0 100644
> --- a/CHANGELOG.md
> +++ b/CHANGELOG.md
> @@ -21,6 +21,17 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/)
>    - qemu-traditional based device models (both, qemu-traditional and ioemu-stubdom) will
>      no longer be built per default. In order to be able to use those, configure needs to
>      be called with "--enable-qemu-traditional" as parameter.
> + - Fixes for credit2 scheduler stability in corner case conditions.
> + - Ongoing improvements in the hypervisor build system.
> + - vtpmmgr miscellaneous fixes in preparation for TPM 2.0 support.
> + - 32bit PV guests only supported in shim mode.
> + - Improved PVH dom0 debug key handling.
> + - Fix booting on some Intel systems without a PIT (i8254).

Missing are:

- don't default to build qemu-traditional
- don't default to build pvgrub

> +
> +### Added
> + - 32bit Arm builds to the automated tests.
> + - New x86 pagetable APIs.
> + - Arm vPMU support.
>   
>   ## [4.15.0](https://xenbits.xen.org/gitweb/?p=xen.git;a=shortlog;h=RELEASE-4.15.0) - 2021-04-08
>   
> 

Juergen

[-- Attachment #1.1.2: OpenPGP public key --]
[-- Type: application/pgp-keys, Size: 3135 bytes --]

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 495 bytes --]

Re: [PATCH for-4.16 1/2] CHANGELOG: set Xen 4.15 release date

[Jan Beulich]

On 17.11.2021 10:53, Roger Pau Monne wrote:
> Signed-off-by: Roger Pau Monné <roger.pau@citrix.com>

Acked-by: Jan Beulich <jbeulich@suse.com>

I'm puzzled how we managed to have missed that.

> --- a/CHANGELOG.md
> +++ b/CHANGELOG.md
> @@ -22,7 +22,7 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/)
>     no longer be built per default. In order to be able to use those, configure needs to
>     be called with "--enable-qemu-traditional" as parameter.
>  
> -## [4.15.0 UNRELEASED](https://xenbits.xen.org/gitweb/?p=xen.git;a=shortlog;h=RELEASE-4.15.0) - TBD
> +## [4.15.0](https://xenbits.xen.org/gitweb/?p=xen.git;a=shortlog;h=RELEASE-4.15.0) - 2021-04-08

One day off is not really a problem, but I wonder how you ended up with the
8th. The tag was made on the 7th, and the commit that was tagged had been
made on the 6th.

Jan

Re: [PATCH for-4.16 1/2] CHANGELOG: set Xen 4.15 release date

[Roger Pau Monné]

On Wed, Nov 17, 2021 at 11:21:34AM +0100, Jan Beulich wrote:
> On 17.11.2021 10:53, Roger Pau Monne wrote:
> > Signed-off-by: Roger Pau Monné <roger.pau@citrix.com>
> 
> Acked-by: Jan Beulich <jbeulich@suse.com>
> 
> I'm puzzled how we managed to have missed that.
> 
> > --- a/CHANGELOG.md
> > +++ b/CHANGELOG.md
> > @@ -22,7 +22,7 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/)
> >     no longer be built per default. In order to be able to use those, configure needs to
> >     be called with "--enable-qemu-traditional" as parameter.
> >  
> > -## [4.15.0 UNRELEASED](https://xenbits.xen.org/gitweb/?p=xen.git;a=shortlog;h=RELEASE-4.15.0) - TBD
> > +## [4.15.0](https://xenbits.xen.org/gitweb/?p=xen.git;a=shortlog;h=RELEASE-4.15.0) - 2021-04-08
> 
> One day off is not really a problem, but I wonder how you ended up with the
> 8th. The tag was made on the 7th, and the commit that was tagged had been
> made on the 6th.

Both the announcement and the blog post where on the 8th, so I think
that's the official release date, not really when the tag was made.

https://lists.xenproject.org/archives/html/xen-announce/2021-04/msg00001.html
https://xenproject.org/2021/04/08/xen-project-hypervisor-4-15/

Thanks, Roger.

Re: [PATCH for-4.16 0/2] CHANGELOG: start to populate entries for 4.16

[Ian Jackson]

Roger Pau Monne writes ("[PATCH for-4.16 0/2] CHANGELOG: start to populate entries for 4.16"):
> The following changes are preparation work for the 4.16 release. This
> is my first pass at the log for the release, it's likely missing more
> entries.
> 
> I'm Ccing all maintainers in this cover letter and also in patch #2 for
> further feedback on missing items.

Thanks!  Dropped the CC list for this mail:

Release-Acked-by: Ian Jackson <iwj@xenproject.org>

for both patches (including future versions).

Ian.

Re: [PATCH for-4.16 1/2] CHANGELOG: set Xen 4.15 release date

[Ian Jackson]

Jan Beulich writes ("Re: [PATCH for-4.16 1/2] CHANGELOG: set Xen 4.15 release date"):
> I'm puzzled how we managed to have missed that.

Manual faff and eyes glazing over I think...

> > --- a/CHANGELOG.md
> > +++ b/CHANGELOG.md
> > @@ -22,7 +22,7 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/)
> >     no longer be built per default. In order to be able to use those, configure needs to
> >     be called with "--enable-qemu-traditional" as parameter.
> >  
> > -## [4.15.0 UNRELEASED](https://xenbits.xen.org/gitweb/?p=xen.git;a=shortlog;h=RELEASE-4.15.0) - TBD
> > +## [4.15.0](https://xenbits.xen.org/gitweb/?p=xen.git;a=shortlog;h=RELEASE-4.15.0) - 2021-04-08
> 
> One day off is not really a problem, but I wonder how you ended up with the
> 8th. The tag was made on the 7th, and the commit that was tagged had been
> made on the 6th.

The announcement was made on the 8th.  The release deliverables are
generally prepped the day before the formal release.

Ian.

Re: [PATCH for-4.16 2/2] CHANGELOG: add missing entries for work during the 4.16 release cycle

[Michal Orzel]

Hi,

On 17.11.2021 11:00, Julien Grall wrote:
> Hi,
> 
> On 17/11/2021 09:53, Roger Pau Monne wrote:
>> Document some of the relevant changes during the 4.16 release cycle,
>> likely more entries are missing.
>>
>> Signed-off-by: Roger Pau Monné <roger.pau@citrix.com>
>> ---
>>   CHANGELOG.md | 11 +++++++++++
>>   1 file changed, 11 insertions(+)
>>
>> diff --git a/CHANGELOG.md b/CHANGELOG.md
>> index ad1a8c2bc2..8b0bdd9cf0 100644
>> --- a/CHANGELOG.md
>> +++ b/CHANGELOG.md
>> @@ -21,6 +21,17 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/)
>>    - qemu-traditional based device models (both, qemu-traditional and ioemu-stubdom) will
>>      no longer be built per default. In order to be able to use those, configure needs to
>>      be called with "--enable-qemu-traditional" as parameter.
>> + - Fixes for credit2 scheduler stability in corner case conditions.
>> + - Ongoing improvements in the hypervisor build system.
>> + - vtpmmgr miscellaneous fixes in preparation for TPM 2.0 support.
>> + - 32bit PV guests only supported in shim mode.
>> + - Improved PVH dom0 debug key handling.
>> + - Fix booting on some Intel systems without a PIT (i8254).
>> +
>> +### Added
>> + - 32bit Arm builds to the automated tests.
>> + - New x86 pagetable APIs.
>> + - Arm vPMU support.
> 
> I would add "Early" or something similar because this can only work in a constrained setup.
> 
+1.
I would say:
"Arm early vPMU support for guests."

> Cheers,
> 

Cheers,
Michal

Re: [PATCH for-4.16 2/2] CHANGELOG: add missing entries for work during the 4.16 release cycle

[Roger Pau Monné]

On Wed, Nov 17, 2021 at 12:44:25PM +0100, Michal Orzel wrote:
> Hi,
> 
> On 17.11.2021 11:00, Julien Grall wrote:
> > Hi,
> > 
> > On 17/11/2021 09:53, Roger Pau Monne wrote:
> >> Document some of the relevant changes during the 4.16 release cycle,
> >> likely more entries are missing.
> >>
> >> Signed-off-by: Roger Pau Monné <roger.pau@citrix.com>
> >> ---
> >>   CHANGELOG.md | 11 +++++++++++
> >>   1 file changed, 11 insertions(+)
> >>
> >> diff --git a/CHANGELOG.md b/CHANGELOG.md
> >> index ad1a8c2bc2..8b0bdd9cf0 100644
> >> --- a/CHANGELOG.md
> >> +++ b/CHANGELOG.md
> >> @@ -21,6 +21,17 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/)
> >>    - qemu-traditional based device models (both, qemu-traditional and ioemu-stubdom) will
> >>      no longer be built per default. In order to be able to use those, configure needs to
> >>      be called with "--enable-qemu-traditional" as parameter.
> >> + - Fixes for credit2 scheduler stability in corner case conditions.
> >> + - Ongoing improvements in the hypervisor build system.
> >> + - vtpmmgr miscellaneous fixes in preparation for TPM 2.0 support.
> >> + - 32bit PV guests only supported in shim mode.
> >> + - Improved PVH dom0 debug key handling.
> >> + - Fix booting on some Intel systems without a PIT (i8254).
> >> +
> >> +### Added
> >> + - 32bit Arm builds to the automated tests.
> >> + - New x86 pagetable APIs.
> >> + - Arm vPMU support.
> > 
> > I would add "Early" or something similar because this can only work in a constrained setup.
> > 
> +1.
> I would say:
> "Arm early vPMU support for guests."

I've used limited instead of early, as using early could be confused
with early boot support or some such IMO.

Thanks, Roger.

Re: [PATCH for-4.16 2/2] CHANGELOG: add missing entries for work during the 4.16 release cycle

[Bertrand Marquis]

Hi Roger,

> On 17 Nov 2021, at 09:53, Roger Pau Monne <roger.pau@citrix.com> wrote:
> 
> Document some of the relevant changes during the 4.16 release cycle,
> likely more entries are missing.
> 
> Signed-off-by: Roger Pau Monné <roger.pau@citrix.com>
> ---
> CHANGELOG.md | 11 +++++++++++
> 1 file changed, 11 insertions(+)
> 
> diff --git a/CHANGELOG.md b/CHANGELOG.md
> index ad1a8c2bc2..8b0bdd9cf0 100644
> --- a/CHANGELOG.md
> +++ b/CHANGELOG.md
> @@ -21,6 +21,17 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/)
>  - qemu-traditional based device models (both, qemu-traditional and ioemu-stubdom) will
>    no longer be built per default. In order to be able to use those, configure needs to
>    be called with "--enable-qemu-traditional" as parameter.
> + - Fixes for credit2 scheduler stability in corner case conditions.
> + - Ongoing improvements in the hypervisor build system.
> + - vtpmmgr miscellaneous fixes in preparation for TPM 2.0 support.
> + - 32bit PV guests only supported in shim mode.
> + - Improved PVH dom0 debug key handling.
> + - Fix booting on some Intel systems without a PIT (i8254).

Missing:
- cpu ID sanitization on arm64
- fix 32/64bit vreg emulation on arm64

> +
> +### Added
> + - 32bit Arm builds to the automated tests.
> + - New x86 pagetable APIs.
> + - Arm vPMU support.

Missing:
- static allocation for dom0less on arm64
- dom0less EFI support on arm64

Cheers
Bertrand

> 
> ## [4.15.0](https://xenbits.xen.org/gitweb/?p=xen.git;a=shortlog;h=RELEASE-4.15.0) - 2021-04-08
> 
> -- 
> 2.33.0
> 

Re: [PATCH for-4.16 2/2] CHANGELOG: add missing entries for work during the 4.16 release cycle

[Wei Chen]

On 2021/11/17 22:07, Bertrand Marquis wrote:
> Hi Roger,
> 
>> On 17 Nov 2021, at 09:53, Roger Pau Monne <roger.pau@citrix.com> wrote:
>>
>> Document some of the relevant changes during the 4.16 release cycle,
>> likely more entries are missing.
>>
>> Signed-off-by: Roger Pau Monné <roger.pau@citrix.com>
>> ---
>> CHANGELOG.md | 11 +++++++++++
>> 1 file changed, 11 insertions(+)
>>
>> diff --git a/CHANGELOG.md b/CHANGELOG.md
>> index ad1a8c2bc2..8b0bdd9cf0 100644
>> --- a/CHANGELOG.md
>> +++ b/CHANGELOG.md
>> @@ -21,6 +21,17 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/)
>>   - qemu-traditional based device models (both, qemu-traditional and ioemu-stubdom) will
>>     no longer be built per default. In order to be able to use those, configure needs to
>>     be called with "--enable-qemu-traditional" as parameter.
>> + - Fixes for credit2 scheduler stability in corner case conditions.
>> + - Ongoing improvements in the hypervisor build system.
>> + - vtpmmgr miscellaneous fixes in preparation for TPM 2.0 support.
>> + - 32bit PV guests only supported in shim mode.
>> + - Improved PVH dom0 debug key handling.
>> + - Fix booting on some Intel systems without a PIT (i8254).
> 
> Missing:
> - cpu ID sanitization on arm64
> - fix 32/64bit vreg emulation on arm64
> 
>> +
>> +### Added
>> + - 32bit Arm builds to the automated tests.
>> + - New x86 pagetable APIs.
>> + - Arm vPMU support.

Maybe miss this one either:
Add GICD_ICPENDR register handler in vGIC to support Zephyr.

> 
> Missing:
> - static allocation for dom0less on arm64
> - dom0less EFI support on arm64
> 
> Cheers
> Bertrand
> 
>>
>> ## [4.15.0](https://xenbits.xen.org/gitweb/?p=xen.git;a=shortlog;h=RELEASE-4.15.0) - 2021-04-08
>>
>> -- 
>> 2.33.0
>>
> 

Re: [PATCH for-4.16 2/2] CHANGELOG: add missing entries for work during the 4.16 release cycle

[Roger Pau Monné]

On Wed, Nov 17, 2021 at 02:07:50PM +0000, Bertrand Marquis wrote:
> Hi Roger,
> 
> > On 17 Nov 2021, at 09:53, Roger Pau Monne <roger.pau@citrix.com> wrote:
> > 
> > Document some of the relevant changes during the 4.16 release cycle,
> > likely more entries are missing.
> > 
> > Signed-off-by: Roger Pau Monné <roger.pau@citrix.com>
> > ---
> > CHANGELOG.md | 11 +++++++++++
> > 1 file changed, 11 insertions(+)
> > 
> > diff --git a/CHANGELOG.md b/CHANGELOG.md
> > index ad1a8c2bc2..8b0bdd9cf0 100644
> > --- a/CHANGELOG.md
> > +++ b/CHANGELOG.md
> > @@ -21,6 +21,17 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/)
> >  - qemu-traditional based device models (both, qemu-traditional and ioemu-stubdom) will
> >    no longer be built per default. In order to be able to use those, configure needs to
> >    be called with "--enable-qemu-traditional" as parameter.
> > + - Fixes for credit2 scheduler stability in corner case conditions.
> > + - Ongoing improvements in the hypervisor build system.
> > + - vtpmmgr miscellaneous fixes in preparation for TPM 2.0 support.
> > + - 32bit PV guests only supported in shim mode.
> > + - Improved PVH dom0 debug key handling.
> > + - Fix booting on some Intel systems without a PIT (i8254).
> 
> Missing:
> - cpu ID sanitization on arm64
> - fix 32/64bit vreg emulation on arm64

Can I get a bit more information about those items? Just a pointer to
the commit messages would be helpful so that I can try to write a more
comprehensive entry (or maybe it's just me not knowing anything about
Arm that fails to understand it).

> > +
> > +### Added
> > + - 32bit Arm builds to the automated tests.
> > + - New x86 pagetable APIs.
> > + - Arm vPMU support.
> 
> Missing:
> - static allocation for dom0less on arm64
> - dom0less EFI support on arm64

Those I can indeed understand :).

Thanks, Roger.

Re: [PATCH for-4.16 2/2] CHANGELOG: add missing entries for work during the 4.16 release cycle

[Bertrand Marquis]

Hi Roger,


> On 18 Nov 2021, at 08:58, Roger Pau Monné <roger.pau@citrix.com> wrote:
> 
> On Wed, Nov 17, 2021 at 02:07:50PM +0000, Bertrand Marquis wrote:
>> Hi Roger,
>> 
>>> On 17 Nov 2021, at 09:53, Roger Pau Monne <roger.pau@citrix.com> wrote:
>>> 
>>> Document some of the relevant changes during the 4.16 release cycle,
>>> likely more entries are missing.
>>> 
>>> Signed-off-by: Roger Pau Monné <roger.pau@citrix.com>
>>> ---
>>> CHANGELOG.md | 11 +++++++++++
>>> 1 file changed, 11 insertions(+)
>>> 
>>> diff --git a/CHANGELOG.md b/CHANGELOG.md
>>> index ad1a8c2bc2..8b0bdd9cf0 100644
>>> --- a/CHANGELOG.md
>>> +++ b/CHANGELOG.md
>>> @@ -21,6 +21,17 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/)
>>> - qemu-traditional based device models (both, qemu-traditional and ioemu-stubdom) will
>>>   no longer be built per default. In order to be able to use those, configure needs to
>>>   be called with "--enable-qemu-traditional" as parameter.
>>> + - Fixes for credit2 scheduler stability in corner case conditions.
>>> + - Ongoing improvements in the hypervisor build system.
>>> + - vtpmmgr miscellaneous fixes in preparation for TPM 2.0 support.
>>> + - 32bit PV guests only supported in shim mode.
>>> + - Improved PVH dom0 debug key handling.
>>> + - Fix booting on some Intel systems without a PIT (i8254).
>> 
>> Missing:
>> - cpu ID sanitization on arm64
>> - fix 32/64bit vreg emulation on arm64
> 
> Can I get a bit more information about those items? Just a pointer to
> the commit messages would be helpful so that I can try to write a more
> comprehensive entry (or maybe it's just me not knowing anything about
> Arm that fails to understand it).

I agree, the text is not quite clear, I will try to come with a better one.
In the meantime here is are links to the series:
https://patchwork.kernel.org/project/xen-devel/list/?series=535805&state=*
https://patchwork.kernel.org/project/xen-devel/list/?series=477151&archive=both

Cheers
Bertrand

> 
>>> +
>>> +### Added
>>> + - 32bit Arm builds to the automated tests.
>>> + - New x86 pagetable APIs.
>>> + - Arm vPMU support.
>> 
>> Missing:
>> - static allocation for dom0less on arm64
>> - dom0less EFI support on arm64
> 
> Those I can indeed understand :).
> 
> Thanks, Roger.

Re: [PATCH for-4.16 2/2] CHANGELOG: add missing entries for work during the 4.16 release cycle

[Roger Pau Monné]

On Thu, Nov 18, 2021 at 10:01:08AM +0000, Bertrand Marquis wrote:
> Hi Roger,
> 
> 
> > On 18 Nov 2021, at 08:58, Roger Pau Monné <roger.pau@citrix.com> wrote:
> > 
> > On Wed, Nov 17, 2021 at 02:07:50PM +0000, Bertrand Marquis wrote:
> >> Hi Roger,
> >> 
> >>> On 17 Nov 2021, at 09:53, Roger Pau Monne <roger.pau@citrix.com> wrote:
> >>> 
> >>> Document some of the relevant changes during the 4.16 release cycle,
> >>> likely more entries are missing.
> >>> 
> >>> Signed-off-by: Roger Pau Monné <roger.pau@citrix.com>
> >>> ---
> >>> CHANGELOG.md | 11 +++++++++++
> >>> 1 file changed, 11 insertions(+)
> >>> 
> >>> diff --git a/CHANGELOG.md b/CHANGELOG.md
> >>> index ad1a8c2bc2..8b0bdd9cf0 100644
> >>> --- a/CHANGELOG.md
> >>> +++ b/CHANGELOG.md
> >>> @@ -21,6 +21,17 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/)
> >>> - qemu-traditional based device models (both, qemu-traditional and ioemu-stubdom) will
> >>>   no longer be built per default. In order to be able to use those, configure needs to
> >>>   be called with "--enable-qemu-traditional" as parameter.
> >>> + - Fixes for credit2 scheduler stability in corner case conditions.
> >>> + - Ongoing improvements in the hypervisor build system.
> >>> + - vtpmmgr miscellaneous fixes in preparation for TPM 2.0 support.
> >>> + - 32bit PV guests only supported in shim mode.
> >>> + - Improved PVH dom0 debug key handling.
> >>> + - Fix booting on some Intel systems without a PIT (i8254).
> >> 
> >> Missing:
> >> - cpu ID sanitization on arm64
> >> - fix 32/64bit vreg emulation on arm64
> > 
> > Can I get a bit more information about those items? Just a pointer to
> > the commit messages would be helpful so that I can try to write a more
> > comprehensive entry (or maybe it's just me not knowing anything about
> > Arm that fails to understand it).
> 
> I agree, the text is not quite clear, I will try to come with a better one.
> In the meantime here is are links to the series:
> https://patchwork.kernel.org/project/xen-devel/list/?series=535805&state=*
> https://patchwork.kernel.org/project/xen-devel/list/?series=477151&archive=both

Thanks. I've added:

 - CPU feature leveling on arm64 platform with heterogeneous cores.
 - Handle register accesses as 32/64bit on Arm depending on the processor
   bitness.

Let me know if that's not accurate.

Roger.

Re: [PATCH for-4.16 2/2] CHANGELOG: add missing entries for work during the 4.16 release cycle

[Bertrand Marquis]

Hi Roger,

> On 18 Nov 2021, at 10:37, Roger Pau Monné <roger.pau@citrix.com> wrote:
> 
> On Thu, Nov 18, 2021 at 10:01:08AM +0000, Bertrand Marquis wrote:
>> Hi Roger,
>> 
>> 
>>> On 18 Nov 2021, at 08:58, Roger Pau Monné <roger.pau@citrix.com> wrote:
>>> 
>>> On Wed, Nov 17, 2021 at 02:07:50PM +0000, Bertrand Marquis wrote:
>>>> Hi Roger,
>>>> 
>>>>> On 17 Nov 2021, at 09:53, Roger Pau Monne <roger.pau@citrix.com> wrote:
>>>>> 
>>>>> Document some of the relevant changes during the 4.16 release cycle,
>>>>> likely more entries are missing.
>>>>> 
>>>>> Signed-off-by: Roger Pau Monné <roger.pau@citrix.com>
>>>>> ---
>>>>> CHANGELOG.md | 11 +++++++++++
>>>>> 1 file changed, 11 insertions(+)
>>>>> 
>>>>> diff --git a/CHANGELOG.md b/CHANGELOG.md
>>>>> index ad1a8c2bc2..8b0bdd9cf0 100644
>>>>> --- a/CHANGELOG.md
>>>>> +++ b/CHANGELOG.md
>>>>> @@ -21,6 +21,17 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/)
>>>>> - qemu-traditional based device models (both, qemu-traditional and ioemu-stubdom) will
>>>>>  no longer be built per default. In order to be able to use those, configure needs to
>>>>>  be called with "--enable-qemu-traditional" as parameter.
>>>>> + - Fixes for credit2 scheduler stability in corner case conditions.
>>>>> + - Ongoing improvements in the hypervisor build system.
>>>>> + - vtpmmgr miscellaneous fixes in preparation for TPM 2.0 support.
>>>>> + - 32bit PV guests only supported in shim mode.
>>>>> + - Improved PVH dom0 debug key handling.
>>>>> + - Fix booting on some Intel systems without a PIT (i8254).
>>>> 
>>>> Missing:
>>>> - cpu ID sanitization on arm64
>>>> - fix 32/64bit vreg emulation on arm64
>>> 
>>> Can I get a bit more information about those items? Just a pointer to
>>> the commit messages would be helpful so that I can try to write a more
>>> comprehensive entry (or maybe it's just me not knowing anything about
>>> Arm that fails to understand it).
>> 
>> I agree, the text is not quite clear, I will try to come with a better one.
>> In the meantime here is are links to the series:
>> https://patchwork.kernel.org/project/xen-devel/list/?series=535805&state=*
>> https://patchwork.kernel.org/project/xen-devel/list/?series=477151&archive=both
> 
> Thanks. I've added:
> 
> - CPU feature leveling on arm64 platform with heterogeneous cores.
Yes
> - Handle register accesses as 32/64bit on Arm depending on the processor
>   bitness.

I would say:
- Fix coprocessor register accesses on Arm to use the proper 32/64bit access size.

Bertrand

> 
> Let me know if that's not accurate.
> 
> Roger.

Re: [PATCH for-4.16 2/2] CHANGELOG: add missing entries for work during the 4.16 release cycle

[Julien Grall]

On 18/11/2021 11:14, Bertrand Marquis wrote:
> Hi Roger,
> 
>> On 18 Nov 2021, at 10:37, Roger Pau Monné <roger.pau@citrix.com> wrote:
>>
>> On Thu, Nov 18, 2021 at 10:01:08AM +0000, Bertrand Marquis wrote:
>>> Hi Roger,
>>>
>>>
>>>> On 18 Nov 2021, at 08:58, Roger Pau Monné <roger.pau@citrix.com> wrote:
>>>>
>>>> On Wed, Nov 17, 2021 at 02:07:50PM +0000, Bertrand Marquis wrote:
>>>>> Hi Roger,
>>>>>
>>>>>> On 17 Nov 2021, at 09:53, Roger Pau Monne <roger.pau@citrix.com> wrote:
>>>>>>
>>>>>> Document some of the relevant changes during the 4.16 release cycle,
>>>>>> likely more entries are missing.
>>>>>>
>>>>>> Signed-off-by: Roger Pau Monné <roger.pau@citrix.com>
>>>>>> ---
>>>>>> CHANGELOG.md | 11 +++++++++++
>>>>>> 1 file changed, 11 insertions(+)
>>>>>>
>>>>>> diff --git a/CHANGELOG.md b/CHANGELOG.md
>>>>>> index ad1a8c2bc2..8b0bdd9cf0 100644
>>>>>> --- a/CHANGELOG.md
>>>>>> +++ b/CHANGELOG.md
>>>>>> @@ -21,6 +21,17 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/)
>>>>>> - qemu-traditional based device models (both, qemu-traditional and ioemu-stubdom) will
>>>>>>   no longer be built per default. In order to be able to use those, configure needs to
>>>>>>   be called with "--enable-qemu-traditional" as parameter.
>>>>>> + - Fixes for credit2 scheduler stability in corner case conditions.
>>>>>> + - Ongoing improvements in the hypervisor build system.
>>>>>> + - vtpmmgr miscellaneous fixes in preparation for TPM 2.0 support.
>>>>>> + - 32bit PV guests only supported in shim mode.
>>>>>> + - Improved PVH dom0 debug key handling.
>>>>>> + - Fix booting on some Intel systems without a PIT (i8254).
>>>>>
>>>>> Missing:
>>>>> - cpu ID sanitization on arm64
>>>>> - fix 32/64bit vreg emulation on arm64
>>>>
>>>> Can I get a bit more information about those items? Just a pointer to
>>>> the commit messages would be helpful so that I can try to write a more
>>>> comprehensive entry (or maybe it's just me not knowing anything about
>>>> Arm that fails to understand it).
>>>
>>> I agree, the text is not quite clear, I will try to come with a better one.
>>> In the meantime here is are links to the series:
>>> https://patchwork.kernel.org/project/xen-devel/list/?series=535805&state=*
>>> https://patchwork.kernel.org/project/xen-devel/list/?series=477151&archive=both
>>
>> Thanks. I've added:
>>
>> - CPU feature leveling on arm64 platform with heterogeneous cores.
> Yes
>> - Handle register accesses as 32/64bit on Arm depending on the processor
>>    bitness.
> 
> I would say:
> - Fix coprocessor register accesses on Arm to use the proper 32/64bit access size.

I would suggest s/coprocessor/system/ because the changes was targeting 
arm64.

Cheers,

-- 
Julien Grall

Re: [PATCH for-4.16 2/2] CHANGELOG: add missing entries for work during the 4.16 release cycle

[Bertrand Marquis]

> On 18 Nov 2021, at 11:28, Julien Grall <julien@xen.org> wrote:
> 
> 
> 
> On 18/11/2021 11:14, Bertrand Marquis wrote:
>> Hi Roger,
>>> On 18 Nov 2021, at 10:37, Roger Pau Monné <roger.pau@citrix.com> wrote:
>>> 
>>> On Thu, Nov 18, 2021 at 10:01:08AM +0000, Bertrand Marquis wrote:
>>>> Hi Roger,
>>>> 
>>>> 
>>>>> On 18 Nov 2021, at 08:58, Roger Pau Monné <roger.pau@citrix.com> wrote:
>>>>> 
>>>>> On Wed, Nov 17, 2021 at 02:07:50PM +0000, Bertrand Marquis wrote:
>>>>>> Hi Roger,
>>>>>> 
>>>>>>> On 17 Nov 2021, at 09:53, Roger Pau Monne <roger.pau@citrix.com> wrote:
>>>>>>> 
>>>>>>> Document some of the relevant changes during the 4.16 release cycle,
>>>>>>> likely more entries are missing.
>>>>>>> 
>>>>>>> Signed-off-by: Roger Pau Monné <roger.pau@citrix.com>
>>>>>>> ---
>>>>>>> CHANGELOG.md | 11 +++++++++++
>>>>>>> 1 file changed, 11 insertions(+)
>>>>>>> 
>>>>>>> diff --git a/CHANGELOG.md b/CHANGELOG.md
>>>>>>> index ad1a8c2bc2..8b0bdd9cf0 100644
>>>>>>> --- a/CHANGELOG.md
>>>>>>> +++ b/CHANGELOG.md
>>>>>>> @@ -21,6 +21,17 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/)
>>>>>>> - qemu-traditional based device models (both, qemu-traditional and ioemu-stubdom) will
>>>>>>>  no longer be built per default. In order to be able to use those, configure needs to
>>>>>>>  be called with "--enable-qemu-traditional" as parameter.
>>>>>>> + - Fixes for credit2 scheduler stability in corner case conditions.
>>>>>>> + - Ongoing improvements in the hypervisor build system.
>>>>>>> + - vtpmmgr miscellaneous fixes in preparation for TPM 2.0 support.
>>>>>>> + - 32bit PV guests only supported in shim mode.
>>>>>>> + - Improved PVH dom0 debug key handling.
>>>>>>> + - Fix booting on some Intel systems without a PIT (i8254).
>>>>>> 
>>>>>> Missing:
>>>>>> - cpu ID sanitization on arm64
>>>>>> - fix 32/64bit vreg emulation on arm64
>>>>> 
>>>>> Can I get a bit more information about those items? Just a pointer to
>>>>> the commit messages would be helpful so that I can try to write a more
>>>>> comprehensive entry (or maybe it's just me not knowing anything about
>>>>> Arm that fails to understand it).
>>>> 
>>>> I agree, the text is not quite clear, I will try to come with a better one.
>>>> In the meantime here is are links to the series:
>>>> https://patchwork.kernel.org/project/xen-devel/list/?series=535805&state=*
>>>> https://patchwork.kernel.org/project/xen-devel/list/?series=477151&archive=both
>>> 
>>> Thanks. I've added:
>>> 
>>> - CPU feature leveling on arm64 platform with heterogeneous cores.
>> Yes
>>> - Handle register accesses as 32/64bit on Arm depending on the processor
>>>   bitness.
>> I would say:
>> - Fix coprocessor register accesses on Arm to use the proper 32/64bit access size.
> 
> I would suggest s/coprocessor/system/ because the changes was targeting arm64.

Agree

Cheers
Bertrand

> 
> Cheers,
> 
> -- 
> Julien Grall

Re: [PATCH for-4.16 2/2] CHANGELOG: add missing entries for work during the 4.16 release cycle

[Oleksandr Tyshchenko]

[-- Attachment #1: Type: text/plain, Size: 1692 bytes --]

On Wed, Nov 17, 2021 at 11:54 AM Roger Pau Monne <roger.pau@citrix.com>
wrote:

Hi Roger, all

[Sorry for the possible format issues]

Document some of the relevant changes during the 4.16 release cycle,
> likely more entries are missing.
>
> Signed-off-by: Roger Pau Monné <roger.pau@citrix.com>
> ---
>  CHANGELOG.md | 11 +++++++++++
>  1 file changed, 11 insertions(+)
>
> diff --git a/CHANGELOG.md b/CHANGELOG.md
> index ad1a8c2bc2..8b0bdd9cf0 100644
> --- a/CHANGELOG.md
> +++ b/CHANGELOG.md
> @@ -21,6 +21,17 @@ The format is based on [Keep a Changelog](
> https://keepachangelog.com/en/1.0.0/)
>   - qemu-traditional based device models (both, qemu-traditional and
> ioemu-stubdom) will
>     no longer be built per default. In order to be able to use those,
> configure needs to
>     be called with "--enable-qemu-traditional" as parameter.
> + - Fixes for credit2 scheduler stability in corner case conditions.
> + - Ongoing improvements in the hypervisor build system.
> + - vtpmmgr miscellaneous fixes in preparation for TPM 2.0 support.
> + - 32bit PV guests only supported in shim mode.
> + - Improved PVH dom0 debug key handling.
> + - Fix booting on some Intel systems without a PIT (i8254).
>

I would add "Various fixes for OP-TEE mediator (Arm)" here and ...



> +
> +### Added
> + - 32bit Arm builds to the automated tests.
> + - New x86 pagetable APIs.
> + - Arm vPMU support.
>

"Extended regions support, device tree only (Arm)" here.

...
The extended regions are ranges of unused address space exposed to domains
as
"safe to use" for special memory mappings.


[snip]


-- 
Regards,

Oleksandr Tyshchenko

[-- Attachment #2: Type: text/html, Size: 3053 bytes --]

Re: [PATCH for-4.16 2/2] CHANGELOG: add missing entries for work during the 4.16 release cycle

[Stefano Stabellini]

[-- Attachment #1: Type: text/plain, Size: 1451 bytes --]

On Wed, 17 Nov 2021, Roger Pau Monne wrote:
> Document some of the relevant changes during the 4.16 release cycle,
> likely more entries are missing.
> 
> Signed-off-by: Roger Pau Monné <roger.pau@citrix.com>
> ---
>  CHANGELOG.md | 11 +++++++++++
>  1 file changed, 11 insertions(+)
> 
> diff --git a/CHANGELOG.md b/CHANGELOG.md
> index ad1a8c2bc2..8b0bdd9cf0 100644
> --- a/CHANGELOG.md
> +++ b/CHANGELOG.md
> @@ -21,6 +21,17 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/)
>   - qemu-traditional based device models (both, qemu-traditional and ioemu-stubdom) will
>     no longer be built per default. In order to be able to use those, configure needs to
>     be called with "--enable-qemu-traditional" as parameter.
> + - Fixes for credit2 scheduler stability in corner case conditions.
> + - Ongoing improvements in the hypervisor build system.
> + - vtpmmgr miscellaneous fixes in preparation for TPM 2.0 support.
> + - 32bit PV guests only supported in shim mode.
> + - Improved PVH dom0 debug key handling.
> + - Fix booting on some Intel systems without a PIT (i8254).
> +
> +### Added
> + - 32bit Arm builds to the automated tests.

I think you can expand this:

- greatly improved gitlab-ci based automated tests:
    - 32-bit Arm builds
    - x86 full system tests

Or with single lines only:

- 32bit Arm builds to the gitlab-ci automated tests
- x86 full system tests to the gitlab-ci automated tests

Re: [PATCH for-4.16 2/2] CHANGELOG: add missing entries for work during the 4.16 release cycle

[Roger Pau Monné]

On Thu, Nov 18, 2021 at 06:11:07PM +0200, Oleksandr Tyshchenko wrote:
> On Wed, Nov 17, 2021 at 11:54 AM Roger Pau Monne <roger.pau@citrix.com>
> wrote:
> 
> Hi Roger, all
> 
> [Sorry for the possible format issues]
> 
> Document some of the relevant changes during the 4.16 release cycle,
> > likely more entries are missing.
> >
> > Signed-off-by: Roger Pau Monné <roger.pau@citrix.com>
> > ---
> >  CHANGELOG.md | 11 +++++++++++
> >  1 file changed, 11 insertions(+)
> >
> > diff --git a/CHANGELOG.md b/CHANGELOG.md
> > index ad1a8c2bc2..8b0bdd9cf0 100644
> > --- a/CHANGELOG.md
> > +++ b/CHANGELOG.md
> > @@ -21,6 +21,17 @@ The format is based on [Keep a Changelog](
> > https://keepachangelog.com/en/1.0.0/)
> >   - qemu-traditional based device models (both, qemu-traditional and
> > ioemu-stubdom) will
> >     no longer be built per default. In order to be able to use those,
> > configure needs to
> >     be called with "--enable-qemu-traditional" as parameter.
> > + - Fixes for credit2 scheduler stability in corner case conditions.
> > + - Ongoing improvements in the hypervisor build system.
> > + - vtpmmgr miscellaneous fixes in preparation for TPM 2.0 support.
> > + - 32bit PV guests only supported in shim mode.
> > + - Improved PVH dom0 debug key handling.
> > + - Fix booting on some Intel systems without a PIT (i8254).
> >
> 
> I would add "Various fixes for OP-TEE mediator (Arm)" here and ...
> 
> 
> 
> > +
> > +### Added
> > + - 32bit Arm builds to the automated tests.
> > + - New x86 pagetable APIs.
> > + - Arm vPMU support.
> >
> 
> "Extended regions support, device tree only (Arm)" here.
> 
> ...
> The extended regions are ranges of unused address space exposed to domains
> as
> "safe to use" for special memory mappings.

I've worded this as:

"Report unpopulated memory regions safe to use for foreign mappings,
Arm and device tree only."

As "extended regions" was IMO too vague. Let me know if that's OK.

Thanks, Roger.

Re: [PATCH for-4.16 2/2] CHANGELOG: add missing entries for work during the 4.16 release cycle

[Roger Pau Monné]

On Thu, Nov 18, 2021 at 09:23:40AM -0800, Stefano Stabellini wrote:
> On Wed, 17 Nov 2021, Roger Pau Monne wrote:
> > Document some of the relevant changes during the 4.16 release cycle,
> > likely more entries are missing.
> > 
> > Signed-off-by: Roger Pau Monné <roger.pau@citrix.com>
> > ---
> >  CHANGELOG.md | 11 +++++++++++
> >  1 file changed, 11 insertions(+)
> > 
> > diff --git a/CHANGELOG.md b/CHANGELOG.md
> > index ad1a8c2bc2..8b0bdd9cf0 100644
> > --- a/CHANGELOG.md
> > +++ b/CHANGELOG.md
> > @@ -21,6 +21,17 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/)
> >   - qemu-traditional based device models (both, qemu-traditional and ioemu-stubdom) will
> >     no longer be built per default. In order to be able to use those, configure needs to
> >     be called with "--enable-qemu-traditional" as parameter.
> > + - Fixes for credit2 scheduler stability in corner case conditions.
> > + - Ongoing improvements in the hypervisor build system.
> > + - vtpmmgr miscellaneous fixes in preparation for TPM 2.0 support.
> > + - 32bit PV guests only supported in shim mode.
> > + - Improved PVH dom0 debug key handling.
> > + - Fix booting on some Intel systems without a PIT (i8254).
> > +
> > +### Added
> > + - 32bit Arm builds to the automated tests.
> 
> I think you can expand this:
> 
> - greatly improved gitlab-ci based automated tests:
>     - 32-bit Arm builds
>     - x86 full system tests
> 
> Or with single lines only:
> 
> - 32bit Arm builds to the gitlab-ci automated tests
> - x86 full system tests to the gitlab-ci automated tests

Added, thanks.

Roger.

Re: [PATCH for-4.16 2/2] CHANGELOG: add missing entries for work during the 4.16 release cycle

[Oleksandr]

On 18.11.21 19:33, Roger Pau Monné wrote:

Hi Roger


> On Thu, Nov 18, 2021 at 06:11:07PM +0200, Oleksandr Tyshchenko wrote:
>> On Wed, Nov 17, 2021 at 11:54 AM Roger Pau Monne <roger.pau@citrix.com>
>> wrote:
>>
>> Hi Roger, all
>>
>> [Sorry for the possible format issues]
>>
>> Document some of the relevant changes during the 4.16 release cycle,
>>> likely more entries are missing.
>>>
>>> Signed-off-by: Roger Pau Monné <roger.pau@citrix.com>
>>> ---
>>>   CHANGELOG.md | 11 +++++++++++
>>>   1 file changed, 11 insertions(+)
>>>
>>> diff --git a/CHANGELOG.md b/CHANGELOG.md
>>> index ad1a8c2bc2..8b0bdd9cf0 100644
>>> --- a/CHANGELOG.md
>>> +++ b/CHANGELOG.md
>>> @@ -21,6 +21,17 @@ The format is based on [Keep a Changelog](
>>> https://keepachangelog.com/en/1.0.0/)
>>>    - qemu-traditional based device models (both, qemu-traditional and
>>> ioemu-stubdom) will
>>>      no longer be built per default. In order to be able to use those,
>>> configure needs to
>>>      be called with "--enable-qemu-traditional" as parameter.
>>> + - Fixes for credit2 scheduler stability in corner case conditions.
>>> + - Ongoing improvements in the hypervisor build system.
>>> + - vtpmmgr miscellaneous fixes in preparation for TPM 2.0 support.
>>> + - 32bit PV guests only supported in shim mode.
>>> + - Improved PVH dom0 debug key handling.
>>> + - Fix booting on some Intel systems without a PIT (i8254).
>>>
>> I would add "Various fixes for OP-TEE mediator (Arm)" here and ...
>>
>>
>>
>>> +
>>> +### Added
>>> + - 32bit Arm builds to the automated tests.
>>> + - New x86 pagetable APIs.
>>> + - Arm vPMU support.
>>>
>> "Extended regions support, device tree only (Arm)" here.
>>
>> ...
>> The extended regions are ranges of unused address space exposed to domains
>> as
>> "safe to use" for special memory mappings.
> I've worded this as:
>
> "Report unpopulated memory regions safe to use for foreign mappings,
> Arm and device tree only."
>
> As "extended regions" was IMO too vague. Let me know if that's OK.

I think, it is OK. Nit: maybe replace "foreign" with "foreign/grant"? I 
would be OK either way.

Thank you.


>
> Thanks, Roger.

-- 
Regards,

Oleksandr Tyshchenko

Re: [PATCH for-4.16 2/2] CHANGELOG: add missing entries for work during the 4.16 release cycle

[Roger Pau Monné]

On Thu, Nov 18, 2021 at 09:04:30PM +0200, Oleksandr wrote:
> 
> On 18.11.21 19:33, Roger Pau Monné wrote:
> 
> Hi Roger
> 
> 
> > On Thu, Nov 18, 2021 at 06:11:07PM +0200, Oleksandr Tyshchenko wrote:
> > > On Wed, Nov 17, 2021 at 11:54 AM Roger Pau Monne <roger.pau@citrix.com>
> > > wrote:
> > > 
> > > Hi Roger, all
> > > 
> > > [Sorry for the possible format issues]
> > > 
> > > Document some of the relevant changes during the 4.16 release cycle,
> > > > likely more entries are missing.
> > > > 
> > > > Signed-off-by: Roger Pau Monné <roger.pau@citrix.com>
> > > > ---
> > > >   CHANGELOG.md | 11 +++++++++++
> > > >   1 file changed, 11 insertions(+)
> > > > 
> > > > diff --git a/CHANGELOG.md b/CHANGELOG.md
> > > > index ad1a8c2bc2..8b0bdd9cf0 100644
> > > > --- a/CHANGELOG.md
> > > > +++ b/CHANGELOG.md
> > > > @@ -21,6 +21,17 @@ The format is based on [Keep a Changelog](
> > > > https://keepachangelog.com/en/1.0.0/)
> > > >    - qemu-traditional based device models (both, qemu-traditional and
> > > > ioemu-stubdom) will
> > > >      no longer be built per default. In order to be able to use those,
> > > > configure needs to
> > > >      be called with "--enable-qemu-traditional" as parameter.
> > > > + - Fixes for credit2 scheduler stability in corner case conditions.
> > > > + - Ongoing improvements in the hypervisor build system.
> > > > + - vtpmmgr miscellaneous fixes in preparation for TPM 2.0 support.
> > > > + - 32bit PV guests only supported in shim mode.
> > > > + - Improved PVH dom0 debug key handling.
> > > > + - Fix booting on some Intel systems without a PIT (i8254).
> > > > 
> > > I would add "Various fixes for OP-TEE mediator (Arm)" here and ...
> > > 
> > > 
> > > 
> > > > +
> > > > +### Added
> > > > + - 32bit Arm builds to the automated tests.
> > > > + - New x86 pagetable APIs.
> > > > + - Arm vPMU support.
> > > > 
> > > "Extended regions support, device tree only (Arm)" here.
> > > 
> > > ...
> > > The extended regions are ranges of unused address space exposed to domains
> > > as
> > > "safe to use" for special memory mappings.
> > I've worded this as:
> > 
> > "Report unpopulated memory regions safe to use for foreign mappings,
> > Arm and device tree only."
> > 
> > As "extended regions" was IMO too vague. Let me know if that's OK.
> 
> I think, it is OK. Nit: maybe replace "foreign" with "foreign/grant"? I
> would be OK either way.

Maybe, I would consider grants as foreign mappings also, ie: the
memory is foreign to the domain, but I can see this being confusing as
we have a specific kind of mappings that are named foreign.

Does replacing foreign with external seem better?

Thanks, Roger.

Re: [PATCH for-4.16 2/2] CHANGELOG: add missing entries for work during the 4.16 release cycle

[Oleksandr]

On 19.11.21 09:59, Roger Pau Monné wrote:

Hi Roger, all


> On Thu, Nov 18, 2021 at 09:04:30PM +0200, Oleksandr wrote:
>> On 18.11.21 19:33, Roger Pau Monné wrote:
>>
>> Hi Roger
>>
>>
>>> On Thu, Nov 18, 2021 at 06:11:07PM +0200, Oleksandr Tyshchenko wrote:
>>>> On Wed, Nov 17, 2021 at 11:54 AM Roger Pau Monne <roger.pau@citrix.com>
>>>> wrote:
>>>>
>>>> Hi Roger, all
>>>>
>>>> [Sorry for the possible format issues]
>>>>
>>>> Document some of the relevant changes during the 4.16 release cycle,
>>>>> likely more entries are missing.
>>>>>
>>>>> Signed-off-by: Roger Pau Monné <roger.pau@citrix.com>
>>>>> ---
>>>>>    CHANGELOG.md | 11 +++++++++++
>>>>>    1 file changed, 11 insertions(+)
>>>>>
>>>>> diff --git a/CHANGELOG.md b/CHANGELOG.md
>>>>> index ad1a8c2bc2..8b0bdd9cf0 100644
>>>>> --- a/CHANGELOG.md
>>>>> +++ b/CHANGELOG.md
>>>>> @@ -21,6 +21,17 @@ The format is based on [Keep a Changelog](
>>>>> https://keepachangelog.com/en/1.0.0/)
>>>>>     - qemu-traditional based device models (both, qemu-traditional and
>>>>> ioemu-stubdom) will
>>>>>       no longer be built per default. In order to be able to use those,
>>>>> configure needs to
>>>>>       be called with "--enable-qemu-traditional" as parameter.
>>>>> + - Fixes for credit2 scheduler stability in corner case conditions.
>>>>> + - Ongoing improvements in the hypervisor build system.
>>>>> + - vtpmmgr miscellaneous fixes in preparation for TPM 2.0 support.
>>>>> + - 32bit PV guests only supported in shim mode.
>>>>> + - Improved PVH dom0 debug key handling.
>>>>> + - Fix booting on some Intel systems without a PIT (i8254).
>>>>>
>>>> I would add "Various fixes for OP-TEE mediator (Arm)" here and ...
>>>>
>>>>
>>>>
>>>>> +
>>>>> +### Added
>>>>> + - 32bit Arm builds to the automated tests.
>>>>> + - New x86 pagetable APIs.
>>>>> + - Arm vPMU support.
>>>>>
>>>> "Extended regions support, device tree only (Arm)" here.
>>>>
>>>> ...
>>>> The extended regions are ranges of unused address space exposed to domains
>>>> as
>>>> "safe to use" for special memory mappings.
>>> I've worded this as:
>>>
>>> "Report unpopulated memory regions safe to use for foreign mappings,
>>> Arm and device tree only."
>>>
>>> As "extended regions" was IMO too vague. Let me know if that's OK.
>> I think, it is OK. Nit: maybe replace "foreign" with "foreign/grant"? I
>> would be OK either way.
> Maybe, I would consider grants as foreign mappings also, ie: the
> memory is foreign to the domain, but I can see this being confusing as
> we have a specific kind of mappings that are named foreign.
>
> Does replacing foreign with external seem better?

Fine with me, thank you.


BTW, I noticed that "Support of generic DT IOMMU bindings for SMMU V2 
(Arm)" is not mentioned. Looks like the support has reached upstream 
during the 4.16 release cycle, so worth adding here.

@Julien, @Stefano?





>
> Thanks, Roger.

-- 
Regards,

Oleksandr Tyshchenko

Re: [PATCH for-4.16 2/2] CHANGELOG: add missing entries for work during the 4.16 release cycle

[Julien Grall]

Hi Oleksandr,

On 19/11/2021 09:34, Oleksandr wrote:
> 
> On 19.11.21 09:59, Roger Pau Monné wrote:
> 
> Hi Roger, all
> 
> 
>> On Thu, Nov 18, 2021 at 09:04:30PM +0200, Oleksandr wrote:
>>> On 18.11.21 19:33, Roger Pau Monné wrote:
>>>
>>> Hi Roger
>>>
>>>
>>>> On Thu, Nov 18, 2021 at 06:11:07PM +0200, Oleksandr Tyshchenko wrote:
>>>>> On Wed, Nov 17, 2021 at 11:54 AM Roger Pau Monne 
>>>>> <roger.pau@citrix.com>
>>>>> wrote:
>>>>>
>>>>> Hi Roger, all
>>>>>
>>>>> [Sorry for the possible format issues]
>>>>>
>>>>> Document some of the relevant changes during the 4.16 release cycle,
>>>>>> likely more entries are missing.
>>>>>>
>>>>>> Signed-off-by: Roger Pau Monné <roger.pau@citrix.com>
>>>>>> ---
>>>>>>    CHANGELOG.md | 11 +++++++++++
>>>>>>    1 file changed, 11 insertions(+)
>>>>>>
>>>>>> diff --git a/CHANGELOG.md b/CHANGELOG.md
>>>>>> index ad1a8c2bc2..8b0bdd9cf0 100644
>>>>>> --- a/CHANGELOG.md
>>>>>> +++ b/CHANGELOG.md
>>>>>> @@ -21,6 +21,17 @@ The format is based on [Keep a Changelog](
>>>>>> https://keepachangelog.com/en/1.0.0/)
>>>>>>     - qemu-traditional based device models (both, qemu-traditional 
>>>>>> and
>>>>>> ioemu-stubdom) will
>>>>>>       no longer be built per default. In order to be able to use 
>>>>>> those,
>>>>>> configure needs to
>>>>>>       be called with "--enable-qemu-traditional" as parameter.
>>>>>> + - Fixes for credit2 scheduler stability in corner case conditions.
>>>>>> + - Ongoing improvements in the hypervisor build system.
>>>>>> + - vtpmmgr miscellaneous fixes in preparation for TPM 2.0 support.
>>>>>> + - 32bit PV guests only supported in shim mode.
>>>>>> + - Improved PVH dom0 debug key handling.
>>>>>> + - Fix booting on some Intel systems without a PIT (i8254).
>>>>>>
>>>>> I would add "Various fixes for OP-TEE mediator (Arm)" here and ...
>>>>>
>>>>>
>>>>>
>>>>>> +
>>>>>> +### Added
>>>>>> + - 32bit Arm builds to the automated tests.
>>>>>> + - New x86 pagetable APIs.
>>>>>> + - Arm vPMU support.
>>>>>>
>>>>> "Extended regions support, device tree only (Arm)" here.
>>>>>
>>>>> ...
>>>>> The extended regions are ranges of unused address space exposed to 
>>>>> domains
>>>>> as
>>>>> "safe to use" for special memory mappings.
>>>> I've worded this as:
>>>>
>>>> "Report unpopulated memory regions safe to use for foreign mappings,
>>>> Arm and device tree only."
>>>>
>>>> As "extended regions" was IMO too vague. Let me know if that's OK.
>>> I think, it is OK. Nit: maybe replace "foreign" with "foreign/grant"? I
>>> would be OK either way.
>> Maybe, I would consider grants as foreign mappings also, ie: the
>> memory is foreign to the domain, but I can see this being confusing as
>> we have a specific kind of mappings that are named foreign.
>>
>> Does replacing foreign with external seem better?
> 
> Fine with me, thank you.
> 
> 
> BTW, I noticed that "Support of generic DT IOMMU bindings for SMMU V2 
> (Arm)" is not mentioned. Looks like the support has reached upstream 
> during the 4.16 release cycle, so worth adding here.

You are right.

> 
> @Julien, @Stefano?

I agree we should mention it.

Cheers,

-- 
Julien Grall

Re: [PATCH for-4.16 2/2] CHANGELOG: add missing entries for work during the 4.16 release cycle

[Bertrand Marquis]

Hi,

> On 19 Nov 2021, at 09:34, Oleksandr <olekstysh@gmail.com> wrote:
> 
> 
> On 19.11.21 09:59, Roger Pau Monné wrote:
> 
> Hi Roger, all
> 
> 
>> On Thu, Nov 18, 2021 at 09:04:30PM +0200, Oleksandr wrote:
>>> On 18.11.21 19:33, Roger Pau Monné wrote:
>>> 
>>> Hi Roger
>>> 
>>> 
>>>> On Thu, Nov 18, 2021 at 06:11:07PM +0200, Oleksandr Tyshchenko wrote:
>>>>> On Wed, Nov 17, 2021 at 11:54 AM Roger Pau Monne <roger.pau@citrix.com>
>>>>> wrote:
>>>>> 
>>>>> Hi Roger, all
>>>>> 
>>>>> [Sorry for the possible format issues]
>>>>> 
>>>>> Document some of the relevant changes during the 4.16 release cycle,
>>>>>> likely more entries are missing.
>>>>>> 
>>>>>> Signed-off-by: Roger Pau Monné <roger.pau@citrix.com>
>>>>>> ---
>>>>>>   CHANGELOG.md | 11 +++++++++++
>>>>>>   1 file changed, 11 insertions(+)
>>>>>> 
>>>>>> diff --git a/CHANGELOG.md b/CHANGELOG.md
>>>>>> index ad1a8c2bc2..8b0bdd9cf0 100644
>>>>>> --- a/CHANGELOG.md
>>>>>> +++ b/CHANGELOG.md
>>>>>> @@ -21,6 +21,17 @@ The format is based on [Keep a Changelog](
>>>>>> https://keepachangelog.com/en/1.0.0/)
>>>>>>    - qemu-traditional based device models (both, qemu-traditional and
>>>>>> ioemu-stubdom) will
>>>>>>      no longer be built per default. In order to be able to use those,
>>>>>> configure needs to
>>>>>>      be called with "--enable-qemu-traditional" as parameter.
>>>>>> + - Fixes for credit2 scheduler stability in corner case conditions.
>>>>>> + - Ongoing improvements in the hypervisor build system.
>>>>>> + - vtpmmgr miscellaneous fixes in preparation for TPM 2.0 support.
>>>>>> + - 32bit PV guests only supported in shim mode.
>>>>>> + - Improved PVH dom0 debug key handling.
>>>>>> + - Fix booting on some Intel systems without a PIT (i8254).
>>>>>> 
>>>>> I would add "Various fixes for OP-TEE mediator (Arm)" here and ...
>>>>> 
>>>>> 
>>>>> 
>>>>>> +
>>>>>> +### Added
>>>>>> + - 32bit Arm builds to the automated tests.
>>>>>> + - New x86 pagetable APIs.
>>>>>> + - Arm vPMU support.
>>>>>> 
>>>>> "Extended regions support, device tree only (Arm)" here.
>>>>> 
>>>>> ...
>>>>> The extended regions are ranges of unused address space exposed to domains
>>>>> as
>>>>> "safe to use" for special memory mappings.
>>>> I've worded this as:
>>>> 
>>>> "Report unpopulated memory regions safe to use for foreign mappings,
>>>> Arm and device tree only."
>>>> 
>>>> As "extended regions" was IMO too vague. Let me know if that's OK.
>>> I think, it is OK. Nit: maybe replace "foreign" with "foreign/grant"? I
>>> would be OK either way.
>> Maybe, I would consider grants as foreign mappings also, ie: the
>> memory is foreign to the domain, but I can see this being confusing as
>> we have a specific kind of mappings that are named foreign.
>> 
>> Does replacing foreign with external seem better?
> 
> Fine with me, thank you.
> 
> 
> BTW, I noticed that "Support of generic DT IOMMU bindings for SMMU V2 (Arm)" is not mentioned. Looks like the support has reached upstream during the 4.16 release cycle, so worth adding here.

Yes you are right this should be added.

Cheers
Bertrand

> 
> @Julien, @Stefano?
> 
> 
> 
> 
> 
>> 
>> Thanks, Roger.
> 
> -- 
> Regards,
> 
> Oleksandr Tyshchenko