|
 |
357786 |
From afdf8a5ba72ac761081edba383699bd45968349e Mon Sep 17 00:00:00 2001
|
|
|
357786 |
From: Kevin Wolf <kwolf@redhat.com>
|
|
|
357786 |
Date: Fri, 21 Sep 2018 12:46:28 +0200
|
|
|
357786 |
Subject: [PATCH 1/3] test-bdrv-drain: Fix outdated comments
|
|
|
357786 |
|
|
|
357786 |
RH-Author: Kevin Wolf <kwolf@redhat.com>
|
|
|
357786 |
Message-id: <20180921124630.29036-2-kwolf@redhat.com>
|
|
|
357786 |
Patchwork-id: 82232
|
|
|
357786 |
O-Subject: [RHV-7.6 qemu-kvm-rhev PATCH 1/3] test-bdrv-drain: Fix outdated comments
|
|
|
357786 |
Bugzilla: 1618584
|
|
|
357786 |
RH-Acked-by: Jeffrey Cody <jcody@redhat.com>
|
|
|
357786 |
RH-Acked-by: Paolo Bonzini <pbonzini@redhat.com>
|
|
|
357786 |
RH-Acked-by: Eric Blake <eblake@redhat.com>
|
|
|
357786 |
RH-Acked-by: John Snow <jsnow@redhat.com>
|
|
|
357786 |
|
|
|
357786 |
Commit 89bd030533e changed the test case from using job_sleep_ns() to
|
|
|
357786 |
using qemu_co_sleep_ns() instead. Also, block_job_sleep_ns() became
|
|
|
357786 |
job_sleep_ns() in commit 5d43e86e11f.
|
|
|
357786 |
|
|
|
357786 |
In both cases, some comments in the test case were not updated. Do that
|
|
|
357786 |
now.
|
|
|
357786 |
|
|
|
357786 |
Reported-by: Max Reitz <mreitz@redhat.com>
|
|
|
357786 |
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
|
|
|
357786 |
Signed-off-by: Miroslav Rezanina <mrezanin@redhat.com>
|
|
|
357786 |
---
|
|
|
357786 |
tests/test-bdrv-drain.c | 10 +++++-----
|
|
|
357786 |
1 file changed, 5 insertions(+), 5 deletions(-)
|
|
|
357786 |
|
|
|
357786 |
diff --git a/tests/test-bdrv-drain.c b/tests/test-bdrv-drain.c
|
|
|
357786 |
index d6202b2..7e7ba9b 100644
|
|
|
357786 |
--- a/tests/test-bdrv-drain.c
|
|
|
357786 |
+++ b/tests/test-bdrv-drain.c
|
|
|
357786 |
@@ -820,9 +820,9 @@ static int coroutine_fn test_job_run(Job *job, Error **errp)
|
|
|
357786 |
|
|
|
357786 |
job_transition_to_ready(&s->common.job);
|
|
|
357786 |
while (!s->should_complete) {
|
|
|
357786 |
- /* Avoid block_job_sleep_ns() because it marks the job as !busy. We
|
|
|
357786 |
- * want to emulate some actual activity (probably some I/O) here so
|
|
|
357786 |
- * that drain has to wait for this acitivity to stop. */
|
|
|
357786 |
+ /* Avoid job_sleep_ns() because it marks the job as !busy. We want to
|
|
|
357786 |
+ * emulate some actual activity (probably some I/O) here so that drain
|
|
|
357786 |
+ * has to wait for this activity to stop. */
|
|
|
357786 |
qemu_co_sleep_ns(QEMU_CLOCK_REALTIME, 100000);
|
|
|
357786 |
job_pause_point(&s->common.job);
|
|
|
357786 |
}
|
|
|
357786 |
@@ -908,7 +908,7 @@ static void test_blockjob_common(enum drain_type drain_type, bool use_iothread,
|
|
|
357786 |
|
|
|
357786 |
g_assert_cmpint(job->job.pause_count, ==, 0);
|
|
|
357786 |
g_assert_false(job->job.paused);
|
|
|
357786 |
- g_assert_true(job->job.busy); /* We're in job_sleep_ns() */
|
|
|
357786 |
+ g_assert_true(job->job.busy); /* We're in qemu_co_sleep_ns() */
|
|
|
357786 |
|
|
|
357786 |
do_drain_begin_unlocked(drain_type, src);
|
|
|
357786 |
|
|
|
357786 |
@@ -956,7 +956,7 @@ static void test_blockjob_common(enum drain_type drain_type, bool use_iothread,
|
|
|
357786 |
|
|
|
357786 |
g_assert_cmpint(job->job.pause_count, ==, 0);
|
|
|
357786 |
g_assert_false(job->job.paused);
|
|
|
357786 |
- g_assert_true(job->job.busy); /* We're in job_sleep_ns() */
|
|
|
357786 |
+ g_assert_true(job->job.busy); /* We're in qemu_co_sleep_ns() */
|
|
|
357786 |
|
|
|
357786 |
aio_context_acquire(ctx);
|
|
|
357786 |
ret = job_complete_sync(&job->job, &error_abort);
|
|
|
357786 |
--
|
|
|
357786 |
1.8.3.1
|
|
|
357786 |
|