cpuallocator: use clustered allocation only with hybrid cores.

Use explicit cluster info based allocation only on hybrid core platforms. In other cases omit it altogether only using cache group based allocation. These should provide identical groups but for cache groups we do have partial allocation implemented while for cluster groups we don't have ATM. Signed-off-by: Krisztian Litkey <[email protected]>
containers · Jun 26, 2024 · 430e7fc · 430e7fc
1 parent 06607a0
commit 430e7fc
Showing 1 changed file with 11 additions and 5 deletions.
diff --git a/pkg/cpuallocator/allocator.go b/pkg/cpuallocator/allocator.go
@@ -1072,11 +1072,17 @@ func (a *allocatorHelper) allocate() cpuset.CPUSet {
 		if (a.flags & AllocIdlePackages) != 0 {
 			a.takeIdlePackages()
 		}
-		if a.cnt > 0 && (a.flags&AllocIdleClusters) != 0 {
-			a.takeIdleClusters()
-		}
-		if a.cnt > 0 && (a.flags&AllocCacheGroups) != 0 {
-			a.takeCacheGroups()
+		if len(a.topology.kind) > 1 {
+			if a.cnt > 0 && (a.flags&AllocIdleClusters) != 0 {
+				a.takeIdleClusters()
+			}
+			if a.cnt > 0 && (a.flags&AllocCacheGroups) != 0 {
+				a.takeCacheGroups()
+			}
+		} else {
+			if a.cnt > 0 && (a.flags&AllocCacheGroups) != 0 {
+				a.takeCacheGroups()
+			}
 		}
 		if a.cnt > 0 && (a.flags&AllocIdleCores) != 0 {
 			a.takeIdleCores()