Re: [libvirt] [PATCH 2/3] qemu: process: Move handling of non-backing files into qemuDomainDetermineDiskChain
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(a)redhat.com>
(for the code regardless of your decision on placement)
Attachments:
- signature.asc (application/pgp-signature — 833 bytes)