[PATCH 09/10] qemuDomainGetStorageSourceByDevstr: Look also in 'mirror' chain

Peter Krempa pkrempa at redhat.com
Tue Jul 21 07:51:49 UTC 2020


On Mon, Jul 20, 2020 at 16:07:25 -0500, Eric Blake wrote:
> On 7/15/20 8:10 AM, Peter Krempa wrote:
> > A disk can have a mirror, look also in it's backing chain.
> 
> its
> 
> (it's is only valid when you can replace with 'it is')
> 
> > 
> > Signed-off-by: Peter Krempa <pkrempa at redhat.com>
> > ---
> >   src/qemu/qemu_domain.c | 13 +++++++++----
> >   1 file changed, 9 insertions(+), 4 deletions(-)
> > 
> 
> Reviewed-by: Eric Blake <eblake at redhat.com>
> 
> Hmm, when doing a pull-mode backup, do we ever want a write-threshold on the
> temporary image?

That's a good question. I didn't yet have a request for it.

> Or is this only for actual block-copy mirroring, and not
> backups?

This one is just for block copy for now. The slight problem is that the
backup scratch is not associated with the disk in the VM xml, but
technically nothing prevents us from adding the event for that as well.
The question is only how to do it API-wise.




More information about the libvir-list mailing list