Re: [Qemu-devel] [PATCH 1/3] qemu-iotests: Many parallel allocating I/O requests

[Kevin Wolf]

Am 24.04.2012 16:38, schrieb Kevin Wolf:
> This test case manages to let qcow2 abort because its cache is used up
> and it can't find free cache entries for new requests any more.
> 
> Signed-off-by: Kevin Wolf <address@hidden>
> ---
>  tests/qemu-iotests/035     |   72 ++++++++
>  tests/qemu-iotests/035.out |  392 
> ++++++++++++++++++++++++++++++++++++++++++++
>  tests/qemu-iotests/group   |    1 +
>  3 files changed, 465 insertions(+), 0 deletions(-)
>  create mode 100755 tests/qemu-iotests/035
>  create mode 100644 tests/qemu-iotests/035.out
> 
> diff --git a/tests/qemu-iotests/035 b/tests/qemu-iotests/035
> new file mode 100755
> index 0000000..297734e
> --- /dev/null
> +++ b/tests/qemu-iotests/035
> @@ -0,0 +1,72 @@
> +#!/bin/bash
> +#
> +# Let a few AIO requests run in parallel and have them access different L2
> +# tables so that the cache has a chance to get used up.
> +#
> +# Copyright (C) 2009 Red Hat, Inc.
> +#
> +# This program is free software; you can redistribute it and/or modify
> +# it under the terms of the GNU General Public License as published by
> +# the Free Software Foundation; either version 2 of the License, or
> +# (at your option) any later version.
> +#
> +# This program is distributed in the hope that it will be useful,
> +# but WITHOUT ANY WARRANTY; without even the implied warranty of
> +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
> +# GNU General Public License for more details.
> +#
> +# You should have received a copy of the GNU General Public License
> +# along with this program.  If not, see <http://www.gnu.org/licenses/>.
> +#
> +
> +# creator
> address@hidden
> +
> +seq=`basename $0`
> +echo "QA output created by $seq"
> +
> +here=`pwd`
> +tmp=/tmp/$$
> +status=1     # failure is the default!
> +
> +_cleanup()
> +{
> +     _cleanup_test_img
> +}
> +trap "_cleanup; exit \$status" 0 1 2 3 15
> +
> +# get standard environment, filters and checks
> +. ./common.rc
> +. ./common.filter
> +
> +_supported_fmt generic
> +_supported_proto generic
> +_supported_os Linux
> +
> +
> +size=6G
> +CLUSTER_SIZE=512
> +
> +echo
> +echo "creating image"
> +_make_test_img $size
> +
> +function generate_requests() {
> +    for i in $(seq 0 63); do
> +        echo "aio_write ${i}M 512"
> +        echo "aio_write ${i}M 512"
> +        echo "aio_write ${i}M 512"
> +    done
> +}
> +
> +generate_requests | ./qemu-io $TEST_IMG | _filter_qemu_io |\

Fixed this locally to say $QEMU_IO instead of ./qemu-io. Unless there
are more problems with it, I won't resend a v2 but just include the
fixed version in the pull request.

Kevin