|
 |
c313de |
From d7a8cd2fdafd168eb1672e7ecde1700463792518 Mon Sep 17 00:00:00 2001
|
|
|
c313de |
Message-Id: <d7a8cd2fdafd168eb1672e7ecde1700463792518@dist-git>
|
|
|
c313de |
From: Pavel Hrdina <phrdina@redhat.com>
|
|
|
c313de |
Date: Mon, 1 Jul 2019 17:05:50 +0200
|
|
|
c313de |
Subject: [PATCH] vircgroup: Extract file link resolving into separate function
|
|
|
c313de |
MIME-Version: 1.0
|
|
|
c313de |
Content-Type: text/plain; charset=UTF-8
|
|
|
c313de |
Content-Transfer-Encoding: 8bit
|
|
|
c313de |
|
|
|
c313de |
Signed-off-by: Pavel Hrdina <phrdina@redhat.com>
|
|
|
c313de |
(cherry picked from commit 00a0085aa17677003e89cdafceb6a0260af43fb8)
|
|
|
c313de |
|
|
|
c313de |
Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1689297
|
|
|
c313de |
|
|
|
c313de |
Signed-off-by: Pavel Hrdina <phrdina@redhat.com>
|
|
|
c313de |
Message-Id: <6d3c5594a53824ceac258a0f54162ec8af5aef37.1561993099.git.phrdina@redhat.com>
|
|
|
c313de |
Reviewed-by: Ján Tomko <jtomko@redhat.com>
|
|
|
c313de |
---
|
|
|
c313de |
src/util/vircgroup.c | 85 +++++++++++++++++++++++++-------------------
|
|
|
c313de |
1 file changed, 48 insertions(+), 37 deletions(-)
|
|
|
c313de |
|
|
|
c313de |
diff --git a/src/util/vircgroup.c b/src/util/vircgroup.c
|
|
|
c313de |
index 5144b33d43..8ddedbfe7c 100644
|
|
|
c313de |
--- a/src/util/vircgroup.c
|
|
|
c313de |
+++ b/src/util/vircgroup.c
|
|
|
c313de |
@@ -356,6 +356,51 @@ virCgroupCopyMounts(virCgroupPtr group,
|
|
|
c313de |
}
|
|
|
c313de |
|
|
|
c313de |
|
|
|
c313de |
+static int
|
|
|
c313de |
+virCgroupResolveMountLink(char *mntDir,
|
|
|
c313de |
+ const char *typeStr,
|
|
|
c313de |
+ virCgroupControllerPtr controller)
|
|
|
c313de |
+{
|
|
|
c313de |
+ VIR_AUTOFREE(char *) linkSrc = NULL;
|
|
|
c313de |
+ char *dirName;
|
|
|
c313de |
+ struct stat sb;
|
|
|
c313de |
+
|
|
|
c313de |
+ dirName = strrchr(mntDir, '/');
|
|
|
c313de |
+ if (!dirName) {
|
|
|
c313de |
+ virReportError(VIR_ERR_INTERNAL_ERROR,
|
|
|
c313de |
+ _("Missing '/' separator in cgroup mount '%s'"), mntDir);
|
|
|
c313de |
+ return -1;
|
|
|
c313de |
+ }
|
|
|
c313de |
+
|
|
|
c313de |
+ if (!strchr(dirName + 1, ','))
|
|
|
c313de |
+ return 0;
|
|
|
c313de |
+
|
|
|
c313de |
+ *dirName = '\0';
|
|
|
c313de |
+ if (virAsprintf(&linkSrc, "%s/%s", mntDir, typeStr) < 0)
|
|
|
c313de |
+ return -1;
|
|
|
c313de |
+ *dirName = '/';
|
|
|
c313de |
+
|
|
|
c313de |
+ if (lstat(linkSrc, &sb) < 0) {
|
|
|
c313de |
+ if (errno == ENOENT) {
|
|
|
c313de |
+ VIR_WARN("Controller %s co-mounted at %s is missing symlink at %s",
|
|
|
c313de |
+ typeStr, mntDir, linkSrc);
|
|
|
c313de |
+ } else {
|
|
|
c313de |
+ virReportSystemError(errno, _("Cannot stat %s"), linkSrc);
|
|
|
c313de |
+ return -1;
|
|
|
c313de |
+ }
|
|
|
c313de |
+ } else {
|
|
|
c313de |
+ if (!S_ISLNK(sb.st_mode)) {
|
|
|
c313de |
+ VIR_WARN("Expecting a symlink at %s for controller %s",
|
|
|
c313de |
+ linkSrc, typeStr);
|
|
|
c313de |
+ } else {
|
|
|
c313de |
+ VIR_STEAL_PTR(controller->linkPoint, linkSrc);
|
|
|
c313de |
+ }
|
|
|
c313de |
+ }
|
|
|
c313de |
+
|
|
|
c313de |
+ return 0;
|
|
|
c313de |
+}
|
|
|
c313de |
+
|
|
|
c313de |
+
|
|
|
c313de |
/*
|
|
|
c313de |
* Process /proc/mounts figuring out what controllers are
|
|
|
c313de |
* mounted and where
|
|
|
c313de |
@@ -397,8 +442,6 @@ virCgroupDetectMountsFromFile(virCgroupPtr group,
|
|
|
c313de |
}
|
|
|
c313de |
|
|
|
c313de |
if (typelen == len && STREQLEN(typestr, tmp, len)) {
|
|
|
c313de |
- struct stat sb;
|
|
|
c313de |
- char *tmp2;
|
|
|
c313de |
|
|
|
c313de |
/* Note that the lines in /proc/mounts have the same
|
|
|
c313de |
* order than the mount operations, and that there may
|
|
|
c313de |
@@ -412,44 +455,12 @@ virCgroupDetectMountsFromFile(virCgroupPtr group,
|
|
|
c313de |
if (VIR_STRDUP(controller->mountPoint, entry.mnt_dir) < 0)
|
|
|
c313de |
goto cleanup;
|
|
|
c313de |
|
|
|
c313de |
- tmp2 = strrchr(entry.mnt_dir, '/');
|
|
|
c313de |
- if (!tmp2) {
|
|
|
c313de |
- virReportError(VIR_ERR_INTERNAL_ERROR,
|
|
|
c313de |
- _("Missing '/' separator in cgroup mount '%s'"),
|
|
|
c313de |
- entry.mnt_dir);
|
|
|
c313de |
- goto cleanup;
|
|
|
c313de |
- }
|
|
|
c313de |
-
|
|
|
c313de |
/* If it is a co-mount it has a filename like "cpu,cpuacct"
|
|
|
c313de |
* and we must identify the symlink path */
|
|
|
c313de |
- if (checkLinks && strchr(tmp2 + 1, ',')) {
|
|
|
c313de |
- VIR_AUTOFREE(char *) linksrc = NULL;
|
|
|
c313de |
-
|
|
|
c313de |
- *tmp2 = '\0';
|
|
|
c313de |
- if (virAsprintf(&linksrc, "%s/%s",
|
|
|
c313de |
- entry.mnt_dir, typestr) < 0)
|
|
|
c313de |
+ if (checkLinks &&
|
|
|
c313de |
+ virCgroupResolveMountLink(entry.mnt_dir, typestr,
|
|
|
c313de |
+ controller) < 0) {
|
|
|
c313de |
goto cleanup;
|
|
|
c313de |
- *tmp2 = '/';
|
|
|
c313de |
-
|
|
|
c313de |
- if (lstat(linksrc, &sb) < 0) {
|
|
|
c313de |
- if (errno == ENOENT) {
|
|
|
c313de |
- VIR_WARN("Controller %s co-mounted at %s is missing symlink at %s",
|
|
|
c313de |
- typestr, entry.mnt_dir, linksrc);
|
|
|
c313de |
- } else {
|
|
|
c313de |
- virReportSystemError(errno,
|
|
|
c313de |
- _("Cannot stat %s"),
|
|
|
c313de |
- linksrc);
|
|
|
c313de |
- goto cleanup;
|
|
|
c313de |
- }
|
|
|
c313de |
- } else {
|
|
|
c313de |
- if (!S_ISLNK(sb.st_mode)) {
|
|
|
c313de |
- VIR_WARN("Expecting a symlink at %s for controller %s",
|
|
|
c313de |
- linksrc, typestr);
|
|
|
c313de |
- } else {
|
|
|
c313de |
- controller->linkPoint = linksrc;
|
|
|
c313de |
- linksrc = NULL;
|
|
|
c313de |
- }
|
|
|
c313de |
- }
|
|
|
c313de |
}
|
|
|
c313de |
}
|
|
|
c313de |
tmp = next;
|
|
|
c313de |
--
|
|
|
c313de |
2.22.0
|
|
|
c313de |
|