diff mbox

qemu-traditional build problem for Xen 4.7.0 RC6

Message ID B8376D2DEA074F45BA033984477C453E034C773C@shsmsx102.ccr.corp.intel.com (mailing list archive)
State New, archived
Headers show

Commit Message

Hao, Xudong June 17, 2016, 3:11 a.m. UTC
QEMU_TAG update to 698d6d6f8d095edadb0c23612b552a89dd3eee4c of traditional qemu in Config.mk, this commit is a branch stable-4.7 commit for qemu traditional. The master commit is 6e20809727261599e8527c456eb078c0e89139a1.
It bring up a build error.

commit 03c716e32912e288929a6e0d9c8729d208462d66
Author: Ian Jackson <ian.jackson@eu.citrix.com>
Date:   Fri Jun 10 11:49:24 2016 +0100

    QEMU_TAG update


Best Regards,
Xudong

Comments

Jan Beulich June 17, 2016, 7:58 a.m. UTC | #1
>>> On 17.06.16 at 05:11, <xudong.hao@intel.com> wrote:
> QEMU_TAG update to 698d6d6f8d095edadb0c23612b552a89dd3eee4c of traditional 
> qemu in Config.mk, this commit is a branch stable-4.7 commit for qemu 
> traditional. The master commit is 6e20809727261599e8527c456eb078c0e89139a1.
> It bring up a build error.

It would help quite a bit if you also said what kind of build error you
observe.

Jan

> commit 03c716e32912e288929a6e0d9c8729d208462d66
> Author: Ian Jackson <ian.jackson@eu.citrix.com>
> Date:   Fri Jun 10 11:49:24 2016 +0100
> 
>     QEMU_TAG update
> 
> diff --git a/Config.mk b/Config.mk
> index bc5c456..1d6522c 100644
> --- a/Config.mk
> +++ b/Config.mk
> @@ -284,9 +284,9 @@ SEABIOS_UPSTREAM_REVISION ?= rel-1.9.2
>  ETHERBOOT_NICS ?= rtl8139 8086100e
> 
> 
> -QEMU_TRADITIONAL_REVISION ?= df553c056104e3dd8a2bd2e72539a57c4c085bae
> -# Thu May 5 11:14:44 2016 +0100
> -# Fix build with newer version of GNUTLS
> +QEMU_TRADITIONAL_REVISION ?= 698d6d6f8d095edadb0c23612b552a89dd3eee4c
> +# Thu May 19 19:38:35 2016 +0100
> +# main loop: Big hammer to fix logfile disk DoS in Xen setups
> 
>  # Specify which qemu-dm to use. This may be `ioemu' to use the old
>  # Mercurial in-tree version, or a local directory, or a git URL.
> 
> Best Regards,
> Xudong
> 
> 
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@lists.xen.org 
> http://lists.xen.org/xen-devel
Hao, Xudong June 17, 2016, 8:46 a.m. UTC | #2
> -----Original Message-----
> From: Jan Beulich [mailto:JBeulich@suse.com]
> Sent: Friday, June 17, 2016 3:59 PM
> To: Hao, Xudong <xudong.hao@intel.com>
> Cc: Wei Liu <wei.liu2@citrix.com>; ian.jackson@eu.citrix.com; xen-
> devel@lists.xenproject.org
> Subject: Re: [Xen-devel] qemu-traditional build problem for Xen 4.7.0 RC6
> 
> >>> On 17.06.16 at 05:11, <xudong.hao@intel.com> wrote:
> > QEMU_TAG update to 698d6d6f8d095edadb0c23612b552a89dd3eee4c of
> > traditional qemu in Config.mk, this commit is a branch stable-4.7
> > commit for qemu traditional. The master commit is
> 6e20809727261599e8527c456eb078c0e89139a1.
> > It bring up a build error.
> 
> It would help quite a bit if you also said what kind of build error you observe.
> 

Actually it's not compile error, it's a repo clone issue. Error log is:

