Re: [PATCH] acpi: Add Windows ACPI Emulated Device Table (WAET)

["Michael S. Tsirkin" <mst@redhat.com>]

On Wed, Mar 11, 2020 at 09:08:56PM +0200, Liran Alon wrote:
> 
> On 11/03/2020 20:59, no-reply@patchew.org wrote:
> > Patchew URL: https://urldefense.com/v3/__https://patchew.org/QEMU/20200311170826.79419-1-liran.alon@oracle.com/__;!!GqivPVa7Brio!L4XXKjkDknE86ihbnytm45vsQI41J-QWVCZRoXEXtPKIAsMmknrGJWVPZpKgLyM$
> > 
> > Hi,
> > 
> > This series failed the docker-quick@centos7 build test. Please find the testing commands and
> > their output below. If you have Docker installed, you can probably reproduce it
> > locally.
> > 
> > === TEST SCRIPT BEGIN ===
> > #!/bin/bash
> > make docker-image-centos7 V=1 NETWORK=1
> > time make docker-test-quick@centos7 SHOW_ENV=1 J=14 NETWORK=1
> > === TEST SCRIPT END ===
> > 
> > Using expected file 'tests/data/acpi/pc/HPET'
> > Looking for expected file 'tests/data/acpi/pc/WAET'
> > **
> > ERROR:/tmp/qemu-test/src/tests/qtest/bios-tables-test.c:367:load_expected_aml: assertion failed: (exp_sdt.aml_file)
> > ERROR - Bail out! ERROR:/tmp/qemu-test/src/tests/qtest/bios-tables-test.c:367:load_expected_aml: assertion failed: (exp_sdt.aml_file)
> 
> My bad. Didn't notice there are tests which verifies ACPI haven't changed
> and requires update for such patch.
> Will submit a patch for this test in v2.
> 
> -Liran
> 

Notice the process as documented in ./tests/qtest/bios-tables-test.c

-- 
MST