Re: [Qemu-devel] [PATCH] main-loop: drop the BQL if the I/O appears to be spinning

2013-04-09 Thread Anthony Liguori
Andreas Färber writes: > Am 08.04.2013 23:55, schrieb Anthony Liguori: >> Applied. Thanks. > > I am seeing "main-loop: WARNING: I/O thread spun for 1000 iterations" > for both check-qtest-i386 and check-qtest-x86_64 now, is that expected? > > PMM just recently cleaned up the noisy arm qtest targ

Re: [Qemu-devel] [PATCH] main-loop: drop the BQL if the I/O appears to be spinning

2013-04-09 Thread Andreas Färber
Am 08.04.2013 23:55, schrieb Anthony Liguori: > Applied. Thanks. I am seeing "main-loop: WARNING: I/O thread spun for 1000 iterations" for both check-qtest-i386 and check-qtest-x86_64 now, is that expected? PMM just recently cleaned up the noisy arm qtest target... :( Andreas -- SUSE LINUX Pr

Re: [Qemu-devel] [PATCH] main-loop: drop the BQL if the I/O appears to be spinning

2013-04-08 Thread Anthony Liguori
Applied. Thanks. Regards, Anthony Liguori

Re: [Qemu-devel] [PATCH] main-loop: drop the BQL if the I/O appears to be spinning

2013-04-06 Thread Peter Crosthwaite
Hi Anthony, On Fri, Apr 5, 2013 at 11:46 PM, Anthony Liguori wrote: > The char-flow refactoring introduced a busy-wait that depended on > an action from the VCPU thread. However, the VCPU thread could > never take that action because the busy-wait starved the VCPU thread > of the BQL because it

Re: [Qemu-devel] [PATCH] main-loop: drop the BQL if the I/O appears to be spinning

2013-04-05 Thread Anthony Liguori
Eric Blake writes: > On 04/05/2013 07:46 AM, Anthony Liguori wrote: >> The char-flow refactoring introduced a busy-wait that depended on >> an action from the VCPU thread. However, the VCPU thread could >> never take that action because the busy-wait starved the VCPU thread >> of the BQL because

Re: [Qemu-devel] [PATCH] main-loop: drop the BQL if the I/O appears to be spinning

2013-04-05 Thread Paolo Bonzini
Il 05/04/2013 15:46, Anthony Liguori ha scritto: > The char-flow refactoring introduced a busy-wait that depended on > an action from the VCPU thread. However, the VCPU thread could > never take that action because the busy-wait starved the VCPU thread > of the BQL because it never dropped the mut

Re: [Qemu-devel] [PATCH] main-loop: drop the BQL if the I/O appears to be spinning

2013-04-05 Thread Eric Blake
On 04/05/2013 07:46 AM, Anthony Liguori wrote: > The char-flow refactoring introduced a busy-wait that depended on > an action from the VCPU thread. However, the VCPU thread could > never take that action because the busy-wait starved the VCPU thread > of the BQL because it never dropped the mutex

[Qemu-devel] [PATCH] main-loop: drop the BQL if the I/O appears to be spinning

2013-04-05 Thread Anthony Liguori
The char-flow refactoring introduced a busy-wait that depended on an action from the VCPU thread. However, the VCPU thread could never take that action because the busy-wait starved the VCPU thread of the BQL because it never dropped the mutex while running select. Paolo doesn't want to drop this