Re: [Xen-devel] [qemu-mainline test] 80469: regressions - FAIL

[Ian Campbell <ian.campbell@xxxxxxxxxx>]

On Wed, 2016-02-10 at 12:03 +0000, Ian Campbell wrote:
> On Wed, 2016-02-10 at 11:53 +0000, Stefano Stabellini wrote:
> > On Wed, 10 Feb 2016, Ian Campbell wrote:
> > > Adding some CCs.Â
> > > 
> > > On Fri, 2016-02-05 at 11:52 +0000, Anthony PERARD wrote:
> > > > On Fri, Feb 05, 2016 at 06:30:25AM +0000, osstest service owner
> > > > wrote:
> > > > > flight 80469 qemu-mainline real [real]
> > > > > http://logs.test-lab.xenproject.org/osstest/logs/80469/
> > > > > 
> > > > > Regressions :-(
> > > > > 
> > > > > Tests which did not succeed and are blocking,
> > > > > including tests which could not be run:
> > > > > Âbuild-i386ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ5 xen-buildÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂfail
> > > > > REGR. vs. 79947
> > > > > Âbuild-amd64ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ5 xen-buildÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂfail
> > > > > REGR. vs. 79947
> > > > > Âbuild-i386-xsmÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ5 xen-buildÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂfail
> > > > > REGR. vs. 79947
> > > > > Âbuild-amd64-xsmÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ5 xen-buildÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂfail
> > > > > REGR. vs. 79947
> > > > > Âbuild-armhfÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ5 xen-buildÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂfail
> > > > > REGR. vs. 79947
> > > > > Âbuild-armhf-xsmÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ5 xen-buildÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂfail
> > > > > REGR. vs. 79947
> > > > 
> > > > ./configure of QEMU fail with:
> > > > "ERROR: invalid trace backends
> > > > ÂÂÂÂÂÂÂÂPlease choose supported trace backends."
> > > > 
> > > > They have remove the "stderr" tracebackend, and replaced it by
> > > > "log".
> > > > Which
> > > > also became the default. I have not look at what to do yet, but the
> > > > configure
> > > > option "--enable-trace-backend=stderr" as became invalid.
> > > 
> > > Paul added this IIRC.
> > > 
> > > If it isn't immediately obvious what an appropriate replacement is
> > > (which
> > > remember needs to work with at least qemu-xen-unstable as well as
> > > qemu-
> > > upstream) then I think we should just remove this option from
> > > xen.git/tools/Makefile for the time being.
> > > 
> > > FWIW the bisector has fingered:
> > > 
> > > Â commit ed7f5f1d8db06fc31352a5ef4f54985e630c575a
> > > Â Author: Paolo Bonzini <pbonzini@xxxxxxxxxx>
> > > Â Date:ÂÂÂThu Jan 7 16:55:30 2016 +0300
> > > Â 
> > > ÂÂÂÂÂÂtrace: convert stderr backend to log
> > > ÂÂÂÂÂÂ
> > > ÂÂÂÂÂÂ[Also update .travis.yml --enable-trace-backends=stderr
> > > ÂÂÂÂÂÂ--Stefan]
> > > ÂÂÂÂÂÂ
> > > ÂÂÂÂÂÂSigned-off-by: Paolo Bonzini <pbonzini@xxxxxxxxxx>
> > > ÂÂÂÂÂÂSigned-off-by: Denis V. Lunev <den@xxxxxxxxxx>
> > > ÂÂÂÂÂÂAcked-by: Christian Borntraeger <borntraeger@xxxxxxxxxx>
> > > ÂÂÂÂÂÂMessage-id: 1452174932-28657-10-git-send-email-den@xxxxxxxxxx
> > > ÂÂÂÂÂÂSigned-off-by: Stefan Hajnoczi <stefanha@xxxxxxxxxx>
> > > 
> > > It looks like s/stderr/log/g would work with qemu-mainline, but we'd
> > > need
> > > that backported to qemu-xen-unstable too.
> > > 
> > > Or given:
> > > ÂÂÂÂgit show --stat -M ed7f5f1d8db06fc31352a5ef4f54985e630c575a
> > > ÂÂÂÂ[...]
> > > ÂÂÂÂÂscripts/tracetool/backend/{stderr.py => log.py} |ÂÂ9 +++++----
> > > 
> > > Maybe xen.git/tools/Makefile could probe for the presence of these
> > > files?
> > Â
> > We can have QEMU list the trace backends available with:
> > 
> > ./scripts/tracetool.py --list-backends
> > 
> > that works on all qemu-xen versions >= 4.3.
> > 
> > Same output:
> 
> Sample?
> 
> > nop, ust, stderr, dtrace, simple
> 
> Here (QEMU Âmaster) it is:
> ÂÂÂÂnop, dtrace, ftrace, log, simple, ust
> 
> Who is going to produce a patch to use this?

Actually, I've just coincidentally tripped over this in what I'm looking at
now, so I suppose I'll whip something up.

Ian.


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel