902636
From 154215041df085271a780a2989f4f481226e3e34 Mon Sep 17 00:00:00 2001
902636
From: Miroslav Rezanina <mrezanin@redhat.com>
902636
Date: Fri, 19 Oct 2018 13:48:41 +0200
902636
Subject: Enable make check
902636
902636
Fixing tests after device disabling and machine types changes and enabling
902636
make check run during build.
902636
902636
Signed-off-by: Miroslav Rezanina <mrezanin@redhat.com>
902636
902636
Rebase changes (4.0.0):
902636
- Remove testing for pseries-2.7 in endianess test
902636
- Disable device-plug-test on s390x as it use disabled device
902636
- Do not run cpu-plug-tests on 7.3 and older machine types
902636
902636
Rebase changes (4.1.0-rc0):
902636
- removed iotests 068
902636
902636
Rebase changes (4.1.0-rc1):
902636
- remove all 205 tests (unstable)
902636
902636
Rebase changes (4.2.0-rc0):
902636
- partially disable hd-geo-test (requires lsi53c895a)
902636
902636
Merged patches (4.0.0):
902636
- f7ffd13 Remove 7 qcow2 and luks iotests that are taking > 25 sec to run during the fast train build proce
902636
902636
Merged patches (4.1.0-rc0):
902636
- 41288ff redhat: Remove raw iotest 205
902636
902636
Signed-off-by: Danilo C. L. de Paula <ddepaula@redhat.com>
902636
---
902636
 redhat/qemu-kvm.spec.template |  2 +-
902636
 tests/Makefile.include        | 10 +++++-----
902636
 tests/boot-serial-test.c      |  6 +++++-
902636
 tests/cpu-plug-test.c         |  4 ++--
902636
 tests/e1000-test.c            |  2 ++
902636
 tests/hd-geo-test.c           |  4 ++++
902636
 tests/prom-env-test.c         |  4 ++++
902636
 tests/qemu-iotests/051        | 12 ++++++------
902636
 tests/qemu-iotests/group      |  4 ++--
902636
 tests/test-x86-cpuid-compat.c |  2 ++
902636
 tests/usb-hcd-xhci-test.c     |  4 ++++
902636
 11 files changed, 37 insertions(+), 17 deletions(-)
902636
902636
diff --git a/tests/Makefile.include b/tests/Makefile.include
902636
index b483790cf3..53bdbdfee0 100644
902636
--- a/tests/Makefile.include
902636
+++ b/tests/Makefile.include
902636
@@ -172,7 +172,7 @@ check-qtest-i386-y += tests/ide-test$(EXESUF)
902636
 check-qtest-i386-y += tests/ahci-test$(EXESUF)
902636
 check-qtest-i386-y += tests/hd-geo-test$(EXESUF)
902636
 check-qtest-i386-y += tests/boot-order-test$(EXESUF)
902636
-check-qtest-i386-y += tests/bios-tables-test$(EXESUF)
902636
+#check-qtest-i386-y += tests/bios-tables-test$(EXESUF)
902636
 check-qtest-i386-$(CONFIG_SGA) += tests/boot-serial-test$(EXESUF)
902636
 check-qtest-i386-$(CONFIG_SLIRP) += tests/pxe-test$(EXESUF)
902636
 check-qtest-i386-y += tests/rtc-test$(EXESUF)
902636
@@ -230,7 +230,7 @@ check-qtest-mips64el-$(CONFIG_VGA) += tests/display-vga-test$(EXESUF)
902636
 check-qtest-moxie-y += tests/boot-serial-test$(EXESUF)
902636
 
902636
 check-qtest-ppc-$(CONFIG_ISA_TESTDEV) = tests/endianness-test$(EXESUF)
902636
-check-qtest-ppc-y += tests/boot-order-test$(EXESUF)
902636
+#check-qtest-ppc-y += tests/boot-order-test$(EXESUF)
902636
 check-qtest-ppc-y += tests/prom-env-test$(EXESUF)
