[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 4/7] blockjob: allow block_job_throttle to take
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH 4/7] blockjob: allow block_job_throttle to take delay_ns |
Date: |
Thu, 14 Dec 2017 18:21:11 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.5.0 |
On 14/12/2017 17:06, John Snow wrote:
>>
>> And another question. After this series there is exactly one
>> block_job_sleep_ns call (in block/mirror.c). Perhaps instead of
>> block_job_throttle, you should refine block_job_sleep_ns?
>>
> Yeah, maybe? "A rose by any other name," though -- I think I might be
> coming for the block/mirror call next because I have one more downstream
> BZ that references this as a job that can cause the warning print.
>
> So maybe we'll just have throttle calls instead of sleep calls from here
> on out.
Ok, shall we wait for v2 where you look at that BZ as well?
Thanks,
Paolo
- Re: [Qemu-devel] [PATCH 3/7] blockjob: create block_job_throttle, (continued)