On 04/26/2017 09:14 AM, Max Reitz wrote: > There is no reason for the qemu-nbd server used for tests not to accept > an arbitrary number of clients. In fact, test 181 will require it to > accept two clients at the same time (and thus it fails before this > patch). > > This patch updates common.rc to launch qemu-nbd with -e 42 which should > be enough for all of our current and future tests.
Perfect place for you to inject some jibe about "life, the universe, and everything" ;) > > Signed-off-by: Max Reitz <[email protected]> > --- > tests/qemu-iotests/common.rc | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/tests/qemu-iotests/common.rc b/tests/qemu-iotests/common.rc > index 7d4781d4ad..31b888d4e9 100644 > --- a/tests/qemu-iotests/common.rc > +++ b/tests/qemu-iotests/common.rc > @@ -168,7 +168,9 @@ _make_test_img() > > # Start an NBD server on the image file, which is what we'll be talking > to > if [ $IMGPROTO = "nbd" ]; then > - eval "$QEMU_NBD -v -t -b 127.0.0.1 -p 10810 -f $IMGFMT > $TEST_IMG_FILE >/dev/null &" > + # Pass a sufficiently high number to -e that should be enough for all > + # tests > + eval "$QEMU_NBD -v -t -b 127.0.0.1 -p 10810 -f $IMGFMT -e 42 > $TEST_IMG_FILE >/dev/null &" Worth dropping the double-space before $TEST_IMG_FILE while touching this? Reviewed-by: Eric Blake <[email protected]> -- Eric Blake, Principal Software Engineer Red Hat, Inc. +1-919-301-3266 Virtualization: qemu.org | libvirt.org
signature.asc
Description: OpenPGP digital signature
