public inbox for linux-ext4@vger.kernel.org
 help / color / mirror / Atom feed
From: Catalin Marinas <catalin.marinas@arm.com>
To: Ingo Molnar <mingo@elte.hu>
Cc: Alexey Fisher <bug-track@fisher-privat.net>,
	"Aneesh Kumar K.V" <aneesh.kumar@linux.vnet.ibm.com>,
	Pekka Enberg <penberg@cs.helsinki.fi>,
	Kernel Testers List <kernel-testers@vger.kernel.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	Sam Ravnborg <sam@ravnborg.org>,
	linux-ext4@vger.kernel.org
Subject: Re: ext4 memory leak (was Re: [PATCH] x86: _edata should include all .data.* sections on X86_64)
Date: Sat, 18 Jul 2009 23:33:35 +0100	[thread overview]
Message-ID: <1247956415.4236.9.camel@toshiba-laptop> (raw)
In-Reply-To: <20090718115556.GA31007@elte.hu>

On Sat, 2009-07-18 at 13:55 +0200, Ingo Molnar wrote:
> * Alexey Fisher <bug-track@fisher-privat.net> wrote:
> 
> > This patch work for me.
> 
> nice. Any leftovers that might be false positives and need 
> annotation?

With the latest mainline all the reports I get look like real leaks but
some of them are pretty difficult to debug. I have a kmemleak
development tree as well which, among other things like more
cond_resched() calls, scans all the task stacks (currently using
for_each_process) but it doesn't reduce the number of reports.

> We learned this with lockdep: the moment a typical x86 distro bootup 
> is 'warnings free', utility of the debugging facility increases 
> dramatically: people can standardize on 'kmemleak should never 
> produce warnings' workflows and distros can also start feeding 
> kmemleak reports into kerneloops.org or so.

Yes. It's also easy to identify recent commits causing leaks but
currently it looks like some of the have been around for some time
(though probably not so serious leaks).

-- 
Catalin


  parent reply	other threads:[~2009-07-18 22:34 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <4A5C20E5.6010203@fisher-privat.net>
     [not found] ` <84144f020907140019g511723dctb541f6333d1a082b@mail.gmail.com>
     [not found]   ` <4A5C41C8.7040904@fisher-privat.net>
     [not found]     ` <1247564356.28240.30.camel@pc1117.cambridge.arm.com>
     [not found]       ` <1247565175.28240.37.camel@pc1117.cambridge.arm.com>
     [not found]         ` <4A5C5A59.5080304@fisher-privat.net>
     [not found]           ` <1247567499.28240.48.camel@pc1117.cambridge.arm.com>
     [not found]             ` <4A5C5FD0.3020204@fisher-privat.net>
     [not found]               ` <4A5C5FD0.3020204-M18mAb7Tlt0yCq4wW13eYl6hYfS7NtTn@public.gmane.org>
2009-07-14 12:26                 ` ext4 memory leak (was Re: [PATCH] x86: _edata should include all .data.* sections on X86_64) Catalin Marinas
2009-07-15  8:03                   ` Aneesh Kumar K.V
2009-07-15  8:54                     ` Alexey Fisher
     [not found]                       ` <4A5D9939.3000500-M18mAb7Tlt0yCq4wW13eYl6hYfS7NtTn@public.gmane.org>
2009-07-18 11:55                         ` Ingo Molnar
     [not found]                           ` <20090718115556.GA31007-X9Un+BFzKDI@public.gmane.org>
2009-07-18 13:30                             ` Alexey Fisher
     [not found]                               ` <4A61CE59.3030905-M18mAb7Tlt0yCq4wW13eYl6hYfS7NtTn@public.gmane.org>
2009-07-18 22:44                                 ` Catalin Marinas
2009-07-18 22:33                           ` Catalin Marinas [this message]
2009-07-15 10:33                     ` 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=1247956415.4236.9.camel@toshiba-laptop \
    --to=catalin.marinas@arm.com \
    --cc=aneesh.kumar@linux.vnet.ibm.com \
    --cc=bug-track@fisher-privat.net \
    --cc=kernel-testers@vger.kernel.org \
    --cc=linux-ext4@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mingo@elte.hu \
    --cc=penberg@cs.helsinki.fi \
    --cc=sam@ravnborg.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