902636
 check-qtest-ppc-y += tests/drive_del-test$(EXESUF)
902636
 check-qtest-ppc-y += tests/boot-serial-test$(EXESUF)
902636
@@ -244,8 +244,8 @@ check-qtest-ppc64-$(CONFIG_PSERIES) += tests/rtas-test$(EXESUF)
902636
 check-qtest-ppc64-$(CONFIG_SLIRP) += tests/pxe-test$(EXESUF)
902636
 check-qtest-ppc64-$(CONFIG_USB_UHCI) += tests/usb-hcd-uhci-test$(EXESUF)
902636
 check-qtest-ppc64-$(CONFIG_USB_XHCI_NEC) += tests/usb-hcd-xhci-test$(EXESUF)
902636
-check-qtest-ppc64-$(CONFIG_SLIRP) += tests/test-netfilter$(EXESUF)
902636
-check-qtest-ppc64-$(CONFIG_POSIX) += tests/test-filter-mirror$(EXESUF)
902636
+#check-qtest-ppc64-$(CONFIG_SLIRP) += tests/test-netfilter$(EXESUF)
902636
+#check-qtest-ppc64-$(CONFIG_POSIX) += tests/test-filter-mirror$(EXESUF)
902636
 check-qtest-ppc64-$(CONFIG_RTL8139_PCI) += tests/test-filter-redirector$(EXESUF)
902636
 check-qtest-ppc64-$(CONFIG_VGA) += tests/display-vga-test$(EXESUF)
902636
 check-qtest-ppc64-y += tests/numa-test$(EXESUF)
902636
@@ -291,7 +291,7 @@ check-qtest-s390x-$(CONFIG_SLIRP) += tests/test-netfilter$(EXESUF)
902636
 check-qtest-s390x-$(CONFIG_POSIX) += tests/test-filter-mirror$(EXESUF)
902636
 check-qtest-s390x-$(CONFIG_POSIX) += tests/test-filter-redirector$(EXESUF)
902636
 check-qtest-s390x-y += tests/drive_del-test$(EXESUF)
902636
-check-qtest-s390x-y += tests/device-plug-test$(EXESUF)
902636
+#check-qtest-s390x-y += tests/device-plug-test$(EXESUF)
902636
 check-qtest-s390x-y += tests/virtio-ccw-test$(EXESUF)
902636
 check-qtest-s390x-y += tests/cpu-plug-test$(EXESUF)
902636
 check-qtest-s390x-y += tests/migration-test$(EXESUF)
