|
|
404507 |
From db2302bc5e5fa8b67384ca6adc85483ce5e24358 Mon Sep 17 00:00:00 2001
|
|
|
404507 |
Message-Id: <db2302bc5e5fa8b67384ca6adc85483ce5e24358@dist-git>
|
|
|
404507 |
From: Martin Kletzander <mkletzan@redhat.com>
|
|
|
404507 |
Date: Wed, 31 Jan 2018 16:32:36 +0100
|
|
|
404507 |
Subject: [PATCH] util: Use "resctrl" instead of "resctrlfs" spelling
|
|
|
404507 |
|
|
|
404507 |
Pointed out during review on one or two places, but it actually appears in lot
|
|
|
404507 |
more places. So let's be consistent.
|
|
|
404507 |
|
|
|
404507 |
Signed-off-by: Martin Kletzander <mkletzan@redhat.com>
|
|
|
404507 |
(cherry picked from commit ebafc603c1ace128927662460d55c1a713ae5225)
|
|
|
404507 |
Signed-off-by: Martin Kletzander <mkletzan@redhat.com>
|
|
|
404507 |
|
|
|
404507 |
https://bugzilla.redhat.com/show_bug.cgi?id=1289368
|
|
|
404507 |
Signed-off-by: Martin Kletzander <mkletzan@redhat.com>
|
|
|
404507 |
---
|
|
|
404507 |
src/qemu/qemu_process.c | 2 +-
|
|
|
404507 |
src/util/virresctrl.c | 14 +++++++-------
|
|
|
404507 |
2 files changed, 8 insertions(+), 8 deletions(-)
|
|
|
404507 |
|
|
|
404507 |
diff --git a/src/qemu/qemu_process.c b/src/qemu/qemu_process.c
|
|
|
404507 |
index 57faf9cd79..29a8d8ba53 100644
|
|
|
404507 |
--- a/src/qemu/qemu_process.c
|
|
|
404507 |
+++ b/src/qemu/qemu_process.c
|
|
|
404507 |
@@ -5932,7 +5932,7 @@ qemuProcessLaunch(virConnectPtr conn,
|
|
|
404507 |
if (qemuProcessSetupEmulator(vm) < 0)
|
|
|
404507 |
goto cleanup;
|
|
|
404507 |
|
|
|
404507 |
- VIR_DEBUG("Setting up resctrlfs");
|
|
|
404507 |
+ VIR_DEBUG("Setting up resctrl");
|
|
|
404507 |
if (qemuProcessResctrlCreate(driver, vm) < 0)
|
|
|
404507 |
goto cleanup;
|
|
|
404507 |
|
|
|
404507 |
diff --git a/src/util/virresctrl.c b/src/util/virresctrl.c
|
|
|
404507 |
index 03218a481d..a1d09c547e 100644
|
|
|
404507 |
--- a/src/util/virresctrl.c
|
|
|
404507 |
+++ b/src/util/virresctrl.c
|
|
|
404507 |
@@ -76,7 +76,7 @@ struct _virResctrlInfoPerType {
|
|
|
404507 |
unsigned int max_cache_id;
|
|
|
404507 |
|
|
|
404507 |
/* In order to be self-sufficient we need size information per cache.
|
|
|
404507 |
- * Funnily enough, one of the outcomes of the resctrlfs design is that it
|
|
|
404507 |
+ * Funnily enough, one of the outcomes of the resctrl design is that it
|
|
|
404507 |
* does not account for different sizes per cache on the same level. So
|
|
|
404507 |
* for the sake of easiness, let's copy that, for now. */
|
|
|
404507 |
unsigned long long size;
|
|
|
404507 |
@@ -302,12 +302,12 @@ virResctrlLockInternal(int op)
|
|
|
404507 |
int fd = open(SYSFS_RESCTRL_PATH, O_DIRECTORY | O_CLOEXEC);
|
|
|
404507 |
|
|
|
404507 |
if (fd < 0) {
|
|
|
404507 |
- virReportSystemError(errno, "%s", _("Cannot open resctrlfs"));
|
|
|
404507 |
+ virReportSystemError(errno, "%s", _("Cannot open resctrl"));
|
|
|
404507 |
return -1;
|
|
|
404507 |
}
|
|
|
404507 |
|
|
|
404507 |
if (flock(fd, op) < 0) {
|
|
|
404507 |
- virReportSystemError(errno, "%s", _("Cannot lock resctrlfs"));
|
|
|
404507 |
+ virReportSystemError(errno, "%s", _("Cannot lock resctrl"));
|
|
|
404507 |
VIR_FORCE_CLOSE(fd);
|
|
|
404507 |
return -1;
|
|
|
404507 |
}
|
|
|
404507 |
@@ -328,7 +328,7 @@ static inline int
|
|
|
404507 |
virResctrlLockWrite(void)
|
|
|
404507 |
{
|
|
|
404507 |
virReportSystemError(ENOSYS, "%s",
|
|
|
404507 |
- _("resctrlfs not supported on this platform"));
|
|
|
404507 |
+ _("resctrl not supported on this platform"));
|
|
|
404507 |
return -1;
|
|
|
404507 |
}
|
|
|
404507 |
|
|
|
404507 |
@@ -347,11 +347,11 @@ virResctrlUnlock(int fd)
|
|
|
404507 |
/* The lock gets unlocked by closing the fd, which we need to do anyway in
|
|
|
404507 |
* order to clean up properly */
|
|
|
404507 |
if (VIR_CLOSE(fd) < 0) {
|
|
|
404507 |
- virReportSystemError(errno, "%s", _("Cannot close resctrlfs"));
|
|
|
404507 |
+ virReportSystemError(errno, "%s", _("Cannot close resctrl"));
|
|
|
404507 |
|
|
|
404507 |
/* Trying to save the already broken */
|
|
|
404507 |
if (flock(fd, LOCK_UN) < 0)
|
|
|
404507 |
- virReportSystemError(errno, "%s", _("Cannot unlock resctrlfs"));
|
|
|
404507 |
+ virReportSystemError(errno, "%s", _("Cannot unlock resctrl"));
|
|
|
404507 |
return -1;
|
|
|
404507 |
}
|
|
|
404507 |
#endif /* ! __linux__ */
|
|
|
404507 |
@@ -486,7 +486,7 @@ virResctrlGetInfo(virResctrlInfoPtr resctrl)
|
|
|
404507 |
|
|
|
404507 |
if (i_level->types[type]) {
|
|
|
404507 |
virReportError(VIR_ERR_INTERNAL_ERROR,
|
|
|
404507 |
- _("Duplicate cache type in resctrlfs for level %u"),
|
|
|
404507 |
+ _("Duplicate cache type in resctrl for level %u"),
|
|
|
404507 |
level);
|
|
|
404507 |
goto cleanup;
|
|
|
404507 |
}
|
|
|
404507 |
--
|
|
|
404507 |
2.16.1
|
|
|
404507 |
|