Re: [PATCH 3/5] Introduce a test suite for the JSON monitor

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

 



On 08/20/2012 07:49 AM, Daniel P. Berrange wrote:
> From: "Daniel P. Berrange" <berrange@xxxxxxxxxx>
> 
> Take advantage of the previously added monitor helpers to
> create a test suite for the QEMU JSON monitor impl. As a
> proof of concept, this tests the 'qemuMonitorGetStatus'
> implementation
> ---
>  .gitignore                  |   1 +
>  tests/Makefile.am           |  11 +++-
>  tests/qemumonitorjsontest.c | 150 ++++++++++++++++++++++++++++++++++++++++++++
>  3 files changed, 161 insertions(+), 1 deletion(-)
>  create mode 100644 tests/qemumonitorjsontest.c

Every so often (and again today), I get a hung testsuite on
qemumonitorjsontest; on IRC, I mentioned this, and you determined it was
a race where the child thread goes away before the parent enters poll(),
so there is no longer anything to wake up the parent.  You said
something about marking the monitor as closed when you detect EOF, but
I'm not sure what you meant.  Is this something we can get fixed before
1.0.0 is released?

-- 
Eric Blake   eblake@xxxxxxxxxx    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature

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