Re: [Qemu-devel] [PULL 0/9] x86 and machine queue, 2017-10-05

[Eduardo Habkost]

On Tue, Oct 10, 2017 at 03:41:56PM +0800, Dou Liyang wrote:
> Hi Eduardo,
> 
> I couldn't reproduce the failure too. But, I have some concern below.
> 
> At 10/06/2017 11:37 PM, Peter Maydell wrote:
> > On 6 October 2017 at 16:27, Eduardo Habkost <address@hidden> wrote:
> > > On Fri, Oct 06, 2017 at 01:17:42PM +0100, Peter Maydell wrote:
> > > > This fails tests, I'm afraid:
> > > [...]
> > > > acpi-test: Warning! DSDT mismatch. Actual [asl:/tmp/asl-6QYK7Y.dsl,
> > > > aml:/tmp/aml-E8YK7Y], Expected [asl:/tmp/asl-1H1I7Y.dsl,
> > > > aml:tests/acpi-test-data/pc/DSDT.memhp].
> 
> This warning is about memory hot-plug tests. DSDT mismatch is too
> strange, I hope to see their difference.
> 
> > > > acpi-test: Warning. not showing difference since no diff utility is
> > > > specified. Set 'DIFF' environment variable to a preferred diff utility
> > > > and run 'make V=1 check' again to see ASL difference.OK
> > > > 
> > > > and same for /i386/acpi/piix4/numamem and for the
> > > > equivalent q35 tests.
> > > > 
> > > > Fails on s390, netbsd, freebsd, aarch32, x86-64.
> > > > 
> > > 
> > > Oops, sorry!
> > > 
> > > I need to find out why it didn't fail on travis-ci:
> > > https://travis-ci.org/ehabkost/qemu/builds/283884670
> > 
> > My guess is that one of {iasl installed, iasl not installed}
> > is broken, but the travis environments don't test both.
> > 
> If we want do the "bios-tables-test", we need iasl support.
> 
> otherwise, when I run "make V=1 check" without iasl, the test
> will report error:
> 
> ...
> TEST: tests/bios-tables-test... (pid=1006)
>   /i386/acpi/piix4:
> Looking for expected file 'tests/acpi-test-data/pc/DSDT'
> 
> Using expected file 'tests/acpi-test-data/pc/DSDT'
> 
> Looking for expected file 'tests/acpi-test-data/pc/APIC'
> 
> Using expected file 'tests/acpi-test-data/pc/APIC'
> 
> Looking for expected file 'tests/acpi-test-data/pc/HPET'
> 
> Using expected file 'tests/acpi-test-data/pc/HPET'
> **
> ERROR:tests/bios-tables-test.c:328:load_asl: assertion failed (error ==
> NULL): Failed to execute child process "iasl" (No such file or directory)
> (g-exec-error-quark, 8)
> FAIL

This shouldn't happen if CONFIG_IASL is not defined.  Did you run
configure again after removing iasl?

-- 
Eduardo