On 03/07/2011 10:18 AM, Daniel P. Berrange wrote:
On Mon, Mar 07, 2011 at 10:09:36AM -0700, Eric Blake wrote:
> From: Wen Congyang <wency(a)cn.fujitsu.com>
>
> When I use newest libvirt to save a domain, libvirtd will be deadlock.
>
> The reason is that we will try to lock some object in callback function, and we may
call event API with locking the same object.
> In the function virEventDispatchHandles(), we unlock eventLoop before calling
callback function. I think we should
> do the same thing in the function virEventCleanupTimeouts() and
virEventCleanupHandles().
>
> Signed-off-by: Wen Congyang <wency(a)cn.fujitsu.com>
> Signed-off-by: Eric Blake <eblake(a)redhat.com>
> ---
>
> v2: incorporate comments from reviewers, and rebase on top of file move
>
ACK
Thanks; pushed.
--
Eric Blake eblake(a)redhat.com +1-801-349-2682
Libvirt virtualization library
http://libvirt.org