On Thu, 07/09 14:02, Stefan Hajnoczi wrote:
> This patch only converts the mirror block job to use the new relax
> function.  The other block jobs (stream, backup, commit) are still using
> a 0 ns delay and are therefore broken.  They should probably be
> converted in the same series.

That's because they all can be perfectly mitigated by setting a reasonable
"speed" that matchs the host horsepower. Thinking about this again, I doubt
that lengthening the duration with a hardcoded value benifits everyone; and
before Alexandre's reply on old server/slow disks, I don't recall any report,
because the coroutines would already yield often enough, when waiting for IO to
complete. So I am not sure whether they're broken in practice.

Fam

Reply via email to