[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

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 ? 
> 
> Yep.
> 
> > 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.
> 
> Bastian
> 
> -- 
> 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)/%,$@)
>  
> +$(BOOT_SPECIAL_TARGETS):
> +	$(Q)$(MAKE) ARCH=ppc64 $(build)=$(boot) $@

Why this ARCH=ppc64 ? 

Friendly,

Sven Luther



Reply to: