[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH RFC v2 2/5] blockdev: combine DriveBackupState and BlockdevBackup
From: |
John Snow |
Subject: |
[PATCH RFC v2 2/5] blockdev: combine DriveBackupState and BlockdevBackupState |
Date: |
Wed, 13 May 2020 23:49:19 -0400 |
They have the same fields -- rename it BlockJobState.
Signed-off-by: John Snow <address@hidden>
---
blockdev.c | 30 ++++++++++++------------------
1 file changed, 12 insertions(+), 18 deletions(-)
diff --git a/blockdev.c b/blockdev.c
index b3c840ec03..d3e8a6ca22 100644
--- a/blockdev.c
+++ b/blockdev.c
@@ -1702,11 +1702,11 @@ static void external_snapshot_clean(BlkActionState
*common)
aio_context_release(aio_context);
}
-typedef struct DriveBackupState {
+typedef struct BlockJobActionState {
BlkActionState common;
BlockDriverState *bs;
BlockJob *job;
-} DriveBackupState;
+} BlockJobActionState;
static BlockJob *do_backup_common(BackupCommon *backup,
BlockDriverState *bs,
@@ -1716,7 +1716,7 @@ static BlockJob *do_backup_common(BackupCommon *backup,
static void drive_backup_prepare(BlkActionState *common, Error **errp)
{
- DriveBackupState *state = DO_UPCAST(DriveBackupState, common, common);
+ BlockJobActionState *state = DO_UPCAST(BlockJobActionState, common,
common);
DriveBackup *backup;
BlockDriverState *bs;
BlockDriverState *target_bs;
@@ -1853,7 +1853,7 @@ out:
static void drive_backup_commit(BlkActionState *common)
{
- DriveBackupState *state = DO_UPCAST(DriveBackupState, common, common);
+ BlockJobActionState *state = DO_UPCAST(BlockJobActionState, common,
common);
AioContext *aio_context;
aio_context = bdrv_get_aio_context(state->bs);
@@ -1867,7 +1867,7 @@ static void drive_backup_commit(BlkActionState *common)
static void drive_backup_abort(BlkActionState *common)
{
- DriveBackupState *state = DO_UPCAST(DriveBackupState, common, common);
+ BlockJobActionState *state = DO_UPCAST(BlockJobActionState, common,
common);
if (state->job) {
AioContext *aio_context;
@@ -1883,7 +1883,7 @@ static void drive_backup_abort(BlkActionState *common)
static void drive_backup_clean(BlkActionState *common)
{
- DriveBackupState *state = DO_UPCAST(DriveBackupState, common, common);
+ BlockJobActionState *state = DO_UPCAST(BlockJobActionState, common,
common);
AioContext *aio_context;
if (!state->bs) {
@@ -1898,15 +1898,9 @@ static void drive_backup_clean(BlkActionState *common)
aio_context_release(aio_context);
}
-typedef struct BlockdevBackupState {
- BlkActionState common;
- BlockDriverState *bs;
- BlockJob *job;
-} BlockdevBackupState;
-
static void blockdev_backup_prepare(BlkActionState *common, Error **errp)
{
- BlockdevBackupState *state = DO_UPCAST(BlockdevBackupState, common,
common);
+ BlockJobActionState *state = DO_UPCAST(BlockJobActionState, common,
common);
BlockdevBackup *backup;
BlockDriverState *bs;
BlockDriverState *target_bs;
@@ -1954,7 +1948,7 @@ static void blockdev_backup_prepare(BlkActionState
*common, Error **errp)
static void blockdev_backup_commit(BlkActionState *common)
{
- BlockdevBackupState *state = DO_UPCAST(BlockdevBackupState, common,
common);
+ BlockJobActionState *state = DO_UPCAST(BlockJobActionState, common,
common);
AioContext *aio_context;
aio_context = bdrv_get_aio_context(state->bs);
@@ -1968,7 +1962,7 @@ static void blockdev_backup_commit(BlkActionState *common)
static void blockdev_backup_abort(BlkActionState *common)
{
- BlockdevBackupState *state = DO_UPCAST(BlockdevBackupState, common,
common);
+ BlockJobActionState *state = DO_UPCAST(BlockJobActionState, common,
common);
if (state->job) {
AioContext *aio_context;
@@ -1984,7 +1978,7 @@ static void blockdev_backup_abort(BlkActionState *common)
static void blockdev_backup_clean(BlkActionState *common)
{
- BlockdevBackupState *state = DO_UPCAST(BlockdevBackupState, common,
common);
+ BlockJobActionState *state = DO_UPCAST(BlockJobActionState, common,
common);
AioContext *aio_context;
if (!state->bs) {
@@ -2265,14 +2259,14 @@ static const BlkActionOps actions[] = {
.clean = external_snapshot_clean,
},
[TRANSACTION_ACTION_KIND_DRIVE_BACKUP] = {
- .instance_size = sizeof(DriveBackupState),
+ .instance_size = sizeof(BlockJobActionState),
.prepare = drive_backup_prepare,
.commit = drive_backup_commit,
.abort = drive_backup_abort,
.clean = drive_backup_clean,
},
[TRANSACTION_ACTION_KIND_BLOCKDEV_BACKUP] = {
- .instance_size = sizeof(BlockdevBackupState),
+ .instance_size = sizeof(BlockJobActionState),
.prepare = blockdev_backup_prepare,
.commit = blockdev_backup_commit,
.abort = blockdev_backup_abort,
--
2.21.1
- [PATCH RFC v2 0/5] block: add block-dirty-bitmap-populate job, John Snow, 2020/05/13
- [PATCH RFC v2 1/5] block: add bitmap-populate job, John Snow, 2020/05/13
- [PATCH RFC v2 2/5] blockdev: combine DriveBackupState and BlockdevBackupState,
John Snow <=
- [PATCH RFC v2 3/5] qmp: expose block-dirty-bitmap-populate, John Snow, 2020/05/13
- [PATCH RFC v2 4/5] iotests: move bitmap helpers into their own file, John Snow, 2020/05/13
- [PATCH RFC v2 5/5] iotests: add 287 for block-dirty-bitmap-populate, John Snow, 2020/05/13
- Re: [PATCH RFC v2 0/5] block: add block-dirty-bitmap-populate job, Peter Krempa, 2020/05/18