]> git.ipfire.org Git - thirdparty/qemu.git/commit - docs/about/deprecated.rst
qapi: deprecate drive-backup
authorVladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
Thu, 4 Nov 2021 08:58:11 +0000 (09:58 +0100)
committerVladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
Tue, 9 Nov 2021 17:21:19 +0000 (18:21 +0100)
commit1084159b31015e003946d199cbfecaec282e0eb2
tree68bd5ae9d818e303ef85d7c06af2296d824633f3
parent24d6cc1fa17745228694a3b2c94e8a5cbd4fc4d9
qapi: deprecate drive-backup

Modern way is using blockdev-add + blockdev-backup, which provides a
lot more control on how target is opened.

As example of drive-backup problems consider the following:

User of drive-backup expects that target will be opened in the same
cache and aio mode as source. Corresponding logic is in
drive_backup_prepare(), where we take bs->open_flags of source.

It works rather bad if source was added by blockdev-add. Assume source
is qcow2 image. On blockdev-add we should specify aio and cache options
for file child of qcow2 node. What happens next:

drive_backup_prepare() looks at bs->open_flags of qcow2 source node.
But there no BDRV_O_NOCAHE neither BDRV_O_NATIVE_AIO: BDRV_O_NOCAHE is
places in bs->file->bs->open_flags, and BDRV_O_NATIVE_AIO is nowhere,
as file-posix parse options and simply set s->use_linux_aio.

The documentation is updated in a minimal way, so that drive-backup is
noted only as a deprecated command, and blockdev-backup used in most of
places.

Signed-off-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
Signed-off-by: Markus Armbruster <armbru@redhat.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
docs/about/deprecated.rst
docs/interop/live-block-operations.rst
qapi/block-core.json
qapi/transaction.json