lkml.org 
[lkml]   [2011]   [Mar]   [31]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: [PATCH] kmemleak: build kmemleak-test.c as a module-only
From
On Wed, Mar 30, 2011 at 6:24 PM, Randy Dunlap <rdunlap@xenotime.net> wrote:
> On Wed, 30 Mar 2011 18:02:02 +0300 Daniel Baluta wrote:
>
>> On Fri, Mar 25, 2011 at 4:05 PM, Daniel Baluta <dbaluta@ixiacom.com> wrote:
>> > On Wed, Mar 23, 2011 at 1:24 PM, Catalin Marinas
>> > <catalin.marinas@arm.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@ixiacom.com>
>> >>
>> >> Sorry for the delay.
>> >>
>> >> Acked-by: Catalin Marinas <catalin.marinas@arm.com>
>> >
>> > Who should pick this up?
>> >
>> > $ scripts/get_maintainer.pl lib/this.patch says:
>> > Andrew Morton <akpm@linux-foundation.org> (commit_signer:8/56=14%)
>> > "Paul E. McKenney" <paulmck@linux.vnet.ibm.com> (commit_signer:7/56=12%)
>> > Ingo Molnar <mingo@elte.hu> (commit_signer:5/56=9%)
>> > Frederic Weisbecker <fweisbec@gmail.com> (commit_signer:5/56=9%)
>> > Randy Dunlap <rdunlap@xenotime.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.
>
> I hadn't seen it until now, but I can merge it.

OK. thanks.


\
 
 \ /
  Last update: 2011-03-31 09:47    [W:0.039 / U:1.104 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site