Re: [Qemu-devel] [PATCH for-1.7 0/5] acpi unit-test: added tests

[Marcel Apfelbaum]

On Thu, 2013-11-21 at 22:20 +0100, Erik Rull wrote:
> Marcel Apfelbaum wrote:
> > Added 2 tests:
> >   1. Basic check of FACS table (missed on prev submission)
> >   2. Compare DSDT and SSDT tables against expected values
> >
> > Test 2:
> >   - runs only if iasl is installed on the host machine.
> >   - the test plan:
> >     1. Dumps the ACPI tables as AML on the disk.
> >     2. Runs iasl to disassembly the tables into ASL files.
> >     3. Compares them with expected offline ASL files.
> >
> >   - the test runs for both default machine and q35.
> >   - in case the test fails, it can be easily tweaked to
> >     show the differences between the ASL files and
> >     understand the issue.
> >
> > Patches:
> >   1/5 - test 1
> >   2/5 - some infrastructure improvements
> >   3/5 - expected asl files for test 2
> >   4/5 - creates links for the expected files
> >         if the build directory is not current
> >   5/5 - test 2
> >
> 
> Which iasl Version is needed for the ACPI compilation and testing? I have 
> an IASL installed on my build machine, but when trying to compile the ACPI 
> stuff, it fails. Maybe it's just too old, but I didn't find a way to 
> disable the iasl access. Must I uninstall iasl on my machine to get qemu 
> compiled again?
I would use the latest version, version 20130823, from 
https://acpica.org/downloads 
or the git from git://github.com/acpica/acpica.git

I don't think you need iasl on your computer to build qemu.

Hope I helped,
Marcel

> The IASL version is:
> Intel ACPI Component Architecture
> ASL Optimizing Compiler version 20060912 [Dec 20 2006]
> Copyright (C) 2000 - 2006 Intel Corporation
> Supports ACPI Specification Revision 3.0a
> 
> Thanks for your support.
> 
> Best regards,
> 
> Erik
> 
> 
>