|
|
9bac43 |
From 5185df6533f3e1f3c428e6d7b11a1fe372d0357b Mon Sep 17 00:00:00 2001
|
|
|
9bac43 |
From: Jeffrey Cody <jcody@redhat.com>
|
|
|
9bac43 |
Date: Thu, 30 Nov 2017 22:49:06 +0100
|
|
|
9bac43 |
Subject: [PATCH 02/21] block: rename bdrv_co_drain to bdrv_co_drain_begin
|
|
|
9bac43 |
|
|
|
9bac43 |
RH-Author: Jeffrey Cody <jcody@redhat.com>
|
|
|
9bac43 |
Message-id: <ec3f5f7be56c7d67bb8e8230d5ef27824ea2bf1f.1511985875.git.jcody@redhat.com>
|
|
|
9bac43 |
Patchwork-id: 78039
|
|
|
9bac43 |
O-Subject: [RHV7.5 qemu-kvm-rhev PATCH 02/11] block: rename bdrv_co_drain to bdrv_co_drain_begin
|
|
|
9bac43 |
Bugzilla: 1506531
|
|
|
9bac43 |
RH-Acked-by: Paolo Bonzini <pbonzini@redhat.com>
|
|
|
9bac43 |
RH-Acked-by: Stefan Hajnoczi <stefanha@redhat.com>
|
|
|
9bac43 |
RH-Acked-by: John Snow <jsnow@redhat.com>
|
|
|
9bac43 |
|
|
|
9bac43 |
From: Manos Pitsidianakis <el13635@mail.ntua.gr>
|
|
|
9bac43 |
|
|
|
9bac43 |
Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>
|
|
|
9bac43 |
Reviewed-by: Fam Zheng <famz@redhat.com>
|
|
|
9bac43 |
Signed-off-by: Manos Pitsidianakis <el13635@mail.ntua.gr>
|
|
|
9bac43 |
Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>
|
|
|
9bac43 |
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
|
|
|
9bac43 |
(cherry picked from commit f8ea8dacf0de636e2c0f13b90c0d75db97dc9b44)
|
|
|
9bac43 |
Signed-off-by: Jeff Cody <jcody@redhat.com>
|
|
|
9bac43 |
Signed-off-by: Miroslav Rezanina <mrezanin@redhat.com>
|
|
|
9bac43 |
---
|
|
|
9bac43 |
block/io.c | 4 ++--
|
|
|
9bac43 |
block/qed.c | 6 +++---
|
|
|
9bac43 |
include/block/block_int.h | 4 ++--
|
|
|
9bac43 |
3 files changed, 7 insertions(+), 7 deletions(-)
|
|
|
9bac43 |
|
|
|
9bac43 |
diff --git a/block/io.c b/block/io.c
|
|
|
9bac43 |
index 93fb802..3a717bc 100644
|
|
|
9bac43 |
--- a/block/io.c
|
|
|
9bac43 |
+++ b/block/io.c
|
|
|
9bac43 |
@@ -162,7 +162,7 @@ static void coroutine_fn bdrv_drain_invoke_entry(void *opaque)
|
|
|
9bac43 |
BlockDriverState *bs = data->bs;
|
|
|
9bac43 |
|
|
|
9bac43 |
if (data->begin) {
|
|
|
9bac43 |
- bs->drv->bdrv_co_drain(bs);
|
|
|
9bac43 |
+ bs->drv->bdrv_co_drain_begin(bs);
|
|
|
9bac43 |
} else {
|
|
|
9bac43 |
bs->drv->bdrv_co_drain_end(bs);
|
|
|
9bac43 |
}
|
|
|
9bac43 |
@@ -176,7 +176,7 @@ static void bdrv_drain_invoke(BlockDriverState *bs, bool begin)
|
|
|
9bac43 |
{
|
|
|
9bac43 |
BdrvCoDrainData data = { .bs = bs, .done = false, .begin = begin};
|
|
|
9bac43 |
|
|
|
9bac43 |
- if (!bs->drv || (begin && !bs->drv->bdrv_co_drain) ||
|
|
|
9bac43 |
+ if (!bs->drv || (begin && !bs->drv->bdrv_co_drain_begin) ||
|
|
|
9bac43 |
(!begin && !bs->drv->bdrv_co_drain_end)) {
|
|
|
9bac43 |
return;
|
|
|
9bac43 |
}
|
|
|
9bac43 |
diff --git a/block/qed.c b/block/qed.c
|
|
|
9bac43 |
index dc54bf4..62d12db 100644
|
|
|
9bac43 |
--- a/block/qed.c
|
|
|
9bac43 |
+++ b/block/qed.c
|
|
|
9bac43 |
@@ -265,7 +265,7 @@ static bool qed_plug_allocating_write_reqs(BDRVQEDState *s)
|
|
|
9bac43 |
assert(!s->allocating_write_reqs_plugged);
|
|
|
9bac43 |
if (s->allocating_acb != NULL) {
|
|
|
9bac43 |
/* Another allocating write came concurrently. This cannot happen
|
|
|
9bac43 |
- * from bdrv_qed_co_drain, but it can happen when the timer runs.
|
|
|
9bac43 |
+ * from bdrv_qed_co_drain_begin, but it can happen when the timer runs.
|
|
|
9bac43 |
*/
|
|
|
9bac43 |
qemu_co_mutex_unlock(&s->table_lock);
|
|
|
9bac43 |
return false;
|
|
|
9bac43 |
@@ -358,7 +358,7 @@ static void bdrv_qed_attach_aio_context(BlockDriverState *bs,
|
|
|
9bac43 |
}
|
|
|
9bac43 |
}
|
|
|
9bac43 |
|
|
|
9bac43 |
-static void coroutine_fn bdrv_qed_co_drain(BlockDriverState *bs)
|
|
|
9bac43 |
+static void coroutine_fn bdrv_qed_co_drain_begin(BlockDriverState *bs)
|
|
|
9bac43 |
{
|
|
|
9bac43 |
BDRVQEDState *s = bs->opaque;
|
|
|
9bac43 |
|
|
|
9bac43 |
@@ -1608,7 +1608,7 @@ static BlockDriver bdrv_qed = {
|
|
|
9bac43 |
.bdrv_check = bdrv_qed_check,
|
|
|
9bac43 |
.bdrv_detach_aio_context = bdrv_qed_detach_aio_context,
|
|
|
9bac43 |
.bdrv_attach_aio_context = bdrv_qed_attach_aio_context,
|
|
|
9bac43 |
- .bdrv_co_drain = bdrv_qed_co_drain,
|
|
|
9bac43 |
+ .bdrv_co_drain_begin = bdrv_qed_co_drain_begin,
|
|
|
9bac43 |
};
|
|
|
9bac43 |
|
|
|
9bac43 |
static void bdrv_qed_init(void)
|
|
|
9bac43 |
diff --git a/include/block/block_int.h b/include/block/block_int.h
|
|
|
9bac43 |
index 98d02ba..e02d540 100644
|
|
|
9bac43 |
--- a/include/block/block_int.h
|
|
|
9bac43 |
+++ b/include/block/block_int.h
|
|
|
9bac43 |
@@ -320,7 +320,7 @@ struct BlockDriver {
|
|
|
9bac43 |
int (*bdrv_probe_geometry)(BlockDriverState *bs, HDGeometry *geo);
|
|
|
9bac43 |
|
|
|
9bac43 |
/**
|
|
|
9bac43 |
- * bdrv_co_drain is called if implemented in the beginning of a
|
|
|
9bac43 |
+ * bdrv_co_drain_begin is called if implemented in the beginning of a
|
|
|
9bac43 |
* drain operation to drain and stop any internal sources of requests in
|
|
|
9bac43 |
* the driver.
|
|
|
9bac43 |
* bdrv_co_drain_end is called if implemented at the end of the drain.
|
|
|
9bac43 |
@@ -329,7 +329,7 @@ struct BlockDriver {
|
|
|
9bac43 |
* requests, or toggle an internal state. After the end of the drain new
|
|
|
9bac43 |
* requests will continue normally.
|
|
|
9bac43 |
*/
|
|
|
9bac43 |
- void coroutine_fn (*bdrv_co_drain)(BlockDriverState *bs);
|
|
|
9bac43 |
+ void coroutine_fn (*bdrv_co_drain_begin)(BlockDriverState *bs);
|
|
|
9bac43 |
void coroutine_fn (*bdrv_co_drain_end)(BlockDriverState *bs);
|
|
|
9bac43 |
|
|
|
9bac43 |
void (*bdrv_add_child)(BlockDriverState *parent, BlockDriverState *child,
|
|
|
9bac43 |
--
|
|
|
9bac43 |
1.8.3.1
|
|
|
9bac43 |
|