[PATCH 0/4] Couple of virnuma cleanups

Michal Privoznik posted 4 patches 2 years, 11 months ago
Test syntax-check failed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/cover.1621255232.git.mprivozn@redhat.com
src/conf/capabilities.c  |   9 ++--
src/conf/numa_conf.c     | 107 ++++++++++++++++++---------------------
src/libvirt_private.syms |   1 +
src/util/virnuma.h       |   2 +-
tests/virnumamock.c      |   7 ++-
5 files changed, 61 insertions(+), 65 deletions(-)
[PATCH 0/4] Couple of virnuma cleanups
Posted by Michal Privoznik 2 years, 11 months ago
I'm looking into NUMA code in capabilities and came up with a couple of
cleanups. Technically, 3/4 is not needed yet, but I'll be introducing
new data to vircaps2xmltest where a NUMA node has no CPUs and that's
why the patch is needed.

Michal Prívozník (4):
  numa_conf: Use virXMLFormatElement() in virDomainNumaDefFormatXML
  virnuma: Export virNumaGetMaxCPUs properly
  virnumamock: Allow CPU-less NUMA nodes
  virCapabilitiesHostNUMAFormat: Bring variables into loops

 src/conf/capabilities.c  |   9 ++--
 src/conf/numa_conf.c     | 107 ++++++++++++++++++---------------------
 src/libvirt_private.syms |   1 +
 src/util/virnuma.h       |   2 +-
 tests/virnumamock.c      |   7 ++-
 5 files changed, 61 insertions(+), 65 deletions(-)

-- 
2.26.3

Re: [PATCH 0/4] Couple of virnuma cleanups
Posted by Ján Tomko 2 years, 11 months ago
On a Monday in 2021, Michal Privoznik wrote:
>I'm looking into NUMA code in capabilities and came up with a couple of
>cleanups. Technically, 3/4 is not needed yet, but I'll be introducing
>new data to vircaps2xmltest where a NUMA node has no CPUs and that's
>why the patch is needed.
>
>Michal Prívozník (4):
>  numa_conf: Use virXMLFormatElement() in virDomainNumaDefFormatXML
>  virnuma: Export virNumaGetMaxCPUs properly
>  virnumamock: Allow CPU-less NUMA nodes
>  virCapabilitiesHostNUMAFormat: Bring variables into loops
>
> src/conf/capabilities.c  |   9 ++--
> src/conf/numa_conf.c     | 107 ++++++++++++++++++---------------------
> src/libvirt_private.syms |   1 +
> src/util/virnuma.h       |   2 +-
> tests/virnumamock.c      |   7 ++-
> 5 files changed, 61 insertions(+), 65 deletions(-)
>

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

Jano