Commit 3fbd3405 authored by Peter Maydell's avatar Peter Maydell
Browse files

Merge remote-tracking branch 'remotes/huth-gitlab/tags/pull-request-2019-08-17' into staging 



- Run the iotest during "make check"

# gpg: Signature made Sat 17 Aug 2019 09:46:13 BST
# gpg:                using RSA key 27B88847EEE0250118F3EAB92ED9D774FE702DB5
# gpg:                issuer "thuth@redhat.com"
# gpg: Good signature from "Thomas Huth <th.huth@gmx.de>" [full]
# gpg:                 aka "Thomas Huth <thuth@redhat.com>" [full]
# gpg:                 aka "Thomas Huth <huth@tuxfamily.org>" [full]
# gpg:                 aka "Thomas Huth <th.huth@posteo.de>" [unknown]
# Primary key fingerprint: 27B8 8847 EEE0 2501 18F3  EAB9 2ED9 D774 FE70 2DB5

* remotes/huth-gitlab/tags/pull-request-2019-08-17:
  gitlab-ci: Remove qcow2 tests that are handled by "make check" already
  tests: Run the iotests during "make check" again
  block: fix NetBSD qemu-iotests failure

Signed-off-by: default avatarPeter Maydell <peter.maydell@linaro.org>
parents 9f3a972e 72e031f3

.gitlab-ci.yml

+4 −9
Original line number Diff line number Diff line
@@ -45,15 +45,10 @@ build-tcg-disabled: 
 - ./check -raw 001 002 003 004 005 008 009 010 011 012 021 025 032 033 048

            052 063 077 086 101 104 106 113 147 148 150 151 152 157 159 160

            163 170 171 183 184 192 194 197 205 208 215 221 222 226 227 236

 - ./check -qcow2 001 002 003 004 005 007 008 009 010 011 012 013 017 018 019

            020 021 022 024 025 027 028 029 031 032 033 034 035 036 037 038

            039 040 042 043 046 047 048 049 050 051 052 053 054 056 057 058

            060 061 062 063 065 066 067 068 069 071 072 073 074 079 080 082

            085 086 089 090 091 095 096 097 098 099 102 103 104 105 107 108

            110 111 114 117 120 122 124 126 127 129 130 132 133 134 137 138

            139 140 141 142 143 144 145 147 150 151 152 154 155 156 157 158

            161 165 170 172 174 176 177 179 184 186 187 190 192 194 195 196

            197 200 202 203 205 208 209 214 215 216 217 218 222 226 227 229 234

 - ./check -qcow2 028 040 051 056 057 058 065 067 068 082 085 091 095 096 102

            122 124 127 129 132 139 142 144 145 147 151 152 155 157 165 194

            196 197 200 202 203 205 208 209 215 216 218 222 227 234 246 247

            248 250 254 255 256



build-user:

 script:

block/file-posix.c

+2 −2
Original line number Diff line number Diff line
@@ -217,7 +217,7 @@ static int raw_normalize_devicepath(const char **filename, Error **errp) 
    fname = *filename;

    dp = strrchr(fname, '/');

    if (lstat(fname, &sb) < 0) {

        error_setg_errno(errp, errno, "%s: stat failed", fname);

        error_setg_file_open(errp, errno, fname);

        return -errno;

    }
