|
From: | Hans de Goede |
Subject: | Re: [Qemu-devel] [PATCH] main-loop: Don't lock starve io-threads when main_loop_tlg has pending events |
Date: | Tue, 08 Oct 2013 21:21:40 +0200 |
User-agent: | Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.0 |
Hi, On 10/08/2013 09:13 PM, Paolo Bonzini wrote:
Il 08/10/2013 21:10, Hans de Goede ha scritto:@@ -480,6 +480,11 @@ int main_loop_wait(int nonblocking) timerlistgroup_deadline_ns( &main_loop_tlg)); + /* When not non-blocking always allow io-threads to acquire the lock */ + if (timeout != 0 && timeout_ns == 0) { + timeout_ns = 1; + } +This _is_ an I/O thread, so I guess this should be changed to "other threads".
Ok.
Also, perhaps timeout_ns can be changed to a higher value such as 250 us that were used up to the offending commit?
Wasn't it 1 ms until the offending commit (note 250 us does sound better to me). More over I wonder how useful is this, if threads are waiting for the lock at his point, they should all get it and do work (and then release it) before this thread will be able to re-aquire it. The only case which I can see where going to sleep will help is when a thread takes the lock, does some thing, releases it, then does something else quite quickly (so within 250 us), and then tries to re-aquire the lock to do more work. Note either solution (1 ns versus 250 us) is fine with me, I'm by no means the expert on this, just let me know which one you think is better and I'll do a v2. Regards, Hans
[Prev in Thread] | Current Thread | [Next in Thread] |