Re: [PATCH 0/2] Revert to blocking behavior of qemuAgentCommand

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

 



On Thu, Aug 30, 2012 at 03:12:44PM +0200, Jiri Denemark wrote:
> Commit 05447e3af44ec153314ff97cd611330d9b4b5730 made changes to
> qemuAgentCommand so that an arbitrary timeout can be passed to it.
> However, it did so partially in a very confusing way and partially in a
> totally wrong way.
> 
> Jiri Denemark (2):
>   qemu: Remove redundant parameter from qemuAgentSend
>   qemu: Revert to blocking behavior of qemuAgentCommand
> 
>  src/qemu/qemu_agent.c | 46 ++++++++++++++++++++++++----------------------
>  1 file changed, 24 insertions(+), 22 deletions(-)

  ACK to the set. The core reason is that the new functionality of
adding arbitrary guest agent command support should not be done at
the detriment of existing functionality, in that case we get an
error when using guest agent based shutdown

   https://bugzilla.redhat.com/show_bug.cgi?id=853002

  Let's try to get the new API fixed for the next release then

Daniel

-- 
Daniel Veillard      | libxml Gnome XML XSLT toolkit  http://xmlsoft.org/
daniel@xxxxxxxxxxxx  | Rpmfind RPM search engine http://rpmfind.net/
http://veillard.com/ | virtualization library  http://libvirt.org/

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list


[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]