[PATCH] kbuild: Fix the wrong format specifier

zhangjiao2 posted 1 patch 1 year, 2 months ago
tools/objtool/check.c | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
[PATCH] kbuild: Fix the wrong format specifier
Posted by zhangjiao2 1 year, 2 months ago
From: zhang jiao <zhangjiao2@cmss.chinamobile.com>

The format specifier of "unsigned long" in printf()
should be "%lu", not "%ld".

Signed-off-by: zhang jiao <zhangjiao2@cmss.chinamobile.com>
---
 tools/objtool/check.c | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/tools/objtool/check.c b/tools/objtool/check.c
index 4ce176ad411f..9c36bb519cf3 100644
--- a/tools/objtool/check.c
+++ b/tools/objtool/check.c
@@ -4934,10 +4934,10 @@ int check(struct objtool_file *file)
 		disas_warned_funcs(file);
 
 	if (opts.stats) {
-		printf("nr_insns_visited: %ld\n", nr_insns_visited);
-		printf("nr_cfi: %ld\n", nr_cfi);
-		printf("nr_cfi_reused: %ld\n", nr_cfi_reused);
-		printf("nr_cfi_cache: %ld\n", nr_cfi_cache);
+		printf("nr_insns_visited: %lu\n", nr_insns_visited);
+		printf("nr_cfi: %lu\n", nr_cfi);
+		printf("nr_cfi_reused: %lu\n", nr_cfi_reused);
+		printf("nr_cfi_cache: %lu\n", nr_cfi_cache);
 	}
 
 out:
-- 
2.33.0
Re: [PATCH] kbuild: Fix the wrong format specifier
Posted by Peter Zijlstra 1 year, 2 months ago
On Mon, Dec 02, 2024 at 12:11:26PM +0800, zhangjiao2 wrote:
> From: zhang jiao <zhangjiao2@cmss.chinamobile.com>
> 
> The format specifier of "unsigned long" in printf()
> should be "%lu", not "%ld".

Your subject is wrong, also, who cares? Why did you write this patch?

> Signed-off-by: zhang jiao <zhangjiao2@cmss.chinamobile.com>
> ---
>  tools/objtool/check.c | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/tools/objtool/check.c b/tools/objtool/check.c
> index 4ce176ad411f..9c36bb519cf3 100644
> --- a/tools/objtool/check.c
> +++ b/tools/objtool/check.c
> @@ -4934,10 +4934,10 @@ int check(struct objtool_file *file)
>  		disas_warned_funcs(file);
>  
>  	if (opts.stats) {
> -		printf("nr_insns_visited: %ld\n", nr_insns_visited);
> -		printf("nr_cfi: %ld\n", nr_cfi);
> -		printf("nr_cfi_reused: %ld\n", nr_cfi_reused);
> -		printf("nr_cfi_cache: %ld\n", nr_cfi_cache);
> +		printf("nr_insns_visited: %lu\n", nr_insns_visited);
> +		printf("nr_cfi: %lu\n", nr_cfi);
> +		printf("nr_cfi_reused: %lu\n", nr_cfi_reused);
> +		printf("nr_cfi_cache: %lu\n", nr_cfi_cache);
>  	}
>  
>  out:
> -- 
> 2.33.0
> 
> 
>