|
From: | Avi Kivity |
Subject: | Re: [Qemu-devel] Re: [PATCH] QEMUFileBuffered: indicate that we're ready when the underlying file is ready |
Date: | Thu, 19 Aug 2010 16:12:01 +0300 |
User-agent: | Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.2.7) Gecko/20100720 Fedora/3.1.1-1.fc13 Lightning/1.0b2pre Thunderbird/3.1.1 |
On 08/03/2010 04:00 PM, Luiz Capitulino wrote:
On Tue, 03 Aug 2010 09:12:52 +0300 Avi Kivity<address@hidden> wrote:On 07/07/2010 07:44 PM, Avi Kivity wrote:QEMUFileBuffered stops writing when the underlying QEMUFile is not ready, and tells its producer so. However, when the underlying QEMUFile becomes ready, it neglects to pass that information along, resulting in stoppage of all data until the next tick (a tenths of a second). Usually this doesn't matter, because most QEMUFiles used with QEMUFileBuffered are almost always ready, but in the case of exec: migration this is not true, due to the small pipe buffers used to connect to the target process. The result is very slow migration. Fix by detecting the readiness notification and propagating it. The detection is a little ugly since QEMUFile overloads put_buffer() to send it, but that's the suject for a different patch.Ping.I've queued this fix and another one in a 'for-0.13' branch and was planning to send a pull request as soon as Anthony called for 0.13 fixes, however this didn't happen yet and I forgot.
Ping again.I don't want to take over Cam's longest-time-to-apply record, so someone apply this please. 0.13 and mainline.
-- error compiling committee.c: too many arguments to function
[Prev in Thread] | Current Thread | [Next in Thread] |