@@ -561,7 +561,7 @@ static int raw_open_common(BlockDriverState *bs, QDict *options, 
    ret = fd < 0 ? -errno : 0;



    if (ret < 0) {

        error_setg_errno(errp, -ret, "Could not open '%s'", filename);

        error_setg_file_open(errp, -ret, filename);

        if (ret == -EROFS) {

            ret = -EACCES;

        }

tests/Makefile.include

+6 −4
Original line number Diff line number Diff line
@@ -142,7 +142,7 @@ check-unit-y += tests/test-uuid$(EXESUF) 
check-unit-y += tests/ptimer-test$(EXESUF)

check-unit-y += tests/test-qapi-util$(EXESUF)



check-block-$(CONFIG_POSIX) += tests/qemu-iotests-quick.sh

check-block-$(call land,$(CONFIG_POSIX),$(CONFIG_SOFTMMU)) += tests/check-block.sh



# All QTests for now are POSIX-only, but the dependencies are

# really in libqtest, not in the testcases themselves.
@@ -1092,8 +1092,10 @@ clean-tcg: $(CLEAN_TCG_TARGET_RULES) 


QEMU_IOTESTS_HELPERS-$(call land,$(CONFIG_SOFTMMU),$(CONFIG_LINUX)) = tests/qemu-iotests/socket_scm_helper$(EXESUF)



.PHONY: check-tests/qemu-iotests-quick.sh

check-tests/qemu-iotests-quick.sh: tests/qemu-iotests-quick.sh qemu-img$(EXESUF) qemu-io$(EXESUF) qemu-nbd$(EXESUF) $(QEMU_IOTESTS_HELPERS-y)

.PHONY: check-tests/check-block.sh

check-tests/check-block.sh: tests/check-block.sh qemu-img$(EXESUF) \

		qemu-io$(EXESUF) qemu-nbd$(EXESUF) $(QEMU_IOTESTS_HELPERS-y) \

		$(patsubst %,%/all,$(filter %-softmmu,$(TARGET_DIRS)))

	$<



.PHONY: $(patsubst %, check-%, $(check-qapi-schema-y))
@@ -1167,7 +1169,7 @@ check-acceptance: check-venv $(TESTS_RESULTS_DIR) 
check-qapi-schema: $(patsubst %,check-%, $(check-qapi-schema-y)) check-tests/qapi-schema/doc-good.texi

check-qtest: $(patsubst %,check-qtest-%, $(QTEST_TARGETS))

check-block: $(patsubst %,check-%, $(check-block-y))

check: check-qapi-schema check-unit check-softfloat check-qtest check-decodetree

check: check-block check-qapi-schema check-unit check-softfloat check-qtest check-decodetree

check-clean:

	rm -rf $(check-unit-y) tests/*.o $(QEMU_IOTESTS_HELPERS-y)

	rm -rf $(sort $(foreach target,$(SYSEMU_TARGET_LIST), $(check-qtest-$(target)-y)) $(check-qtest-generic-y))

tests/check-block.sh

+34 −10
Original line number Diff line number Diff line 
#!/bin/sh



FORMAT_LIST="raw qcow2 qed vmdk vpc"

# Honor the SPEED environment variable, just like we do it for the qtests.

if [ "$SPEED" = "slow" ]; then

    format_list="raw qcow2"

    group=

elif [ "$SPEED" = "thorough" ]; then

    format_list="raw qcow2 qed vmdk vpc"

    group=

else

    format_list=qcow2

    group="-g auto"

fi



if [ "$#" -ne 0 ]; then

    FORMAT_LIST="$@"

    format_list="$@"

fi



if grep -q "TARGET_GPROF=y" *-softmmu/config-target.mak 2>/dev/null ; then

    echo "GPROF is enabled ==> Not running the qemu-iotests."

    exit 0

fi



export QEMU_PROG="$PWD/x86_64-softmmu/qemu-system-x86_64"

export QEMU_IMG_PROG="$PWD/qemu-img"

export QEMU_IO_PROG="$PWD/qemu-io"

if [ -z "$(find . -name 'qemu-system-*' -print)" ]; then

    echo "No qemu-system binary available ==> Not running the qemu-iotests."

    exit 0

fi



if ! command -v bash >/dev/null 2>&1 ; then

    echo "bash not available ==> Not running the qemu-iotests."

    exit 0

fi



if [ ! -x $QEMU_PROG ]; then

    echo "'make check-block' requires qemu-system-x86_64"

    exit 1

if ! (sed --version | grep 'GNU sed') > /dev/null 2>&1 ; then

    if ! command -v gsed >/dev/null 2>&1; then

        echo "GNU sed not available ==> Not running the qemu-iotests."

        exit 0

    fi

fi



cd tests/qemu-iotests



ret=0

for FMT in $FORMAT_LIST ; do

    ./check -T -nocache -$FMT || ret=1

for fmt in $format_list ; do

    ./check -makecheck -$fmt $group || ret=1

done



exit $ret

tests/qemu-iotests-quick.sh

deleted100755 → 0
+0 −8
Original line number Diff line number Diff line 
#!/bin/sh



cd tests/qemu-iotests



ret=0

TEST_DIR=${TEST_DIR:-/tmp/qemu-iotests-quick-$$} ./check -T -qcow2 -g quick || ret=1



exit $ret