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

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



> > 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


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