diff options
author | John Snow <jsnow@redhat.com> | 2019-07-29 16:35:53 -0400 |
---|---|---|
committer | John Snow <jsnow@redhat.com> | 2019-08-16 16:28:02 -0400 |
commit | cf0cd293c6311e0714739a95d022b262e1d9d798 (patch) | |
tree | ac87e5810f46d3a8a71b5cc4ce9450c59678d6ef /block | |
parent | c8b56501780615f709d96419fd718ca562d41c46 (diff) | |
download | qemu-cf0cd293c6311e0714739a95d022b262e1d9d798.zip qemu-cf0cd293c6311e0714739a95d022b262e1d9d798.tar.gz qemu-cf0cd293c6311e0714739a95d022b262e1d9d798.tar.bz2 |
block/backup: add 'never' policy to bitmap sync mode
This adds a "never" policy for bitmap synchronization. Regardless of if
the job succeeds or fails, we never update the bitmap. This can be used
to perform differential backups, or simply to avoid the job modifying a
bitmap.
Signed-off-by: John Snow <jsnow@redhat.com>
Reviewed-by: Max Reitz <mreitz@redhat.com>
Message-id: 20190709232550.10724-7-jsnow@redhat.com
Signed-off-by: John Snow <jsnow@redhat.com>
Diffstat (limited to 'block')
-rw-r--r-- | block/backup.c | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/block/backup.c b/block/backup.c index 2b4c5c2..d07b838 100644 --- a/block/backup.c +++ b/block/backup.c @@ -274,8 +274,11 @@ static void backup_cleanup_sync_bitmap(BackupBlockJob *job, int ret) BdrvDirtyBitmap *bm; BlockDriverState *bs = blk_bs(job->common.blk); - if (ret < 0) { - /* Merge the successor back into the parent, delete nothing. */ + if (ret < 0 || job->bitmap_mode == BITMAP_SYNC_MODE_NEVER) { + /* + * Failure, or we don't want to synchronize the bitmap. + * Merge the successor back into the parent, delete nothing. + */ bm = bdrv_reclaim_dirty_bitmap(bs, job->sync_bitmap, NULL); assert(bm); } else { |