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

Re: [libvirt] [PATCH] qemuDriver: check for migrate command



On Wed, Jul 23, 2008 at 03:20:31PM -0400, Guido Günther wrote:
> Hi,
> qemu doesn't know about "migrate" used to save a qemu/kvm domain.
> Current CVS fails silently - this patch checks if the command if
> available.

  I would have expected the previous qemudMonitorCommand() to fail
instead. Patch looks fine and safe to me, applied and commited. Maybe
this should be instead caught up more generally one level up in 
qemudMonitorCommand() looking for 'unknow command' in info, but
that's fine as is I think.

  thanks !

Daniel

-- 
Red Hat Virtualization group http://redhat.com/virtualization/
Daniel Veillard      | virtualization library  http://libvirt.org/
veillard redhat com  | libxml GNOME XML XSLT toolkit  http://xmlsoft.org/
http://veillard.com/ | Rpmfind RPM search engine  http://rpmfind.net/


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