render / rpms / libvirt

Forked from rpms/libvirt 5 months ago
Clone
Pablo Greco 40546a
From e624d2040a5a8e16e670accc52e110393d9fb2b8 Mon Sep 17 00:00:00 2001
Pablo Greco 40546a
Message-Id: <e624d2040a5a8e16e670accc52e110393d9fb2b8@dist-git>
Pablo Greco 40546a
From: Andrea Bolognani <abologna@redhat.com>
Pablo Greco 40546a
Date: Tue, 11 Jun 2019 10:55:02 +0200
Pablo Greco 40546a
Subject: [PATCH] util: Introduce virNumaNodesetToCPUset()
Pablo Greco 40546a
MIME-Version: 1.0
Pablo Greco 40546a
Content-Type: text/plain; charset=UTF-8
Pablo Greco 40546a
Content-Transfer-Encoding: 8bit
Pablo Greco 40546a
Pablo Greco 40546a
This helper converts a set of NUMA node to the set of CPUs
Pablo Greco 40546a
they contain.
Pablo Greco 40546a
Pablo Greco 40546a
Signed-off-by: Andrea Bolognani <abologna@redhat.com>
Pablo Greco 40546a
Reviewed-by: Ján Tomko <jtomko@redhat.com>
Pablo Greco 40546a
(cherry picked from commit 2d2b26f96fed1e95dd4495168cee73c5c4092634)
Pablo Greco 40546a
Pablo Greco 40546a
https://bugzilla.redhat.com/show_bug.cgi?id=1716908
Pablo Greco 40546a
Pablo Greco 40546a
Signed-off-by: Andrea Bolognani <abologna@redhat.com>
Pablo Greco 40546a
Message-Id: <20190611085506.12564-3-abologna@redhat.com>
Pablo Greco 40546a
Reviewed-by: Ján Tomko <jtomko@redhat.com>
Pablo Greco 40546a
---
Pablo Greco 40546a
 src/libvirt_private.syms |  1 +
Pablo Greco 40546a
 src/util/virnuma.c       | 55 ++++++++++++++++++++++++++++++++++++++++
Pablo Greco 40546a
 src/util/virnuma.h       |  2 ++
Pablo Greco 40546a
 3 files changed, 58 insertions(+)
Pablo Greco 40546a
Pablo Greco 40546a
diff --git a/src/libvirt_private.syms b/src/libvirt_private.syms
Pablo Greco 40546a
index 05c96c29e2..7c48908a54 100644
Pablo Greco 40546a
--- a/src/libvirt_private.syms
Pablo Greco 40546a
+++ b/src/libvirt_private.syms
Pablo Greco 40546a
@@ -2485,6 +2485,7 @@ virNumaGetPages;
Pablo Greco 40546a
 virNumaIsAvailable;
Pablo Greco 40546a
 virNumaNodeIsAvailable;
Pablo Greco 40546a
 virNumaNodesetIsAvailable;
Pablo Greco 40546a
+virNumaNodesetToCPUset;
Pablo Greco 40546a
 virNumaSetPagePoolSize;
Pablo Greco 40546a
 virNumaSetupMemoryPolicy;
Pablo Greco 40546a
 
