public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Ingo Molnar <mingo@elte.hu>
To: Catalin Marinas <catalin.marinas@gmail.com>
Cc: linux-kernel@vger.kernel.org
Subject: Re: [PATCH 2.6.20-rc1 00/10] Kernel memory leak detector 0.13
Date: Sun, 17 Dec 2006 10:41:43 +0100	[thread overview]
Message-ID: <20061217094143.GA15372@elte.hu> (raw)
In-Reply-To: <20061217092828.GA14181@elte.hu>


it would also be nice to have more information than this:

 unreferenced object 0xf76f5af8 (size 512):
  [<c0191f23>] memleak_alloc
  [<c018eeaa>] kmem_cache_zalloc
  [<c03277a7>] probe_hwif
  [<c032870c>] probe_hwif_init_with_fixup
  [<c032aea1>] ide_setup_pci_device
  [<c0312564>] amd74xx_probe
  [<c069c4b4>] ide_scan_pcidev
  [<c069c505>] ide_scan_pcibus
  [<c069bdca>] ide_init
  [<c0100532>] init
  [<c0105da3>] kernel_thread_helper
  [<ffffffff>]

it would be nice to record 1) the jiffies value at the time of 
allocation, 2) the PID and the comm of the task that did the allocation. 
The jiffies timestamp would be useful to see the age of the allocation, 
and the PID/comm is useful for context.

plus it would be nice to have a kernel thread (running at nice 19) that 
scans everything every 10 minutes or so, which would output some 
overview 'delta' information into the syslog, along the lines of:

	kmemleak: 2 new object leaks detected, see /debug/memleak for details

that way users could see (and report) new leaks in the dmesg, without 
having to do anything. I'd like to enable kmemleak in the -rt yum 
kernels, so it all has to be as automatic and as informative as 
possible.

	Ingo

  reply	other threads:[~2006-12-17  9:43 UTC|newest]

Thread overview: 41+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-12-16 15:34 [PATCH 2.6.20-rc1 00/10] Kernel memory leak detector 0.13 Catalin Marinas
2006-12-16 15:34 ` [PATCH 2.6.20-rc1 01/10] Base support for kmemleak Catalin Marinas
2006-12-16 15:34 ` [PATCH 2.6.20-rc1 02/10] Kmemleak documentation Catalin Marinas
2006-12-16 15:34 ` [PATCH 2.6.20-rc1 03/10] Add the memory allocation/freeing hooks for kmemleak Catalin Marinas
2006-12-16 15:34 ` [PATCH 2.6.20-rc1 04/10] Modules support " Catalin Marinas
2006-12-16 15:35 ` [PATCH 2.6.20-rc1 05/10] Add kmemleak support for i386 Catalin Marinas
2006-12-16 15:35 ` [PATCH 2.6.20-rc1 06/10] Add kmemleak support for ARM Catalin Marinas
2006-12-16 15:35 ` [PATCH 2.6.20-rc1 07/10] Remove some of the kmemleak false positives Catalin Marinas
2006-12-16 15:35 ` [PATCH 2.6.20-rc1 08/10] Keep the __init functions after initialization Catalin Marinas
2006-12-16 15:35 ` [PATCH 2.6.20-rc1 09/10] Testing module for kmemleak Catalin Marinas
2006-12-16 15:36 ` [PATCH 2.6.20-rc1 10/10] Update the MAINTAINERS file " Catalin Marinas
2006-12-16 16:57 ` [PATCH 2.6.20-rc1 00/10] Kernel memory leak detector 0.13 Ingo Molnar
2006-12-16 23:39   ` Catalin Marinas
2006-12-17  8:58     ` Ingo Molnar
2006-12-17  9:09       ` Ingo Molnar
2006-12-17  9:28         ` Ingo Molnar
2006-12-17  9:41           ` Ingo Molnar [this message]
2006-12-17  9:49             ` Ingo Molnar
2006-12-17 12:05             ` Catalin Marinas
2006-12-27 16:14             ` Catalin Marinas
2006-12-27 16:23               ` Arjan van de Ven
2006-12-27 16:30                 ` Catalin Marinas
2006-12-27 16:47               ` Ingo Molnar
2006-12-27 16:56                 ` Ingo Molnar
2006-12-27 17:02                 ` Catalin Marinas
2006-12-17 11:58           ` Catalin Marinas
2006-12-17 13:28             ` Ingo Molnar
2006-12-17 11:09       ` Mike Galbraith
2006-12-17 23:05       ` Catalin Marinas
2006-12-18  7:29         ` Ingo Molnar
2006-12-18 10:28           ` Catalin Marinas
2006-12-18 11:21             ` Ingo Molnar
2006-12-18 12:26               ` Catalin Marinas
2006-12-18 19:56                 ` Ingo Molnar
2006-12-19  9:36                   ` Catalin Marinas
2006-12-27 13:52           ` Catalin Marinas
2006-12-27 15:08             ` Ingo Molnar
2006-12-27 17:30               ` Ingo Molnar
2006-12-28  0:15                 ` Catalin Marinas
2006-12-28  9:44                   ` Ingo Molnar
2006-12-28 11:50                     ` Catalin Marinas

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20061217094143.GA15372@elte.hu \
    --to=mingo@elte.hu \
    --cc=catalin.marinas@gmail.com \
    --cc=linux-kernel@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox