Re: [PATCH 2/3] qemu: process: Move handling of non-backing files into qemuDomainDetermineDiskChain

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

 



On Wed, Nov 29, 2017 at 21:25:45 -0500, John Ferlan wrote:
> 
> 
> On 11/24/2017 07:21 AM, Peter Krempa wrote:
> > Until now we would skip loading of the backing chain for files which
> > don't support backing chains only when starting up the VM. Move the
> > check from qemuProcessPrepareHostStorage with some adaptations so that's
> > always applied.
> > ---
> >  src/qemu/qemu_domain.c  | 17 +++++++++++++++++
> >  src/qemu/qemu_process.c | 10 ----------
> >  2 files changed, 17 insertions(+), 10 deletions(-)
> > 
> 
> Does it matter or should the check go before the
> virStorageSourceBackingStoreClear... Up through this point we did it
> before anyway.

Well. It does not matter at this precise point. If the file does not
exist (since it's the top level image), we would fail and clearing or
not clearing of the terminator does not matter.

It simplifies the code for the next patch though. If we are nuking the
rest of the backing chain (as we do in all cases today), we want to nuke
the rest of the chain for the raw disk (since it would be invalid).

After that we will always need to terminate the chain.

So yes, change in the ordering is desired.

> 
> Reviewed-by: John Ferlan <jferlan@xxxxxxxxxx>
> 
> (for the code regardless of your decision on placement)
> 

Attachment: signature.asc
Description: PGP signature

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[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