Re: Announce: Kernel Build for 2.5, Release 1.1 is available.
Tom Rini (trini@kernel.crashing.org)
Sat, 11 Aug 2001 18:44:47 -0700
On Sun, Aug 12, 2001 at 11:21:22AM +1000, Keith Owens wrote:
> On Sat, 11 Aug 2001 15:02:55 -0700, 
> Tom Rini <trini@kernel.crashing.org> wrote:
> >On Sun, Aug 12, 2001 at 01:03:00AM +1000, Keith Owens wrote:
> >
> >> Changes from Release 1.
> >[snip]
> >>   Document kbuild targets and C to assembler conversions.  As always,
> >>   Documentation/kbuild/kbuild-2.5.txt is your friend.
> >
> >Okay, I've played with this a bit on PPC, and got it working to boot :)
> >Now here's what I see as the slight problems with it.  At least on PPC
> >what we do is generate the offset bits, and then have another file,
> >arch/ppc/kernel/ppc_asm.h include that file and have some other useful
> >macros for .S files.  So any of the .S files which include ppc_asm.h
> >would need an additional
> >extra_aflags(foo.o $(src_includelist /arch/$(ARCH)))
> 
> That will be required for all asm code that includes offsets.h, on all
> architectures, I doubt there will be more than 10 on any arch.
Hopefully not much more than 10, I hope.
> The alternative of having code in some arch directory updating
> include/asm-$(ARCH)/offsets.h is worse.  It is a terrible design to
> have code in one makefile updating files in another directory.  It is a
> layer violation which is always a bad idea.
Right.  I figured it'd be worth pointing out, if nothing else.
-- 
Tom Rini (TR1265)
http://gate.crashing.org/~trini/
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/