|
 |
05bba0 |
From 228e49fabffa644ab7a6a03e98205f293115dc89 Mon Sep 17 00:00:00 2001
|
|
|
05bba0 |
Message-Id: <228e49fabffa644ab7a6a03e98205f293115dc89.1445029944.git.jen@redhat.com>
|
|
|
605027 |
From: John Snow <jsnow@redhat.com>
|
|
|
05bba0 |
Date: Fri, 16 Oct 2015 16:26:24 -0500
|
|
|
605027 |
Subject: [CHANGE] qtest/ide-test: disable flush-test
|
|
|
605027 |
To: rhvirt-patches@redhat.com,
|
|
|
605027 |
jen@redhat.com
|
|
|
605027 |
|
|
|
605027 |
RH-Author: John Snow <jsnow@redhat.com>
|
|
|
05bba0 |
Message-id: <1445012784-22604-2-git-send-email-jsnow@redhat.com>
|
|
|
05bba0 |
Patchwork-id: 68169
|
|
|
05bba0 |
O-Subject: [RHEL-7.2 qemu-kvm PATCH 1/1] qtest/ide-test: disable flush-test
|
|
|
05bba0 |
Bugzilla: 1270341
|
|
|
605027 |
RH-Acked-by: Laszlo Ersek <lersek@redhat.com>
|
|
|
05bba0 |
RH-Acked-by: Wei Huang <wei@redhat.com>
|
|
|
05bba0 |
RH-Acked-by: Jeff Nelson <jenelson@redhat.com>
|
|
|
605027 |
|
|
|
605027 |
One of the tests downstream causes a race that can result in build
|
|
|
05bba0 |
failures. For 7.3, we intend to fix the test properly, but for 7.2,
|
|
|
05bba0 |
we disable the test as a workaround.
|
|
|
605027 |
|
|
|
605027 |
Signed-off-by: John Snow <jsnow@redhat.com>
|
|
|
605027 |
Signed-off-by: Jeff E. Nelson <jen@redhat.com>
|
|
|
605027 |
---
|
|
|
605027 |
tests/ide-test.c | 38 --------------------------------------
|
|
|
605027 |
1 file changed, 38 deletions(-)
|
|
|
605027 |
|
|
|
605027 |
diff --git a/tests/ide-test.c b/tests/ide-test.c
|
|
|
605027 |
index 51f9239..43b7fd6 100644
|
|
|
605027 |
--- a/tests/ide-test.c
|
|
|
605027 |
+++ b/tests/ide-test.c
|
|
|
605027 |
@@ -425,43 +425,6 @@ static void test_identify(void)
|
|
|
605027 |
ide_test_quit();
|
|
|
605027 |
}
|
|
|
605027 |
|
|
|
605027 |
-static void test_flush(void)
|
|
|
605027 |
-{
|
|
|
605027 |
- uint8_t data;
|
|
|
605027 |
-
|
|
|
605027 |
- ide_test_start(
|
|
|
605027 |
- "-vnc none "
|
|
|
605027 |
- "-drive file=blkdebug::%s,if=ide,cache=writeback",
|
|
|
605027 |
- tmp_path);
|
|
|
605027 |
-
|
|
|
605027 |
- /* Delay the completion of the flush request until we explicitly do it */
|
|
|
605027 |
- qmp("{'execute':'human-monitor-command', 'arguments': { "
|
|
|
605027 |
- "'command-line': 'qemu-io ide0-hd0 \"break flush_to_os A\"'} }");
|
|
|
605027 |
-
|
|
|
605027 |
- /* FLUSH CACHE command on device 0*/
|
|
|
605027 |
- outb(IDE_BASE + reg_device, 0);
|
|
|
605027 |
- outb(IDE_BASE + reg_command, CMD_FLUSH_CACHE);
|
|
|
605027 |
-
|
|
|
605027 |
- /* Check status while request is in flight*/
|
|
|
605027 |
- data = inb(IDE_BASE + reg_status);
|
|
|
605027 |
- assert_bit_set(data, BSY | DRDY);
|
|
|
605027 |
- assert_bit_clear(data, DF | ERR | DRQ);
|
|
|
605027 |
-
|
|
|
605027 |
- /* Complete the command */
|
|
|
605027 |
- qmp("{'execute':'human-monitor-command', 'arguments': { "
|
|
|
605027 |
- "'command-line': 'qemu-io ide0-hd0 \"resume A\"'} }");
|
|
|
605027 |
-
|
|
|
605027 |
- /* Check registers */
|
|
|
605027 |
- data = inb(IDE_BASE + reg_device);
|
|
|
605027 |
- g_assert_cmpint(data & DEV, ==, 0);
|
|
|
605027 |
-
|
|
|
605027 |
- data = inb(IDE_BASE + reg_status);
|
|
|
605027 |
- assert_bit_set(data, DRDY);
|
|
|
605027 |
- assert_bit_clear(data, BSY | DF | ERR | DRQ);
|
|
|
605027 |
-
|
|
|
605027 |
- ide_test_quit();
|
|
|
605027 |
-}
|
|
|
605027 |
-
|
|
|
605027 |
static void test_flush_nodev(void)
|
|
|
605027 |
{
|
|
|
605027 |
ide_test_start("");
|
|
|
605027 |
@@ -505,7 +468,6 @@ int main(int argc, char **argv)
|
|
|
605027 |
qtest_add_func("/ide/bmdma/long_prdt", test_bmdma_long_prdt);
|
|
|
605027 |
qtest_add_func("/ide/bmdma/teardown", test_bmdma_teardown);
|
|
|
605027 |
|
|
|
605027 |
- qtest_add_func("/ide/flush", test_flush);
|
|
|
605027 |
qtest_add_func("/ide/flush_nodev", test_flush_nodev);
|
|
|
605027 |
|
|
|
605027 |
ret = g_test_run();
|
|
|
605027 |
--
|
|
|
605027 |
2.4.3
|
|
|
605027 |
|