902636
diff --git a/tests/boot-serial-test.c b/tests/boot-serial-test.c
902636
index d3a54a0ba5..33ce72b89c 100644
902636
--- a/tests/boot-serial-test.c
902636
+++ b/tests/boot-serial-test.c
902636
@@ -108,19 +108,23 @@ static testdef_t tests[] = {
902636
     { "ppc", "g3beige", "", "PowerPC,750" },
902636
     { "ppc", "mac99", "", "PowerPC,G4" },
902636
     { "ppc", "sam460ex", "-m 256", "DRAM:  256 MiB" },
902636
+#if 0 /* Disabled for Red Hat Enterprise Linux */
902636
     { "ppc64", "ppce500", "", "U-Boot" },
902636
     { "ppc64", "40p", "-m 192", "Memory: 192M" },
902636
     { "ppc64", "mac99", "", "PowerPC,970FX" },
902636
+#endif
902636
     { "ppc64", "pseries",
902636
       "-machine cap-cfpc=broken,cap-sbbc=broken,cap-ibs=broken",
902636
       "Open Firmware" },
902636
+#if 0 /* Disabled for Red Hat Enterprise Linux */
902636
     { "ppc64", "powernv8", "", "OPAL" },
902636
     { "ppc64", "powernv9", "", "OPAL" },
902636
     { "ppc64", "sam460ex", "-device e1000", "8086  100e" },
902636
+#endif
902636
     { "i386", "isapc", "-cpu qemu32 -device sga", "SGABIOS" },
902636
     { "i386", "pc", "-device sga", "SGABIOS" },
902636
     { "i386", "q35", "-device sga", "SGABIOS" },
902636
-    { "x86_64", "isapc", "-cpu qemu32 -device sga", "SGABIOS" },
902636
+    { "x86_64", "pc", "-cpu qemu32 -device sga", "SGABIOS" },
902636
     { "x86_64", "q35", "-device sga", "SGABIOS" },
902636
     { "sparc", "LX", "", "TMS390S10" },
902636
     { "sparc", "SS-4", "", "MB86904" },
902636
diff --git a/tests/cpu-plug-test.c b/tests/cpu-plug-test.c
902636
index 30e514bbfb..a04beae1c6 100644
902636
--- a/tests/cpu-plug-test.c
902636
+++ b/tests/cpu-plug-test.c
902636
@@ -185,8 +185,8 @@ static void add_pseries_test_case(const char *mname)
902636
     char *path;
902636
     PlugTestData *data;
902636
 
902636
-    if (!g_str_has_prefix(mname, "pseries-") ||
902636
-        (g_str_has_prefix(mname, "pseries-2.") && atoi(&mname[10]) < 7)) {
902636
+    if (!g_str_has_prefix(mname, "pseries-rhel") ||
902636
+        (g_str_has_prefix(mname, "pseries-rhel7.") && atoi(&mname[14]) < 4)) {
902636
         return;
902636
     }
902636
     data = g_new(PlugTestData, 1);
902636
diff --git a/tests/e1000-test.c b/tests/e1000-test.c
902636
index c387984ef6..c89112d6f8 100644
902636
--- a/tests/e1000-test.c
902636
+++ b/tests/e1000-test.c
902636
@@ -22,9 +22,11 @@ struct QE1000 {
902636
 
902636
 static const char *models[] = {
902636
     "e1000",
902636
+#if 0 /* Disabled for Red Hat Enterprise Linux */
902636
     "e1000-82540em",
902636
     "e1000-82544gc",
902636
     "e1000-82545em",
902636
+#endif
902636
 };
902636
 
902636
 static void *e1000_get_driver(void *obj, const char *interface)
902636
diff --git a/tests/hd-geo-test.c b/tests/hd-geo-test.c
902636
index 7e86c5416c..cc068bad87 100644
902636
--- a/tests/hd-geo-test.c
902636
+++ b/tests/hd-geo-test.c
902636
@@ -732,6 +732,7 @@ static void test_override_ide(void)
902636
     test_override(args, expected);
902636
 }
902636
 
902636
+#if 0 /* Require lsi53c895a - not supported on RHEL */
902636
 static void test_override_scsi(void)
902636
 {
902636
     TestArgs *args = create_args();
902636
@@ -776,6 +777,7 @@ static void test_override_scsi_2_controllers(void)
902636
     add_scsi_disk(args, 3, 1, 0, 1, 2, 0, 1, 0);
902636
     test_override(args, expected);
902636
 }
902636
+#endif
902636
 
902636
 static void test_override_virtio_blk(void)
902636
 {
902636
@@ -951,9 +953,11 @@ int main(int argc, char **argv)
902636
     qtest_add_func("hd-geo/ide/device/user/chst", test_ide_device_user_chst);
902636
     if (have_qemu_img()) {
902636
         qtest_add_func("hd-geo/override/ide", test_override_ide);
902636
+#if 0 /* Require lsi53c895a - not supported on RHEL */
902636
         qtest_add_func("hd-geo/override/scsi", test_override_scsi);
902636
         qtest_add_func("hd-geo/override/scsi_2_controllers",
902636
                        test_override_scsi_2_controllers);
902636
+#endif
902636
         qtest_add_func("hd-geo/override/virtio_blk", test_override_virtio_blk);
902636
         qtest_add_func("hd-geo/override/zero_chs", test_override_zero_chs);
902636
         qtest_add_func("hd-geo/override/scsi_hot_unplug",
902636
diff --git a/tests/prom-env-test.c b/tests/prom-env-test.c
902636
index 61bc1d1e7b..028d45c7d7 100644
902636
--- a/tests/prom-env-test.c
902636
+++ b/tests/prom-env-test.c
902636
@@ -88,10 +88,14 @@ int main(int argc, char *argv[])
902636
     if (!strcmp(arch, "ppc")) {
902636
         add_tests(ppc_machines);
902636
     } else if (!strcmp(arch, "ppc64")) {
902636
+#if 0 /* Disabled for Red Hat Enterprise Linux */
902636
         add_tests(ppc_machines);
902636
         if (g_test_slow()) {
902636
+#endif
902636
             qtest_add_data_func("prom-env/pseries", "pseries", test_machine);
902636
+#if 0 /* Disabled for Red Hat Enterprise Linux */
902636
         }
902636
+#endif
902636
     } else if (!strcmp(arch, "sparc")) {
902636
         add_tests(sparc_machines);
902636
     } else if (!strcmp(arch, "sparc64")) {
902636
diff --git a/tests/qemu-iotests/051 b/tests/qemu-iotests/051
902636
index 53bcdbc911..b387e0c233 100755
902636
--- a/tests/qemu-iotests/051
902636
+++ b/tests/qemu-iotests/051
902636
@@ -181,11 +181,11 @@ run_qemu -drive if=virtio
902636
 case "$QEMU_DEFAULT_MACHINE" in
902636
     pc)
902636
         run_qemu -drive if=none,id=disk -device ide-cd,drive=disk
902636
-        run_qemu -drive if=none,id=disk -device lsi53c895a -device scsi-cd,drive=disk
902636
+#        run_qemu -drive if=none,id=disk -device lsi53c895a -device scsi-cd,drive=disk
902636
         run_qemu -drive if=none,id=disk -device ide-drive,drive=disk
902636
         run_qemu -drive if=none,id=disk -device ide-hd,drive=disk
902636
-        run_qemu -drive if=none,id=disk -device lsi53c895a -device scsi-disk,drive=disk
902636
-        run_qemu -drive if=none,id=disk -device lsi53c895a -device scsi-hd,drive=disk
902636
+#        run_qemu -drive if=none,id=disk -device lsi53c895a -device scsi-disk,drive=disk
902636
+#        run_qemu -drive if=none,id=disk -device lsi53c895a -device scsi-hd,drive=disk
902636
         ;;
902636
      *)
902636
         ;;
902636
@@ -234,11 +234,11 @@ run_qemu -drive file="$TEST_IMG",if=virtio,readonly=on
902636
 case "$QEMU_DEFAULT_MACHINE" in
902636
     pc)
902636
         run_qemu -drive file="$TEST_IMG",if=none,id=disk,readonly=on -device ide-cd,drive=disk
902636
-        run_qemu -drive file="$TEST_IMG",if=none,id=disk,readonly=on -device lsi53c895a -device scsi-cd,drive=disk
902636
+#        run_qemu -drive file="$TEST_IMG",if=none,id=disk,readonly=on -device lsi53c895a -device scsi-cd,drive=disk
902636
         run_qemu -drive file="$TEST_IMG",if=none,id=disk,readonly=on -device ide-drive,drive=disk
902636
         run_qemu -drive file="$TEST_IMG",if=none,id=disk,readonly=on -device ide-hd,drive=disk
902636
-        run_qemu -drive file="$TEST_IMG",if=none,id=disk,readonly=on -device lsi53c895a -device scsi-disk,drive=disk
902636
-        run_qemu -drive file="$TEST_IMG",if=none,id=disk,readonly=on -device lsi53c895a -device scsi-hd,drive=disk
902636
+#        run_qemu -drive file="$TEST_IMG",if=none,id=disk,readonly=on -device lsi53c895a -device scsi-disk,drive=disk
902636
+#        run_qemu -drive file="$TEST_IMG",if=none,id=disk,readonly=on -device lsi53c895a -device scsi-hd,drive=disk
902636
         ;;
902636
      *)
902636
         ;;
902636
diff --git a/tests/qemu-iotests/group b/tests/qemu-iotests/group
902636
index 6b10a6a762..06cc734b26 100644
902636
--- a/tests/qemu-iotests/group
902636
+++ b/tests/qemu-iotests/group
902636
@@ -92,7 +92,7 @@
902636
 068 rw quick
902636
 069 rw auto quick
902636
 070 rw quick
902636
-071 rw auto quick
902636
+# 071 rw auto quick -- requires whitelisted blkverify
902636
 072 rw auto quick
902636
 073 rw auto quick
902636
 074 rw auto quick
902636
@@ -120,7 +120,7 @@
902636
 096 rw quick
902636
 097 rw auto backing
902636
 098 rw auto backing quick
902636
-099 rw auto quick
902636
+# 099 rw auto quick  -- requires whitelisted blkverify
902636
 # 100 was removed, do not reuse
902636
 101 rw quick
902636
 102 rw quick
902636
diff --git a/tests/test-x86-cpuid-compat.c b/tests/test-x86-cpuid-compat.c
902636
index 772287bdb4..e7c075ed98 100644
902636
--- a/tests/test-x86-cpuid-compat.c
902636
+++ b/tests/test-x86-cpuid-compat.c
902636
@@ -300,6 +300,7 @@ int main(int argc, char **argv)
902636
                    "-cpu 486,xlevel2=0xC0000002,+xstore",
902636
                    "xlevel2", 0xC0000002);
902636
 
902636
+#if 0 /* Disabled in Red Hat Enterprise Linux */
902636
     /* Check compatibility of old machine-types that didn't
902636
      * auto-increase level/xlevel/xlevel2: */
902636
 
902636
@@ -350,6 +351,7 @@ int main(int argc, char **argv)
902636
     add_cpuid_test("x86/cpuid/xlevel-compat/pc-i440fx-2.4/npt-on",
902636
                    "-machine pc-i440fx-2.4 -cpu SandyBridge,+npt",
902636
                    "xlevel", 0x80000008);
902636
+#endif
902636
 
902636
     /* Test feature parsing */
902636
     add_feature_test("x86/cpuid/features/plus",
902636
diff --git a/tests/usb-hcd-xhci-test.c b/tests/usb-hcd-xhci-test.c
902636
index 10ef9d2a91..3855873050 100644
902636
--- a/tests/usb-hcd-xhci-test.c
902636
+++ b/tests/usb-hcd-xhci-test.c
902636
@@ -21,6 +21,7 @@ static void test_xhci_hotplug(void)
902636
     usb_test_hotplug(global_qtest, "xhci", "1", NULL);
902636
 }
902636
 
902636
+#if 0 /* Disabled for Red Hat Enterprise Linux */
902636
 static void test_usb_uas_hotplug(void)
902636
 {
902636
     QTestState *qts = global_qtest;
902636
@@ -36,6 +37,7 @@ static void test_usb_uas_hotplug(void)
902636
     qtest_qmp_device_del(qts, "scsihd");
902636
     qtest_qmp_device_del(qts, "uas");
902636
 }
902636
+#endif
902636
 
902636
 static void test_usb_ccid_hotplug(void)
902636
 {
902636
@@ -56,7 +58,9 @@ int main(int argc, char **argv)
902636
 
902636
     qtest_add_func("/xhci/pci/init", test_xhci_init);
902636
     qtest_add_func("/xhci/pci/hotplug", test_xhci_hotplug);
902636
+#if 0 /* Disabled for Red Hat Enterprise Linux */
902636
     qtest_add_func("/xhci/pci/hotplug/usb-uas", test_usb_uas_hotplug);
902636
+#endif
902636
     qtest_add_func("/xhci/pci/hotplug/usb-ccid", test_usb_ccid_hotplug);
902636
 
902636
     qtest_start("-device nec-usb-xhci,id=xhci"
902636
-- 
902636
2.21.0
902636