[PATCH] proc/softirqs: change softirqs info from possile_cpu to online_cpu

[PATCH] proc/softirqs: change softirqs info from possile_cpu to online_cpu

[enlin.mu]

From: Enlin Mu <enlin.mu@unisoc.com>

like /proc/interrupts,/proc/softirqs which shows
the number of softirq for each online CPU

Signed-off-by: Enlin Mu <enlin.mu@unisoc.com>
---
 fs/proc/softirqs.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/fs/proc/softirqs.c b/fs/proc/softirqs.c
index 04bb29721419..01698b8f3898 100644
--- a/fs/proc/softirqs.c
+++ b/fs/proc/softirqs.c
@@ -13,13 +13,13 @@ static int show_softirqs(struct seq_file *p, void *v)
 	int i, j;
 
 	seq_puts(p, "                    ");
-	for_each_possible_cpu(i)
+	for_each_online_cpu(i)
 		seq_printf(p, "CPU%-8d", i);
 	seq_putc(p, '\n');
 
 	for (i = 0; i < NR_SOFTIRQS; i++) {
 		seq_printf(p, "%12s:", softirq_to_name[i]);
-		for_each_possible_cpu(j)
+		for_each_online_cpu(j)
 			seq_put_decimal_ull_width(p, " ", kstat_softirqs_cpu(i, j), 10);
 		seq_putc(p, '\n');
 	}
-- 
2.25.1

Re: [PATCH] proc/softirqs: change softirqs info from possile_cpu to online_cpu

[Thomas Gleixner]

On Sat, Nov 09 2024 at 00:22, enlin mu wrote:

> From: Enlin Mu <enlin.mu@unisoc.com>
>
> like /proc/interrupts,/proc/softirqs which shows
> the number of softirq for each online CPU

This changes the output format of a file which is used by various
tools, i.e. this is an ABI change.

Did you validate that none of these tools relies on consecutive CPU
numbers even when CPUs in the middle of the number space are offline?

Thanks,

        tglx