Re: [PATCH] scripts: kallsyms: Use %zu to print 'size_t'
On 8.11.2013 03:45, Fabio Estevam wrote: > From: Fabio Estevam > > 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 Thanks, applied to kbuild.git#kbuild. Michal -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] scripts: kallsyms: Use %zu to print 'size_t'
On 8.11.2013 03:45, Fabio Estevam wrote: From: Fabio Estevam fabio.este...@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.este...@freescale.com Thanks, applied to kbuild.git#kbuild. Michal -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[PATCH] scripts: kallsyms: Use %zu to print 'size_t'
From: Fabio Estevam
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
---
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
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[PATCH] scripts: kallsyms: Use %zu to print 'size_t'
From: Fabio Estevam fabio.este...@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.este...@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
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
