Re: [libvirt PATCH v2 13/17] ch_driver, ch_domain: vcpupin callback in ch driver

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

 



On a Thursday in 2021, Praveen K Paladugu wrote:
From: Vineeth Pillai <viremana@xxxxxxxxxxxxxxxxxxx>

Signed-off-by: Vineeth Pillai <viremana@xxxxxxxxxxxxxxxxxxx>
Signed-off-by: Praveen K Paladugu <prapal@xxxxxxxxxxxxxxxxxxx>
---
src/ch/ch_domain.c |  30 +++++++++
src/ch/ch_domain.h |   1 +
src/ch/ch_driver.c | 151 +++++++++++++++++++++++++++++++++++++++++++++
3 files changed, 182 insertions(+)

@@ -1129,6 +1130,154 @@ chDomainGetVcpus(virDomainPtr dom,
    return ret;
}

+static int
+chDomainPinVcpuLive(virDomainObj *vm,
+                    virDomainDef *def,
+                    int vcpu,
+                    virCHDriver *driver,
+                    virCHDriverConfig *cfg,
+                    virBitmap *cpumap)
+{
+    virBitmap *tmpmap = NULL;

You can use
    g_autoptr(virBitmap) tmpmap = NULL;
to automatically free the memory when it goes out of scope
instead of manually calling virBitmapFree


+    virDomainVcpuDef *vcpuinfo;
+    virCHDomainObjPrivate *priv = vm->privateData;
+    virCgroup *cgroup_vcpu = NULL;
+    g_autofree char *str = NULL;
+    int ret = -1;
+
+    if (!virCHDomainHasVcpuPids(vm)) {
+

+    if (persistentDef) {
+        virBitmapFree(vcpuinfo->cpumask);
+        vcpuinfo->cpumask = pcpumap;
+        pcpumap = NULL;
+
+        // ret = virDomainDefSave(persistentDef, driver->xmlopt, cfg->configDir);

There's no need for this comment.

+        goto endjob;

Jano

Attachment: signature.asc
Description: PGP signature


[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