[PATCH] NEWS: Document my contributions for upcoming release
Signed-off-by: Michal Privoznik <mprivozn@redhat.com> --- NEWS.rst | 17 +++++++++++++++++ 1 file changed, 17 insertions(+) diff --git a/NEWS.rst b/NEWS.rst index 1de8314a61..0dc909c849 100644 --- a/NEWS.rst +++ b/NEWS.rst @@ -33,8 +33,25 @@ v9.6.0 (unreleased) ``/disk/target@bus='scsi'``) supports the ``removable`` attribute at ``/disk/target@removable```. + * qemu: Add NUMA node automatically for memory hotplug + + When enabling memory hotplug, libvirt required at least one guest NUMA to + be specified in the domain XML. With this release, libvirt adds it + automatically when needed. + * **Bug fixes** + * qemu: Adapt to new way of specifying PC speaker + + Because the way PC speaker is specified on QEMU command line changed, + libvirt was unable to start a domain with PC speaker. This is now fixed. + + * qemu_tpm: Try harder to create emulator state + + If a per-domain SWTPM state directory exists but is empty libvirt would + consider it a valid state and skipped running 'swtpm_setup'. This is now + fixed. + v9.5.0 (2023-07-03) =================== -- 2.41.0
On Fri, Jul 28, 2023 at 04:40:50PM +0200, Michal Privoznik wrote:
Signed-off-by: Michal Privoznik <mprivozn@redhat.com> --- NEWS.rst | 17 +++++++++++++++++ 1 file changed, 17 insertions(+)
diff --git a/NEWS.rst b/NEWS.rst index 1de8314a61..0dc909c849 100644 --- a/NEWS.rst +++ b/NEWS.rst @@ -33,8 +33,25 @@ v9.6.0 (unreleased) ``/disk/target@bus='scsi'``) supports the ``removable`` attribute at ``/disk/target@removable```.
+ * qemu: Add NUMA node automatically for memory hotplug + + When enabling memory hotplug, libvirt required at least one guest NUMA to + be specified in the domain XML. With this release, libvirt adds it + automatically when needed. +
I think it sounds better if we say what libvirt does since this release rather than what used to happen before. I would suggest something like: Users no longer need to specify guest NUMA node in the domain XML during hotplug, libvirt automatically adds one when it is missing.
* **Bug fixes**
+ * qemu: Adapt to new way of specifying PC speaker + + Because the way PC speaker is specified on QEMU command line changed, + libvirt was unable to start a domain with PC speaker. This is now fixed. +
PC speaker is now usable again with newer QEMU since the change of how it is specified on the command line.
+ * qemu_tpm: Try harder to create emulator state + + If a per-domain SWTPM state directory exists but is empty libvirt would + consider it a valid state and skipped running 'swtpm_setup'. This is now + fixed. +
Libvirt no longer considers empty directory valid SWTPM state and setup is now run properly in such case. What do you think?
v9.5.0 (2023-07-03) =================== -- 2.41.0
On 7/31/23 09:29, Martin Kletzander wrote:
On Fri, Jul 28, 2023 at 04:40:50PM +0200, Michal Privoznik wrote:
Signed-off-by: Michal Privoznik <mprivozn@redhat.com> --- NEWS.rst | 17 +++++++++++++++++ 1 file changed, 17 insertions(+)
diff --git a/NEWS.rst b/NEWS.rst index 1de8314a61..0dc909c849 100644 --- a/NEWS.rst +++ b/NEWS.rst @@ -33,8 +33,25 @@ v9.6.0 (unreleased) ``/disk/target@bus='scsi'``) supports the ``removable`` attribute at ``/disk/target@removable```.
+ * qemu: Add NUMA node automatically for memory hotplug + + When enabling memory hotplug, libvirt required at least one guest NUMA to + be specified in the domain XML. With this release, libvirt adds it + automatically when needed. +
I think it sounds better if we say what libvirt does since this release rather than what used to happen before. I would suggest something like:
Users no longer need to specify guest NUMA node in the domain XML during hotplug, libvirt automatically adds one when it is missing.
Good point. But we need better phrasing, because "memory hotplug" is basically two things: enabling it in domain XML (e.g. by specifying <maxMemory/> or <maxMemory slots=''/> - as examined by virDomainDefHasMemoryHotplug()), and then there's actual hotplug whilst the domain is running. The NUMA node is added only for the first part. If domain is running we can not add a NUMA node, obviously. Let me fix that in v2, just like the rest of your (valuable) comments. Thanks! Michal
participants (3)
-
Martin Kletzander -
Michal Privoznik -
Michal Prívozník