* [PATCH] xen: arm: make zImage the default target which we install @ 2013-07-12 11:59 Ian Campbell 2013-07-15 8:38 ` [PATCH] xen: allow architecture to choose whether to compress installed xen binary Ian Campbell 2013-07-18 12:05 ` [PATCH] xen: arm: make zImage the default target which we install Julien Grall 0 siblings, 2 replies; 15+ messages in thread From: Ian Campbell @ 2013-07-12 11:59 UTC (permalink / raw) To: xen-devel; +Cc: julien.grall, Wookey, tim, Ian Campbell, stefano.stabellini From: Ian Campbell <ian.campbell@citrix.com> The zImage compatible binary is the useful one on real hardware. The relocated ELF thing is only really useful when booting directly on Fast Models. The customary suffix for that case is .axf so provide that as a target. Signed-off-by: Ian Campbell <ian.campbell@citrix.com> Cc: Wookey <wookey@linaro.org> --- xen/arch/arm/Makefile | 13 ++++--------- 1 file changed, 4 insertions(+), 9 deletions(-) diff --git a/xen/arch/arm/Makefile b/xen/arch/arm/Makefile index 87fabe1..0fba75c 100644 --- a/xen/arch/arm/Makefile +++ b/xen/arch/arm/Makefile @@ -40,20 +40,15 @@ endif ALL_OBJS := $(TARGET_SUBARCH)/head.o $(ALL_OBJS) -$(TARGET): $(TARGET)-syms $(TARGET).bin +$(TARGET): $(TARGET)-syms $(TARGET).axf + $(OBJCOPY) -O binary -S $< $@ + +$(TARGET).axf: $(TARGET)-syms # XXX: VE model loads by VMA so instead of # making a proper ELF we link with LMA == VMA and adjust crudely $(OBJCOPY) --change-addresses +0x80000000 $< $@ $(STRIP) $@ -# -$(TARGET).bin: $(TARGET)-syms - $(OBJCOPY) -O binary -S $< $@ - -#$(TARGET): $(TARGET)-syms $(efi-y) boot/mkelf32 -# ./boot/mkelf32 $(TARGET)-syms $(TARGET) 0x100000 \ -# `$(NM) -nr $(TARGET)-syms | head -n 1 | sed -e 's/^\([^ ]*\).*/0x\1/'` - ifeq ($(lto),y) # Gather all LTO objects together prelink_lto.o: $(ALL_OBJS) -- 1.8.3.2 ^ permalink raw reply related [flat|nested] 15+ messages in thread
* [PATCH] xen: allow architecture to choose whether to compress installed xen binary 2013-07-12 11:59 [PATCH] xen: arm: make zImage the default target which we install Ian Campbell @ 2013-07-15 8:38 ` Ian Campbell 2013-07-15 9:44 ` Jan Beulich 2013-07-18 12:05 ` [PATCH] xen: arm: make zImage the default target which we install Julien Grall 1 sibling, 1 reply; 15+ messages in thread From: Ian Campbell @ 2013-07-15 8:38 UTC (permalink / raw) To: xen-devel Cc: keir, Ian Campbell, stefano.stabellini, tim, julien.grall, jbeulich From: Ian Campbell <ian.campbell@citrix.com> This is a follow up to "xen: arm: make zImage the default target which we install". On ARM the xen.gz binary installed into /boot is not immediately useful because bootloaders (e.g. u-boot) do not unconditionally support decompression (except via the uImage wrapper, which we currently do not support via our build system) Signed-off-by: Ian Campbell <ian.campbell@citrix.com> --- config/arm32.mk | 2 ++ config/arm64.mk | 2 ++ config/x86_64.mk | 2 ++ xen/Makefile | 26 ++++++++++++++++---------- 4 files changed, 22 insertions(+), 10 deletions(-) diff --git a/config/arm32.mk b/config/arm32.mk index d8e958b..f38ca3a 100644 --- a/config/arm32.mk +++ b/config/arm32.mk @@ -2,6 +2,8 @@ CONFIG_ARM := y CONFIG_ARM_32 := y CONFIG_ARM_$(XEN_OS) := y +CONFIG_INSTALL_COMPRESSED_XEN := n + # -march= -mcpu= # Explicitly specifiy 32-bit ARM ISA since toolchain default can be -mthumb: diff --git a/config/arm64.mk b/config/arm64.mk index b2457eb..c625d02 100644 --- a/config/arm64.mk +++ b/config/arm64.mk @@ -2,6 +2,8 @@ CONFIG_ARM := y CONFIG_ARM_64 := y CONFIG_ARM_$(XEN_OS) := y +CONFIG_INSTALL_COMPRESSED_XEN := n + CFLAGS += #-marm -march= -mcpu= etc HAS_PL011 := y diff --git a/config/x86_64.mk b/config/x86_64.mk index 4ec6cc1..be3273d 100644 --- a/config/x86_64.mk +++ b/config/x86_64.mk @@ -8,6 +8,8 @@ CONFIG_MIGRATE := y CONFIG_XCUTILS := y CONFIG_IOEMU ?= y +CONFIG_INSTALL_COMPRESSED_XEN := y + CFLAGS += -m64 SunOS_LIBDIR = $(SunOS_LIBDIR_x86_64) diff --git a/xen/Makefile b/xen/Makefile index 854a370..8898d3f 100644 --- a/xen/Makefile +++ b/xen/Makefile @@ -28,16 +28,22 @@ else echo "*** Xen x86/32 target no longer supported!" endif +ifeq ($(CONFIG_INSTALL_COMPRESSED_XEN),y) +GZ := .gz +else +GZ := +endif + .PHONY: _build -_build: $(TARGET).gz +_build: $(TARGET)$(GZ) .PHONY: _install -_install: $(TARGET).gz +_install: $(TARGET)$(GZ) [ -d $(DESTDIR)/boot ] || $(INSTALL_DIR) $(DESTDIR)/boot - $(INSTALL_DATA) $(TARGET).gz $(DESTDIR)/boot/$(notdir $(TARGET))-$(XEN_FULLVERSION).gz - ln -f -s $(notdir $(TARGET))-$(XEN_FULLVERSION).gz $(DESTDIR)/boot/$(notdir $(TARGET))-$(XEN_VERSION).$(XEN_SUBVERSION).gz - ln -f -s $(notdir $(TARGET))-$(XEN_FULLVERSION).gz $(DESTDIR)/boot/$(notdir $(TARGET))-$(XEN_VERSION).gz - ln -f -s $(notdir $(TARGET))-$(XEN_FULLVERSION).gz $(DESTDIR)/boot/$(notdir $(TARGET)).gz + $(INSTALL_DATA) $(TARGET)$(GZ) $(DESTDIR)/boot/$(notdir $(TARGET))-$(XEN_FULLVERSION)$(GZ) + ln -f -s $(notdir $(TARGET))-$(XEN_FULLVERSION)$(GZ) $(DESTDIR)/boot/$(notdir $(TARGET))-$(XEN_VERSION).$(XEN_SUBVERSION)$(GZ) + ln -f -s $(notdir $(TARGET))-$(XEN_FULLVERSION)$(GZ) $(DESTDIR)/boot/$(notdir $(TARGET))-$(XEN_VERSION)$(GZ) + ln -f -s $(notdir $(TARGET))-$(XEN_FULLVERSION)$(GZ) $(DESTDIR)/boot/$(notdir $(TARGET))$(GZ) $(INSTALL_DATA) $(TARGET)-syms $(DESTDIR)/boot/$(notdir $(TARGET))-syms-$(XEN_FULLVERSION) if [ -r $(TARGET).efi -a -n '$(EFI_DIR)' ]; then \ [ -d $(DESTDIR)$(EFI_DIR) ] || $(INSTALL_DIR) $(DESTDIR)$(EFI_DIR); \ @@ -56,10 +62,10 @@ _install: $(TARGET).gz _uninstall: D=$(DESTDIR) _uninstall: T=$(notdir $(TARGET)) _uninstall: - rm -f $(D)/boot/$(T)-$(XEN_FULLVERSION).gz - rm -f $(D)/boot/$(T)-$(XEN_VERSION).$(XEN_SUBVERSION).gz - rm -f $(D)/boot/$(T)-$(XEN_VERSION).gz - rm -f $(D)/boot/$(T).gz + rm -f $(D)/boot/$(T)-$(XEN_FULLVERSION)$(GZ) + rm -f $(D)/boot/$(T)-$(XEN_VERSION).$(XEN_SUBVERSION)$(GZ) + rm -f $(D)/boot/$(T)-$(XEN_VERSION)$(GZ) + rm -f $(D)/boot/$(T)$(GZ) rm -f $(D)/boot/$(T)-syms-$(XEN_FULLVERSION) rm -f $(D)$(EFI_DIR)/$(T)-$(XEN_FULLVERSION).efi rm -f $(D)$(EFI_DIR)/$(T)-$(XEN_VERSION).$(XEN_SUBVERSION).efi -- 1.8.3.2 ^ permalink raw reply related [flat|nested] 15+ messages in thread
* Re: [PATCH] xen: allow architecture to choose whether to compress installed xen binary 2013-07-15 8:38 ` [PATCH] xen: allow architecture to choose whether to compress installed xen binary Ian Campbell @ 2013-07-15 9:44 ` Jan Beulich 2013-07-16 9:15 ` Ian Campbell 0 siblings, 1 reply; 15+ messages in thread From: Jan Beulich @ 2013-07-15 9:44 UTC (permalink / raw) To: Ian Campbell Cc: keir, Ian Campbell, stefano.stabellini, tim, xen-devel, julien.grall >>> On 15.07.13 at 10:38, Ian Campbell <ijc@hellion.org.uk> wrote: > This is a follow up to "xen: arm: make zImage the default target which we > install". > > On ARM the xen.gz binary installed into /boot is not immediately useful > because > bootloaders (e.g. u-boot) do not unconditionally support decompression > (except > via the uImage wrapper, which we currently do not support via our build > system) There are two changes I'd like to suggest: > --- a/config/arm32.mk > +++ b/config/arm32.mk > @@ -2,6 +2,8 @@ CONFIG_ARM := y > CONFIG_ARM_32 := y > CONFIG_ARM_$(XEN_OS) := y > > +CONFIG_INSTALL_COMPRESSED_XEN := n Rename it suitably, leave it empty for ARM and set it to .gz on x86, allowing it to also be used for other purposes (like a different suffix if e.g. another compression method is supported on some platform). > -_install: $(TARGET).gz > +_install: $(TARGET)$(GZ) > [ -d $(DESTDIR)/boot ] || $(INSTALL_DIR) $(DESTDIR)/boot > - $(INSTALL_DATA) $(TARGET).gz $(DESTDIR)/boot/$(notdir > $(TARGET))-$(XEN_FULLVERSION).gz > - ln -f -s $(notdir $(TARGET))-$(XEN_FULLVERSION).gz $(DESTDIR)/boot/$(notdir > $(TARGET))-$(XEN_VERSION).$(XEN_SUBVERSION).gz > - ln -f -s $(notdir $(TARGET))-$(XEN_FULLVERSION).gz $(DESTDIR)/boot/$(notdir > $(TARGET))-$(XEN_VERSION).gz > - ln -f -s $(notdir $(TARGET))-$(XEN_FULLVERSION).gz $(DESTDIR)/boot/$(notdir > $(TARGET)).gz > + $(INSTALL_DATA) $(TARGET)$(GZ) $(DESTDIR)/boot/$(notdir > $(TARGET))-$(XEN_FULLVERSION)$(GZ) > + ln -f -s $(notdir $(TARGET))-$(XEN_FULLVERSION)$(GZ) $(DESTDIR)/boot/$(notdir > $(TARGET))-$(XEN_VERSION).$(XEN_SUBVERSION)$(GZ) > + ln -f -s $(notdir $(TARGET))-$(XEN_FULLVERSION)$(GZ) $(DESTDIR)/boot/$(notdir > $(TARGET))-$(XEN_VERSION)$(GZ) > + ln -f -s $(notdir $(TARGET))-$(XEN_FULLVERSION)$(GZ) $(DESTDIR)/boot/$(notdir As you're touching this block already, let's drop the .gz extension on the destination altogether - under /boot that's pretty meaningless anyway. Jan ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH] xen: allow architecture to choose whether to compress installed xen binary 2013-07-15 9:44 ` Jan Beulich @ 2013-07-16 9:15 ` Ian Campbell 2013-07-16 9:20 ` Jan Beulich 0 siblings, 1 reply; 15+ messages in thread From: Ian Campbell @ 2013-07-16 9:15 UTC (permalink / raw) To: Jan Beulich; +Cc: julien.grall, tim, xen-devel, keir, stefano.stabellini On Mon, 2013-07-15 at 10:44 +0100, Jan Beulich wrote: > >>> On 15.07.13 at 10:38, Ian Campbell <ijc@hellion.org.uk> wrote: > > This is a follow up to "xen: arm: make zImage the default target which we > > install". > > > > On ARM the xen.gz binary installed into /boot is not immediately useful > > because > > bootloaders (e.g. u-boot) do not unconditionally support decompression > > (except > > via the uImage wrapper, which we currently do not support via our build > > system) > > There are two changes I'd like to suggest: > > > --- a/config/arm32.mk > > +++ b/config/arm32.mk > > @@ -2,6 +2,8 @@ CONFIG_ARM := y > > CONFIG_ARM_32 := y > > CONFIG_ARM_$(XEN_OS) := y > > > > +CONFIG_INSTALL_COMPRESSED_XEN := n > > Rename it suitably, leave it empty for ARM and set it to .gz on x86, > allowing it to also be used for other purposes (like a different > suffix if e.g. another compression method is supported on some > platform). Heh, my first (unposted) version did exactly that but I decided people would prefer this way as a less intrusive change. I will change it back and use CONFIG_INSTALL_XEN_SUFFIX perhaps? > > > -_install: $(TARGET).gz > > +_install: $(TARGET)$(GZ) > > [ -d $(DESTDIR)/boot ] || $(INSTALL_DIR) $(DESTDIR)/boot > > - $(INSTALL_DATA) $(TARGET).gz $(DESTDIR)/boot/$(notdir > > $(TARGET))-$(XEN_FULLVERSION).gz > > - ln -f -s $(notdir $(TARGET))-$(XEN_FULLVERSION).gz $(DESTDIR)/boot/$(notdir > > $(TARGET))-$(XEN_VERSION).$(XEN_SUBVERSION).gz > > - ln -f -s $(notdir $(TARGET))-$(XEN_FULLVERSION).gz $(DESTDIR)/boot/$(notdir > > $(TARGET))-$(XEN_VERSION).gz > > - ln -f -s $(notdir $(TARGET))-$(XEN_FULLVERSION).gz $(DESTDIR)/boot/$(notdir > > $(TARGET)).gz > > + $(INSTALL_DATA) $(TARGET)$(GZ) $(DESTDIR)/boot/$(notdir > > $(TARGET))-$(XEN_FULLVERSION)$(GZ) > > + ln -f -s $(notdir $(TARGET))-$(XEN_FULLVERSION)$(GZ) $(DESTDIR)/boot/$(notdir > > $(TARGET))-$(XEN_VERSION).$(XEN_SUBVERSION)$(GZ) > > + ln -f -s $(notdir $(TARGET))-$(XEN_FULLVERSION)$(GZ) $(DESTDIR)/boot/$(notdir > > $(TARGET))-$(XEN_VERSION)$(GZ) > > + ln -f -s $(notdir $(TARGET))-$(XEN_FULLVERSION)$(GZ) $(DESTDIR)/boot/$(notdir > > As you're touching this block already, let's drop the .gz extension > on the destination altogether - under /boot that's pretty > meaningless anyway. I'm slightly concerned about breaking bootloaders (or more specifically their tooling like update-grub, grubby etc) here. I'm happy to make the change but would prefer to do it as a separate patch (which negates your "As you're touching" comment). Ian. ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH] xen: allow architecture to choose whether to compress installed xen binary 2013-07-16 9:15 ` Ian Campbell @ 2013-07-16 9:20 ` Jan Beulich 2013-07-17 11:13 ` Ian Campbell 2013-07-17 11:13 ` [PATCH] xen: x86: drop the ".gz" suffix when installing Ian Campbell 0 siblings, 2 replies; 15+ messages in thread From: Jan Beulich @ 2013-07-16 9:20 UTC (permalink / raw) To: Ian Campbell; +Cc: julien.grall, tim, xen-devel, keir, stefano.stabellini >>> On 16.07.13 at 11:15, Ian Campbell <ijc@hellion.org.uk> wrote: > On Mon, 2013-07-15 at 10:44 +0100, Jan Beulich wrote: >> >>> On 15.07.13 at 10:38, Ian Campbell <ijc@hellion.org.uk> wrote: >> > This is a follow up to "xen: arm: make zImage the default target which we >> > install". >> > >> > On ARM the xen.gz binary installed into /boot is not immediately useful >> > because >> > bootloaders (e.g. u-boot) do not unconditionally support decompression >> > (except >> > via the uImage wrapper, which we currently do not support via our build >> > system) >> >> There are two changes I'd like to suggest: >> >> > --- a/config/arm32.mk >> > +++ b/config/arm32.mk >> > @@ -2,6 +2,8 @@ CONFIG_ARM := y >> > CONFIG_ARM_32 := y >> > CONFIG_ARM_$(XEN_OS) := y >> > >> > +CONFIG_INSTALL_COMPRESSED_XEN := n >> >> Rename it suitably, leave it empty for ARM and set it to .gz on x86, >> allowing it to also be used for other purposes (like a different >> suffix if e.g. another compression method is supported on some >> platform). > > Heh, my first (unposted) version did exactly that but I decided people > would prefer this way as a less intrusive change. I will change it back > and use CONFIG_INSTALL_XEN_SUFFIX perhaps? Yes, please. >> > -_install: $(TARGET).gz >> > +_install: $(TARGET)$(GZ) >> > [ -d $(DESTDIR)/boot ] || $(INSTALL_DIR) $(DESTDIR)/boot >> > - $(INSTALL_DATA) $(TARGET).gz $(DESTDIR)/boot/$(notdir >> > $(TARGET))-$(XEN_FULLVERSION).gz >> > - ln -f -s $(notdir $(TARGET))-$(XEN_FULLVERSION).gz > $(DESTDIR)/boot/$(notdir >> > $(TARGET))-$(XEN_VERSION).$(XEN_SUBVERSION).gz >> > - ln -f -s $(notdir $(TARGET))-$(XEN_FULLVERSION).gz > $(DESTDIR)/boot/$(notdir >> > $(TARGET))-$(XEN_VERSION).gz >> > - ln -f -s $(notdir $(TARGET))-$(XEN_FULLVERSION).gz > $(DESTDIR)/boot/$(notdir >> > $(TARGET)).gz >> > + $(INSTALL_DATA) $(TARGET)$(GZ) $(DESTDIR)/boot/$(notdir >> > $(TARGET))-$(XEN_FULLVERSION)$(GZ) >> > + ln -f -s $(notdir $(TARGET))-$(XEN_FULLVERSION)$(GZ) > $(DESTDIR)/boot/$(notdir >> > $(TARGET))-$(XEN_VERSION).$(XEN_SUBVERSION)$(GZ) >> > + ln -f -s $(notdir $(TARGET))-$(XEN_FULLVERSION)$(GZ) > $(DESTDIR)/boot/$(notdir >> > $(TARGET))-$(XEN_VERSION)$(GZ) >> > + ln -f -s $(notdir $(TARGET))-$(XEN_FULLVERSION)$(GZ) > $(DESTDIR)/boot/$(notdir >> >> As you're touching this block already, let's drop the .gz extension >> on the destination altogether - under /boot that's pretty >> meaningless anyway. > > I'm slightly concerned about breaking bootloaders (or more specifically > their tooling like update-grub, grubby etc) here. I'm happy to make the > change but would prefer to do it as a separate patch (which negates your > "As you're touching" comment). Fine with me. Jan ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH] xen: allow architecture to choose whether to compress installed xen binary 2013-07-16 9:20 ` Jan Beulich @ 2013-07-17 11:13 ` Ian Campbell 2013-07-17 11:31 ` Jan Beulich 2013-07-17 11:13 ` [PATCH] xen: x86: drop the ".gz" suffix when installing Ian Campbell 1 sibling, 1 reply; 15+ messages in thread From: Ian Campbell @ 2013-07-17 11:13 UTC (permalink / raw) To: Jan Beulich; +Cc: julien.grall, keir, stefano.stabellini, tim, xen-devel On Tue, 2013-07-16 at 10:20 +0100, Jan Beulich wrote: > >>> On 16.07.13 at 11:15, Ian Campbell <ijc@hellion.org.uk> wrote: > > On Mon, 2013-07-15 at 10:44 +0100, Jan Beulich wrote: > >> >>> On 15.07.13 at 10:38, Ian Campbell <ijc@hellion.org.uk> wrote: > >> > This is a follow up to "xen: arm: make zImage the default target which we > >> > install". > >> > > >> > On ARM the xen.gz binary installed into /boot is not immediately useful > >> > because > >> > bootloaders (e.g. u-boot) do not unconditionally support decompression > >> > (except > >> > via the uImage wrapper, which we currently do not support via our build > >> > system) > >> > >> There are two changes I'd like to suggest: > >> > >> > --- a/config/arm32.mk > >> > +++ b/config/arm32.mk > >> > @@ -2,6 +2,8 @@ CONFIG_ARM := y > >> > CONFIG_ARM_32 := y > >> > CONFIG_ARM_$(XEN_OS) := y > >> > > >> > +CONFIG_INSTALL_COMPRESSED_XEN := n > >> > >> Rename it suitably, leave it empty for ARM and set it to .gz on x86, > >> allowing it to also be used for other purposes (like a different > >> suffix if e.g. another compression method is supported on some > >> platform). > > > > Heh, my first (unposted) version did exactly that but I decided people > > would prefer this way as a less intrusive change. I will change it back > > and use CONFIG_INSTALL_XEN_SUFFIX perhaps? > > Yes, please. In the end I thought CONFIG_XEN_INSTALL_SUFFIX was more natural, here it is. 8<------------------------------------ commit e814a4a403d4e0e1e76e9c80f9906fe972ad0a2d Author: Ian Campbell <ian.campbell@citrix.com> Date: Fri Jul 12 13:15:35 2013 +0100 xen: allow architecture to choose how/whether to compress installed xen binary This is a follow up to "xen: arm: make zImage the default target which we install". On ARM the xen.gz binary installed into /boot is not immediately useful because bootloaders (e.g. u-boot) do not unconditionally support decompression (except via the uImage wrapper, which we currently do not support via our build system) Signed-off-by: Ian Campbell <ian.campbell@citrix.com> --- v2: Allowarch to specify the compression suffix to use diff --git a/config/arm32.mk b/config/arm32.mk index d8e958b..3bc642d 100644 --- a/config/arm32.mk +++ b/config/arm32.mk @@ -2,6 +2,8 @@ CONFIG_ARM := y CONFIG_ARM_32 := y CONFIG_ARM_$(XEN_OS) := y +CONFIG_XEN_INSTALL_SUFFIX := + # -march= -mcpu= # Explicitly specifiy 32-bit ARM ISA since toolchain default can be -mthumb: diff --git a/config/arm64.mk b/config/arm64.mk index b2457eb..1130630 100644 --- a/config/arm64.mk +++ b/config/arm64.mk @@ -2,6 +2,8 @@ CONFIG_ARM := y CONFIG_ARM_64 := y CONFIG_ARM_$(XEN_OS) := y +CONFIG_XEN_INSTALL_SUFFIX := + CFLAGS += #-marm -march= -mcpu= etc HAS_PL011 := y diff --git a/config/x86_64.mk b/config/x86_64.mk index 4ec6cc1..70c0d8d 100644 --- a/config/x86_64.mk +++ b/config/x86_64.mk @@ -8,6 +8,8 @@ CONFIG_MIGRATE := y CONFIG_XCUTILS := y CONFIG_IOEMU ?= y +CONFIG_XEN_INSTALL_SUFFIX := .gz + CFLAGS += -m64 SunOS_LIBDIR = $(SunOS_LIBDIR_x86_64) diff --git a/xen/Makefile b/xen/Makefile index 854a370..343e789 100644 --- a/xen/Makefile +++ b/xen/Makefile @@ -28,16 +28,19 @@ else echo "*** Xen x86/32 target no longer supported!" endif +# Define as an alias to avoid amazingly long lines +Z := $(CONFIG_XEN_INSTALL_SUFFIX) + .PHONY: _build -_build: $(TARGET).gz +_build: $(TARGET)$(Z) .PHONY: _install -_install: $(TARGET).gz +_install: $(TARGET)$(Z) [ -d $(DESTDIR)/boot ] || $(INSTALL_DIR) $(DESTDIR)/boot - $(INSTALL_DATA) $(TARGET).gz $(DESTDIR)/boot/$(notdir $(TARGET))-$(XEN_FULLVERSION).gz - ln -f -s $(notdir $(TARGET))-$(XEN_FULLVERSION).gz $(DESTDIR)/boot/$(notdir $(TARGET))-$(XEN_VERSION).$(XEN_SUBVERSION).gz - ln -f -s $(notdir $(TARGET))-$(XEN_FULLVERSION).gz $(DESTDIR)/boot/$(notdir $(TARGET))-$(XEN_VERSION).gz - ln -f -s $(notdir $(TARGET))-$(XEN_FULLVERSION).gz $(DESTDIR)/boot/$(notdir $(TARGET)).gz + $(INSTALL_DATA) $(TARGET)$(Z) $(DESTDIR)/boot/$(notdir $(TARGET))-$(XEN_FULLVERSION)$(Z) + ln -f -s $(notdir $(TARGET))-$(XEN_FULLVERSION)$(Z) $(DESTDIR)/boot/$(notdir $(TARGET))-$(XEN_VERSION).$(XEN_SUBVERSION)$(Z) + ln -f -s $(notdir $(TARGET))-$(XEN_FULLVERSION)$(Z) $(DESTDIR)/boot/$(notdir $(TARGET))-$(XEN_VERSION)$(Z) + ln -f -s $(notdir $(TARGET))-$(XEN_FULLVERSION)$(Z) $(DESTDIR)/boot/$(notdir $(TARGET))$(Z) $(INSTALL_DATA) $(TARGET)-syms $(DESTDIR)/boot/$(notdir $(TARGET))-syms-$(XEN_FULLVERSION) if [ -r $(TARGET).efi -a -n '$(EFI_DIR)' ]; then \ [ -d $(DESTDIR)$(EFI_DIR) ] || $(INSTALL_DIR) $(DESTDIR)$(EFI_DIR); \ @@ -56,10 +59,10 @@ _install: $(TARGET).gz _uninstall: D=$(DESTDIR) _uninstall: T=$(notdir $(TARGET)) _uninstall: - rm -f $(D)/boot/$(T)-$(XEN_FULLVERSION).gz - rm -f $(D)/boot/$(T)-$(XEN_VERSION).$(XEN_SUBVERSION).gz - rm -f $(D)/boot/$(T)-$(XEN_VERSION).gz - rm -f $(D)/boot/$(T).gz + rm -f $(D)/boot/$(T)-$(XEN_FULLVERSION)$(Z) + rm -f $(D)/boot/$(T)-$(XEN_VERSION).$(XEN_SUBVERSION)$(Z) + rm -f $(D)/boot/$(T)-$(XEN_VERSION)$(Z) + rm -f $(D)/boot/$(T)$(Z) rm -f $(D)/boot/$(T)-syms-$(XEN_FULLVERSION) rm -f $(D)$(EFI_DIR)/$(T)-$(XEN_FULLVERSION).efi rm -f $(D)$(EFI_DIR)/$(T)-$(XEN_VERSION).$(XEN_SUBVERSION).efi ^ permalink raw reply related [flat|nested] 15+ messages in thread
* Re: [PATCH] xen: allow architecture to choose whether to compress installed xen binary 2013-07-17 11:13 ` Ian Campbell @ 2013-07-17 11:31 ` Jan Beulich 2013-07-17 11:37 ` Ian Campbell 0 siblings, 1 reply; 15+ messages in thread From: Jan Beulich @ 2013-07-17 11:31 UTC (permalink / raw) To: Ian Campbell; +Cc: julien.grall, tim, xen-devel, keir, stefano.stabellini >>> On 17.07.13 at 13:13, Ian Campbell <ijc@hellion.org.uk> wrote: > On Tue, 2013-07-16 at 10:20 +0100, Jan Beulich wrote: > In the end I thought CONFIG_XEN_INSTALL_SUFFIX was more natural, here it > is. Ah, here it comes (out of order with the other one for some reason). > --- a/xen/Makefile > +++ b/xen/Makefile > @@ -28,16 +28,19 @@ else > echo "*** Xen x86/32 target no longer supported!" > endif > > +# Define as an alias to avoid amazingly long lines > +Z := $(CONFIG_XEN_INSTALL_SUFFIX) I'd prefer this to be limited in scope to _install and _uninstall (just like T and D get set for _uninstall [but for an unknown reason the same isn't being done for _install - perhaps that's something you could clean up as you go without risking any breakage]): > @@ -56,10 +59,10 @@ _install: $(TARGET).gz > _uninstall: D=$(DESTDIR) > _uninstall: T=$(notdir $(TARGET)) Jan ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH] xen: allow architecture to choose whether to compress installed xen binary 2013-07-17 11:31 ` Jan Beulich @ 2013-07-17 11:37 ` Ian Campbell 0 siblings, 0 replies; 15+ messages in thread From: Ian Campbell @ 2013-07-17 11:37 UTC (permalink / raw) To: Jan Beulich; +Cc: julien.grall, keir, stefano.stabellini, tim, xen-devel On Wed, 2013-07-17 at 12:31 +0100, Jan Beulich wrote: > >>> On 17.07.13 at 13:13, Ian Campbell <ijc@hellion.org.uk> wrote: > > On Tue, 2013-07-16 at 10:20 +0100, Jan Beulich wrote: > > In the end I thought CONFIG_XEN_INSTALL_SUFFIX was more natural, here it > > is. > > Ah, here it comes (out of order with the other one for some > reason). Yeah, sorry about that. > > --- a/xen/Makefile > > +++ b/xen/Makefile > > @@ -28,16 +28,19 @@ else > > echo "*** Xen x86/32 target no longer supported!" > > endif > > > > +# Define as an alias to avoid amazingly long lines > > +Z := $(CONFIG_XEN_INSTALL_SUFFIX) > > I'd prefer this to be limited in scope to _install and _uninstall (just > like T and D get set for _uninstall [but for an unknown reason the > same isn't being done for _install - perhaps that's something you > could clean up as you go without risking any breakage]): I'll look into it. BTW this one is needed by _build too. Ian. ^ permalink raw reply [flat|nested] 15+ messages in thread
* [PATCH] xen: x86: drop the ".gz" suffix when installing 2013-07-16 9:20 ` Jan Beulich 2013-07-17 11:13 ` Ian Campbell @ 2013-07-17 11:13 ` Ian Campbell 2013-07-17 11:26 ` Jan Beulich 1 sibling, 1 reply; 15+ messages in thread From: Ian Campbell @ 2013-07-17 11:13 UTC (permalink / raw) To: xen-devel; +Cc: keir, Ian Campbell, jbeulich As Jan says it is pretty meaningless under /boot anyway. However I am slightly concerned about breaking bootloaders (or more specifically their help scripts which automatically generate config files). By inspection at leastr grub 2's update-grub script (as present in Debian Wheezy) seems to cope (it matches on xen* not xen*.gz) Signed-off-by: Ian Campbell <ian.campbell@citrix.com> Cc: jbeulich@suse.com Cc: keir@xen.org --- xen/Makefile | 16 ++++++++-------- 1 files changed, 8 insertions(+), 8 deletions(-) diff --git a/xen/Makefile b/xen/Makefile index 343e789..ef8a1cd 100644 --- a/xen/Makefile +++ b/xen/Makefile @@ -37,10 +37,10 @@ _build: $(TARGET)$(Z) .PHONY: _install _install: $(TARGET)$(Z) [ -d $(DESTDIR)/boot ] || $(INSTALL_DIR) $(DESTDIR)/boot - $(INSTALL_DATA) $(TARGET)$(Z) $(DESTDIR)/boot/$(notdir $(TARGET))-$(XEN_FULLVERSION)$(Z) - ln -f -s $(notdir $(TARGET))-$(XEN_FULLVERSION)$(Z) $(DESTDIR)/boot/$(notdir $(TARGET))-$(XEN_VERSION).$(XEN_SUBVERSION)$(Z) - ln -f -s $(notdir $(TARGET))-$(XEN_FULLVERSION)$(Z) $(DESTDIR)/boot/$(notdir $(TARGET))-$(XEN_VERSION)$(Z) - ln -f -s $(notdir $(TARGET))-$(XEN_FULLVERSION)$(Z) $(DESTDIR)/boot/$(notdir $(TARGET))$(Z) + $(INSTALL_DATA) $(TARGET)$(Z) $(DESTDIR)/boot/$(notdir $(TARGET))-$(XEN_FULLVERSION) + ln -f -s $(notdir $(TARGET))-$(XEN_FULLVERSION) $(DESTDIR)/boot/$(notdir $(TARGET))-$(XEN_VERSION).$(XEN_SUBVERSION) + ln -f -s $(notdir $(TARGET))-$(XEN_FULLVERSION) $(DESTDIR)/boot/$(notdir $(TARGET))-$(XEN_VERSION) + ln -f -s $(notdir $(TARGET))-$(XEN_FULLVERSION) $(DESTDIR)/boot/$(notdir $(TARGET)) $(INSTALL_DATA) $(TARGET)-syms $(DESTDIR)/boot/$(notdir $(TARGET))-syms-$(XEN_FULLVERSION) if [ -r $(TARGET).efi -a -n '$(EFI_DIR)' ]; then \ [ -d $(DESTDIR)$(EFI_DIR) ] || $(INSTALL_DIR) $(DESTDIR)$(EFI_DIR); \ @@ -59,10 +59,10 @@ _install: $(TARGET)$(Z) _uninstall: D=$(DESTDIR) _uninstall: T=$(notdir $(TARGET)) _uninstall: - rm -f $(D)/boot/$(T)-$(XEN_FULLVERSION)$(Z) - rm -f $(D)/boot/$(T)-$(XEN_VERSION).$(XEN_SUBVERSION)$(Z) - rm -f $(D)/boot/$(T)-$(XEN_VERSION)$(Z) - rm -f $(D)/boot/$(T)$(Z) + rm -f $(D)/boot/$(T)-$(XEN_FULLVERSION) + rm -f $(D)/boot/$(T)-$(XEN_VERSION).$(XEN_SUBVERSION) + rm -f $(D)/boot/$(T)-$(XEN_VERSION) + rm -f $(D)/boot/$(T) rm -f $(D)/boot/$(T)-syms-$(XEN_FULLVERSION) rm -f $(D)$(EFI_DIR)/$(T)-$(XEN_FULLVERSION).efi rm -f $(D)$(EFI_DIR)/$(T)-$(XEN_VERSION).$(XEN_SUBVERSION).efi -- 1.7.2.5 ^ permalink raw reply related [flat|nested] 15+ messages in thread
* Re: [PATCH] xen: x86: drop the ".gz" suffix when installing 2013-07-17 11:13 ` [PATCH] xen: x86: drop the ".gz" suffix when installing Ian Campbell @ 2013-07-17 11:26 ` Jan Beulich 2013-07-17 11:35 ` Ian Campbell 0 siblings, 1 reply; 15+ messages in thread From: Jan Beulich @ 2013-07-17 11:26 UTC (permalink / raw) To: Ian Campbell; +Cc: keir, xen-devel >>> On 17.07.13 at 13:13, Ian Campbell <ian.campbell@citrix.com> wrote: > As Jan says it is pretty meaningless under /boot anyway. However I am > slightly > concerned about breaking bootloaders (or more specifically their help > scripts > which automatically generate config files). By inspection at leastr grub 2's > update-grub script (as present in Debian Wheezy) seems to cope (it matches on > xen* not xen*.gz) Fine with me (obviously), but ... > --- a/xen/Makefile > +++ b/xen/Makefile > @@ -37,10 +37,10 @@ _build: $(TARGET)$(Z) > .PHONY: _install > _install: $(TARGET)$(Z) ... where are all those $(Z) coming from? The staging tree doesn't have this, and I also don't recall having see a respective patch (and you also don't mention such a patch as prerequisite). Jan > [ -d $(DESTDIR)/boot ] || $(INSTALL_DIR) $(DESTDIR)/boot > - $(INSTALL_DATA) $(TARGET)$(Z) $(DESTDIR)/boot/$(notdir > $(TARGET))-$(XEN_FULLVERSION)$(Z) > - ln -f -s $(notdir $(TARGET))-$(XEN_FULLVERSION)$(Z) $(DESTDIR)/boot/$(notdir > $(TARGET))-$(XEN_VERSION).$(XEN_SUBVERSION)$(Z) > - ln -f -s $(notdir $(TARGET))-$(XEN_FULLVERSION)$(Z) $(DESTDIR)/boot/$(notdir > $(TARGET))-$(XEN_VERSION)$(Z) > - ln -f -s $(notdir $(TARGET))-$(XEN_FULLVERSION)$(Z) $(DESTDIR)/boot/$(notdir > $(TARGET))$(Z) > + $(INSTALL_DATA) $(TARGET)$(Z) $(DESTDIR)/boot/$(notdir > $(TARGET))-$(XEN_FULLVERSION) > + ln -f -s $(notdir $(TARGET))-$(XEN_FULLVERSION) $(DESTDIR)/boot/$(notdir > $(TARGET))-$(XEN_VERSION).$(XEN_SUBVERSION) > + ln -f -s $(notdir $(TARGET))-$(XEN_FULLVERSION) $(DESTDIR)/boot/$(notdir > $(TARGET))-$(XEN_VERSION) > + ln -f -s $(notdir $(TARGET))-$(XEN_FULLVERSION) $(DESTDIR)/boot/$(notdir > $(TARGET)) > $(INSTALL_DATA) $(TARGET)-syms $(DESTDIR)/boot/$(notdir > $(TARGET))-syms-$(XEN_FULLVERSION) > if [ -r $(TARGET).efi -a -n '$(EFI_DIR)' ]; then \ > [ -d $(DESTDIR)$(EFI_DIR) ] || $(INSTALL_DIR) $(DESTDIR)$(EFI_DIR); \ > @@ -59,10 +59,10 @@ _install: $(TARGET)$(Z) > _uninstall: D=$(DESTDIR) > _uninstall: T=$(notdir $(TARGET)) > _uninstall: > - rm -f $(D)/boot/$(T)-$(XEN_FULLVERSION)$(Z) > - rm -f $(D)/boot/$(T)-$(XEN_VERSION).$(XEN_SUBVERSION)$(Z) > - rm -f $(D)/boot/$(T)-$(XEN_VERSION)$(Z) > - rm -f $(D)/boot/$(T)$(Z) > + rm -f $(D)/boot/$(T)-$(XEN_FULLVERSION) > + rm -f $(D)/boot/$(T)-$(XEN_VERSION).$(XEN_SUBVERSION) > + rm -f $(D)/boot/$(T)-$(XEN_VERSION) > + rm -f $(D)/boot/$(T) > rm -f $(D)/boot/$(T)-syms-$(XEN_FULLVERSION) > rm -f $(D)$(EFI_DIR)/$(T)-$(XEN_FULLVERSION).efi > rm -f $(D)$(EFI_DIR)/$(T)-$(XEN_VERSION).$(XEN_SUBVERSION).efi > -- > 1.7.2.5 ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH] xen: x86: drop the ".gz" suffix when installing 2013-07-17 11:26 ` Jan Beulich @ 2013-07-17 11:35 ` Ian Campbell 0 siblings, 0 replies; 15+ messages in thread From: Ian Campbell @ 2013-07-17 11:35 UTC (permalink / raw) To: Jan Beulich; +Cc: keir, xen-devel On Wed, 2013-07-17 at 12:26 +0100, Jan Beulich wrote: > >>> On 17.07.13 at 13:13, Ian Campbell <ian.campbell@citrix.com> wrote: > > As Jan says it is pretty meaningless under /boot anyway. However I am > > slightly > > concerned about breaking bootloaders (or more specifically their help > > scripts > > which automatically generate config files). By inspection at leastr grub 2's > > update-grub script (as present in Debian Wheezy) seems to cope (it matches on > > xen* not xen*.gz) > > Fine with me (obviously), but ... > > > --- a/xen/Makefile > > +++ b/xen/Makefile > > @@ -37,10 +37,10 @@ _build: $(TARGET)$(Z) > > .PHONY: _install > > _install: $(TARGET)$(Z) > > ... where are all those $(Z) coming from? The staging tree doesn't > have this, and I also don't recall having see a respective patch > (and you also don't mention such a patch as prerequisite). It was my reworked "xen: allow architecture to choose whether to compress installed xen binary" to which this was a reply (or perhaps a sibling). Anyway, it was <1374059599.8960.32.camel@kazak.uk.xensource.com> Sorry I should have made this clearer. > > Jan > > > [ -d $(DESTDIR)/boot ] || $(INSTALL_DIR) $(DESTDIR)/boot > > - $(INSTALL_DATA) $(TARGET)$(Z) $(DESTDIR)/boot/$(notdir > > $(TARGET))-$(XEN_FULLVERSION)$(Z) > > - ln -f -s $(notdir $(TARGET))-$(XEN_FULLVERSION)$(Z) $(DESTDIR)/boot/$(notdir > > $(TARGET))-$(XEN_VERSION).$(XEN_SUBVERSION)$(Z) > > - ln -f -s $(notdir $(TARGET))-$(XEN_FULLVERSION)$(Z) $(DESTDIR)/boot/$(notdir > > $(TARGET))-$(XEN_VERSION)$(Z) > > - ln -f -s $(notdir $(TARGET))-$(XEN_FULLVERSION)$(Z) $(DESTDIR)/boot/$(notdir > > $(TARGET))$(Z) > > + $(INSTALL_DATA) $(TARGET)$(Z) $(DESTDIR)/boot/$(notdir > > $(TARGET))-$(XEN_FULLVERSION) > > + ln -f -s $(notdir $(TARGET))-$(XEN_FULLVERSION) $(DESTDIR)/boot/$(notdir > > $(TARGET))-$(XEN_VERSION).$(XEN_SUBVERSION) > > + ln -f -s $(notdir $(TARGET))-$(XEN_FULLVERSION) $(DESTDIR)/boot/$(notdir > > $(TARGET))-$(XEN_VERSION) > > + ln -f -s $(notdir $(TARGET))-$(XEN_FULLVERSION) $(DESTDIR)/boot/$(notdir > > $(TARGET)) > > $(INSTALL_DATA) $(TARGET)-syms $(DESTDIR)/boot/$(notdir > > $(TARGET))-syms-$(XEN_FULLVERSION) > > if [ -r $(TARGET).efi -a -n '$(EFI_DIR)' ]; then \ > > [ -d $(DESTDIR)$(EFI_DIR) ] || $(INSTALL_DIR) $(DESTDIR)$(EFI_DIR); \ > > @@ -59,10 +59,10 @@ _install: $(TARGET)$(Z) > > _uninstall: D=$(DESTDIR) > > _uninstall: T=$(notdir $(TARGET)) > > _uninstall: > > - rm -f $(D)/boot/$(T)-$(XEN_FULLVERSION)$(Z) > > - rm -f $(D)/boot/$(T)-$(XEN_VERSION).$(XEN_SUBVERSION)$(Z) > > - rm -f $(D)/boot/$(T)-$(XEN_VERSION)$(Z) > > - rm -f $(D)/boot/$(T)$(Z) > > + rm -f $(D)/boot/$(T)-$(XEN_FULLVERSION) > > + rm -f $(D)/boot/$(T)-$(XEN_VERSION).$(XEN_SUBVERSION) > > + rm -f $(D)/boot/$(T)-$(XEN_VERSION) > > + rm -f $(D)/boot/$(T) > > rm -f $(D)/boot/$(T)-syms-$(XEN_FULLVERSION) > > rm -f $(D)$(EFI_DIR)/$(T)-$(XEN_FULLVERSION).efi > > rm -f $(D)$(EFI_DIR)/$(T)-$(XEN_VERSION).$(XEN_SUBVERSION).efi > > -- > > 1.7.2.5 > > > ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH] xen: arm: make zImage the default target which we install 2013-07-12 11:59 [PATCH] xen: arm: make zImage the default target which we install Ian Campbell 2013-07-15 8:38 ` [PATCH] xen: allow architecture to choose whether to compress installed xen binary Ian Campbell @ 2013-07-18 12:05 ` Julien Grall 2013-07-18 12:10 ` Ian Campbell 1 sibling, 1 reply; 15+ messages in thread From: Julien Grall @ 2013-07-18 12:05 UTC (permalink / raw) To: Ian Campbell Cc: Ian Campbell, stefano.stabellini, tim, xen-devel, julien.grall, Wookey On 07/12/2013 12:59 PM, Ian Campbell wrote: > From: Ian Campbell <ian.campbell@citrix.com> > > The zImage compatible binary is the useful one on real hardware. The relocated > ELF thing is only really useful when booting directly on Fast Models. The > customary suffix for that case is .axf so provide that as a target. > > Signed-off-by: Ian Campbell <ian.campbell@citrix.com> > Cc: Wookey <wookey@linaro.org> > --- > xen/arch/arm/Makefile | 13 ++++--------- > 1 file changed, 4 insertions(+), 9 deletions(-) > > diff --git a/xen/arch/arm/Makefile b/xen/arch/arm/Makefile > index 87fabe1..0fba75c 100644 > --- a/xen/arch/arm/Makefile > +++ b/xen/arch/arm/Makefile > @@ -40,20 +40,15 @@ endif > > ALL_OBJS := $(TARGET_SUBARCH)/head.o $(ALL_OBJS) > > -$(TARGET): $(TARGET)-syms $(TARGET).bin > +$(TARGET): $(TARGET)-syms $(TARGET).axf > + $(OBJCOPY) -O binary -S $< $@ > + > +$(TARGET).axf: $(TARGET)-syms > # XXX: VE model loads by VMA so instead of > # making a proper ELF we link with LMA == VMA and adjust crudely > $(OBJCOPY) --change-addresses +0x80000000 $< $@ > $(STRIP) $@ > > -# > -$(TARGET).bin: $(TARGET)-syms > - $(OBJCOPY) -O binary -S $< $@ > - > -#$(TARGET): $(TARGET)-syms $(efi-y) boot/mkelf32 > -# ./boot/mkelf32 $(TARGET)-syms $(TARGET) 0x100000 \ > -# `$(NM) -nr $(TARGET)-syms | head -n 1 | sed -e 's/^\([^ ]*\).*/0x\1/'` > - > ifeq ($(lto),y) > # Gather all LTO objects together > prelink_lto.o: $(ALL_OBJS) > Is it possible to remove xen.axf when "make clean" is called? -- Julien ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH] xen: arm: make zImage the default target which we install 2013-07-18 12:05 ` [PATCH] xen: arm: make zImage the default target which we install Julien Grall @ 2013-07-18 12:10 ` Ian Campbell 2013-07-18 12:12 ` Julien Grall 0 siblings, 1 reply; 15+ messages in thread From: Ian Campbell @ 2013-07-18 12:10 UTC (permalink / raw) To: Julien Grall; +Cc: julien.grall, Wookey, stefano.stabellini, tim, xen-devel On Thu, 2013-07-18 at 13:05 +0100, Julien Grall wrote: > On 07/12/2013 12:59 PM, Ian Campbell wrote: > > From: Ian Campbell <ian.campbell@citrix.com> > > > > The zImage compatible binary is the useful one on real hardware. The relocated > > ELF thing is only really useful when booting directly on Fast Models. The > > customary suffix for that case is .axf so provide that as a target. > > > > Signed-off-by: Ian Campbell <ian.campbell@citrix.com> > > Cc: Wookey <wookey@linaro.org> > > --- > > xen/arch/arm/Makefile | 13 ++++--------- > > 1 file changed, 4 insertions(+), 9 deletions(-) > > > > diff --git a/xen/arch/arm/Makefile b/xen/arch/arm/Makefile > > index 87fabe1..0fba75c 100644 > > --- a/xen/arch/arm/Makefile > > +++ b/xen/arch/arm/Makefile > > @@ -40,20 +40,15 @@ endif > > > > ALL_OBJS := $(TARGET_SUBARCH)/head.o $(ALL_OBJS) > > > > -$(TARGET): $(TARGET)-syms $(TARGET).bin > > +$(TARGET): $(TARGET)-syms $(TARGET).axf > > + $(OBJCOPY) -O binary -S $< $@ > > + > > +$(TARGET).axf: $(TARGET)-syms > > # XXX: VE model loads by VMA so instead of > > # making a proper ELF we link with LMA == VMA and adjust crudely > > $(OBJCOPY) --change-addresses +0x80000000 $< $@ > > $(STRIP) $@ > > > > -# > > -$(TARGET).bin: $(TARGET)-syms > > - $(OBJCOPY) -O binary -S $< $@ > > - > > -#$(TARGET): $(TARGET)-syms $(efi-y) boot/mkelf32 > > -# ./boot/mkelf32 $(TARGET)-syms $(TARGET) 0x100000 \ > > -# `$(NM) -nr $(TARGET)-syms | head -n 1 | sed -e 's/^\([^ ]*\).*/0x\1/'` > > - > > ifeq ($(lto),y) > > # Gather all LTO objects together > > prelink_lto.o: $(ALL_OBJS) > > > > Is it possible to remove xen.axf when "make clean" is called? Sure. 8<-------------------------------------------- >From 2c805196a0a4cfdac3ce185d04336cca11357d63 Mon Sep 17 00:00:00 2001 From: Ian Campbell <ian.campbell@citrix.com> Date: Fri, 12 Jul 2013 12:54:42 +0100 Subject: [PATCH] xen: arm: make zImage the default target which we install The zImage compatible binary is the useful one on real hardware. The relocated ELF thing is only really useful when booting directly on Fast Models. The customary suffix for that case is .axf so provide that as a target. Signed-off-by: Ian Campbell <ian.campbell@citrix.com> Cc: Wookey <wookey@linaro.org> --- v2: Remove $(TARGET).axf on clean --- xen/arch/arm/Makefile | 14 +++++--------- 1 files changed, 5 insertions(+), 9 deletions(-) diff --git a/xen/arch/arm/Makefile b/xen/arch/arm/Makefile index 87fabe1..5ae5831 100644 --- a/xen/arch/arm/Makefile +++ b/xen/arch/arm/Makefile @@ -40,20 +40,15 @@ endif ALL_OBJS := $(TARGET_SUBARCH)/head.o $(ALL_OBJS) -$(TARGET): $(TARGET)-syms $(TARGET).bin +$(TARGET): $(TARGET)-syms $(TARGET).axf + $(OBJCOPY) -O binary -S $< $@ + +$(TARGET).axf: $(TARGET)-syms # XXX: VE model loads by VMA so instead of # making a proper ELF we link with LMA == VMA and adjust crudely $(OBJCOPY) --change-addresses +0x80000000 $< $@ $(STRIP) $@ -# -$(TARGET).bin: $(TARGET)-syms - $(OBJCOPY) -O binary -S $< $@ - -#$(TARGET): $(TARGET)-syms $(efi-y) boot/mkelf32 -# ./boot/mkelf32 $(TARGET)-syms $(TARGET) 0x100000 \ -# `$(NM) -nr $(TARGET)-syms | head -n 1 | sed -e 's/^\([^ ]*\).*/0x\1/'` - ifeq ($(lto),y) # Gather all LTO objects together prelink_lto.o: $(ALL_OBJS) @@ -97,3 +92,4 @@ dtb.o: $(CONFIG_DTB_FILE) clean:: rm -f asm-offsets.s xen.lds rm -f $(BASEDIR)/.xen-syms.[0-9]* + rm -f $(TARGET).axf -- 1.7.2.5 ^ permalink raw reply related [flat|nested] 15+ messages in thread
* Re: [PATCH] xen: arm: make zImage the default target which we install 2013-07-18 12:10 ` Ian Campbell @ 2013-07-18 12:12 ` Julien Grall 2013-07-19 14:18 ` Ian Campbell 0 siblings, 1 reply; 15+ messages in thread From: Julien Grall @ 2013-07-18 12:12 UTC (permalink / raw) To: Ian Campbell; +Cc: Wookey, stefano.stabellini, tim, xen-devel On 07/18/2013 01:10 PM, Ian Campbell wrote: > On Thu, 2013-07-18 at 13:05 +0100, Julien Grall wrote: >> On 07/12/2013 12:59 PM, Ian Campbell wrote: >>> From: Ian Campbell <ian.campbell@citrix.com> >>> >>> The zImage compatible binary is the useful one on real hardware. The relocated >>> ELF thing is only really useful when booting directly on Fast Models. The >>> customary suffix for that case is .axf so provide that as a target. >>> >>> Signed-off-by: Ian Campbell <ian.campbell@citrix.com> >>> Cc: Wookey <wookey@linaro.org> >>> --- >>> xen/arch/arm/Makefile | 13 ++++--------- >>> 1 file changed, 4 insertions(+), 9 deletions(-) >>> >>> diff --git a/xen/arch/arm/Makefile b/xen/arch/arm/Makefile >>> index 87fabe1..0fba75c 100644 >>> --- a/xen/arch/arm/Makefile >>> +++ b/xen/arch/arm/Makefile >>> @@ -40,20 +40,15 @@ endif >>> >>> ALL_OBJS := $(TARGET_SUBARCH)/head.o $(ALL_OBJS) >>> >>> -$(TARGET): $(TARGET)-syms $(TARGET).bin >>> +$(TARGET): $(TARGET)-syms $(TARGET).axf >>> + $(OBJCOPY) -O binary -S $< $@ >>> + >>> +$(TARGET).axf: $(TARGET)-syms >>> # XXX: VE model loads by VMA so instead of >>> # making a proper ELF we link with LMA == VMA and adjust crudely >>> $(OBJCOPY) --change-addresses +0x80000000 $< $@ >>> $(STRIP) $@ >>> >>> -# >>> -$(TARGET).bin: $(TARGET)-syms >>> - $(OBJCOPY) -O binary -S $< $@ >>> - >>> -#$(TARGET): $(TARGET)-syms $(efi-y) boot/mkelf32 >>> -# ./boot/mkelf32 $(TARGET)-syms $(TARGET) 0x100000 \ >>> -# `$(NM) -nr $(TARGET)-syms | head -n 1 | sed -e 's/^\([^ ]*\).*/0x\1/'` >>> - >>> ifeq ($(lto),y) >>> # Gather all LTO objects together >>> prelink_lto.o: $(ALL_OBJS) >>> >> >> Is it possible to remove xen.axf when "make clean" is called? > > Sure. > > 8<-------------------------------------------- > > From 2c805196a0a4cfdac3ce185d04336cca11357d63 Mon Sep 17 00:00:00 2001 > From: Ian Campbell <ian.campbell@citrix.com> > Date: Fri, 12 Jul 2013 12:54:42 +0100 > Subject: [PATCH] xen: arm: make zImage the default target which we install > > The zImage compatible binary is the useful one on real hardware. The relocated > ELF thing is only really useful when booting directly on Fast Models. The > customary suffix for that case is .axf so provide that as a target. > > Signed-off-by: Ian Campbell <ian.campbell@citrix.com> > Cc: Wookey <wookey@linaro.org> Acked-by: Julien Grall <julien.grall@linaro.org> > --- > v2: Remove $(TARGET).axf on clean > --- > xen/arch/arm/Makefile | 14 +++++--------- > 1 files changed, 5 insertions(+), 9 deletions(-) > > diff --git a/xen/arch/arm/Makefile b/xen/arch/arm/Makefile > index 87fabe1..5ae5831 100644 > --- a/xen/arch/arm/Makefile > +++ b/xen/arch/arm/Makefile > @@ -40,20 +40,15 @@ endif > > ALL_OBJS := $(TARGET_SUBARCH)/head.o $(ALL_OBJS) > > -$(TARGET): $(TARGET)-syms $(TARGET).bin > +$(TARGET): $(TARGET)-syms $(TARGET).axf > + $(OBJCOPY) -O binary -S $< $@ > + > +$(TARGET).axf: $(TARGET)-syms > # XXX: VE model loads by VMA so instead of > # making a proper ELF we link with LMA == VMA and adjust crudely > $(OBJCOPY) --change-addresses +0x80000000 $< $@ > $(STRIP) $@ > > -# > -$(TARGET).bin: $(TARGET)-syms > - $(OBJCOPY) -O binary -S $< $@ > - > -#$(TARGET): $(TARGET)-syms $(efi-y) boot/mkelf32 > -# ./boot/mkelf32 $(TARGET)-syms $(TARGET) 0x100000 \ > -# `$(NM) -nr $(TARGET)-syms | head -n 1 | sed -e 's/^\([^ ]*\).*/0x\1/'` > - > ifeq ($(lto),y) > # Gather all LTO objects together > prelink_lto.o: $(ALL_OBJS) > @@ -97,3 +92,4 @@ dtb.o: $(CONFIG_DTB_FILE) > clean:: > rm -f asm-offsets.s xen.lds > rm -f $(BASEDIR)/.xen-syms.[0-9]* > + rm -f $(TARGET).axf > ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH] xen: arm: make zImage the default target which we install 2013-07-18 12:12 ` Julien Grall @ 2013-07-19 14:18 ` Ian Campbell 0 siblings, 0 replies; 15+ messages in thread From: Ian Campbell @ 2013-07-19 14:18 UTC (permalink / raw) To: Julien Grall; +Cc: Wookey, stefano.stabellini, tim, xen-devel On Thu, 2013-07-18 at 13:12 +0100, Julien Grall wrote: > > From 2c805196a0a4cfdac3ce185d04336cca11357d63 Mon Sep 17 00:00:00 2001 > > From: Ian Campbell <ian.campbell@citrix.com> > > Date: Fri, 12 Jul 2013 12:54:42 +0100 > > Subject: [PATCH] xen: arm: make zImage the default target which we install > > > > The zImage compatible binary is the useful one on real hardware. The relocated > > ELF thing is only really useful when booting directly on Fast Models. The > > customary suffix for that case is .axf so provide that as a target. > > > > Signed-off-by: Ian Campbell <ian.campbell@citrix.com> > > Cc: Wookey <wookey@linaro.org> > Acked-by: Julien Grall <julien.grall@linaro.org> Applied, thanks. ^ permalink raw reply [flat|nested] 15+ messages in thread
end of thread, other threads:[~2013-07-19 14:18 UTC | newest] Thread overview: 15+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2013-07-12 11:59 [PATCH] xen: arm: make zImage the default target which we install Ian Campbell 2013-07-15 8:38 ` [PATCH] xen: allow architecture to choose whether to compress installed xen binary Ian Campbell 2013-07-15 9:44 ` Jan Beulich 2013-07-16 9:15 ` Ian Campbell 2013-07-16 9:20 ` Jan Beulich 2013-07-17 11:13 ` Ian Campbell 2013-07-17 11:31 ` Jan Beulich 2013-07-17 11:37 ` Ian Campbell 2013-07-17 11:13 ` [PATCH] xen: x86: drop the ".gz" suffix when installing Ian Campbell 2013-07-17 11:26 ` Jan Beulich 2013-07-17 11:35 ` Ian Campbell 2013-07-18 12:05 ` [PATCH] xen: arm: make zImage the default target which we install Julien Grall 2013-07-18 12:10 ` Ian Campbell 2013-07-18 12:12 ` Julien Grall 2013-07-19 14:18 ` Ian Campbell
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.