From: Dario Faggioli <raistlin@linux.it>
To: Ian Jackson <Ian.Jackson@eu.citrix.com>
Cc: Stefano Stabellini <Stefano.Stabellini@eu.citrix.com>,
"Keir (Xen.org)" <keir@xen.org>,
Ian Campbell <Ian.Campbell@citrix.com>,
Jan Beulich <JBeulich@suse.com>,
xen-devel <xen-devel@lists.xen.org>
Subject: Re: [xen-unstable bisection] complete test-amd64-amd64-xl-win7-amd64
Date: Wed, 20 Jun 2012 16:01:19 +0200 [thread overview]
Message-ID: <1340200879.20225.11.camel@Solace> (raw)
In-Reply-To: <20449.54129.523732.173929@mariner.uk.xensource.com>
[-- Attachment #1.1: Type: text/plain, Size: 2807 bytes --]
On Wed, 2012-06-20 at 14:43 +0100, Ian Jackson wrote:
> Ian Campbell writes ("Re: [Xen-devel] [xen-unstable bisection] complete test-amd64-amd64-xl-win7-amd64"):
> > It's an odd one -- the logs seem to show the guest has started but the
> > patch in question would (or should!) cause an early error return during
> > build. It's possible that the recent test system outage has caused the
> > bisector to get confused but it has fingered this changeset at least
> > twice now.
>
> The bisector has repro'd this on the same host with 6 tests:
> alternately failures for 9d1fd58ff602 and successes for 32034d1914a6.
>
Which means it might be unrelated from that specific commit (sorry if
it's a stupid question, just want to be sure I'm getting things
correct :-/ )
> So if it's a heisenbug we've been moderately (but not outrageously)
> unlucky that it actually fingered a specific changeset rather than
> just shrugging its shoulders and moving on. (It does the latter quite
> a lot.)
>
Just to be sure, this part of the bisector report:
branch xen-unstable
xen branch xen-unstable
job test-amd64-amd64-xl-win7-amd64
test windows-install
means that the test that failed was 'job
test-amd64-amd64-xl-win7-amd64' (and, specifically, during
'windows-install' phase), right?
I'm asking because I actually found a bug in that change, _but_ it only
comes into play if the SEDF scheduler is being used. In fact, this test
also fails:
http://www.chiark.greenend.org.uk/~xensrcts/logs/13053/test-amd64-amd64-xl-sedf/info.html
and it does by saying this:
2012-06-16 07:37:47 Z executing ssh ... root@10.80.248.105 xl create /etc/xen/debian.guest.osstest.cfg
libxl: error: libxl.c:3619:sched_sedf_domain_set: setting domain sched sedf: Invalid argument
libxl: error: libxl_create.c:710:domcreate_bootloader_done: cannot (re-)build domain: -3
Parsing config from /etc/xen/debian.guest.osstest.cfg
Which is actually related to my patch, and for which I already have a
fix ready.
OTOH, I really don't see how that patch might cause that HVM windows
domain to die, given the log for 'test-amd64-amd64-xl-sedf' says this:
Parsing config from /etc/xen/win.guest.osstest.cfg
Daemon running with PID 2642
Which seems to mean, at least to me, that the check happened without
causing any issues (as it, conversely, does when SEDF is the scheduler).
Anyway, I'll send the patch for the SEDF case and try to reproduce the
bug for the credit+Win7+HVM case.
Thanks and Regards,
Dario
--
<<This happens because I choose it to happen!>> (Raistlin Majere)
-----------------------------------------------------------------
Dario Faggioli, Ph.D, http://retis.sssup.it/people/faggioli
Senior Software Engineer, Citrix Systems R&D Ltd., Cambridge (UK)
[-- Attachment #1.2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 198 bytes --]
[-- Attachment #2: Type: text/plain, Size: 126 bytes --]
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel
next prev parent reply other threads:[~2012-06-20 14:01 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-06-20 2:40 [xen-unstable bisection] complete test-amd64-amd64-xl-win7-amd64 xen.org
2012-06-20 7:51 ` Jan Beulich
2012-06-20 9:06 ` Ian Campbell
2012-06-20 10:34 ` Dario Faggioli
2012-06-20 13:43 ` Ian Jackson
2012-06-20 14:01 ` Dario Faggioli [this message]
2012-06-20 15:32 ` Ian Jackson
2012-06-20 17:12 ` Dario Faggioli
2012-06-20 17:18 ` Dario Faggioli
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1340200879.20225.11.camel@Solace \
--to=raistlin@linux.it \
--cc=Ian.Campbell@citrix.com \
--cc=Ian.Jackson@eu.citrix.com \
--cc=JBeulich@suse.com \
--cc=Stefano.Stabellini@eu.citrix.com \
--cc=keir@xen.org \
--cc=xen-devel@lists.xen.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).