linux-kernel - [PATCH] irq/matrix: simplify CPU selection logic

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [day] [month] [year] [list]

Message-ID: <20260121030549.45257-1-zhanxusheng@xiaomi.com>
Date: Wed, 21 Jan 2026 11:05:49 +0800
From: Zhan Xusheng <zhanxusheng1024@...il.com>
To: linux-kernel@...r.kernel.org
Cc: mingo@...nel.org,
	tglx@...utronix.de,
	Zhan Xusheng <zhanxusheng@...omi.com>
Subject: [PATCH] irq/matrix: simplify CPU selection logic

The functions matrix_find_best_cpu() and matrix_find_best_cpu_managed()
were refactored to simplify the CPU selection logic and improve
readability.

- In matrix_find_best_cpu(), the selection logic was clarified by ensuring
  that the `best_cpu` is only updated when a CPU has more available IRQ
  vectors than the previously selected CPU.

- In matrix_find_best_cpu_managed(), the logic was modified to select the
  CPU with the fewest `managed_allocated` IRQ vectors, ensuring that the
  `best_cpu` is only updated when a CPU has a strictly fewer number of
  allocated vectors. Previously, the `best_cpu` was updated even when the
  `managed_allocated` values were equal, which was unnecessary and
  inconsistent.

This change improves code clarity and ensures that CPU selection is more
predictable, especially in high-load or resource-constrained scenarios.

Impact:
- The change improves the readability of both functions and makes the CPU
  selection behavior more explicit and predictable.
- There is no functional change under normal operating conditions. The
  behavior change only affects scenarios where the `managed_allocated`
  value is equal between CPUs, which is rare and would have previously led
  to unnecessary updates to `best_cpu`.

Signed-off-by: Zhan Xusheng <zhanxusheng@...omi.com>
---
 kernel/irq/matrix.c | 18 ++++++++++++------
 1 file changed, 12 insertions(+), 6 deletions(-)

diff --git a/kernel/irq/matrix.c b/kernel/irq/matrix.c
index a50f2305a8dc..ed4b1e44dc1e 100644
--- a/kernel/irq/matrix.c
+++ b/kernel/irq/matrix.c
@@ -140,14 +140,17 @@ static unsigned int matrix_find_best_cpu(struct irq_matrix *m,
 
 	best_cpu = UINT_MAX;
 
+	/* Select the online CPU with the most available vectors */
 	for_each_cpu(cpu, msk) {
 		cm = per_cpu_ptr(m->maps, cpu);
 
-		if (!cm->online || cm->available <= maxavl)
+		if (!cm->online)
 			continue;
 
-		best_cpu = cpu;
-		maxavl = cm->available;
+		if (cm->available > maxavl) {
+			best_cpu = cpu;
+			maxavl = cm->available;
+		}
 	}
 	return best_cpu;
 }
@@ -161,14 +164,17 @@ static unsigned int matrix_find_best_cpu_managed(struct irq_matrix *m,
 
 	best_cpu = UINT_MAX;
 
+	/* Select the online CPU with the fewest managed allocated vectors */
 	for_each_cpu(cpu, msk) {
 		cm = per_cpu_ptr(m->maps, cpu);
 
-		if (!cm->online || cm->managed_allocated > allocated)
+		if (!cm->online)
 			continue;
 
-		best_cpu = cpu;
-		allocated = cm->managed_allocated;
+		if (cm->managed_allocated < allocated) {
+			best_cpu = cpu;
+			allocated = cm->managed_allocated;
+		}
 	}
 	return best_cpu;
 }
-- 
2.43.0