[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH V7 0/5] Continuous Leaky Bucket Throttling
From: |
Stefan Hajnoczi |
Subject: |
Re: [Qemu-devel] [PATCH V7 0/5] Continuous Leaky Bucket Throttling |
Date: |
Fri, 30 Aug 2013 16:21:04 +0200 |
On Fri, Aug 30, 2013 at 12:06 PM, Paolo Bonzini <address@hidden> wrote:
> Il 30/08/2013 11:53, Stefan Hajnoczi ha scritto:
>>
>> Resources are refilled as time passes, not by completing requests, so
>> there should be no need to act when a request completes.
>
> Resources are refilled also by completing requests. iops=0, iops_max=10
> is a valid setting that will allow at most 10 I/O operations to be
> active at the same time.
Where do you see that in the code? LeakyBucket.level is lowered only
as time passes, not by request completion.
Stefan
- [Qemu-devel] [PATCH V7 1/5] throttle: Add a new t hrottling API implementing continuous leaky bucket., (continued)
- [Qemu-devel] [PATCH V7 1/5] throttle: Add a new t hrottling API implementing continuous leaky bucket., Benoît Canet, 2013/08/28
- [Qemu-devel] [PATCH V7 2/5] throttle: Add units t ests, Benoît Canet, 2013/08/28
- [Qemu-devel] [PATCH V7 3/5] block: Enable the new throttling code in the block layer., Benoît Canet, 2013/08/28
- [Qemu-devel] [PATCH V7 4/5] block: Add support fo r throttling burst max in QMP and the command line., Benoît Canet, 2013/08/28
- [Qemu-devel] [PATCH V7 5/5] block: Add iops_siz e to do the iops accounting for a given io size., Benoît Canet, 2013/08/28
- Re: [Qemu-devel] [PATCH V7 0/5] Continuous Leaky Bucket Throttling, Stefan Hajnoczi, 2013/08/29
- Re: [Qemu-devel] [PATCH V7 0/5] Continuous Leaky Bucket Throttling, Benoît Canet, 2013/08/29