[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20070713204817.GA3529@uranus.ravnborg.org>
Date: Fri, 13 Jul 2007 22:48:17 +0200
From: Sam Ravnborg <sam@...nborg.org>
To: Rusty Russell <rusty@...tcorp.com.au>
Cc: Matt Mackall <mpm@...enic.com>,
Jeremy Fitzhardinge <jeremy@...p.org>,
David Rientjes <rientjes@...gle.com>,
Andrew Morton <akpm@...ux-foundation.org>,
lkml - Kernel Mailing List <linux-kernel@...r.kernel.org>
Subject: Re: maps2-move-the-page-walker-code-to-lib.patch
On Tue, Jul 10, 2007 at 12:06:35PM +1000, Rusty Russell wrote:
> On Mon, 2007-07-09 at 15:22 -0500, Matt Mackall wrote:
> > On Sun, Jul 08, 2007 at 03:49:56PM +1000, Rusty Russell wrote:
> > > Matt wrote:
> > > > This lets it get shared outside of proc/ and linked in only when needed.
> > >
> > > Erk, this really belongs in the mm dir. You can use "lib-y +=
> > > pagewalk.o" there, and it seems to work as well as doing it in lib.
> >
> > Are you sure? I tested that a few months ago and it didn't work for
> > me. I seem to recall it wanted a chunk of makefile surgery.
>
> Well, I tested here and it worked unless I screwed something (producing
> lib.a in mm/ and linking it in). And when I turned off all the CONFIG
> options which required it, the symbols disappeared from the vmlinux...
I think you had some additional change somewhere...
kbuild will only look for lib.a files (produced from all files
listed with lib-y _except_ files listed with obj-y too).
kbuild by default know about lib/ but almost all archs
add the directory: arch/$(ARCH)/lib but no-one
add mm/.
See assignment to libs-y.
Add see the bad documentation in Documentation/kbuild/makefiles.txt
Sam
-
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