[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20120519214545.GA26721@arm.com>
Date: Sat, 19 May 2012 22:45:45 +0100
From: Catalin Marinas <catalin.marinas@....com>
To: Greg KH <gregkh@...uxfoundation.org>
Cc: Christoph Biedl <linux-kernel.bfrz@...chmal.in-ulm.de>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
"stable@...r.kernel.org" <stable@...r.kernel.org>,
"torvalds@...ux-foundation.org" <torvalds@...ux-foundation.org>,
"akpm@...ux-foundation.org" <akpm@...ux-foundation.org>,
"alan@...rguk.ukuu.org.uk" <alan@...rguk.ukuu.org.uk>,
Sami Liedes <sami.liedes@....fi>, Tejun Heo <tj@...nel.org>,
Christoph Lameter <cl@...ux-foundation.org>
Subject: Re: [ 31/54] kmemleak: Fix the kmemleak tracking of the percpu
areas with !SMP
On Sat, May 19, 2012 at 03:46:52PM +0100, Greg KH wrote:
> On Sat, May 19, 2012 at 03:27:33PM +0200, Christoph Biedl wrote:
> > Greg KH wrote...
> >
> > > 3.0-stable review patch. If anyone has any objections, please let me know.
> >
> > > From: Catalin Marinas <catalin.marinas@....com>
> > >
> > > commit 100d13c3b5b9410f604b86f5e0a34da64b8cf659 upstream.
> >
> > > + /* kmemleak tracks the percpu allocations separately */
> > > + kmemleak_free(fc);
> >
> > That one causes:
> >
> > mm/percpu.c: In function ‘setup_per_cpu_areas’:
> > mm/percpu.c:1877: error: implicit declaration of function ‘kmemleak_free’
> > make[1]: *** [mm/percpu.o] Error 1
> > make: *** [mm] Error 2
>
> Odd, it doesn't cause that problem here for me, what ARCH are you
> building for? I'm on x86-64.
This patch propagated to far in the past. Support for per-cpu areas in
kmemleak was added with commit f528f0b8e5 (kmemleak: Handle percpu
memory allocation) in 3.3-rc1. That patch introduced the correct
#include for kmemleak.h.
Commit 100d13c should only be applied to stable 3.3 and not earlier.
Thanks.
--
Catalin
--
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