Re: [PATCH v4 2/5] blockdev: combine DriveBackupState and BlockdevBackup

qemu-devel

From:	Vladimir Sementsov-Ogievskiy
Subject:	Re: [PATCH v4 2/5] blockdev: combine DriveBackupState and BlockdevBackupState
Date:	Thu, 3 Sep 2020 09:55:33 +0300
User-agent:	Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.12.0

02.09.2020 21:18, Eric Blake wrote:

From: John Snow<jsnow@redhat.com>

They have the same fields -- rename it BlockJobActionState.


commit/abort/clean functions are identical after it. I think better to combine 
them as well here

(no changes from v3)

--
Best regards,
Vladimir

Current Thread

[Next in Thread]

[PATCH v4 0/5] block: add block-dirty-bitmap-populate job, Eric Blake, 2020/09/02
- [PATCH v4 3/5] qmp: expose block-dirty-bitmap-populate, Eric Blake, 2020/09/02
- [PATCH v4 2/5] blockdev: combine DriveBackupState and BlockdevBackupState, Eric Blake, 2020/09/02
  - Re: [PATCH v4 2/5] blockdev: combine DriveBackupState and BlockdevBackupState, Vladimir Sementsov-Ogievskiy <=
- [PATCH v4 1/5] block: add bitmap-populate job, Eric Blake, 2020/09/02
  - Re: [PATCH v4 1/5] block: add bitmap-populate job, Markus Armbruster, 2020/09/08
- [PATCH v4 4/5] iotests: move bitmap helpers into their own file, Eric Blake, 2020/09/02
  - Re: [PATCH v4 4/5] iotests: move bitmap helpers into their own file, Vladimir Sementsov-Ogievskiy, 2020/09/03
- [PATCH v4 5/5] iotests: add 298 for block-dirty-bitmap-populate, Eric Blake, 2020/09/02
- Re: [PATCH v4 0/5] block: add block-dirty-bitmap-populate job, Vladimir Sementsov-Ogievskiy, 2020/09/03