The patch titled mm: convert sprintf_symbol to %pS has been added to the -mm tree. Its filename is mm-convert-sprintf_symbol-to-%ps.patch Before you just go and hit "reply", please: a) Consider who else should be cc'ed b) Prefer to cc a suitable mailing list as well c) Ideally: find the original patch on the mailing list and do a reply-to-all to that, adding suitable additional cc's *** Remember to use Documentation/SubmitChecklist when testing your code *** See http://userweb.kernel.org/~akpm/stuff/added-to-mm.txt to find out what to do about this The current -mm tree may be found at http://userweb.kernel.org/~akpm/mmotm/ ------------------------------------------------------ Subject: mm: convert sprintf_symbol to %pS From: Joe Perches <joe@xxxxxxxxxxx> Signed-off-by: Joe Perches <joe@xxxxxxxxxxx> Acked-by: Pekka Enberg <penberg@xxxxxxxxxx> Cc: Jiri Kosina <trivial@xxxxxxxxxx> Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx> --- mm/slub.c | 11 ++++------- mm/vmalloc.c | 9 ++------- 2 files changed, 6 insertions(+), 14 deletions(-) diff -puN mm/slub.c~mm-convert-sprintf_symbol-to-%ps mm/slub.c --- a/mm/slub.c~mm-convert-sprintf_symbol-to-%ps +++ a/mm/slub.c @@ -3676,7 +3676,7 @@ static int list_locations(struct kmem_ca len += sprintf(buf + len, "%7ld ", l->count); if (l->addr) - len += sprint_symbol(buf + len, (unsigned long)l->addr); + len += sprintf(buf + len, "%pS", (void *)l->addr); else len += sprintf(buf + len, "<not-available>"); @@ -3986,12 +3986,9 @@ SLAB_ATTR(min_partial); static ssize_t ctor_show(struct kmem_cache *s, char *buf) { - if (s->ctor) { - int n = sprint_symbol(buf, (unsigned long)s->ctor); - - return n + sprintf(buf + n, "\n"); - } - return 0; + if (!s->ctor) + return 0; + return sprintf(buf, "%pS\n", s->ctor); } SLAB_ATTR_RO(ctor); diff -puN mm/vmalloc.c~mm-convert-sprintf_symbol-to-%ps mm/vmalloc.c --- a/mm/vmalloc.c~mm-convert-sprintf_symbol-to-%ps +++ a/mm/vmalloc.c @@ -2507,13 +2507,8 @@ static int s_show(struct seq_file *m, vo seq_printf(m, "0x%p-0x%p %7ld", v->addr, v->addr + v->size, v->size); - if (v->caller) { - char buff[KSYM_SYMBOL_LEN]; - - seq_putc(m, ' '); - sprint_symbol(buff, (unsigned long)v->caller); - seq_puts(m, buff); - } + if (v->caller) + seq_printf(m, " %pS", v->caller); if (v->nr_pages) seq_printf(m, " pages=%d", v->nr_pages); _ Patches currently in -mm which might be from joe@xxxxxxxxxxx are origin.patch linux-next.patch mm-convert-sprintf_symbol-to-%ps.patch scripts-get_maintainerpl-make-rolestats-the-default.patch maintainers-intel-gfx-is-a-subscribers-only-mailing-list.patch fs-proc-basec-kernel-latencytopc-convert-sprintf_symbol-to-%ps.patch -- To unsubscribe from this list: send the line "unsubscribe mm-commits" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html