Skip to content

Commit 52c5d2b

Browse files
jimcakpm00
authored andcommitted
kmemleak: add checksum to backtrace report
Change /sys/kernel/debug/kmemleak report format slightly, adding "(extra info)" to the backtrace header: from: " backtrace:" to: " backtrace (crc <cksum>):" The <cksum> allows a user to see recurring backtraces without detailed/careful reading of multiline stacks. So after cycling kmemleak-test a few times, I know some leaks are repeating. bash-5.2# grep backtrace /sys/kernel/debug/kmemleak | wc 62 186 1792 bash-5.2# grep backtrace /sys/kernel/debug/kmemleak | sort -u | wc 37 111 1067 syzkaller parses kmemleak for "unreferenced object" only, so is unaffected by this change. Other github repos are moribund. Link: https://lkml.kernel.org/r/[email protected] Signed-off-by: Jim Cromie <[email protected]> Reviewed-by: Catalin Marinas <[email protected]> Signed-off-by: Andrew Morton <[email protected]>
1 parent 88f9ee2 commit 52c5d2b

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

mm/kmemleak.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -362,7 +362,7 @@ static void print_unreferenced(struct seq_file *seq,
362362
warn_or_seq_printf(seq, " comm \"%s\", pid %d, jiffies %lu\n",
363363
object->comm, object->pid, object->jiffies);
364364
hex_dump_object(seq, object);
365-
warn_or_seq_printf(seq, " backtrace:\n");
365+
warn_or_seq_printf(seq, " backtrace (crc %x):\n", object->checksum);
366366

367367
for (i = 0; i < nr_entries; i++) {
368368
void *ptr = (void *)entries[i];

0 commit comments

Comments
 (0)