[PATCH v1 0/4] revert latest PPC64 NVDIMM changes

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi,

In [1], Daniel pointed out that the changes made by [2]
are uncompliant with the Libvirt design of not doing
anything that can compromise guests in the wild.

This series aims to solve that by reverting the changes
made by [2]. It also enhances the auto-alignment of ppc64
NVDIMM in a way that we can have consistency between the sizes
reported in the domain XML and what QEMU will see, while not
hurting any running guests that are already using the auto-alignment
feature.

For simplicity, first patch contains the revert of all relevant
patches from [2] in a single commit.


[1] https://www.redhat.com/archives/libvir-list/2020-September/msg00572.html
[2] https://www.redhat.com/archives/libvir-list/2020-July/msg02010.html


Daniel Henrique Barboza (4):
  qemu: revert latest pSeries NVDIMM design changes
  conf, qemu: move qemuDomainNVDimmAlignSizePseries to domain_conf.c
  domain_conf.c: auto-align pSeries NVDIMM in
    virDomainMemoryDefParseXML()
  NEWS.rst: document the pSeries NVDIMM auto-alignment revival

 NEWS.rst                                      | 12 ++++
 docs/formatdomain.rst                         |  6 +-
 src/conf/domain_conf.c                        | 71 +++++++++++++++----
 src/conf/domain_conf.h                        |  3 +
 src/libvirt_private.syms                      |  1 +
 src/qemu/qemu_domain.c                        | 19 +++--
 src/qemu/qemu_domain.h                        |  6 +-
 src/qemu/qemu_hotplug.c                       |  6 +-
 src/qemu/qemu_validate.c                      | 42 ++---------
 ...ory-hotplug-nvdimm-ppc64.ppc64-latest.args |  2 +-
 .../memory-hotplug-nvdimm-ppc64.xml           |  2 +-
 .../memory-hotplug-nvdimm-ppc64.xml           |  2 +-
 12 files changed, 105 insertions(+), 67 deletions(-)

-- 
2.26.2




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux