On 04/28/2015 06:40 AM, John Ferlan wrote:
v1 here: http://www.redhat.com/archives/libvir-list/2015-April/msg01382.html
v2 changes:
Patch 1 - Add a !STRPREFIX(tmp, "iothread") of the returned iothreads to ignore anything that doesn't start with "iothread".
Patch 2 - already ACKed, just difficult to extract.
Patch 3 - NEW: remove qemuMonitorIOThreadInfoFree as requested from review of patch 1.
John Ferlan (3): qemu: Remove need for qemuDomainParseIOThreadAlias qemu: qemuProcessDetectIOThreadPIDs invert checks qemu: Remove need for qemuMonitorIOThreadInfoFree
src/qemu/qemu_command.c | 17 ----------------- src/qemu/qemu_command.h | 3 --- src/qemu/qemu_driver.c | 15 +++++---------- src/qemu/qemu_monitor.c | 10 ---------- src/qemu/qemu_monitor.h | 4 +--- src/qemu/qemu_monitor_json.c | 22 +++++++++++++++------- src/qemu/qemu_process.c | 25 +++++++++++-------------- tests/qemumonitorjsontest.c | 14 +++++++------- 8 files changed, 39 insertions(+), 71 deletions(-)
Thanks for the reviews - now pushed. John FYI: This cover letter touched on why patch 2 was a repeat... It was just easier to not extract it - perhaps the git merge would have worked, perhaps it would not have.