Re: mkvmlinuz patches broken
On Fri, May 25, 2007 at 08:00:30PM +0200, Bastian Blank wrote:
> On Fri, May 25, 2007 at 06:32:00PM +0200, Sven Luther wrote:
> > This is a problem only in your new kernel-package less build
> > infrastructure, or something related to newer upstreams, or something
> > which was present previously and not noted ?
> The problem is in upstream. All targets in arch/powerpc/boot depends
> against vmlinux which produces this modpost call.
> > I guess this is the modpost line which is problematic, right ?
> > Bastian, if i understand this right, it worked previously, because the
> > mkvmlinuz_support_install call was done *AFTER* all the installation
> > happened, while you do it earlier ?
> It did not show up as problem as kpkg always did make modules before
> make modules_install, which trashes a lot of io time.
> The attached patch seems to fix it by adding a new boot target class
> which don't depends against vmlinux.
> First study the enemy. Seek weakness.
> -- Romulan Commander, "Balance of Terror", stardate 1709.2
> diff --git a/arch/powerpc/Makefile b/arch/powerpc/Makefile
> index 6238b58..26003cf 100644
> --- a/arch/powerpc/Makefile
> +++ b/arch/powerpc/Makefile
> @@ -150,14 +150,19 @@ all: $(KBUILD_IMAGE)
> CPPFLAGS_vmlinux.lds := -Upowerpc
> BOOT_TARGETS = zImage zImage.initrd zImage.dts zImage.dts_initrd uImage
> +BOOT_SPECIAL_TARGETS = mkvmlinuz_support_install
> PHONY += $(BOOT_TARGETS)
> +PHONY += $(BOOT_SPECIAL_TARGETS)
> boot := arch/$(ARCH)/boot
> $(BOOT_TARGETS): vmlinux
> $(Q)$(MAKE) ARCH=ppc64 $(build)=$(boot) $(patsubst %,$(boot)/%,$@)
> + $(Q)$(MAKE) ARCH=ppc64 $(build)=$(boot) $@
Why this ARCH=ppc64 ?