mirror of
https://gitlab.com/libvirt/libvirt.git
synced 2025-03-07 17:28:15 +00:00
qemu: cgroup: Kill qemuSetupCgroupVcpuPin()
The function doesn't make sense. There's a simpler way to achieve the same.
This commit is contained in:
parent
8a81264b18
commit
6afb0d04fe
@ -942,22 +942,6 @@ qemuSetupCgroupVcpuBW(virCgroupPtr cgroup,
|
|||||||
return -1;
|
return -1;
|
||||||
}
|
}
|
||||||
|
|
||||||
int
|
|
||||||
qemuSetupCgroupVcpuPin(virCgroupPtr cgroup,
|
|
||||||
virDomainPinDefPtr *vcpupin,
|
|
||||||
int nvcpupin,
|
|
||||||
int vcpuid)
|
|
||||||
{
|
|
||||||
size_t i;
|
|
||||||
|
|
||||||
for (i = 0; i < nvcpupin; i++) {
|
|
||||||
if (vcpuid == vcpupin[i]->id)
|
|
||||||
return qemuSetupCgroupCpusetCpus(cgroup, vcpupin[i]->cpumask);
|
|
||||||
}
|
|
||||||
|
|
||||||
return -1;
|
|
||||||
}
|
|
||||||
|
|
||||||
|
|
||||||
int
|
int
|
||||||
qemuSetupCgroupCpusetCpus(virCgroupPtr cgroup,
|
qemuSetupCgroupCpusetCpus(virCgroupPtr cgroup,
|
||||||
|
@ -52,10 +52,6 @@ int qemuSetupCpusetMems(virDomainObjPtr vm);
|
|||||||
int qemuSetupCgroupVcpuBW(virCgroupPtr cgroup,
|
int qemuSetupCgroupVcpuBW(virCgroupPtr cgroup,
|
||||||
unsigned long long period,
|
unsigned long long period,
|
||||||
long long quota);
|
long long quota);
|
||||||
int qemuSetupCgroupVcpuPin(virCgroupPtr cgroup,
|
|
||||||
virDomainPinDefPtr *vcpupin,
|
|
||||||
int nvcpupin,
|
|
||||||
int vcpuid);
|
|
||||||
int qemuSetupCgroupCpusetCpus(virCgroupPtr cgroup, virBitmapPtr cpumask);
|
int qemuSetupCgroupCpusetCpus(virCgroupPtr cgroup, virBitmapPtr cpumask);
|
||||||
int qemuSetupCgroupForVcpu(virDomainObjPtr vm);
|
int qemuSetupCgroupForVcpu(virDomainObjPtr vm);
|
||||||
int qemuSetupCgroupForIOThreads(virDomainObjPtr vm);
|
int qemuSetupCgroupForIOThreads(virDomainObjPtr vm);
|
||||||
|
@ -4775,9 +4775,8 @@ static int qemuDomainHotplugVcpus(virQEMUDriverPtr driver,
|
|||||||
}
|
}
|
||||||
|
|
||||||
if (cgroup_vcpu) {
|
if (cgroup_vcpu) {
|
||||||
if (qemuSetupCgroupVcpuPin(cgroup_vcpu,
|
if (qemuSetupCgroupCpusetCpus(cgroup_vcpu,
|
||||||
vm->def->cputune.vcpupin,
|
vcpupin->cpumask) < 0) {
|
||||||
vm->def->cputune.nvcpupin, i) < 0) {
|
|
||||||
virReportError(VIR_ERR_OPERATION_INVALID,
|
virReportError(VIR_ERR_OPERATION_INVALID,
|
||||||
_("failed to set cpuset.cpus in cgroup"
|
_("failed to set cpuset.cpus in cgroup"
|
||||||
" for vcpu %zu"), i);
|
" for vcpu %zu"), i);
|
||||||
@ -5143,8 +5142,7 @@ qemuDomainPinVcpuFlags(virDomainPtr dom,
|
|||||||
if (virCgroupHasController(priv->cgroup, VIR_CGROUP_CONTROLLER_CPUSET)) {
|
if (virCgroupHasController(priv->cgroup, VIR_CGROUP_CONTROLLER_CPUSET)) {
|
||||||
if (virCgroupNewVcpu(priv->cgroup, vcpu, false, &cgroup_vcpu) < 0)
|
if (virCgroupNewVcpu(priv->cgroup, vcpu, false, &cgroup_vcpu) < 0)
|
||||||
goto endjob;
|
goto endjob;
|
||||||
if (qemuSetupCgroupVcpuPin(cgroup_vcpu, newVcpuPin, newVcpuPinNum,
|
if (qemuSetupCgroupCpusetCpus(cgroup_vcpu, pcpumap) < 0) {
|
||||||
vcpu) < 0) {
|
|
||||||
virReportError(VIR_ERR_OPERATION_INVALID,
|
virReportError(VIR_ERR_OPERATION_INVALID,
|
||||||
_("failed to set cpuset.cpus in cgroup"
|
_("failed to set cpuset.cpus in cgroup"
|
||||||
" for vcpu %d"), vcpu);
|
" for vcpu %d"), vcpu);
|
||||||
|
Loading…
x
Reference in New Issue
Block a user