From c16723f606c4fc3af4de9af96949e2e03d45926c Mon Sep 17 00:00:00 2001 From: John Ferlan Date: Tue, 10 Mar 2015 20:24:04 -0400 Subject: [PATCH] Convert virDomainVcpuPinDefCopy into virDomainPinDefCopy Since both Vcpu and IOThreads code use the same API's, alter the naming of the API's to remove the "Vcpu" specific reference --- src/conf/domain_conf.c | 8 ++++---- src/conf/domain_conf.h | 4 ++-- src/libvirt_private.syms | 2 +- src/qemu/qemu_driver.c | 9 ++++----- 4 files changed, 11 insertions(+), 12 deletions(-) diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c index 3fb8ea8863..c01b8952a0 100644 --- a/src/conf/domain_conf.c +++ b/src/conf/domain_conf.c @@ -2004,15 +2004,15 @@ virDomainClockDefClear(virDomainClockDefPtr def) } virDomainPinDefPtr * -virDomainVcpuPinDefCopy(virDomainPinDefPtr *src, int nvcpupin) +virDomainPinDefCopy(virDomainPinDefPtr *src, int npin) { size_t i; virDomainPinDefPtr *ret = NULL; - if (VIR_ALLOC_N(ret, nvcpupin) < 0) + if (VIR_ALLOC_N(ret, npin) < 0) goto error; - for (i = 0; i < nvcpupin; i++) { + for (i = 0; i < npin; i++) { if (VIR_ALLOC(ret[i]) < 0) goto error; ret[i]->id = src[i]->id; @@ -2024,7 +2024,7 @@ virDomainVcpuPinDefCopy(virDomainPinDefPtr *src, int nvcpupin) error: if (ret) { - for (i = 0; i < nvcpupin; i++) { + for (i = 0; i < npin; i++) { if (ret[i]) { virBitmapFree(ret[i]->cpumask); VIR_FREE(ret[i]); diff --git a/src/conf/domain_conf.h b/src/conf/domain_conf.h index 341be5f01e..f991927f95 100644 --- a/src/conf/domain_conf.h +++ b/src/conf/domain_conf.h @@ -1910,8 +1910,8 @@ struct _virDomainPinDef { void virDomainPinDefFree(virDomainPinDefPtr def); void virDomainPinDefArrayFree(virDomainPinDefPtr *def, int npin); -virDomainPinDefPtr *virDomainVcpuPinDefCopy(virDomainPinDefPtr *src, - int nvcpupin); +virDomainPinDefPtr *virDomainPinDefCopy(virDomainPinDefPtr *src, + int npin); int virDomainVcpuPinIsDuplicate(virDomainPinDefPtr *def, int nvcpupin, diff --git a/src/libvirt_private.syms b/src/libvirt_private.syms index cbc3aea972..ed18629038 100644 --- a/src/libvirt_private.syms +++ b/src/libvirt_private.syms @@ -385,6 +385,7 @@ virDomainParseMemory; virDomainPausedReasonTypeFromString; virDomainPausedReasonTypeToString; virDomainPinDefArrayFree; +virDomainPinDefCopy; virDomainPinDefFree; virDomainPMSuspendedReasonTypeFromString; virDomainPMSuspendedReasonTypeToString; @@ -438,7 +439,6 @@ virDomainTPMDefFree; virDomainTPMModelTypeFromString; virDomainTPMModelTypeToString; virDomainVcpuPinAdd; -virDomainVcpuPinDefCopy; virDomainVcpuPinDel; virDomainVcpuPinFindByVcpu; virDomainVcpuPinIsDuplicate; diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c index d20118703a..66ef95fbc2 100644 --- a/src/qemu/qemu_driver.c +++ b/src/qemu/qemu_driver.c @@ -5111,8 +5111,8 @@ qemuDomainPinVcpuFlags(virDomainPtr dom, } if (vm->def->cputune.vcpupin) { - newVcpuPin = virDomainVcpuPinDefCopy(vm->def->cputune.vcpupin, - vm->def->cputune.nvcpupin); + newVcpuPin = virDomainPinDefCopy(vm->def->cputune.vcpupin, + vm->def->cputune.nvcpupin); if (!newVcpuPin) goto endjob; @@ -5984,10 +5984,9 @@ qemuDomainPinIOThread(virDomainPtr dom, } if (vm->def->cputune.iothreadspin) { - /* The VcpuPinDefCopy works for IOThreads too */ newIOThreadsPin = - virDomainVcpuPinDefCopy(vm->def->cputune.iothreadspin, - vm->def->cputune.niothreadspin); + virDomainPinDefCopy(vm->def->cputune.iothreadspin, + vm->def->cputune.niothreadspin); if (!newIOThreadsPin) goto endjob; -- GitLab