[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-block] [PATCH 2/4] block/block-backend: blk_iostatus_reset: drop u
From: |
Vladimir Sementsov-Ogievskiy |
Subject: |
[Qemu-block] [PATCH 2/4] block/block-backend: blk_iostatus_reset: drop usage of bs->job |
Date: |
Thu, 6 Jun 2019 18:41:30 +0300 |
We are going to remove bs->job pointer. Drop it's usage in
blk_iostatus_reset.
blk_iostatus_reset() has only two callers:
1. blk_attach_dev(). This doesn't have anything to do with jobs and
attaching a new guest device won't solve any problem the job
encountered, so no reason to reset the iostatus for the job.
2. qmp_cont(). This resets the iostatus for everything. We can just
call block_job_iostatus_reset() for all block jobs instead of going
through BlockBackend.
Suggested-by: Kevin Wolf <address@hidden>
Signed-off-by: Vladimir Sementsov-Ogievskiy <address@hidden>
---
block/block-backend.c | 4 ----
qmp.c | 5 +++++
2 files changed, 5 insertions(+), 4 deletions(-)
diff --git a/block/block-backend.c b/block/block-backend.c
index f5d9407d20..a8d160fd5d 100644
--- a/block/block-backend.c
+++ b/block/block-backend.c
@@ -1073,11 +1073,7 @@ void blk_iostatus_disable(BlockBackend *blk)
void blk_iostatus_reset(BlockBackend *blk)
{
if (blk_iostatus_is_enabled(blk)) {
- BlockDriverState *bs = blk_bs(blk);
blk->iostatus = BLOCK_DEVICE_IO_STATUS_OK;
- if (bs && bs->job) {
- block_job_iostatus_reset(bs->job);
- }
}
}
diff --git a/qmp.c b/qmp.c
index fa1b3c1577..89cf10e28e 100644
--- a/qmp.c
+++ b/qmp.c
@@ -142,6 +142,7 @@ void qmp_x_exit_preconfig(Error **errp)
void qmp_cont(Error **errp)
{
BlockBackend *blk;
+ BlockJob *job;
Error *local_err = NULL;
/* if there is a dump in background, we should wait until the dump
@@ -165,6 +166,10 @@ void qmp_cont(Error **errp)
blk_iostatus_reset(blk);
}
+ for (job = block_job_next(NULL); job; job = block_job_next(job)) {
+ block_job_iostatus_reset(job);
+ }
+
/* Continuing after completed migration. Images have been inactivated to
* allow the destination to take control. Need to get control back now.
*
--
2.18.0
- [Qemu-block] [PATCH 0/4] block: drop bs->job, Vladimir Sementsov-Ogievskiy, 2019/06/06
- [Qemu-block] [PATCH 2/4] block/block-backend: blk_iostatus_reset: drop usage of bs->job,
Vladimir Sementsov-Ogievskiy <=
- [Qemu-block] [PATCH 4/4] block: drop bs->job, Vladimir Sementsov-Ogievskiy, 2019/06/06
- [Qemu-block] [PATCH 1/4] block/replication: drop usage of bs->job, Vladimir Sementsov-Ogievskiy, 2019/06/06
- [Qemu-block] [PATCH 3/4] blockdev: blockdev_mark_auto_del: drop usage of bs->job, Vladimir Sementsov-Ogievskiy, 2019/06/06
- Re: [Qemu-block] [Qemu-devel] [PATCH 0/4] block: drop bs->job, no-reply, 2019/06/06
- Re: [Qemu-block] [PATCH 0/4] block: drop bs->job, Kevin Wolf, 2019/06/14