[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: |
Fam Zheng |
Subject: |
Re: [Qemu-devel] [PATCH v7 0/5] block: Add a qemu-iotests case for IO throttling |
Date: |
Fri, 13 Feb 2015 10:55:50 +0800 |
User-agent: |
Mutt/1.5.23 (2014-03-12) |
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