[PATCH 0/5] remove NVDIMM auto-alignment for pSeries guests

Daniel Henrique Barboza posted 5 patches 1 week ago
Test syntax-check failed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/20200730194804.491732-1-danielhb413@gmail.com
docs/formatdomain.html.in                     |  6 +-
src/qemu/qemu_domain.c                        | 57 ++-----------------
src/qemu/qemu_domain.h                        |  7 ++-
src/qemu/qemu_hotplug.c                       |  6 +-
src/qemu/qemu_validate.c                      | 43 ++++++++++++--
...ory-hotplug-nvdimm-ppc64.ppc64-latest.args |  2 +-
.../memory-hotplug-nvdimm-ppc64.xml           |  2 +-
.../memory-hotplug-nvdimm-ppc64.xml           |  2 +-
8 files changed, 55 insertions(+), 70 deletions(-)

[PATCH 0/5] remove NVDIMM auto-alignment for pSeries guests

Posted by Daniel Henrique Barboza 1 week ago
Hi,

In [1] Andrea proposed that we should *not* auto-align down
the NVDIMM size for pSeries guests. Instead we should error
out and provide users with a suggested aligned value. This
patch implements it.

[1] https://www.redhat.com/archives/libvir-list/2020-July/msg01471.html


Daniel Henrique Barboza (5):
  qemu_domain.c: make qemuDomainGetMemorySizeAlignment() public
  qemu_validate.c: add pSeries NVDIMM size alignment validation
  qemu_domain.c: do not auto-align ppc64 NVDIMMs
  qemu_domain.c: change qemuDomainMemoryDeviceAlignSize() return type
  Revert "formatdomain.html.in: mention pSeries NVDIMM 'align down'
    mechanic"

 docs/formatdomain.html.in                     |  6 +-
 src/qemu/qemu_domain.c                        | 57 ++-----------------
 src/qemu/qemu_domain.h                        |  7 ++-
 src/qemu/qemu_hotplug.c                       |  6 +-
 src/qemu/qemu_validate.c                      | 43 ++++++++++++--
 ...ory-hotplug-nvdimm-ppc64.ppc64-latest.args |  2 +-
 .../memory-hotplug-nvdimm-ppc64.xml           |  2 +-
 .../memory-hotplug-nvdimm-ppc64.xml           |  2 +-
 8 files changed, 55 insertions(+), 70 deletions(-)

-- 
2.26.2

Re: [PATCH 0/5] remove NVDIMM auto-alignment for pSeries guests

Posted by Daniel Henrique Barboza 1 day ago
Ping

On 7/30/20 4:47 PM, Daniel Henrique Barboza wrote:
> Hi,
> 
> In [1] Andrea proposed that we should *not* auto-align down
> the NVDIMM size for pSeries guests. Instead we should error
> out and provide users with a suggested aligned value. This
> patch implements it.
> 
> [1] https://www.redhat.com/archives/libvir-list/2020-July/msg01471.html
> 
> 
> Daniel Henrique Barboza (5):
>    qemu_domain.c: make qemuDomainGetMemorySizeAlignment() public
>    qemu_validate.c: add pSeries NVDIMM size alignment validation
>    qemu_domain.c: do not auto-align ppc64 NVDIMMs
>    qemu_domain.c: change qemuDomainMemoryDeviceAlignSize() return type
>    Revert "formatdomain.html.in: mention pSeries NVDIMM 'align down'
>      mechanic"
> 
>   docs/formatdomain.html.in                     |  6 +-
>   src/qemu/qemu_domain.c                        | 57 ++-----------------
>   src/qemu/qemu_domain.h                        |  7 ++-
>   src/qemu/qemu_hotplug.c                       |  6 +-
>   src/qemu/qemu_validate.c                      | 43 ++++++++++++--
>   ...ory-hotplug-nvdimm-ppc64.ppc64-latest.args |  2 +-
>   .../memory-hotplug-nvdimm-ppc64.xml           |  2 +-
>   .../memory-hotplug-nvdimm-ppc64.xml           |  2 +-
>   8 files changed, 55 insertions(+), 70 deletions(-)
>