...
Initialized empty Git repository in /home/www/builds_xen_unstable/xen-src-xen-4.7.0-rc6-20160616/tools/qemu-xen-traditional-dir-remote.tmp/.git/
fatal: reference is not a tree: 698d6d6f8d095edadb0c23612b552a89dd3eee4c
make[2]: *** [qemu-xen-traditional-dir-find] Error 128
...

The issue get root caused and resolved now. 
The failure is that we set up a local repo mirror for qemu-xen-traditional, but the mirror only sync "master" branch, not include "stable-4.7" branch, so commit 698d6d6* didn't be identified when doing checkout. 

Thanks,
-Xudong

> Jan
> 
> > commit 03c716e32912e288929a6e0d9c8729d208462d66
> > Author: Ian Jackson <ian.jackson@eu.citrix.com>
> > Date:   Fri Jun 10 11:49:24 2016 +0100
> >
> >     QEMU_TAG update
> >
> > diff --git a/Config.mk b/Config.mk
> > index bc5c456..1d6522c 100644
> > --- a/Config.mk
> > +++ b/Config.mk
> > @@ -284,9 +284,9 @@ SEABIOS_UPSTREAM_REVISION ?= rel-1.9.2
> > ETHERBOOT_NICS ?= rtl8139 8086100e
> >
> >
> > -QEMU_TRADITIONAL_REVISION ?=
> df553c056104e3dd8a2bd2e72539a57c4c085bae
> > -# Thu May 5 11:14:44 2016 +0100
> > -# Fix build with newer version of GNUTLS
> > +QEMU_TRADITIONAL_REVISION ?=
> 698d6d6f8d095edadb0c23612b552a89dd3eee4c
> > +# Thu May 19 19:38:35 2016 +0100
> > +# main loop: Big hammer to fix logfile disk DoS in Xen setups
> >
> >  # Specify which qemu-dm to use. This may be `ioemu' to use the old  #
> > Mercurial in-tree version, or a local directory, or a git URL.
> >
> > Best Regards,
> > Xudong
> >
> >
> > _______________________________________________
> > Xen-devel mailing list
> > Xen-devel@lists.xen.org
> > http://lists.xen.org/xen-devel
> 
>
Ian Jackson June 17, 2016, 2:31 p.m. UTC | #3
Hao, Xudong writes ("RE: [Xen-devel] qemu-traditional build problem for Xen 4.7.0 RC6"):
> Actually it's not compile error, it's a repo clone issue. Error log is:
> 
> ...
> Initialized empty Git repository in /home/www/builds_xen_unstable/xen-src-xen-4.7.0-rc6-20160616/tools/qemu-xen-traditional-dir-remote.tmp/.git/
> fatal: reference is not a tree: 698d6d6f8d095edadb0c23612b552a89dd3eee4c
> make[2]: *** [qemu-xen-traditional-dir-find] Error 128
> ...
> 
> The issue get root caused and resolved now. 
> The failure is that we set up a local repo mirror for qemu-xen-traditional, but the mirror only sync "master" branch, not include "stable-4.7" branch, so commit 698d6d6* didn't be identified when doing checkout. 

OK, thanks for clearing that up.

Indeed, the "master" branch won't (in general) contain all the
relevant commits, because stable branches like 4.7 do diverge.

Regards,
Ian.
diff mbox

Patch

diff --git a/Config.mk b/Config.mk
index bc5c456..1d6522c 100644
--- a/Config.mk
+++ b/Config.mk
@@ -284,9 +284,9 @@  SEABIOS_UPSTREAM_REVISION ?= rel-1.9.2
 ETHERBOOT_NICS ?= rtl8139 8086100e


-QEMU_TRADITIONAL_REVISION ?= df553c056104e3dd8a2bd2e72539a57c4c085bae
-# Thu May 5 11:14:44 2016 +0100
-# Fix build with newer version of GNUTLS
+QEMU_TRADITIONAL_REVISION ?= 698d6d6f8d095edadb0c23612b552a89dd3eee4c
+# Thu May 19 19:38:35 2016 +0100
+# main loop: Big hammer to fix logfile disk DoS in Xen setups

 # Specify which qemu-dm to use. This may be `ioemu' to use the old
 # Mercurial in-tree version, or a local directory, or a git URL.