From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:56312) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Vjbzb-0003Kf-KT for qemu-devel@nongnu.org; Thu, 21 Nov 2013 16:41:01 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1VjbzV-0002Yy-BS for qemu-devel@nongnu.org; Thu, 21 Nov 2013 16:40:55 -0500 Received: from moutng.kundenserver.de ([212.227.17.9]:60383) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1VjbzV-0002Yu-1N for qemu-devel@nongnu.org; Thu, 21 Nov 2013 16:40:49 -0500 Message-ID: <528E7DDB.8000105@rdsoftware.de> Date: Thu, 21 Nov 2013 22:40:43 +0100 From: Erik Rull MIME-Version: 1.0 References: <1385062405-3800-1-git-send-email-marcel.a@redhat.com> <528E791E.2010105@rdsoftware.de> <1385069438.5379.18.camel@localhost.localdomain> In-Reply-To: <1385069438.5379.18.camel@localhost.localdomain> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH for-1.7 0/5] acpi unit-test: added tests List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Marcel Apfelbaum Cc: qemu-devel@nongnu.org, mst@redhat.com Marcel Apfelbaum wrote: > 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 Thanks. But then I don't understand the error that appears: CPP x86_64-softmmu/acpi-dsdt.dsl.i.orig ACPI_PREPROCESS x86_64-softmmu/acpi-dsdt.dsl.i IASL x86_64-softmmu/acpi-dsdt.dsl.i make[1]: *** [hw/i386/acpi-dsdt.hex] Error 1 make: *** [subdir-x86_64-softmmu] Error 2 I don't find a chance to disable this access/compilation within configure. If I just missed a possible option, it would be great to point me at it. I found also when grep'ing through the sources that there is an "if" for check whether iasl is present or not. But setting --iasl= (empty) to force a removal of iasl for the qemu compilation gives a configure error. Best regards, Erik > >> 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 >> >> >> > > >