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

[Michael S. Tsirkin]

On Wed, Mar 11, 2020 at 09:08:56PM +0200, Liran Alon wrote:
> 
> On 11/03/2020 20:59, address@hidden wrote:
> > Patchew URL: 
> > https://urldefense.com/v3/__https://patchew.org/QEMU/address@hidden/__;!!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