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

[Dou Liyang]

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
...

Thanks,
        dou.
> thanks
> -- PMM