[libvirt] [PATCH 0/3] Fixes for issues caused by IOThreads

shyu shyu at redhat.com
Tue Sep 16 07:44:01 UTC 2014


Hi John

With your patch and latest qemu-kvm, I can't start guest.

# rpm -q qemu-kvm
qemu-kvm-1.5.3-70.el7.x86_64

# virsh start test
error: Failed to start domain test
error: internal error: unable to execute QEMU command 'query-iothreads': 
The command query-iothreads has not been found

On 09/16/2014 08:55 AM, Eric Blake wrote:
> On 09/15/2014 06:13 PM, John Ferlan wrote:
>> Well I guess it happens to everyone - hopefully only once though...
> Yep, the brown bag of shame lets you learn from mistakes rather quickly :)
>
>> The IOThreads code really messed up a few things - git bisection for
>> the build is broken as of 5f6ad32c733a3bd158938aecabb0508a434ece95,
>> but that's resolved by 938fb12fad6d15c9fdb73f998c4e0ec1e278721f
>> which adds the 'niothreadspin && iothreadspin' definitions.
>>
>> Secondarily it seems things got worse, because guests weren't able to
>> be started because of a very bad logic error (<= 0 vs. < 0 and specific
>> check that 0 is "OK" (meaning no IOThreads).
>>
>> How that got by own self testing I'm still not quite sure, but it did
>> and it's a mea culpa for that.
>>
>> Anyway, patch 1 fixes the running issue... Patch 2 fixes some spacing
>> issues that Eric pointed out privately (prefer "i + 1" vs. "i+1").
>> Patch 3 just makes sure to use the right cgroup setup/init parameters
>> since there is no "iothread0"
>>
>> John Ferlan (3):
>>    qemu: Fix iothreads issue
>>    qemu_cgroup: Adjust spacing around incrementor
>>    qemu: Fix call in qemuDomainSetNumaParamsLive for virCgroupNewIOThread
> Thanks for the fast fixes.  I confirm that patch 1 fixes the regression
> in starting my dummy transient domain.  ACK series.
>
>
>
> --
> libvir-list mailing list
> libvir-list at redhat.com
> https://www.redhat.com/mailman/listinfo/libvir-list

-- 
Regards
shyu

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20140916/f3ffd165/attachment-0001.htm>


More information about the libvir-list mailing list