Messages in this thread | | | Date | Thu, 31 Mar 2011 10:44:45 +0300 | Subject | Re: [PATCH] kmemleak: build kmemleak-test.c as a module-only | From | Daniel Baluta <> |
| |
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.
| |