github-mirror/linux
1
0
Fork 0
mirror of https://github.com/torvalds/linux.git synced 2026-11-07 06:17:02 +01:00
Code Issues Actions Packages Projects Releases Wiki Activity Graph

mm/vmalloc: use %p for pointer formatting

Browse Source 
Commit 45ec16908e ("mm: use %pK for /proc/vmallocinfo") introduced the
%pK in order not to leak kernel pointers.  Since commit ad67b74d24
("printk: hash addresses printed with %p") pointers are hashed by default
and the behaviour can be controlled by `hash_pointers' boot argument.  The
policy on %p is to not introduce new ones.  Removing %pK makes it possible
to remove its handling from the library.

Looking at the output, having the pointer in the output makes it
possible to distinguish the individual entries.

Use %p instead %pK.

/proc/vmallocinfo has mode 0400, so this change doesn't make kernel
pointer information available to unprivileged userspace.

Link: https://lore.kernel.org/20260918105013.UpdykT6j@linutronix.de
Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Reviewed-by: SJ Park <sj@kernel.org>
Reviewed-by: Uladzislau Rezki (Sony) <urezki@gmail.com>
This commit is contained in:
Sebastian Andrzej Siewior 2026-09-18 12:50:13 +02:00 committed by Andrew Morton
parent bcf7b0c3a0
commit 25aa09142e
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
mm/vmalloc.c
View File

@ -5323,7 +5323,7 @@ static void show_purge_info(struct seq_file *m)
for_each_vmap_node(vn) {
spin_lock(&vn->lazy.lock);
list_for_each_entry(va, &vn->lazy.head, list) {
seq_printf(m, "0x%pK-0x%pK %7ld unpurged vm_area\n",
seq_printf(m, "0x%p-0x%p %7ld unpurged vm_area\n",
(void *)va->va_start, (void *)va->va_end,
va_size(va));
}
@ -5346,7 +5346,7 @@ static void show_busy_info(struct seq_file *m)
list_for_each_entry(va, &vn->busy.head, list) {
if (!va->vm) {
if (va->flags & VMAP_RAM)
seq_printf(m, "0x%pK-0x%pK %7ld vm_map_ram\n",
seq_printf(m, "0x%p-0x%p %7ld vm_map_ram\n",
(void *)va->va_start, (void *)va->va_end,
va_size(va));
@ -5360,7 +5360,7 @@ static void show_busy_info(struct seq_file *m)
/* Pair with smp_wmb() in clear_vm_uninitialized_flag() */
smp_rmb();
seq_printf(m, "0x%pK-0x%pK %7ld",
seq_printf(m, "0x%p-0x%p %7ld",
v->addr, v->addr + v->size, v->size);
if (v->caller)