Messages in this thread | | | Date | Wed, 30 Mar 2011 08:24:19 -0700 | From | Randy Dunlap <> | Subject | Re: [PATCH] kmemleak: build kmemleak-test.c as a module-only |
| |
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.
thanks.
--- ~Randy *** Remember to use Documentation/SubmitChecklist when testing your code ***
| |