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 <[email protected]>
>> ---
>>  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
[email protected]
https://www.redhat.com/mailman/listinfo/libvir-list

Reply via email to