|
 |
c313de |
From 164b5885c6916d39a71a4778da9782c9dc6cadf0 Mon Sep 17 00:00:00 2001
|
|
|
c313de |
Message-Id: <164b5885c6916d39a71a4778da9782c9dc6cadf0@dist-git>
|
|
|
c313de |
From: Pavel Hrdina <phrdina@redhat.com>
|
|
|
c313de |
Date: Fri, 6 Sep 2019 09:29:41 +0200
|
|
|
c313de |
Subject: [PATCH] vircgroupv2: fix setting cpu.max period
|
|
|
c313de |
|
|
|
c313de |
When we set cpu.max period we need to parse the cpu.max file first as
|
|
|
c313de |
it contains both quota and period values separated by space. When only
|
|
|
c313de |
a single number is written to that file it will set quota. However,
|
|
|
c313de |
in order to change period we need to write both values.
|
|
|
c313de |
|
|
|
c313de |
The code was prepared for that but mistakenly used new line to end the
|
|
|
c313de |
string with the first value.
|
|
|
c313de |
|
|
|
c313de |
Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1749227
|
|
|
c313de |
|
|
|
c313de |
Signed-off-by: Pavel Hrdina <phrdina@redhat.com>
|
|
|
c313de |
Reviewed-by: Erik Skultety <eskultet@redhat.com>
|
|
|
c313de |
(cherry picked from commit 0bd4ad193d8ba7f0104f4739f19f2731e7cf9f56)
|
|
|
c313de |
Signed-off-by: Pavel Hrdina <phrdina@redhat.com>
|
|
|
c313de |
Message-Id: <3a1021b39f7b5add388e98eae4caa63975b60fc3.1567754949.git.phrdina@redhat.com>
|
|
|
c313de |
Reviewed-by: Erik Skultety <eskultet@redhat.com>
|
|
|
c313de |
---
|
|
|
c313de |
src/util/vircgroupv2.c | 2 +-
|
|
|
c313de |
1 file changed, 1 insertion(+), 1 deletion(-)
|
|
|
c313de |
|
|
|
c313de |
diff --git a/src/util/vircgroupv2.c b/src/util/vircgroupv2.c
|
|
|
c313de |
index e7b9093754..13a51e67a4 100644
|
|
|
c313de |
--- a/src/util/vircgroupv2.c
|
|
|
c313de |
+++ b/src/util/vircgroupv2.c
|
|
|
c313de |
@@ -1507,7 +1507,7 @@ virCgroupV2SetCpuCfsPeriod(virCgroupPtr group,
|
|
|
c313de |
_("Invalid 'cpu.max' data."));
|
|
|
c313de |
return -1;
|
|
|
c313de |
}
|
|
|
c313de |
- *tmp = '\n';
|
|
|
c313de |
+ *tmp = '\0';
|
|
|
c313de |
|
|
|
c313de |
if (virAsprintf(&value, "%s %llu", str, cfs_period) < 0)
|
|
|
c313de |
return -1;
|
|
|
c313de |
--
|
|
|
c313de |
2.23.0
|
|
|
c313de |
|