[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PULL v2 09/35] acpi unit-test: extract iasl executable fro
From: |
Michael S. Tsirkin |
Subject: |
[Qemu-devel] [PULL v2 09/35] acpi unit-test: extract iasl executable from configuration |
Date: |
Sun, 26 Jan 2014 18:05:45 +0200 |
From: Marcel Apfelbaum <address@hidden>
The test checked if iasl is installed by running "iasl"
and checking the error output.
It is better to use the iasl executable as appears
in configuration.
Signed-off-by: Marcel Apfelbaum <address@hidden>
Signed-off-by: Michael S. Tsirkin <address@hidden>
---
tests/acpi-test.c | 31 ++++++++-----------------------
1 file changed, 8 insertions(+), 23 deletions(-)
diff --git a/tests/acpi-test.c b/tests/acpi-test.c
index c3fc603..83e5aed 100644
--- a/tests/acpi-test.c
+++ b/tests/acpi-test.c
@@ -127,6 +127,11 @@ static uint8_t boot_sector[0x7e000] = {
static const char *disk = "tests/acpi-test-disk.raw";
static const char *data_dir = "tests/acpi-test-data";
+#ifdef CONFIG_IASL
+static const char *iasl = stringify(CONFIG_IASL);
+#else
+static const char *iasl;
+#endif
static void free_test_data(test_data *data)
{
@@ -358,26 +363,6 @@ static void test_acpi_ssdt_tables(test_data *data)
}
}
-static bool iasl_installed(void)
-{
- gchar *out = NULL, *out_err = NULL;
- bool ret;
-
- /* pass 'out' and 'out_err' in order to be redirected */
- ret = g_spawn_command_line_sync("iasl", &out, &out_err, NULL, NULL);
-
- if (out_err) {
- ret = ret && (out_err[0] == '\0');
- g_free(out_err);
- }
-
- if (out) {
- g_free(out);
- }
-
- return ret;
-}
-
static void dump_aml_files(test_data *data)
{
AcpiSdtTable *sdt;
@@ -406,7 +391,7 @@ static void load_asl(GArray *sdts, AcpiSdtTable *sdt)
{
AcpiSdtTable *temp;
GError *error = NULL;
- GString *command_line = g_string_new("'iasl' ");
+ GString *command_line = g_string_new(iasl);
gint fd;
gchar *out, *out_err;
gboolean ret;
@@ -417,7 +402,7 @@ static void load_asl(GArray *sdts, AcpiSdtTable *sdt)
close(fd);
/* build command line */
- g_string_append_printf(command_line, "-p %s ", sdt->asl_file);
+ g_string_append_printf(command_line, " -p %s ", sdt->asl_file);
for (i = 0; i < 2; ++i) { /* reference DSDT and SSDT */
temp = &g_array_index(sdts, AcpiSdtTable, i);
g_string_append_printf(command_line, "-e %s ", temp->aml_file);
@@ -571,7 +556,7 @@ static void test_acpi_one(const char *params, test_data
*data)
test_acpi_dsdt_table(data);
test_acpi_ssdt_tables(data);
- if (iasl_installed()) {
+ if (iasl) {
test_acpi_asl(data);
}
--
MST
- [Qemu-devel] [PULL v2 00/35] acpi, pci, pc, virtio fixes and enhancements, Michael S. Tsirkin, 2014/01/26
- [Qemu-devel] [PULL v2 01/35] Python-lang gdb script to extract x86_64 guest vmcore from qemu coredump, Michael S. Tsirkin, 2014/01/26
- [Qemu-devel] [PULL v2 02/35] Add DSDT node for AppleSMC, Michael S. Tsirkin, 2014/01/26
- [Qemu-devel] [PULL v2 03/35] ACPI: Fix AppleSMC _STA size, Michael S. Tsirkin, 2014/01/26
- [Qemu-devel] [PULL v2 04/35] virtio: Fix return value for dummy function vhost_net_virtqueue_pending, Michael S. Tsirkin, 2014/01/26
- [Qemu-devel] [PULL v2 05/35] acpi unit-test: add test files, Michael S. Tsirkin, 2014/01/26
- [Qemu-devel] [PULL v2 06/35] configure: added acpi unit-test files, Michael S. Tsirkin, 2014/01/26
- [Qemu-devel] [PULL v2 07/35] acpi unit-test: compare DSDT and SSDT tables against expected values, Michael S. Tsirkin, 2014/01/26
- [Qemu-devel] [PULL v2 08/35] configure: add CONFIG_IASL to config-host.h, Michael S. Tsirkin, 2014/01/26
- [Qemu-devel] [PULL v2 10/35] acpi unit-test: added script to rebuild the expected aml files, Michael S. Tsirkin, 2014/01/26
- [Qemu-devel] [PULL v2 09/35] acpi unit-test: extract iasl executable from configuration,
Michael S. Tsirkin <=
- [Qemu-devel] [PULL v2 11/35] acpi unit-test: hook to rebuild expected aml files, Michael S. Tsirkin, 2014/01/26
- [Qemu-devel] [PULL v2 12/35] tests: fix acpi to work on bigendian host, Michael S. Tsirkin, 2014/01/26
- [Qemu-devel] [PULL v2 13/35] acpi unit-test: renamed ssdt_tables to tables, Michael S. Tsirkin, 2014/01/26
- [Qemu-devel] [PULL v2 14/35] acpi unit-test: resolved iasl crash, Michael S. Tsirkin, 2014/01/26
- [Qemu-devel] [PULL v2 16/35] pc: make: fix dependencies: rebuild when included file is changed, Michael S. Tsirkin, 2014/01/26
- [Qemu-devel] [PULL v2 15/35] acpi unit-test: do not fail on asl mismatch, Michael S. Tsirkin, 2014/01/26
- [Qemu-devel] [PULL v2 17/35] pci: add pci_for_each_bus_depth_first, Michael S. Tsirkin, 2014/01/26
- [Qemu-devel] [PULL v2 18/35] pcihp: generalization of piix4 acpi, Michael S. Tsirkin, 2014/01/26
- [Qemu-devel] [PULL v2 19/35] piix4: add acpi pci hotplug support, Michael S. Tsirkin, 2014/01/26
- [Qemu-devel] [PULL v2 20/35] acpi-build: enable hotplug for PCI bridges, Michael S. Tsirkin, 2014/01/26