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

[Ian Campbell <ian.campbell@xxxxxxxxxx>]

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?

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