Re: [libvirt] [PATCH 5/9] qemu: blockjob: Split out update of persistent XML disk's source on mirror jobs

Both active block commit and block copy modify the disk source of the active definition and thus also must modify the corresponding inactive definition source so that the VM starts up later. This is currently implemented in the legacy block job handler but the logic will be ueful

also for the new handlers. Split it out which also simplifies it. Signed-off-by: Peter Krempa <pkrempa(a)redhat.com&gt; --- src/qemu/qemu_blockjob.c | 64 ++++++++++++++++++++++++---------------- 1 file changed, 39 insertions(+), 25 deletions(-)

diff --git a/src/qemu/qemu_blockjob.c b/src/qemu/qemu_blockjob.c index a3109d3934..f9e79db131 100644 --- a/src/qemu/qemu_blockjob.c +++ b/src/qemu/qemu_blockjob.c @@ -434,6 +434,44 @@ qemuBlockJobEmitEvents(virQEMUDriverPtr driver, } +/** + * qemuBlockJobRewriteConfigDiskSource: + * @vm: domain object + * @disk: live definition disk + * @newsrc: new source which should be also considered for the new disk + * + * For block jobs which modify the running disk source it is required that we + * try our best to update the config XML's disk source as well in most cases. + * + * This helper finds the disk from the persistent definition corresponding to + * @disk and updates it's source to @newsrc.