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

[Fam Zheng]

On Thu, 02/12 16:50, Stefan Hajnoczi wrote:
> 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

Could you apply this as a dependency?

http://lists.gnu.org/archive/html/qemu-stable/2015-01/msg00134.html

(sorry, I forgot to update the pointer in cover letter)

Fam