Pablo Greco 40546a
diff --git a/src/util/virnuma.c b/src/util/virnuma.c
Pablo Greco 40546a
index fd15714553..75f80dcd30 100644
Pablo Greco 40546a
--- a/src/util/virnuma.c
Pablo Greco 40546a
+++ b/src/util/virnuma.c
Pablo Greco 40546a
@@ -311,6 +311,49 @@ virNumaGetNodeCPUs(int node,
Pablo Greco 40546a
 # undef MASK_CPU_ISSET
Pablo Greco 40546a
 # undef n_bits
Pablo Greco 40546a
 
Pablo Greco 40546a
+/**
Pablo Greco 40546a
+ * virNumaNodesetToCPUset:
Pablo Greco 40546a
+ * @nodeset: bitmap containing a set of NUMA nodes
Pablo Greco 40546a
+ * @cpuset: return location for a bitmap containing a set of CPUs
Pablo Greco 40546a
+ *
Pablo Greco 40546a
+ * Convert a set of NUMA node to the set of CPUs they contain.
Pablo Greco 40546a
+ *
Pablo Greco 40546a
+ * Returns 0 on success, <0 on failure.
Pablo Greco 40546a
+ */
Pablo Greco 40546a
+int
Pablo Greco 40546a
+virNumaNodesetToCPUset(virBitmapPtr nodeset,
Pablo Greco 40546a
+                       virBitmapPtr *cpuset)
Pablo Greco 40546a
+{
Pablo Greco 40546a
+    VIR_AUTOPTR(virBitmap) allNodesCPUs = NULL;
Pablo Greco 40546a
+    size_t nodesetSize;
Pablo Greco 40546a
+    size_t i;
Pablo Greco 40546a
+
Pablo Greco 40546a
+    *cpuset = NULL;
Pablo Greco 40546a
+
Pablo Greco 40546a
+    if (!nodeset)
Pablo Greco 40546a
+        return 0;
Pablo Greco 40546a
+
Pablo Greco 40546a
+    allNodesCPUs = virBitmapNewEmpty();
Pablo Greco 40546a
+    nodesetSize = virBitmapSize(nodeset);
Pablo Greco 40546a
+
Pablo Greco 40546a
+    for (i = 0; i < nodesetSize; i++) {
Pablo Greco 40546a
+        VIR_AUTOPTR(virBitmap) nodeCPUs = NULL;
Pablo Greco 40546a
+
Pablo Greco 40546a
+        if (!virBitmapIsBitSet(nodeset, i))
Pablo Greco 40546a
+            continue;
Pablo Greco 40546a
+
Pablo Greco 40546a
+        if (virNumaGetNodeCPUs(i, &nodeCPUs) < 0)
Pablo Greco 40546a
+            return -1;
Pablo Greco 40546a
+
Pablo Greco 40546a
+        if (virBitmapUnion(allNodesCPUs, nodeCPUs) < 0)
Pablo Greco 40546a
+            return -1;
Pablo Greco 40546a
+    }
Pablo Greco 40546a
+
Pablo Greco 40546a
+    VIR_STEAL_PTR(*cpuset, allNodesCPUs);
Pablo Greco 40546a
+
Pablo Greco 40546a
+    return 0;
Pablo Greco 40546a
+}
Pablo Greco 40546a
+
Pablo Greco 40546a
 #else /* !WITH_NUMACTL */
Pablo Greco 40546a
 
Pablo Greco 40546a
 int
Pablo Greco 40546a
@@ -365,6 +408,18 @@ virNumaGetNodeCPUs(int node ATTRIBUTE_UNUSED,
Pablo Greco 40546a
                    _("NUMA isn't available on this host"));
Pablo Greco 40546a
     return -1;
Pablo Greco 40546a
 }
Pablo Greco 40546a
+
Pablo Greco 40546a
+int
Pablo Greco 40546a
+virNumaNodesetToCPUset(virBitmapPtr nodeset ATTRIBUTE_UNUSED,
Pablo Greco 40546a
+                       virBitmapPtr *cpuset)
Pablo Greco 40546a
+{
Pablo Greco 40546a
+    *cpuset = NULL;
Pablo Greco 40546a
+
Pablo Greco 40546a
+    virReportError(VIR_ERR_INTERNAL_ERROR, "%s",
Pablo Greco 40546a
+                   _("NUMA isn't available on this host"));
Pablo Greco 40546a
+    return -1;
Pablo Greco 40546a
+}
Pablo Greco 40546a
+
Pablo Greco 40546a
 #endif /* !WITH_NUMACTL */
Pablo Greco 40546a
 
Pablo Greco 40546a
 /**
Pablo Greco 40546a
diff --git a/src/util/virnuma.h b/src/util/virnuma.h
Pablo Greco 40546a
index a3ffb6d6c7..df7911db4e 100644
Pablo Greco 40546a
--- a/src/util/virnuma.h
Pablo Greco 40546a
+++ b/src/util/virnuma.h
Pablo Greco 40546a
@@ -48,6 +48,8 @@ int virNumaGetNodeMemory(int node,
Pablo Greco 40546a
 unsigned int virNumaGetMaxCPUs(void);
Pablo Greco 40546a
 
Pablo Greco 40546a
 int virNumaGetNodeCPUs(int node, virBitmapPtr *cpus) ATTRIBUTE_NOINLINE;
Pablo Greco 40546a
+int virNumaNodesetToCPUset(virBitmapPtr nodeset,
Pablo Greco 40546a
+                           virBitmapPtr *cpuset);
Pablo Greco 40546a
 
Pablo Greco 40546a
 int virNumaGetPageInfo(int node,
Pablo Greco 40546a
                        unsigned int page_size,
Pablo Greco 40546a
-- 
Pablo Greco 40546a
2.22.0
Pablo Greco 40546a