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

Re: [libvirt] [Qemu-devel] live snapshot wiki updated

On 07/21/2011 03:34 AM, Jes Sorensen wrote:
On 07/21/11 11:34, Daniel P. Berrange wrote:
QEMU is *not* yet running at the time libvirt reads the file metadata.

Of course it is: hotplug
In the case of hotplug, the hotplug monitor commands have not yet been
invoked when libvirt access the file metadata, or the drive has already
been unplugged, so there is still no concurrent access to the file by
QEMU and libvirt.

Unless someone tries to hotplug an image that relies on a backing file
already used by another image. While unlikely, it is possible.

Backing images should be treated as read-only by qemu, right? That is, if I have file c.img which uses ca.img as its backing file, then qemu only needs O_RDONLY access to ca.img, right? My understanding is that you never want to have a qcow2 image whose backing file is being simultaneously edited by another external process, otherwise you risk data corruption from the point of view of the qemu process that is trying to refer to that backing file.

Given that restriction, if I want to hotplug file d.img that _also_ uses ca.img as its backing file, then that's not an issue. Libvirt _still_ reads the metadata of d.img, and learns of the backing file of ca.img, all before calling the hotplug command, even if ca.img is already in use by qemu, with no complications.

You still haven't managed to convince me that there is ever any situation where qemu needs to open a backing file that is not already determined /a priori/ by reading just the metadata of the files being handed to qemu in the first place, or by being aware of the metadata relationship being requested of qemu in the first place (such as the relationship implied by calling snapshot_blkdev to create a qcow2 file with an existing file as backing).

Eric Blake   eblake redhat com    +1-801-349-2682
Libvirt virtualization library http://libvirt.org

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