[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v7 0/5] block: Add a qemu-iotests case for IO th
From: |
Stefan Hajnoczi |
Subject: |
Re: [Qemu-devel] [PATCH v7 0/5] block: Add a qemu-iotests case for IO throttling |
Date: |
Thu, 12 Feb 2015 16:50:16 +0000 |
User-agent: |
Mutt/1.5.23 (2014-03-12) |
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
pgpAEISY9CgI0.pgp
Description: PGP signature
- Re: [Qemu-devel] [PATCH v7 0/5] block: Add a qemu-iotests case for IO throttling,
Stefan Hajnoczi <=