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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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@xxxxxxxxxx>
> > ---
> >   src/qemu/qemu_domain.c | 13 +++++++++----
> >   1 file changed, 9 insertions(+), 4 deletions(-)
> > 
> 
> Reviewed-by: Eric Blake <eblake@xxxxxxxxxx>
> 
> 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.




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux