[PATCH] NEWS: Mention stuff I've dealt with for 7.0.0

Michal Privoznik posted 1 patch 1 week, 5 days ago
Test syntax-check failed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/21579d970d1d4851436903f08db06374fd853b09.1610529801.git.mprivozn@redhat.com
NEWS.rst | 47 +++++++++++++++++++++++++++++++++++++++++++++++
1 file changed, 47 insertions(+)

[PATCH] NEWS: Mention stuff I've dealt with for 7.0.0

Posted by Michal Privoznik 1 week, 5 days ago
Signed-off-by: Michal Privoznik <mprivozn@redhat.com>
---
 NEWS.rst | 47 +++++++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 47 insertions(+)

diff --git a/NEWS.rst b/NEWS.rst
index 07cec19c2b..b9a3d94eb2 100644
--- a/NEWS.rst
+++ b/NEWS.rst
@@ -48,6 +48,28 @@ v7.0.0 (unreleased)
     an usage name which would not pass the XML validation. Relax the requirement
     to make such XMLs valid.
 
+  * virnetdevopenvswitch: Various improvements
+
+    The code that handles ``<interface type='vhostuser'/>`` was given various
+    improvements. So far, libvirt assumed vhostuser interfaces are handled
+    exclusively by OpenVSwitch and refused to start a guest if it was not so.
+    Now a guest can be started successfully even if the interface is created by
+    some other tool (e.g. ``dpdk-testpmd``). Also, the code that detects the
+    interface name was adopted to new versions of OpenVSwitch and thus can
+    detect name more reliably.
+
+  * qemu: Report guest disks informations in ``virDomainGetGuestInfo``
+
+    Libvirt is now able to report disks and filesystems from the guest's
+    perspective (using guest agent). And with sufficiently new guest agent
+    (5.3.0 or newer) the API also handles disks on CCW bus.
+
+  * conf: Add support for keeping TPM emulator state
+
+    Currently, swtpm TPM state file is removed when a transient domain is
+    powered off or undefined. Add per-TPM emulator option ``persistent_sate``
+    for keeping TPM state.
+
 * **Bug fixes**
 
   * qemu: Fix logic bug in inactive snapshot deletion
@@ -69,6 +91,31 @@ v7.0.0 (unreleased)
     amount of initial memory. Existing guests that migrate from an older
     libvirt version to 7.0.0 will not be affected by this change.
 
+  * qemu: Don't cache NUMA caps
+
+    ``virsh capabilities`` contains ``<topology/>`` section which reports NUMA
+    topology among with amount of free hugepages per each NUMA node. However,
+    these amounts were not updated between calls.
+
+  * networkGetDHCPLeases: Handle leases with infinite expiry time
+
+    Since libvirt-6.3.0 it is possible to configure expiry time for DHCP
+    leases. If the expiry time was infinite then ``virsh net-dhcp-leases``
+    and NSS plugins refused to work.
+
+  * qemu: Don't prealloc mem for real NVDIMMs
+
+    If a real life NVDIMM is assigned to a guest via ``<memory model='nvdimm'/>``
+    then QEMU is no longer instructed to preallocate memory
+    for it. This prevents unnecessary wear off of the NVDIMM.
+
+  * network: Introduce mutex for bridge name generation
+
+    When new libvirt network is defined or created and the input XML does not
+    contain any bridge name, libvirt generates one. However, it might have
+    happened that the same name would be generated for different networks if
+    two or more networks were defined/created at once.
+
 
 v6.10.0 (2020-12-01)
 ====================
-- 
2.26.2

Re: [PATCH] NEWS: Mention stuff I've dealt with for 7.0.0

Posted by Andrea Bolognani 1 week, 5 days ago
On Wed, 2021-01-13 at 10:23 +0100, Michal Privoznik wrote:
> +  * virnetdevopenvswitch: Various improvements
> +
> +    The code that handles ``<interface type='vhostuser'/>`` was given various
> +    improvements. So far, libvirt assumed vhostuser interfaces are handled
> +    exclusively by OpenVSwitch and refused to start a guest if it was not so.
> +    Now a guest can be started successfully even if the interface is created by
> +    some other tool (e.g. ``dpdk-testpmd``). Also, the code that detects the
> +    interface name was adopted to new versions of OpenVSwitch and thus can

*adapted

> +  * conf: Add support for keeping TPM emulator state
> +
> +    Currently, swtpm TPM state file is removed when a transient domain is
> +    powered off or undefined. Add per-TPM emulator option ``persistent_sate``

*persistent_state

With these typos fixed

  Reviewed-by: Andrea Bolognani <abologna@redhat.com>

and safe for freeze.

Thanks for doing your part in keeping the release notes useful!

