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 · 05afcb5 · 05afcb5
1 parent ffbd73b
commit 05afcb5
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()