|
Pablo Greco |
40546a |
From 0628a57a5b4a28d644970a2ad82cfadc14a3cd9f Mon Sep 17 00:00:00 2001
|
|
Pablo Greco |
40546a |
Message-Id: <0628a57a5b4a28d644970a2ad82cfadc14a3cd9f@dist-git>
|
|
Pablo Greco |
40546a |
From: Pavel Hrdina <phrdina@redhat.com>
|
|
Pablo Greco |
40546a |
Date: Mon, 1 Jul 2019 17:08:02 +0200
|
|
Pablo Greco |
40546a |
Subject: [PATCH] vircgrouptest: add hybrid tests
|
|
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 |
Signed-off-by: Pavel Hrdina <phrdina@redhat.com>
|
|
Pablo Greco |
40546a |
(cherry picked from commit a26de856f98119bf651ed9928c64b6f96f78f185)
|
|
Pablo Greco |
40546a |
|
|
Pablo Greco |
40546a |
Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1689297
|
|
Pablo Greco |
40546a |
|
|
Pablo Greco |
40546a |
Signed-off-by: Pavel Hrdina <phrdina@redhat.com>
|
|
Pablo Greco |
40546a |
Message-Id: <434692bd0df2a6f28c93e3cc071187144820d836.1561993100.git.phrdina@redhat.com>
|
|
Pablo Greco |
40546a |
Reviewed-by: Ján Tomko <jtomko@redhat.com>
|
|
Pablo Greco |
40546a |
---
|
|
Pablo Greco |
40546a |
tests/vircgrouptest.c | 48 +++++++++++++++++++++++++++++++++++++++++++
|
|
Pablo Greco |
40546a |
1 file changed, 48 insertions(+)
|
|
Pablo Greco |
40546a |
|
|
Pablo Greco |
40546a |
diff --git a/tests/vircgrouptest.c b/tests/vircgrouptest.c
|
|
Pablo Greco |
40546a |
index 3b844f0fbf..7fde50b463 100644
|
|
Pablo Greco |
40546a |
--- a/tests/vircgrouptest.c
|
|
Pablo Greco |
40546a |
+++ b/tests/vircgrouptest.c
|
|
Pablo Greco |
40546a |
@@ -605,6 +605,45 @@ static int testCgroupNewForSelfUnified(const void *args ATTRIBUTE_UNUSED)
|
|
Pablo Greco |
40546a |
}
|
|
Pablo Greco |
40546a |
|
|
Pablo Greco |
40546a |
|
|
Pablo Greco |
40546a |
+static int testCgroupNewForSelfHybrid(const void *args ATTRIBUTE_UNUSED)
|
|
Pablo Greco |
40546a |
+{
|
|
Pablo Greco |
40546a |
+ virCgroupPtr cgroup = NULL;
|
|
Pablo Greco |
40546a |
+ int ret = -1;
|
|
Pablo Greco |
40546a |
+ const char *empty[VIR_CGROUP_CONTROLLER_LAST] = { 0 };
|
|
Pablo Greco |
40546a |
+ const char *mounts[VIR_CGROUP_CONTROLLER_LAST] = {
|
|
Pablo Greco |
40546a |
+ [VIR_CGROUP_CONTROLLER_CPUSET] = "/not/really/sys/fs/cgroup/cpuset",
|
|
Pablo Greco |
40546a |
+ [VIR_CGROUP_CONTROLLER_DEVICES] = "/not/really/sys/fs/cgroup/devices",
|
|
Pablo Greco |
40546a |
+ [VIR_CGROUP_CONTROLLER_FREEZER] = "/not/really/sys/fs/cgroup/freezer",
|
|
Pablo Greco |
40546a |
+ [VIR_CGROUP_CONTROLLER_NET_CLS] = "/not/really/sys/fs/cgroup/net_cls",
|
|
Pablo Greco |
40546a |
+ [VIR_CGROUP_CONTROLLER_PERF_EVENT] = "/not/really/sys/fs/cgroup/perf_event",
|
|
Pablo Greco |
40546a |
+ };
|
|
Pablo Greco |
40546a |
+ const char *placement[VIR_CGROUP_CONTROLLER_LAST] = {
|
|
Pablo Greco |
40546a |
+ [VIR_CGROUP_CONTROLLER_CPUSET] = "/",
|
|
Pablo Greco |
40546a |
+ [VIR_CGROUP_CONTROLLER_DEVICES] = "/",
|
|
Pablo Greco |
40546a |
+ [VIR_CGROUP_CONTROLLER_FREEZER] = "/",
|
|
Pablo Greco |
40546a |
+ [VIR_CGROUP_CONTROLLER_NET_CLS] = "/",
|
|
Pablo Greco |
40546a |
+ [VIR_CGROUP_CONTROLLER_PERF_EVENT] = "/",
|
|
Pablo Greco |
40546a |
+ };
|
|
Pablo Greco |
40546a |
+ unsigned int controllers =
|
|
Pablo Greco |
40546a |
+ (1 << VIR_CGROUP_CONTROLLER_CPU) |
|
|
Pablo Greco |
40546a |
+ (1 << VIR_CGROUP_CONTROLLER_CPUACCT) |
|
|
Pablo Greco |
40546a |
+ (1 << VIR_CGROUP_CONTROLLER_MEMORY) |
|
|
Pablo Greco |
40546a |
+ (1 << VIR_CGROUP_CONTROLLER_BLKIO);
|
|
Pablo Greco |
40546a |
+
|
|
Pablo Greco |
40546a |
+ if (virCgroupNewSelf(&cgroup) < 0) {
|
|
Pablo Greco |
40546a |
+ fprintf(stderr, "Cannot create cgroup for self\n");
|
|
Pablo Greco |
40546a |
+ goto cleanup;
|
|
Pablo Greco |
40546a |
+ }
|
|
Pablo Greco |
40546a |
+
|
|
Pablo Greco |
40546a |
+ ret = validateCgroup(cgroup, "", mounts, empty, placement,
|
|
Pablo Greco |
40546a |
+ "/not/really/sys/fs/cgroup/unified", "/", controllers);
|
|
Pablo Greco |
40546a |
+
|
|
Pablo Greco |
40546a |
+ cleanup:
|
|
Pablo Greco |
40546a |
+ virCgroupFree(&cgroup);
|
|
Pablo Greco |
40546a |
+ return ret;
|
|
Pablo Greco |
40546a |
+}
|
|
Pablo Greco |
40546a |
+
|
|
Pablo Greco |
40546a |
+
|
|
Pablo Greco |
40546a |
static int testCgroupAvailable(const void *args)
|
|
Pablo Greco |
40546a |
{
|
|
Pablo Greco |
40546a |
bool got = virCgroupAvailable();
|
|
Pablo Greco |
40546a |
@@ -1024,6 +1063,15 @@ mymain(void)
|
|
Pablo Greco |
40546a |
ret = -1;
|
|
Pablo Greco |
40546a |
cleanupFakeFS(fakerootdir);
|
|
Pablo Greco |
40546a |
|
|
Pablo Greco |
40546a |
+ /* cgroup hybrid */
|
|
Pablo Greco |
40546a |
+
|
|
Pablo Greco |
40546a |
+ fakerootdir = initFakeFS("hybrid", "hybrid");
|
|
Pablo Greco |
40546a |
+ if (virTestRun("New cgroup for self (hybrid)", testCgroupNewForSelfHybrid, NULL) < 0)
|
|
Pablo Greco |
40546a |
+ ret = -1;
|
|
Pablo Greco |
40546a |
+ if (virTestRun("Cgroup available (hybrid)", testCgroupAvailable, (void*)0x1) < 0)
|
|
Pablo Greco |
40546a |
+ ret = -1;
|
|
Pablo Greco |
40546a |
+ cleanupFakeFS(fakerootdir);
|
|
Pablo Greco |
40546a |
+
|
|
Pablo Greco |
40546a |
return ret == 0 ? EXIT_SUCCESS : EXIT_FAILURE;
|
|
Pablo Greco |
40546a |
}
|
|
Pablo Greco |
40546a |
|
|
Pablo Greco |
40546a |
--
|
|
Pablo Greco |
40546a |
2.22.0
|
|
Pablo Greco |
40546a |
|