[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v9 0/6] monitor: enable OOB by default
From: |
Eric Blake |
Subject: |
Re: [Qemu-devel] [PATCH v9 0/6] monitor: enable OOB by default |
Date: |
Wed, 10 Oct 2018 15:27:34 -0500 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.0 |
On 10/10/18 2:26 PM, Eric Blake wrote:
On the other hand, when I'm trying to use a qemu binary with these
patches applied, libvirt is hanging when trying to probe the
capabilities of the binary, waiting for a response to
"qmp_capabilities". I'll try and bisect which patch is causing the
problem, and figure out why it is happening for libvirt and not running
by hand (perhaps is it a tty vs. Unix socket thing?)
Bisect didn't help much; it landed on:
monitor: remove "x-oob", turn oob on by default
as the cause of libvirt hanging. I didn't have time to investigate
further, other than the command line that is hanging:
/home/eblake/qemu/x86_64-softmmu/qemu-system-x86_64 -S -no-user-config
-nodefaults -nographic -machine none,accel=kvm:tcg -qmp
unix:/var/lib/libvirt/qemu/capabilities.monitor.sock,server,nowait
-pidfile /var/lib/libvirt/qemu/capabilities.pidfile -daemonize
And I suspect it is the -daemonize that is causing the hang I'm seeing
when run by libvirt.
Based-on: <address@hidden>
Based-on: <address@hidden>
(this series is based on Markus's monitor-next tree)
Am I missing any prerequisite patches? Markus' monitor-next tree is
currently a subset of git master (merge efd1d522).
--
Eric Blake, Principal Software Engineer
Red Hat, Inc. +1-919-301-3266
Virtualization: qemu.org | libvirt.org
- [Qemu-devel] [PATCH v9 2/6] monitor: resume the monitor earlier if needed, (continued)
- [Qemu-devel] [PATCH v9 3/6] monitor: remove "x-oob", turn oob on by default, Peter Xu, 2018/10/09
- [Qemu-devel] [PATCH v9 4/6] Revert "tests: Add parameter to qtest_init_without_qmp_handshake", Peter Xu, 2018/10/09
- [Qemu-devel] [PATCH v9 5/6] tests: add oob functional test for test-qmp-cmds, Peter Xu, 2018/10/09
- [Qemu-devel] [PATCH v9 6/6] tests: qmp-test: add queue full test, Peter Xu, 2018/10/09
- Re: [Qemu-devel] [PATCH v9 0/6] monitor: enable OOB by default, Eric Blake, 2018/10/10
Re: [Qemu-devel] [PATCH v9 0/6] monitor: enable OOB by default, Markus Armbruster, 2018/10/31