* [PATCH] scripts: kallsyms: Use %zu to print 'size_t'
@ 2013-11-08 2:45 Fabio Estevam
2013-11-08 9:12 ` Michal Marek
0 siblings, 1 reply; 2+ messages in thread
From: Fabio Estevam @ 2013-11-08 2:45 UTC (permalink / raw)
To: mmarek; +Cc: andi, linux-kernel, Fabio Estevam
From: Fabio Estevam <fabio.estevam@freescale.com>
Commit f3462aa95 (Kbuild: Handle longer symbols in kallsyms.c) introduced the
following warning on ARM:
scripts/kallsyms.c:121:4: warning: format '%lu' expects argument of type 'long unsigned int', but argument 4 has type 'size_t' [-Wformat]
Use %zu to print 'size_t'.
Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
---
scripts/kallsyms.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/scripts/kallsyms.c b/scripts/kallsyms.c
index 11552e0..990695c 100644
--- a/scripts/kallsyms.c
+++ b/scripts/kallsyms.c
@@ -116,7 +116,7 @@ static int read_symbol(FILE *in, struct sym_entry *s)
return -1;
}
if (strlen(str) > KSYM_NAME_LEN) {
- fprintf(stderr, "Symbol %s too long for kallsyms (%lu vs %d).\n"
+ fprintf(stderr, "Symbol %s too long for kallsyms (%zu vs %d).\n"
"Please increase KSYM_NAME_LEN both in kernel and kallsyms.c\n",
str, strlen(str), KSYM_NAME_LEN);
return -1;
--
1.8.1.2
^ permalink raw reply related [flat|nested] 2+ messages in thread* Re: [PATCH] scripts: kallsyms: Use %zu to print 'size_t'
2013-11-08 2:45 [PATCH] scripts: kallsyms: Use %zu to print 'size_t' Fabio Estevam
@ 2013-11-08 9:12 ` Michal Marek
0 siblings, 0 replies; 2+ messages in thread
From: Michal Marek @ 2013-11-08 9:12 UTC (permalink / raw)
To: Fabio Estevam; +Cc: andi, linux-kernel, Fabio Estevam
On 8.11.2013 03:45, Fabio Estevam wrote:
> From: Fabio Estevam <fabio.estevam@freescale.com>
>
> Commit f3462aa95 (Kbuild: Handle longer symbols in kallsyms.c) introduced the
> following warning on ARM:
>
> scripts/kallsyms.c:121:4: warning: format '%lu' expects argument of type 'long unsigned int', but argument 4 has type 'size_t' [-Wformat]
Actually, every 32bit glibc seems to define size_t as unsigned int.
>
> Use %zu to print 'size_t'.
>
> Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
Thanks, applied to kbuild.git#kbuild.
Michal
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2013-11-08 9:12 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-11-08 2:45 [PATCH] scripts: kallsyms: Use %zu to print 'size_t' Fabio Estevam
2013-11-08 9:12 ` Michal Marek
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox