From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:48651) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1e1u1Z-0002bH-7b for qemu-devel@nongnu.org; Tue, 10 Oct 2017 08:52:45 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1e1u1V-0003Tw-5O for qemu-devel@nongnu.org; Tue, 10 Oct 2017 08:52:41 -0400 Received: from mx1.redhat.com ([209.132.183.28]:56868) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1e1u1U-0003TQ-OD for qemu-devel@nongnu.org; Tue, 10 Oct 2017 08:52:37 -0400 Date: Tue, 10 Oct 2017 09:52:31 -0300 From: Eduardo Habkost Message-ID: <20171010125231.GD4015@localhost.localdomain> References: <20171005203638.19255-1-ehabkost@redhat.com> <20171006152751.GR4015@localhost.localdomain> <4479e30f-801d-0d8e-712d-06f2dd987528@cn.fujitsu.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <4479e30f-801d-0d8e-712d-06f2dd987528@cn.fujitsu.com> Subject: Re: [Qemu-devel] [PULL 0/9] x86 and machine queue, 2017-10-05 List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Dou Liyang Cc: Peter Maydell , QEMU Developers 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 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