From: Erik Rull <erik.rull@rdsoftware.de>
To: Marcel Apfelbaum <marcel.a@redhat.com>, qemu-devel@nongnu.org
Cc: mst@redhat.com
Subject: Re: [Qemu-devel] [PATCH for-1.7 0/5] acpi unit-test: added tests
Date: Thu, 21 Nov 2013 22:20:30 +0100 [thread overview]
Message-ID: <528E791E.2010105@rdsoftware.de> (raw)
In-Reply-To: <1385062405-3800-1-git-send-email-marcel.a@redhat.com>
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?
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
next prev parent reply other threads:[~2013-11-21 21:20 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-11-21 19:33 [Qemu-devel] [PATCH for-1.7 0/5] acpi unit-test: added tests Marcel Apfelbaum
2013-11-21 19:33 ` [Qemu-devel] [PATCH for-1.7 1/5] acpi unit-test: load and check facs table Marcel Apfelbaum
2013-12-10 16:42 ` Michael S. Tsirkin
2013-11-21 19:33 ` [Qemu-devel] [PATCH for-1.7 2/5] acpi unit-test: adjust the test data structure for better handling Marcel Apfelbaum
2013-12-10 16:42 ` [Qemu-devel] (no subject) Michael S. Tsirkin
2013-11-21 19:33 ` [Qemu-devel] [PATCH for-1.7 3/5] acpi unit-test: add test files Marcel Apfelbaum
2013-12-10 16:45 ` Michael S. Tsirkin
2013-12-11 9:23 ` Marcel Apfelbaum
2013-12-11 10:35 ` Michael S. Tsirkin
2013-11-21 19:33 ` [Qemu-devel] [PATCH for-1.7 4/5] configure: added acpi unit-test files Marcel Apfelbaum
2013-11-21 19:33 ` [Qemu-devel] [PATCH for-1.7 5/5] acpi unit-test: compare DSDT and SSDT tables against expected values Marcel Apfelbaum
2013-11-21 21:20 ` Erik Rull [this message]
2013-11-21 21:30 ` [Qemu-devel] [PATCH for-1.7 0/5] acpi unit-test: added tests Marcel Apfelbaum
2013-11-21 21:40 ` Erik Rull
2013-11-24 11:32 ` Michael S. Tsirkin
2013-11-24 11:33 ` Michael S. Tsirkin
2013-11-24 11:37 ` Marcel Apfelbaum
2013-11-24 11:46 ` Michael S. Tsirkin
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=528E791E.2010105@rdsoftware.de \
--to=erik.rull@rdsoftware.de \
--cc=marcel.a@redhat.com \
--cc=mst@redhat.com \
--cc=qemu-devel@nongnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.