yeahuh / rpms / qemu-kvm

Forked from rpms/qemu-kvm 2 years ago
Clone
9ae3a8
From 3f51f5d57b5a4c5de484db56175c58e1825d6aa9 Mon Sep 17 00:00:00 2001
9ae3a8
Message-Id: <3f51f5d57b5a4c5de484db56175c58e1825d6aa9.1387288155.git.minovotn@redhat.com>
9ae3a8
In-Reply-To: <527da6c2ce2c09d0183aa8595fc95f136f61b6df.1387288155.git.minovotn@redhat.com>
9ae3a8
References: <527da6c2ce2c09d0183aa8595fc95f136f61b6df.1387288155.git.minovotn@redhat.com>
9ae3a8
From: Stefan Hajnoczi <stefanha@redhat.com>
9ae3a8
Date: Thu, 12 Dec 2013 16:21:28 +0100
9ae3a8
Subject: [PATCH 8/8] qtest: Use -display none by default
9ae3a8
MIME-Version: 1.0
9ae3a8
Content-Type: text/plain; charset=UTF-8
9ae3a8
Content-Transfer-Encoding: 8bit
9ae3a8
9ae3a8
RH-Author: Stefan Hajnoczi <stefanha@redhat.com>
9ae3a8
Message-id: <1386865288-1575-9-git-send-email-stefanha@redhat.com>
9ae3a8
Patchwork-id: 56262
9ae3a8
O-Subject: [RHEL7 qemu-kvm PATCH 8/8] qtest: Use -display none by default
9ae3a8
Bugzilla: 1003773
9ae3a8
RH-Acked-by: Paolo Bonzini <pbonzini@redhat.com>
9ae3a8
RH-Acked-by: Jeffrey Cody <jcody@redhat.com>
9ae3a8
RH-Acked-by: Markus Armbruster <armbru@redhat.com>
9ae3a8
9ae3a8
From: Andreas Färber <afaerber@suse.de>
9ae3a8
9ae3a8
This avoids each test needing to add it to suppress windows popping up.
9ae3a8
9ae3a8
[Commit 7ceeedd016facf8d58e14a0d1417fa7225d71072 ("blockdev-test: add
9ae3a8
test case for drive_add duplicate IDs") and commit
9ae3a8
43cd209803d6cffb1e1a028c9ff2fd0ff4fce954 ("qdev-monitor-test: add
9ae3a8
device_add leak test cases") added qtest tests without specifying
9ae3a8
-display none.
9ae3a8
9ae3a8
As a result, "make check" now tries to use graphics (GTK or SDL).  Since
9ae3a8
graphics are not used by the test and inappropriate for headless "make
9ae3a8
check" runs, add the missing -display none.
9ae3a8
9ae3a8
This fixes "make check" in the QEMU buildbot.
9ae3a8
-- Stefan]
9ae3a8
9ae3a8
Signed-off-by: Andreas Färber <afaerber@suse.de>
9ae3a8
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
9ae3a8
(cherry picked from commit 2ad645d2854746b55ddfd1d8e951f689cca5d78f)
9ae3a8
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
9ae3a8
9ae3a8
Conflicts:
9ae3a8
	tests/boot-order-test.c
9ae3a8
	tests/endianness-test.c
9ae3a8
	tests/qom-test.c
9ae3a8
9ae3a8
qom-test.c and endianness-test.c do not exist in RHEL.
9ae3a8
9ae3a8
boot-order-test.c is slightly older but the conflict is simple to
9ae3a8
resolve.
9ae3a8
---
9ae3a8
 tests/boot-order-test.c | 2 +-
9ae3a8
 tests/fw_cfg-test.c     | 3 +--
9ae3a8
 tests/hd-geo-test.c     | 2 +-
9ae3a8
 tests/i440fx-test.c     | 2 +-
9ae3a8
 tests/libqtest.c        | 1 +
9ae3a8
 tests/m48t59-test.c     | 2 +-
9ae3a8
 tests/rtc-test.c        | 2 +-
9ae3a8
 tests/tmp105-test.c     | 2 +-
9ae3a8
 8 files changed, 8 insertions(+), 8 deletions(-)
9ae3a8
9ae3a8
Signed-off-by: Michal Novotny <minovotn@redhat.com>
9ae3a8
---
9ae3a8
 tests/boot-order-test.c | 2 +-
9ae3a8
 tests/fw_cfg-test.c     | 3 +--
9ae3a8
 tests/hd-geo-test.c     | 2 +-
9ae3a8
 tests/i440fx-test.c     | 2 +-
9ae3a8
 tests/libqtest.c        | 1 +
9ae3a8
 tests/m48t59-test.c     | 2 +-
9ae3a8
 tests/rtc-test.c        | 2 +-
9ae3a8
 tests/tmp105-test.c     | 2 +-
9ae3a8
 8 files changed, 8 insertions(+), 8 deletions(-)
9ae3a8
9ae3a8
diff --git a/tests/boot-order-test.c b/tests/boot-order-test.c
9ae3a8
index a35e9f7..f386600 100644
9ae3a8
--- a/tests/boot-order-test.c
9ae3a8
+++ b/tests/boot-order-test.c
9ae3a8
@@ -32,7 +32,7 @@ static void test_pc_with_args(const char *test_args,
9ae3a8
                               uint8_t boot1, uint8_t boot2,
9ae3a8
                               uint8_t reboot1, uint8_t reboot2)
9ae3a8
 {
9ae3a8
-    char *args = g_strdup_printf("-nodefaults -display none %s", test_args);
9ae3a8
+    char *args = g_strdup_printf("-nodefaults %s", test_args);
9ae3a8
 
9ae3a8
     qtest_start(args);
9ae3a8
     test_pc_cmos(boot1, boot2);
9ae3a8
diff --git a/tests/fw_cfg-test.c b/tests/fw_cfg-test.c
9ae3a8
index c284c4d..3428dca 100644
9ae3a8
--- a/tests/fw_cfg-test.c
9ae3a8
+++ b/tests/fw_cfg-test.c
9ae3a8
@@ -126,8 +126,7 @@ int main(int argc, char **argv)
9ae3a8
     g_test_add_func("/fw_cfg/numa", test_fw_cfg_numa);
9ae3a8
     g_test_add_func("/fw_cfg/boot_menu", test_fw_cfg_boot_menu);
9ae3a8
 
9ae3a8
-    cmdline = g_strdup_printf("-display none "
9ae3a8
-                              "-uuid 4600cb32-38ec-4b2f-8acb-81c6ea54f2d8 ");
9ae3a8
+    cmdline = g_strdup_printf("-uuid 4600cb32-38ec-4b2f-8acb-81c6ea54f2d8 ");
9ae3a8
     s = qtest_start(cmdline);
9ae3a8
     g_free(cmdline);
9ae3a8
 
9ae3a8
diff --git a/tests/hd-geo-test.c b/tests/hd-geo-test.c
9ae3a8
index b72042e..c84d1e7 100644
9ae3a8
--- a/tests/hd-geo-test.c
9ae3a8
+++ b/tests/hd-geo-test.c
9ae3a8
@@ -171,7 +171,7 @@ static int setup_common(char *argv[], int argv_sz)
9ae3a8
 {
9ae3a8
     memset(cur_ide, 0, sizeof(cur_ide));
9ae3a8
     return append_arg(0, argv, argv_sz,
9ae3a8
-                      g_strdup("-nodefaults -display none"));
9ae3a8
+                      g_strdup("-nodefaults"));
9ae3a8
 }
9ae3a8
 
9ae3a8
 static void setup_mbr(int img_idx, MBRcontents mbr)
9ae3a8
diff --git a/tests/i440fx-test.c b/tests/i440fx-test.c
9ae3a8
index 08ce820..65c786c 100644
9ae3a8
--- a/tests/i440fx-test.c
9ae3a8
+++ b/tests/i440fx-test.c
9ae3a8
@@ -265,7 +265,7 @@ int main(int argc, char **argv)
9ae3a8
 
9ae3a8
     data.num_cpus = 1;
9ae3a8
 
9ae3a8
-    cmdline = g_strdup_printf("-display none -smp %d", data.num_cpus);
9ae3a8
+    cmdline = g_strdup_printf("-smp %d", data.num_cpus);
9ae3a8
     s = qtest_start(cmdline);
9ae3a8
     g_free(cmdline);
9ae3a8
 
9ae3a8
diff --git a/tests/libqtest.c b/tests/libqtest.c
9ae3a8
index 83424c3..359d571 100644
9ae3a8
--- a/tests/libqtest.c
9ae3a8
+++ b/tests/libqtest.c
9ae3a8
@@ -135,6 +135,7 @@ QTestState *qtest_init(const char *extra_args)
9ae3a8
                                   "-qmp unix:%s,nowait "
9ae3a8
                                   "-pidfile %s "
9ae3a8
                                   "-machine accel=qtest "
9ae3a8
+                                  "-display none "
9ae3a8
                                   "%s", qemu_binary, s->socket_path,
9ae3a8
                                   s->qmp_socket_path, pid_file,
9ae3a8
                                   extra_args ?: "");
9ae3a8
diff --git a/tests/m48t59-test.c b/tests/m48t59-test.c
9ae3a8
index 4081a5f..6abc4c8 100644
9ae3a8
--- a/tests/m48t59-test.c
9ae3a8
+++ b/tests/m48t59-test.c
9ae3a8
@@ -249,7 +249,7 @@ int main(int argc, char **argv)
9ae3a8
 
9ae3a8
     g_test_init(&argc, &argv, NULL);
9ae3a8
 
9ae3a8
-    s = qtest_start("-display none -rtc clock=vm");
9ae3a8
+    s = qtest_start("-rtc clock=vm");
9ae3a8
 
9ae3a8
     qtest_add_func("/rtc/bcd/check-time", bcd_check_time);
9ae3a8
     qtest_add_func("/rtc/fuzz-registers", fuzz_registers);
9ae3a8
diff --git a/tests/rtc-test.c b/tests/rtc-test.c
9ae3a8
index 3395d7f..f1b123f 100644
9ae3a8
--- a/tests/rtc-test.c
9ae3a8
+++ b/tests/rtc-test.c
9ae3a8
@@ -552,7 +552,7 @@ int main(int argc, char **argv)
9ae3a8
 
9ae3a8
     g_test_init(&argc, &argv, NULL);
9ae3a8
 
9ae3a8
-    s = qtest_start("-display none -rtc clock=vm");
9ae3a8
+    s = qtest_start("-rtc clock=vm");
9ae3a8
     qtest_irq_intercept_in(s, "ioapic");
9ae3a8
 
9ae3a8
     qtest_add_func("/rtc/check-time/bcd", bcd_check_time);
9ae3a8
diff --git a/tests/tmp105-test.c b/tests/tmp105-test.c
9ae3a8
index fecd6dc..5ac48e2 100644
9ae3a8
--- a/tests/tmp105-test.c
9ae3a8
+++ b/tests/tmp105-test.c
9ae3a8
@@ -59,7 +59,7 @@ int main(int argc, char **argv)
9ae3a8
 
9ae3a8
     g_test_init(&argc, &argv, NULL);
9ae3a8
 
9ae3a8
-    s = qtest_start("-display none -machine n800");
9ae3a8
+    s = qtest_start("-machine n800");
9ae3a8
     i2c = omap_i2c_create(OMAP2_I2C_1_BASE);
9ae3a8
     addr = N8X0_ADDR;
9ae3a8
 
9ae3a8
-- 
9ae3a8
1.7.11.7
9ae3a8