[libvirt] [PATCHv4 00/51] shared snapshot meta-data
Philipp Hahn
hahn at univention.de
Fri Sep 2 19:24:37 UTC 2011
Hello,
On Friday 02 September 2011 11:57:15 Daniel Veillard wrote:
> Review:
> 29 Isn't there a way to save the domain snapshot on shared storage when
> available to try to avoid the problem ? It wouldn't work all the
> time but might be simpler than rolling out a v4. or consider the
> snapshot data as extra domain resource that could be migrated on
> the fly like we can do for disk images in some cases.
We are currently doing this and it works more or less.
1. You have to put /var/lib/libvirt/qemu/snapshot/ on a shared storage, but
only that and not /var/lib/libvirt/qemu/ itself, because there the UNIX
domain monitor sockets are located; if you do that, you'll get problems
because on migration two KVM instances will use the same path name for their
monitor sockets.
2. You have to hook qemuDomainSnapshotLoad() to qemudDomainMigrateFinish2()
(and qemudDomainDefine()) to get already existing snapshots associated to
newly migrated domains, either via migrate() for live domains, or via
define() for non-live domains.
3. You need do remove the locking from qemudDomainDefine(), because the two
functions mentioned above alread hold the domain lock.
That is what get me started to work on "Fix memory leak while scanning
snapshots", "killall -SIGHUP libvirtd / virStateReload" and "Reload snapshots
on SIGHUP". At leas our patched 0.8.7 is working fine with those changes.
Sincerely
Philipp
--
Philipp Hahn Open Source Software Engineer hahn at univention.de
Univention GmbH Linux for Your Business fon: +49 421 22 232- 0
Mary-Somerville-Str.1 D-28359 Bremen fax: +49 421 22 232-99
http://www.univention.de/
----------------------------------------------------------------------------
Treffen Sie Univention auf der IT&Business vom 20. bis 22. September 2011
auf dem Gemeinschaftsstand der Open Source Business Alliance in Stuttgart in
Halle 3 Stand 3D27-7.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 76_snapshot_migrate.patch
Type: text/x-diff
Size: 2737 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20110902/af6b529d/attachment-0001.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 197 bytes
Desc: This is a digitally signed message part.
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20110902/af6b529d/attachment-0001.sig>
More information about the libvir-list
mailing list