xen.org
2013-Jul-21 03:26 UTC
[xen-unstable bisection] complete test-amd64-i386-rhel6hvm-intel
branch xen-unstable xen branch xen-unstable job test-amd64-i386-rhel6hvm-intel test xen-install Tree: linux git://xenbits.xen.org/linux-pvops.git Tree: linuxfirmware git://xenbits.xen.org/osstest/linux-firmware.git Tree: qemu git://xenbits.xen.org/staging/qemu-xen-unstable.git Tree: qemuu git://xenbits.xen.org/staging/qemu-upstream-unstable.git Tree: xen git://xenbits.xen.org/xen.git *** Found and reproduced problem changeset *** Bug is in tree: xen git://xenbits.xen.org/xen.git Bug introduced: 524b93def23b9f75fd7851063f5291886e63d1ed Bug not present: 09a08ef52a21d171cc48b54a975f13e7704c912f commit 524b93def23b9f75fd7851063f5291886e63d1ed Author: Ian Campbell <ian.campbell@citrix.com> Date: Thu Jul 18 09:41:41 2013 +0100 xen: x86: drop the ".gz" suffix when installing 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 least 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> Acked-by: Keir Fraser <keir@xen.org> Acked-by: Julien Grall <julien.grall@linaro.org> Acked-by: Jan Beulich <jbeulich@suse.com> For bisection revision-tuple graph see: http://www.chiark.greenend.org.uk/~xensrcts/results/bisect.xen-unstable.test-amd64-i386-rhel6hvm-intel.xen-install.html Revision IDs in each graph node refer, respectively, to the Trees above. ---------------------------------------- Searching for failure / basis pass: 18537 fail [host=bush-cricket] / 18485 [host=gall-mite] 18484 [host=field-cricket] 18466 [host=itch-mite] 18454 [host=earwig] 18431 ok. Failure / basis pass flights: 18537 / 18431 Tree: linux git://xenbits.xen.org/linux-pvops.git Tree: linuxfirmware git://xenbits.xen.org/osstest/linux-firmware.git Tree: qemu git://xenbits.xen.org/staging/qemu-xen-unstable.git Tree: qemuu git://xenbits.xen.org/staging/qemu-upstream-unstable.git Tree: xen git://xenbits.xen.org/xen.git Latest a938a246d34912423c560f475ccf1ce0c71d9d00 c530a75c1e6a472b0eb9558310b518f0dfcd8860 13c144d96e825f145e5b37f97e5f6210c2c645e9 7483e7f15139603380c45ebcd8cc2a57dda5583c 2a6327bf2bfaf5de5e07aed583d2c337c9d368c0 Basis pass a938a246d34912423c560f475ccf1ce0c71d9d00 c530a75c1e6a472b0eb9558310b518f0dfcd8860 13c144d96e825f145e5b37f97e5f6210c2c645e9 7483e7f15139603380c45ebcd8cc2a57dda5583c 5d0ca62156d734a757656b9bcb6bf17ee76d37b4 Generating revisions with ./adhoc-revtuple-generator git://xenbits.xen.org/linux-pvops.git#a938a246d34912423c560f475ccf1ce0c71d9d00-a938a246d34912423c560f475ccf1ce0c71d9d00 git://xenbits.xen.org/osstest/linux-firmware.git#c530a75c1e6a472b0eb9558310b518f0dfcd8860-c530a75c1e6a472b0eb9558310b518f0dfcd8860 git://xenbits.xen.org/staging/qemu-xen-unstable.git#13c144d96e825f145e5b37f97e5f6210c2c645e9-13c144d96e825f145e5b37f97e5f6210c2c645e9 git://xenbits.xen.org/staging/qemu-upstream-unstable.git#7483e7f15139603380c45ebcd8cc2a57dda5583c-7483e7f15139603380c45ebcd8cc2a57dda5583c git://xenbits.xen.org/xen.git#5d0ca62156d734a757656b9bcb6bf17ee76d37b4-2a6327bf2bfaf5de5e07aed583d2c337c9d368c0 using cache /export/home/osstest/repos/git-cache... using cache /export/home/osstest/repos/git-cache... locked cache /export/home/osstest/repos/git-cache... processing ./cacheing-git clone --bare git://xenbits.xen.org/xen.git /export/home/osstest/repos/xen... Initialized empty Git repository in /export/home/osstest/repos/xen/ updating cache /export/home/osstest/repos/git-cache xen... using cache /export/home/osstest/repos/git-cache... using cache /export/home/osstest/repos/git-cache... locked cache /export/home/osstest/repos/git-cache... processing ./cacheing-git clone --bare git://xenbits.xen.org/xen.git /export/home/osstest/repos/xen... Initialized empty Git repository in /export/home/osstest/repos/xen/ updating cache /export/home/osstest/repos/git-cache xen... Loaded 1001 nodes in revision graph Searching for test results: 18431 pass a938a246d34912423c560f475ccf1ce0c71d9d00 c530a75c1e6a472b0eb9558310b518f0dfcd8860 13c144d96e825f145e5b37f97e5f6210c2c645e9 7483e7f15139603380c45ebcd8cc2a57dda5583c 5d0ca62156d734a757656b9bcb6bf17ee76d37b4 18433 [] 18430 pass a938a246d34912423c560f475ccf1ce0c71d9d00 c530a75c1e6a472b0eb9558310b518f0dfcd8860 13c144d96e825f145e5b37f97e5f6210c2c645e9 7483e7f15139603380c45ebcd8cc2a57dda5583c 5d0ca62156d734a757656b9bcb6bf17ee76d37b4 18466 [host=itch-mite] 18438 [] 18454 [host=earwig] 18546 fail a938a246d34912423c560f475ccf1ce0c71d9d00 c530a75c1e6a472b0eb9558310b518f0dfcd8860 13c144d96e825f145e5b37f97e5f6210c2c645e9 7483e7f15139603380c45ebcd8cc2a57dda5583c 524b93def23b9f75fd7851063f5291886e63d1ed 18528 [] 18479 [] 18480 [] 18483 [] 18496 fail a938a246d34912423c560f475ccf1ce0c71d9d00 c530a75c1e6a472b0eb9558310b518f0dfcd8860 13c144d96e825f145e5b37f97e5f6210c2c645e9 7483e7f15139603380c45ebcd8cc2a57dda5583c 2a6327bf2bfaf5de5e07aed583d2c337c9d368c0 18484 [host=field-cricket] 18485 [host=gall-mite] 18536 pass a938a246d34912423c560f475ccf1ce0c71d9d00 c530a75c1e6a472b0eb9558310b518f0dfcd8860 13c144d96e825f145e5b37f97e5f6210c2c645e9 7483e7f15139603380c45ebcd8cc2a57dda5583c 09a08ef52a21d171cc48b54a975f13e7704c912f 18491 fail a938a246d34912423c560f475ccf1ce0c71d9d00 c530a75c1e6a472b0eb9558310b518f0dfcd8860 13c144d96e825f145e5b37f97e5f6210c2c645e9 7483e7f15139603380c45ebcd8cc2a57dda5583c 2a6327bf2bfaf5de5e07aed583d2c337c9d368c0 18519 fail a938a246d34912423c560f475ccf1ce0c71d9d00 c530a75c1e6a472b0eb9558310b518f0dfcd8860 13c144d96e825f145e5b37f97e5f6210c2c645e9 7483e7f15139603380c45ebcd8cc2a57dda5583c 2a6327bf2bfaf5de5e07aed583d2c337c9d368c0 18529 pass a938a246d34912423c560f475ccf1ce0c71d9d00 c530a75c1e6a472b0eb9558310b518f0dfcd8860 13c144d96e825f145e5b37f97e5f6210c2c645e9 7483e7f15139603380c45ebcd8cc2a57dda5583c 5d0ca62156d734a757656b9bcb6bf17ee76d37b4 18530 fail a938a246d34912423c560f475ccf1ce0c71d9d00 c530a75c1e6a472b0eb9558310b518f0dfcd8860 13c144d96e825f145e5b37f97e5f6210c2c645e9 7483e7f15139603380c45ebcd8cc2a57dda5583c 2a6327bf2bfaf5de5e07aed583d2c337c9d368c0 18531 blocked a938a246d34912423c560f475ccf1ce0c71d9d00 c530a75c1e6a472b0eb9558310b518f0dfcd8860 13c144d96e825f145e5b37f97e5f6210c2c645e9 7483e7f15139603380c45ebcd8cc2a57dda5583c 4816f9a7d47f985dfa796dc632771201b10858e8 18532 pass a938a246d34912423c560f475ccf1ce0c71d9d00 c530a75c1e6a472b0eb9558310b518f0dfcd8860 13c144d96e825f145e5b37f97e5f6210c2c645e9 7483e7f15139603380c45ebcd8cc2a57dda5583c 09a08ef52a21d171cc48b54a975f13e7704c912f 18533 blocked a938a246d34912423c560f475ccf1ce0c71d9d00 c530a75c1e6a472b0eb9558310b518f0dfcd8860 13c144d96e825f145e5b37f97e5f6210c2c645e9 7483e7f15139603380c45ebcd8cc2a57dda5583c 2f044a6a6e4cb0ea24c856c1615e3fb878af2cfb 18537 fail a938a246d34912423c560f475ccf1ce0c71d9d00 c530a75c1e6a472b0eb9558310b518f0dfcd8860 13c144d96e825f145e5b37f97e5f6210c2c645e9 7483e7f15139603380c45ebcd8cc2a57dda5583c 2a6327bf2bfaf5de5e07aed583d2c337c9d368c0 18534 blocked a938a246d34912423c560f475ccf1ce0c71d9d00 c530a75c1e6a472b0eb9558310b518f0dfcd8860 13c144d96e825f145e5b37f97e5f6210c2c645e9 7483e7f15139603380c45ebcd8cc2a57dda5583c c57c50c1de759583d5de629fec205254280da4f0 18535 fail a938a246d34912423c560f475ccf1ce0c71d9d00 c530a75c1e6a472b0eb9558310b518f0dfcd8860 13c144d96e825f145e5b37f97e5f6210c2c645e9 7483e7f15139603380c45ebcd8cc2a57dda5583c 524b93def23b9f75fd7851063f5291886e63d1ed 18544 fail a938a246d34912423c560f475ccf1ce0c71d9d00 c530a75c1e6a472b0eb9558310b518f0dfcd8860 13c144d96e825f145e5b37f97e5f6210c2c645e9 7483e7f15139603380c45ebcd8cc2a57dda5583c 524b93def23b9f75fd7851063f5291886e63d1ed 18545 pass a938a246d34912423c560f475ccf1ce0c71d9d00 c530a75c1e6a472b0eb9558310b518f0dfcd8860 13c144d96e825f145e5b37f97e5f6210c2c645e9 7483e7f15139603380c45ebcd8cc2a57dda5583c 09a08ef52a21d171cc48b54a975f13e7704c912f Searching for interesting versions Result found: flight 18430 (pass), for basis pass Result found: flight 18491 (fail), for basis failure Repro found: flight 18529 (pass), for basis pass Repro found: flight 18530 (fail), for basis failure 0 revisions at a938a246d34912423c560f475ccf1ce0c71d9d00 c530a75c1e6a472b0eb9558310b518f0dfcd8860 13c144d96e825f145e5b37f97e5f6210c2c645e9 7483e7f15139603380c45ebcd8cc2a57dda5583c 09a08ef52a21d171cc48b54a975f13e7704c912f No revisions left to test, checking graph state. Result found: flight 18532 (pass), for last pass Result found: flight 18535 (fail), for first failure Repro found: flight 18536 (pass), for last pass Repro found: flight 18544 (fail), for first failure Repro found: flight 18545 (pass), for last pass Repro found: flight 18546 (fail), for first failure *** Found and reproduced problem changeset *** Bug is in tree: xen git://xenbits.xen.org/xen.git Bug introduced: 524b93def23b9f75fd7851063f5291886e63d1ed Bug not present: 09a08ef52a21d171cc48b54a975f13e7704c912f using cache /export/home/osstest/repos/git-cache... using cache /export/home/osstest/repos/git-cache... locked cache /export/home/osstest/repos/git-cache... processing ./cacheing-git clone --bare git://xenbits.xen.org/xen.git /export/home/osstest/repos/xen... Initialized empty Git repository in /export/home/osstest/repos/xen/ updating cache /export/home/osstest/repos/git-cache xen... commit 524b93def23b9f75fd7851063f5291886e63d1ed Author: Ian Campbell <ian.campbell@citrix.com> Date: Thu Jul 18 09:41:41 2013 +0100 xen: x86: drop the ".gz" suffix when installing 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 least 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> Acked-by: Keir Fraser <keir@xen.org> Acked-by: Julien Grall <julien.grall@linaro.org> Acked-by: Jan Beulich <jbeulich@suse.com> Revision graph left in /home/xc_osstest/results/bisect.xen-unstable.test-amd64-i386-rhel6hvm-intel.xen-install.{dot,ps,png,html}. ---------------------------------------- 18546: tolerable ALL FAIL flight 18546 xen-unstable real-bisect [real] http://www.chiark.greenend.org.uk/~xensrcts/logs/18546/ Failures :-/ but no regressions. Tests which did not succeed, including tests which could not be run: test-amd64-i386-rhel6hvm-intel 4 xen-install fail baseline untested jobs: test-amd64-i386-rhel6hvm-intel fail ------------------------------------------------------------ sg-report-flight on woking.cam.xci-test.com logs: /home/xc_osstest/logs images: /home/xc_osstest/images Logs, config files, etc. are available at http://www.chiark.greenend.org.uk/~xensrcts/logs Test harness code can be found at http://xenbits.xensource.com/gitweb?p=osstest.git;a=summary
Ian Campbell
2013-Jul-21 05:30 UTC
Re: [xen-unstable bisection] complete test-amd64-i386-rhel6hvm-intel
On Sun, 2013-07-21 at 04:26 +0100, xen.org wrote:> commit 524b93def23b9f75fd7851063f5291886e63d1ed > Author: Ian Campbell <ian.campbell@citrix.com> > Date: Thu Jul 18 09:41:41 2013 +0100 > > xen: x86: drop the ".gz" suffix when installing > > 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 least grub 2''s > update-grub script (as present in Debian Wheezy) seems to cope (it matches on > xen* not xen*.gz)Looks like update-grub in Squeeze doesn''t handle this case well. http://www.chiark.greenend.org.uk/~xensrcts/logs/18537/test-amd64-amd64-xl/4.ts-xen-install.log shows osstest failing to find any hypervisor stanzas in grub.cfg and http://www.chiark.greenend.org.uk/~xensrcts/logs/18537/test-amd64-amd64-xl/field-cricket--grub.cfg.1 shows that there are indeed none present... Lets just undo things for now: 8<------------------------------ From 5f7b0c68d3721fd2eef80f7e23466425b55d21af Mon Sep 17 00:00:00 2001 From: Ian Campbell <ian.campbell@citrix.com> Date: Sun, 21 Jul 2013 06:24:30 +0100 Subject: [PATCH] xen: x86: put back .gz suffix on installed hypervisor binary. This reverts the effect of 524b93def23b "xen: x86: drop the ".gz" suffix when installing" which broke things in osstest (Debian Squeeze update-grub apparently can''t cope). It is not a direct revert because of other changes made since. We continue to omit the suffix on ARM. Signed-off-by: Ian Campbell <ian.campbell@citrix.com> Cc: jbeulich@suse.com --- xen/Makefile | 18 ++++++++++-------- 1 files changed, 10 insertions(+), 8 deletions(-) diff --git a/xen/Makefile b/xen/Makefile index 2abfa58..597972d 100644 --- a/xen/Makefile +++ b/xen/Makefile @@ -34,12 +34,13 @@ _build: $(TARGET)$(CONFIG_XEN_INSTALL_SUFFIX) .PHONY: _install _install: D=$(DESTDIR) _install: T=$(notdir $(TARGET)) +_install: Z=$(CONFIG_XEN_INSTALL_SUFFIX) _install: $(TARGET)$(CONFIG_XEN_INSTALL_SUFFIX) [ -d $(D)/boot ] || $(INSTALL_DIR) $(D)/boot - $(INSTALL_DATA) $(TARGET)$(CONFIG_XEN_INSTALL_SUFFIX) $(D)/boot/$(T)-$(XEN_FULLVERSION) - ln -f -s $(T)-$(XEN_FULLVERSION) $(D)/boot/$(T)-$(XEN_VERSION).$(XEN_SUBVERSION) - ln -f -s $(T)-$(XEN_FULLVERSION) $(D)/boot/$(T)-$(XEN_VERSION) - ln -f -s $(T)-$(XEN_FULLVERSION) $(D)/boot/$(T) + $(INSTALL_DATA) $(TARGET)$(Z) $(D)/boot/$(T)-$(XEN_FULLVERSION)$(Z) + ln -f -s $(T)-$(XEN_FULLVERSION)$(Z) $(D)/boot/$(T)-$(XEN_VERSION).$(XEN_SUBVERSION)$(Z) + ln -f -s $(T)-$(XEN_FULLVERSION)$(Z) $(D)/boot/$(T)-$(XEN_VERSION)$(Z) + ln -f -s $(T)-$(XEN_FULLVERSION)$(Z) $(D)/boot/$(T)$(Z) $(INSTALL_DATA) $(TARGET)-syms $(D)/boot/$(T)-syms-$(XEN_FULLVERSION) if [ -r $(TARGET).efi -a -n ''$(EFI_DIR)'' ]; then \ [ -d $(D)$(EFI_DIR) ] || $(INSTALL_DIR) $(D)$(EFI_DIR); \ @@ -57,11 +58,12 @@ _install: $(TARGET)$(CONFIG_XEN_INSTALL_SUFFIX) .PHONY: _uninstall _uninstall: D=$(DESTDIR) _uninstall: T=$(notdir $(TARGET)) +_uninstall: Z=$(CONFIG_XEN_INSTALL_SUFFIX) _uninstall: - 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)-$(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 -- 1.7.2.5
Ian Campbell
2013-Jul-21 15:15 UTC
Re: [xen-unstable bisection] complete test-amd64-i386-rhel6hvm-intel
On Sun, 2013-07-21 at 06:30 +0100, Ian Campbell wrote:> 8<------------------------------ > > From 5f7b0c68d3721fd2eef80f7e23466425b55d21af Mon Sep 17 00:00:00 2001 > From: Ian Campbell <ian.campbell@citrix.com> > Date: Sun, 21 Jul 2013 06:24:30 +0100 > Subject: [PATCH] xen: x86: put back .gz suffix on installed hypervisor binary. > > This reverts the effect of 524b93def23b "xen: x86: drop the ".gz" suffix when > installing" which broke things in osstest (Debian Squeeze update-grub apparently > can''t cope). It is not a direct revert because of other changes made since.Since this is (functionally if not literally) a revert which fixes the tests I''ve gone ahead and pushed it without any acks.> > We continue to omit the suffix on ARM. > > Signed-off-by: Ian Campbell <ian.campbell@citrix.com> > Cc: jbeulich@suse.com > --- > xen/Makefile | 18 ++++++++++-------- > 1 files changed, 10 insertions(+), 8 deletions(-) > > diff --git a/xen/Makefile b/xen/Makefile > index 2abfa58..597972d 100644 > --- a/xen/Makefile > +++ b/xen/Makefile > @@ -34,12 +34,13 @@ _build: $(TARGET)$(CONFIG_XEN_INSTALL_SUFFIX) > .PHONY: _install > _install: D=$(DESTDIR) > _install: T=$(notdir $(TARGET)) > +_install: Z=$(CONFIG_XEN_INSTALL_SUFFIX) > _install: $(TARGET)$(CONFIG_XEN_INSTALL_SUFFIX) > [ -d $(D)/boot ] || $(INSTALL_DIR) $(D)/boot > - $(INSTALL_DATA) $(TARGET)$(CONFIG_XEN_INSTALL_SUFFIX) $(D)/boot/$(T)-$(XEN_FULLVERSION) > - ln -f -s $(T)-$(XEN_FULLVERSION) $(D)/boot/$(T)-$(XEN_VERSION).$(XEN_SUBVERSION) > - ln -f -s $(T)-$(XEN_FULLVERSION) $(D)/boot/$(T)-$(XEN_VERSION) > - ln -f -s $(T)-$(XEN_FULLVERSION) $(D)/boot/$(T) > + $(INSTALL_DATA) $(TARGET)$(Z) $(D)/boot/$(T)-$(XEN_FULLVERSION)$(Z) > + ln -f -s $(T)-$(XEN_FULLVERSION)$(Z) $(D)/boot/$(T)-$(XEN_VERSION).$(XEN_SUBVERSION)$(Z) > + ln -f -s $(T)-$(XEN_FULLVERSION)$(Z) $(D)/boot/$(T)-$(XEN_VERSION)$(Z) > + ln -f -s $(T)-$(XEN_FULLVERSION)$(Z) $(D)/boot/$(T)$(Z) > $(INSTALL_DATA) $(TARGET)-syms $(D)/boot/$(T)-syms-$(XEN_FULLVERSION) > if [ -r $(TARGET).efi -a -n ''$(EFI_DIR)'' ]; then \ > [ -d $(D)$(EFI_DIR) ] || $(INSTALL_DIR) $(D)$(EFI_DIR); \ > @@ -57,11 +58,12 @@ _install: $(TARGET)$(CONFIG_XEN_INSTALL_SUFFIX) > .PHONY: _uninstall > _uninstall: D=$(DESTDIR) > _uninstall: T=$(notdir $(TARGET)) > +_uninstall: Z=$(CONFIG_XEN_INSTALL_SUFFIX) > _uninstall: > - 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)-$(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