qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Marcel Apfelbaum <marcel.a@redhat.com>
To: qemu-devel@nongnu.org
Cc: mst@redhat.com
Subject: [Qemu-devel]  [PATCH for-1.7 0/5] acpi unit-test: added tests
Date: Thu, 21 Nov 2013 21:33:20 +0200	[thread overview]
Message-ID: <1385062405-3800-1-git-send-email-marcel.a@redhat.com> (raw)

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

Marcel Apfelbaum (5):
  acpi unit-test: load and check facs table
  acpi unit-test: adjust the test data structure for better handling
  acpi unit-test: add test files
  configure: added acpi unit-test files
  acpi unit-test: compare DSDT and SSDT tables against expected values

 configure                         |    4 +
 tests/acpi-test-data/pc/APIC.dsl  |  103 ++
 tests/acpi-test-data/pc/DSDT.dsl  | 1870 ++++++++++++++++++++++
 tests/acpi-test-data/pc/FACP.dsl  |   99 ++
 tests/acpi-test-data/pc/FACS.dsl  |   32 +
 tests/acpi-test-data/pc/HPET.dsl  |   43 +
 tests/acpi-test-data/pc/SSDT.dsl  |  634 ++++++++
 tests/acpi-test-data/q35/APIC.dsl |  103 ++
 tests/acpi-test-data/q35/DSDT.dsl | 3197 +++++++++++++++++++++++++++++++++++++
 tests/acpi-test-data/q35/FACP.dsl |   99 ++
 tests/acpi-test-data/q35/FACS.dsl |   32 +
 tests/acpi-test-data/q35/HPET.dsl |   43 +
 tests/acpi-test-data/q35/MCFG.dsl |   36 +
 tests/acpi-test-data/q35/SSDT.dsl |  665 ++++++++
 tests/acpi-test.c                 |  282 +++-
 15 files changed, 7210 insertions(+), 32 deletions(-)
 create mode 100644 tests/acpi-test-data/pc/APIC.dsl
 create mode 100644 tests/acpi-test-data/pc/DSDT.dsl
 create mode 100644 tests/acpi-test-data/pc/FACP.dsl
 create mode 100644 tests/acpi-test-data/pc/FACS.dsl
 create mode 100644 tests/acpi-test-data/pc/HPET.dsl
 create mode 100644 tests/acpi-test-data/pc/SSDT.dsl
 create mode 100644 tests/acpi-test-data/q35/APIC.dsl
 create mode 100644 tests/acpi-test-data/q35/DSDT.dsl
 create mode 100644 tests/acpi-test-data/q35/FACP.dsl
 create mode 100644 tests/acpi-test-data/q35/FACS.dsl
 create mode 100644 tests/acpi-test-data/q35/HPET.dsl
 create mode 100644 tests/acpi-test-data/q35/MCFG.dsl
 create mode 100644 tests/acpi-test-data/q35/SSDT.dsl

-- 
1.8.3.1

             reply	other threads:[~2013-11-21 19:36 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-11-21 19:33 Marcel Apfelbaum [this message]
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 ` [Qemu-devel] [PATCH for-1.7 0/5] acpi unit-test: added tests Erik Rull
2013-11-21 21:30   ` 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=1385062405-3800-1-git-send-email-marcel.a@redhat.com \
    --to=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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).