Re: [Qemu-devel] [PATCH v7 0/5] block: Add a qemu-iotests case for IO throttling

[Stefan Hajnoczi <stefanha@gmail.com>]

[-- Attachment #1: Type: text/plain, Size: 1224 bytes --]

On Fri, Jan 30, 2015 at 10:49:41AM +0800, Fam Zheng wrote:
> v7: Remove any "{iops,bps}_max" in 093 completely. (Max)
> 
> v6: Less resource demanding patch 5. (Max)
>     Add rev-by of Max to other patches.
> 
> v5: Rebase and improve the test. Please review again.
> 
>     Patch dependencies:
> 
>     This test depends on the qtest timer fix to run correctly.
>     http://lists.gnu.org/archive/html/qemu-devel/2015-01/msg01865.html
> 
>     Also depends on the os check fix to run at all:
>     http://lists.gnu.org/archive/html/qemu-devel/2015-01/msg01848.html
> 
> Original cover letter
> ---------------------
> 
> There is a change in qemu-io sub-commands "aio_read" and "aio_write", which
> makes the aio requests accounted and the statistics reflected in blockstats.
> 
> Note that IO throttling implementation allows overcommiting of requests, so the
> actual IO happened in a time unit may be a bit larger than given limits. In the
> test case, the stats numbers are compared with a 10% error tolerance, to make
> room for such flexibility in order to improve determinism.

Tried to merge this but unfortunately it hangs using 100% CPU:

  $ cd tests/qemu-iotests
  $ ./check 093

[-- Attachment #2: Type: application/pgp-signature, Size: 473 bytes --]