On 19.11.2013 15:53, Eric Blake wrote: > On 11/19/2013 07:44 AM, Michal Privoznik wrote: >> On the domain startup, this function is called to dump some info about >> the CPUs. At the beginning of the function we check if we aren't running >> older qemu which is not exposing the CPUs via 'qom-list'. However, we >> are not checking for even older qemus, which throw 'CommandNotFound' >> error. >> >> Signed-off-by: Michal Privoznik <mprivozn@xxxxxxxxxx> >> --- >> src/qemu/qemu_monitor_json.c | 12 +++++++----- >> 1 file changed, 7 insertions(+), 5 deletions(-) > > Any chance we can enhance the testsuite to cover this? At any rate, ACK. > > I'll push this one, and post another patch adding a test case. Thanks! Michal -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list