Re: [GIT PULL (updated)] kbuild updates

[Markus Trippelsdorf]

On Mon, Oct 15, 2007 at 10:50:18PM +0200, Sam Ravnborg wrote:
> 
> The kbuild patches has been rebased on top of -linus after
> the x86 merge.
> The patch that caused ARCH to be unset has been withdrawn for
> now. It had additional issues and I need to do additional changes
> to get it included - in other words it will await next merge window.
> 
> The following patches does almost clear my patch queue. I have some
> fixes pending but need some more time to check them out.
> On top of this I have a few reports from -mm that needs attention too.
> So in other words I expect a smaller set of patches for -rc1 or -rc2
> as my time permits.
> 
> In addition to latest pull request this one includes:
> - rename of
> 	CFLAGS => KBUILD_CFLAGS
> 	AFLAGS => KBUILD_AFLAGS
> 	CPPFLAGS => KBUILD_CPPFLAGS
>     rationale behind this was two-fold:
>     1) Do not let the CFLAGS environment variable impact all builds
>     2) Allow one to specify addition gcc options on the commandline
>        as make CLFAGS=-Os ...
> - introducing ccflags-y, asflags-y and ldflags-y as the future
>   replacement of the EXTRA_* variants.
> 
> 
> The majority (line wise) is update of generated files.
> The rest is almost all one-liners so nothing controversial.
> The file with most lines changed is makefiles.txt!
> 

Bisecting shows that:

commit f77bf01425b11947eeb3b5b54685212c302741b8
	Author: Sam Ravnborg <sam@neptun.(none)>
 	Date:   Mon Oct 15 22:25:06 2007 +0200

    kbuild: introduce ccflags-y, asflags-y and ldflags-y

breaks booting with grub here. Grub stops with error 28: 
Selected item cannot fit into memory.

Reverting the commit fixes the problem.
-- 
Markus
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/