linux-kernel - Re: [PATCH 03/13] sparc/irq: use seq_put_decimal_ull

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>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <69e24f81-c9fd-4815-b803-c6d094b84e9d@gaisler.com>
Date: Fri, 17 Jan 2025 16:28:29 +0100
From: Andreas Larsson <andreas@...sler.com>
To: David Wang <00107082@....com>, davem@...emloft.net
Cc: linux-kernel@...r.kernel.org, sparclinux@...r.kernel.org
Subject: Re: [PATCH 03/13] sparc/irq: use seq_put_decimal_ull_width() for
 decimal values

On 2024-11-08 17:11, David Wang wrote:
> Performance improvement for reading /proc/interrupts on arch sparc
> 
> Signed-off-by: David Wang <00107082@....com>
> ---
>  arch/sparc/kernel/irq_32.c | 12 ++++++------
>  arch/sparc/kernel/irq_64.c |  4 ++--
>  2 files changed, 8 insertions(+), 8 deletions(-)
> 
> diff --git a/arch/sparc/kernel/irq_32.c b/arch/sparc/kernel/irq_32.c
> index 8605dd710f3c..5210991429d5 100644
> --- a/arch/sparc/kernel/irq_32.c
> +++ b/arch/sparc/kernel/irq_32.c
> @@ -199,18 +199,18 @@ int arch_show_interrupts(struct seq_file *p, int prec)
>  	int j;
>  
>  #ifdef CONFIG_SMP
> -	seq_printf(p, "RES: ");
> +	seq_printf(p, "RES:");
>  	for_each_online_cpu(j)
> -		seq_printf(p, "%10u ", cpu_data(j).irq_resched_count);
> +		seq_put_decimal_ull_width(p, " ", cpu_data(j).irq_resched_count, 10);
>  	seq_printf(p, "     IPI rescheduling interrupts\n");
> -	seq_printf(p, "CAL: ");
> +	seq_printf(p, "CAL:");
>  	for_each_online_cpu(j)
> -		seq_printf(p, "%10u ", cpu_data(j).irq_call_count);
> +		seq_put_decimal_ull_width(p, " ", cpu_data(j).irq_call_count, 10);
>  	seq_printf(p, "     IPI function call interrupts\n");
>  #endif
> -	seq_printf(p, "NMI: ");
> +	seq_printf(p, "NMI:");
>  	for_each_online_cpu(j)
> -		seq_printf(p, "%10u ", cpu_data(j).counter);
> +		seq_put_decimal_ull_width(p, " ", cpu_data(j).counter, 10);
>  	seq_printf(p, "     Non-maskable interrupts\n");
>  	return 0;
>  }
> diff --git a/arch/sparc/kernel/irq_64.c b/arch/sparc/kernel/irq_64.c
> index 01ee800efde3..9ab6e79b617b 100644
> --- a/arch/sparc/kernel/irq_64.c
> +++ b/arch/sparc/kernel/irq_64.c
> @@ -304,9 +304,9 @@ int arch_show_interrupts(struct seq_file *p, int prec)
>  {
>  	int j;
>  
> -	seq_printf(p, "NMI: ");
> +	seq_printf(p, "NMI:");
>  	for_each_online_cpu(j)
> -		seq_printf(p, "%10u ", cpu_data(j).__nmi_count);
> +		seq_put_decimal_ull_width(p, " ", cpu_data(j).__nmi_count, 10);
>  	seq_printf(p, "     Non-maskable interrupts\n");
>  	return 0;
>  }

Reviewed-by: Andreas Larsson <andreas@...sler.com>
Tested-by: Andreas Larsson <andreas@...sler.com>

Picking this up to my for-next.

Thanks,
Andreas