[libvirt] [PATCH 6/7] vircgroup: Simplify if conditions in virCgroupMakeGroup
Pavel Hrdina
phrdina at redhat.com
Fri Aug 24 13:45:56 UTC 2018
Signed-off-by: Pavel Hrdina <phrdina at redhat.com>
---
src/util/vircgroup.c | 20 ++++++++------------
1 file changed, 8 insertions(+), 12 deletions(-)
diff --git a/src/util/vircgroup.c b/src/util/vircgroup.c
index dde9ed21a2..02de33f74f 100644
--- a/src/util/vircgroup.c
+++ b/src/util/vircgroup.c
@@ -1079,24 +1079,20 @@ virCgroupMakeGroup(virCgroupPtr parent,
goto error;
}
}
- if (group->controllers[VIR_CGROUP_CONTROLLER_CPUSET].mountPoint != NULL &&
- (i == VIR_CGROUP_CONTROLLER_CPUSET ||
- STREQ(group->controllers[i].mountPoint,
- group->controllers[VIR_CGROUP_CONTROLLER_CPUSET].mountPoint))) {
- if (virCgroupCpuSetInherit(parent, group) < 0)
- goto error;
+ if (i == VIR_CGROUP_CONTROLLER_CPUSET &&
+ group->controllers[i].mountPoint != NULL &&
+ virCgroupCpuSetInherit(parent, group) < 0) {
+ goto error;
}
/*
* Note that virCgroupSetMemoryUseHierarchy should always be
* called prior to creating subcgroups and attaching tasks.
*/
if ((flags & VIR_CGROUP_MEM_HIERACHY) &&
- (group->controllers[VIR_CGROUP_CONTROLLER_MEMORY].mountPoint != NULL) &&
- (i == VIR_CGROUP_CONTROLLER_MEMORY ||
- STREQ(group->controllers[i].mountPoint,
- group->controllers[VIR_CGROUP_CONTROLLER_MEMORY].mountPoint))) {
- if (virCgroupSetMemoryUseHierarchy(group) < 0)
- goto error;
+ i == VIR_CGROUP_CONTROLLER_MEMORY &&
+ group->controllers[i].mountPoint != NULL &&
+ virCgroupSetMemoryUseHierarchy(group) < 0) {
+ goto error;
}
}
}
--
2.17.1
More information about the libvir-list
mailing list