-- 
Andrea Bolognani / Red Hat / Virtualization

Re: [PATCH] NEWS: Mention stuff I've dealt with for 7.0.0

Posted by Ján Tomko 1 week, 5 days ago
On a Wednesday in 2021, Michal Privoznik wrote:
>Signed-off-by: Michal Privoznik <mprivozn@redhat.com>
>---
> NEWS.rst | 47 +++++++++++++++++++++++++++++++++++++++++++++++
> 1 file changed, 47 insertions(+)
>
>diff --git a/NEWS.rst b/NEWS.rst
>index 07cec19c2b..b9a3d94eb2 100644
>--- a/NEWS.rst
>+++ b/NEWS.rst
>@@ -48,6 +48,28 @@ v7.0.0 (unreleased)
>     an usage name which would not pass the XML validation. Relax the requirement
>     to make such XMLs valid.
>
>+  * virnetdevopenvswitch: Various improvements
>+
>+    The code that handles ``<interface type='vhostuser'/>`` was given various
>+    improvements. So far, libvirt assumed vhostuser interfaces are handled
>+    exclusively by OpenVSwitch and refused to start a guest if it was not so.
>+    Now a guest can be started successfully even if the interface is created by
>+    some other tool (e.g. ``dpdk-testpmd``). Also, the code that detects the
>+    interface name was adopted to new versions of OpenVSwitch and thus can

s/adopted/adapted/

>+    detect name more reliably.
>+
>+  * qemu: Report guest disks informations in ``virDomainGetGuestInfo``
>+
>+    Libvirt is now able to report disks and filesystems from the guest's
>+    perspective (using guest agent). And with sufficiently new guest agent
>+    (5.3.0 or newer) the API also handles disks on CCW bus.
>+


>+  * conf: Add support for keeping TPM emulator state
>+

Is this a new feature, maybe?

>+    Currently, swtpm TPM state file is removed when a transient domain is
>+    powered off or undefined. Add per-TPM emulator option ``persistent_sate``

Ouch, did we introduce another XML element using an underscore?

s/sate/state/

>+    for keeping TPM state.

>+
> * **Bug fixes**
>
>   * qemu: Fix logic bug in inactive snapshot deletion
>@@ -69,6 +91,31 @@ v7.0.0 (unreleased)
>     amount of initial memory. Existing guests that migrate from an older
>     libvirt version to 7.0.0 will not be affected by this change.
>
>+  * qemu: Don't cache NUMA caps
>+
>+    ``virsh capabilities`` contains ``<topology/>`` section which reports NUMA
>+    topology among with amount of free hugepages per each NUMA node. However,
>+    these amounts were not updated between calls.
>+
>+  * networkGetDHCPLeases: Handle leases with infinite expiry time
>+
>+    Since libvirt-6.3.0 it is possible to configure expiry time for DHCP
>+    leases. If the expiry time was infinite then ``virsh net-dhcp-leases``
>+    and NSS plugins refused to work.
>+
>+  * qemu: Don't prealloc mem for real NVDIMMs
>+
>+    If a real life NVDIMM is assigned to a guest via ``<memory model='nvdimm'/>``
>+    then QEMU is no longer instructed to preallocate memory
>+    for it. This prevents unnecessary wear off of the NVDIMM.

s/wear off of/wear on/

>+
>+  * network: Introduce mutex for bridge name generation
>+
>+    When new libvirt network is defined or created and the input XML does not
>+    contain any bridge name, libvirt generates one. However, it might have
>+    happened that the same name would be generated for different networks if
>+    two or more networks were defined/created at once.
>+
>

Reviewed-by: Ján Tomko <jtomko@redhat.com>

Jano

Re: [PATCH] NEWS: Mention stuff I've dealt with for 7.0.0

Posted by Michal Privoznik 1 week, 5 days ago
On 1/13/21 10:50 AM, Ján Tomko wrote:
> On a Wednesday in 2021, Michal Privoznik wrote:
>> Signed-off-by: Michal Privoznik <mprivozn@redhat.com>
>> ---
>> NEWS.rst | 47 +++++++++++++++++++++++++++++++++++++++++++++++
>> 1 file changed, 47 insertions(+)
>>

>> +  * conf: Add support for keeping TPM emulator state
>> +
> 
> Is this a new feature, maybe?
> 
>> +    Currently, swtpm TPM state file is removed when a transient 
>> domain is
>> +    powered off or undefined. Add per-TPM emulator option 
>> ``persistent_sate``
> 
> Ouch, did we introduce another XML element using an underscore?

Indeed we did. I thought that what we don't like is 'persistent-state' 
but allow 'persistent_state' or 'persistentState'. Do we have preference 
over the two? If so, I can still fix it before it's released.

Michal