[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <AANLkTim3-6vrS8uEEm+jsuSJ64z_Edq40+MqF-m2ve5=@mail.gmail.com>
Date: Wed, 30 Mar 2011 18:02:02 +0300
From: Daniel Baluta <daniel.baluta@...il.com>
To: Catalin Marinas <catalin.marinas@....com>, rdunlap@...otime.net
Cc: paulmck@...ux.vnet.ibm.com, akpm@...ux-foundation.org,
mingo@...e.hu, fweisbec@...il.com,
Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
penberg@...nel.org
Subject: Re: [PATCH] kmemleak: build kmemleak-test.c as a module-only
On Fri, Mar 25, 2011 at 4:05 PM, Daniel Baluta <dbaluta@...acom.com> wrote:
> On Wed, Mar 23, 2011 at 1:24 PM, Catalin Marinas
> <catalin.marinas@....com> wrote:
>> On Wed, 2011-03-16 at 18:16 +0000, Daniel Baluta wrote:
>>> mm/kmemleak-test.c is used to provide an example of how kmemleak
>>> tool works.
>>>
>>> Memory is leaked at module unload-time, so building the test
>>> in kernel (Y) makes the leaks impossible and the test useless.
>>>
>>> Qualify DEBUG_KMEMLEAK_TEST config symbol with "depends on m",
>>> to restrict module-only building.
>>>
>>> Signed-off-by: Daniel Baluta <dbaluta@...acom.com>
>>
>> Sorry for the delay.
>>
>> Acked-by: Catalin Marinas <catalin.marinas@....com>
>
> Who should pick this up?
>
> $ scripts/get_maintainer.pl lib/this.patch says:
> Andrew Morton <akpm@...ux-foundation.org> (commit_signer:8/56=14%)
> "Paul E. McKenney" <paulmck@...ux.vnet.ibm.com> (commit_signer:7/56=12%)
> Ingo Molnar <mingo@...e.hu> (commit_signer:5/56=9%)
> Frederic Weisbecker <fweisbec@...il.com> (commit_signer:5/56=9%)
> Randy Dunlap <rdunlap@...otime.net> (commit_signer:3/56=5%)
Catalin, Randy since lib/Kconfig.debug has no official maintainer
I think this should go up through your trees.
thanks,
Daniel.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists