Re: [Libguestfs] [PATCH libnbd] copy: Test --request-size option

On Mon, Mar 1, 2021 at 6:38 PM Richard W.M. Jones <rjones(a)redhat.com&gt; wrote: > > On Mon, Mar 01, 2021 at 06:35:04PM +0200, Nir Soffer wrote: > > Add simple test similar to copy-sparse-no-extents testing that > > --request-size is respected. > > > > Signed-off-by: Nir Soffer <nsoffer(a)redhat.com&gt; > > --- > > copy/Makefile.am | 2 + > > copy/copy-sparse-request-size.sh | 92 ++++++++++++++++++++++++++++++++ > > 2 files changed, 94 insertions(+) > > create mode 100755 copy/copy-sparse-request-size.sh > > > > diff --git a/copy/Makefile.am b/copy/Makefile.am > > index c8b28ac..d85ceff 100644 > > --- a/copy/Makefile.am > > +++ b/copy/Makefile.am > > @@ -36,6 +36,7 @@ EXTRA_DIST = \ > > copy-sparse.sh \ > > copy-sparse-allocated.sh \ > > copy-sparse-no-extents.sh \ > > + copy-sparse-request-size.sh \ > > copy-sparse-to-stream.sh \ > > copy-stdin-to-nbd.sh \ > > copy-stdin-to-null.sh \ > > @@ -122,6 +123,7 @@ TESTS += \ > > copy-sparse.sh \ > > copy-sparse-allocated.sh \ > > copy-sparse-no-extents.sh \ > > + copy-sparse-request-size.sh \ > > copy-sparse-to-stream.sh \ > > $(ROOT_TESTS) \ > > test-long-options.sh \ > > diff --git a/copy/copy-sparse-request-size.sh b/copy/copy-sparse-request-size.sh > > new file mode 100755 > > index 0000000..b539f5e > > --- /dev/null > > +++ b/copy/copy-sparse-request-size.sh > > @@ -0,0 +1,92 @@ > > +#!/usr/bin/env bash > > +# nbd client library in userspace > > +# Copyright (C) 2021 Red Hat Inc. > > +# > > +# This library is free software; you can redistribute it and/or > > +# modify it under the terms of the GNU Lesser General Public > > +# License as published by the Free Software Foundation; either > > +# version 2 of the License, or (at your option) any later version. > > +# > > +# This library 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 > > +# Lesser General Public License for more details. > > +# > > +# You should have received a copy of the GNU Lesser General Public > > +# License along with this library; if not, write to the Free Software > > +# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA > > + > > +# Adapted from copy-sparse-no-extents.sh > > +# > > +# This test depends on the nbdkit default sparse block size (32K). > > + > > +. ../tests/functions.sh > > + > > +set -e > > +set -x > > + > > +# Skip this test under valgrind, it takes too long. > > +if [ "x$LIBNBD_VALGRIND" = "x1" ]; then > > + echo "$0: test skipped under valgrind" > > + exit 77 > > +fi > > + > > +requires nbdkit --version > > +requires nbdkit --exit-with-parent --version > > +requires nbdkit data --version > > +requires nbdkit eval --version > > + > > +out=copy-sparse-request-size.out > > Copy and paste problem - unless temporary files have a unique name > they will conflict when we run the tests in parallel (which we do > normally). Isn't "copy-sparse-request-size.out" unique?

> > Rich. > > > +cleanup_fn rm -f $out > > + > > +$VG nbdcopy --no-extents -S 0 --request-size=1048576 -- \ > > + [ nbdkit --exit-with-parent data data=' > > + 1 > > + @33554431 1 > > + ' ] \ > > + [ nbdkit --exit-with-parent eval \ > > + get_size=' echo 33554432 ' \ > > + pwrite=" echo \$@ >> $out " \ > > + trim=" echo \$@ >> $out " \ > > + zero=" echo \$@ >> $out " ] > > + > > +sort -n -o $out $out > > + > > +echo Output: > > +cat $out > > + > > +if [ "$(cat $out)" != "pwrite 1048576 0 > > +pwrite 1048576 1048576 > > +pwrite 1048576 10485760 > > +pwrite 1048576 11534336 > > +pwrite 1048576 12582912 > > +pwrite 1048576 13631488 > > +pwrite 1048576 14680064 > > +pwrite 1048576 15728640 > > +pwrite 1048576 16777216 > > +pwrite 1048576 17825792 > > +pwrite 1048576 18874368 > > +pwrite 1048576 19922944 > > +pwrite 1048576 2097152 > > +pwrite 1048576 20971520 > > +pwrite 1048576 22020096 > > +pwrite 1048576 23068672 > > +pwrite 1048576 24117248 > > +pwrite 1048576 25165824 > > +pwrite 1048576 26214400 > > +pwrite 1048576 27262976 > > +pwrite 1048576 28311552 > > +pwrite 1048576 29360128 > > +pwrite 1048576 30408704 > > +pwrite 1048576 3145728 > > +pwrite 1048576 31457280 > > +pwrite 1048576 32505856 > > +pwrite 1048576 4194304 > > +pwrite 1048576 5242880 > > +pwrite 1048576 6291456 > > +pwrite 1048576 7340032 > > +pwrite 1048576 8388608 > > +pwrite 1048576 9437184" ]; then > > + echo "$0: output does not match expected" > > + exit 1 > > +fi > > -- > > 2.26.2 > > -- > Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones > Read my programming and virtualization blog: http://rwmj.wordpress.com > virt-top is 'top' for virtual machines. Tiny program with many > powerful monitoring features, net stats, disk stats, logging, etc. > http://people.redhat.com/~rjones/virt-top >