* gnu/local.mk (dist_patch_DATA): Add new patch file. * gnu/packages/patches/libvirt-create-machine-cgroup.patch: New patch, submitted to upstream for upstream bug 1760233. * gnu/packages/virtualization.scm (libvirt): Update version to 5.8.0. Include patch. Avoid execution of failing tests qemuxml2argvtest and qemuhotplugtest. Replace python by python-wrapper to avoid warnings on patch-shebangs phase. Signed-off-by: Ludovic Courtès <ludo@gnu.org>
		
			
				
	
	
		
			48 lines
		
	
	
	
		
			1.6 KiB
		
	
	
	
		
			Diff
		
	
	
	
	
	
			
		
		
	
	
			48 lines
		
	
	
	
		
			1.6 KiB
		
	
	
	
		
			Diff
		
	
	
	
	
	
From 9ca0b2955edea162b255b428e493cd8ffac52167 Mon Sep 17 00:00:00 2001
 | 
						|
From: =?UTF-8?q?Miguel=20=C3=81ngel=20Arruga=20Vivas?=
 | 
						|
 <rosen644835@gmail.com>
 | 
						|
Date: Fri, 1 Nov 2019 17:29:00 +0100
 | 
						|
Subject: [PATCH] vircgroup: Ensure /machine group is associated with its
 | 
						|
 parent.
 | 
						|
 | 
						|
Call first virCgroupNew on the parent group virCgroupNewPartition if
 | 
						|
it is available on before the creation of the child group.  This
 | 
						|
ensures that the creation of a first level group on the unified
 | 
						|
architecture, as the check at virCgroupV2ParseControllersFile as the
 | 
						|
parent file is there.
 | 
						|
 | 
						|
Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1760233
 | 
						|
---
 | 
						|
 src/util/vircgroup.c | 8 +++++---
 | 
						|
 1 file changed, 5 insertions(+), 3 deletions(-)
 | 
						|
 | 
						|
diff --git a/src/util/vircgroup.c b/src/util/vircgroup.c
 | 
						|
index b46f20abfd..33c61f2d45 100644
 | 
						|
--- a/src/util/vircgroup.c
 | 
						|
+++ b/src/util/vircgroup.c
 | 
						|
@@ -855,9 +855,6 @@ virCgroupNewPartition(const char *path,
 | 
						|
     if (virCgroupSetPartitionSuffix(path, &newPath) < 0)
 | 
						|
         goto cleanup;
 | 
						|
 
 | 
						|
-    if (virCgroupNew(-1, newPath, NULL, controllers, group) < 0)
 | 
						|
-        goto cleanup;
 | 
						|
-
 | 
						|
     if (STRNEQ(newPath, "/")) {
 | 
						|
         char *tmp;
 | 
						|
         parentPath = g_strdup(newPath);
 | 
						|
@@ -868,7 +865,12 @@ virCgroupNewPartition(const char *path,
 | 
						|
 
 | 
						|
         if (virCgroupNew(-1, parentPath, NULL, controllers, &parent) < 0)
 | 
						|
             goto cleanup;
 | 
						|
+    }
 | 
						|
 
 | 
						|
+    if (virCgroupNew(-1, newPath, parent, controllers, group) < 0)
 | 
						|
+        goto cleanup;
 | 
						|
+
 | 
						|
+    if (parent) {
 | 
						|
         if (virCgroupMakeGroup(parent, *group, create, VIR_CGROUP_NONE) < 0)
 | 
						|
             goto cleanup;
 | 
						|
     }
 | 
						|
-- 
 | 
						|
2.23.0
 | 
						|
 |