[libvirt] [PATCH 2/2] add --live support to "virsh dump"

Jiri Denemark jdenemar at redhat.com
Mon Nov 30 14:58:08 UTC 2009


> > In Xen it is buggy, so I chose to not support it.
> 
> Well some versions of Xen are buggy, but not neccessarily all of
> them. IMHO we should support it in libvirt because it is a trivial
> change to make. This at least lets users try the functionality and
> see if there are bugs in their version of Xen
> 
> eg, our code already does
> 
>     return xend_op(domain->conn, domain->name, "op", "dump", "file", filename,
>                    "live", "0", "crash", "0", NULL);
> 
> so it is easy to set   "live", "1"   based on flags

Well, the live dump implementation in xen is just bogus even upstream but I
agree we should support it in xen driver because xen claims to support it and
it's not our problem that their code for that is buggy. Furthermore they could
fix it in the future and then it will just work.

Jirka




More information about the libvir-list mailing list