From: "Philippe Mathieu-Daudé" <f4bug@amsat.org>
To: Kevin Wolf <kwolf@redhat.com>, Max Reitz <mreitz@redhat.com>
Cc: "Philippe Mathieu-Daudé" <f4bug@amsat.org>,
qemu-devel@nongnu.org, qemu-block@nongnu.org,
"Stefan Hajnoczi" <stefanha@redhat.com>,
"Paolo Bonzini" <pbonzini@redhat.com>,
"Wenchao Xia" <xiawenc@linux.vnet.ibm.com>,
"Stratos Psomadakis" <psomas@grnet.gr>,
"Eric Blake" <eblake@redhat.com>
Subject: [Qemu-devel] [PATCH 2/2] qemu-iotests: check socket_scm_helper is available
Date: Tue, 19 Dec 2017 13:37:13 -0300 [thread overview]
Message-ID: <20171219163713.31383-3-f4bug@amsat.org> (raw)
In-Reply-To: <20171219163713.31383-1-f4bug@amsat.org>
This now throws a more helpful error message when not found:
tests/qemu-iotests$ ./check 045
QEMU -- "/qemu/build/x86_64-softmmu/qemu-system-x86_64" -nodefaults -machine accel=qtest
QEMU_IMG -- "/qemu/build/qemu-img"
QEMU_IO -- "/qemu/build/qemu-io" --cache writeback -f raw
QEMU_NBD -- "/qemu/build/qemu-nbd"
TEST_DIR -- /qemu/build/tests/qemu-iotests/scratch
SOCKET_SCM_HELPER --
045 1s ... [failed, exit status 1] - output mismatch (see 045.out.bad)
-...........
+......EE.EE
+======================================================================
+ERROR: test_add_fd (__main__.TestSCMFd)
+----------------------------------------------------------------------
+Traceback (most recent call last):
+ File "045", line 147, in test_add_fd
+ self._send_fd_by_SCM()
+ File "045", line 143, in _send_fd_by_SCM
+ ret = self.vm.send_fd_scm(image0)
+ File "/source/qemu/tests/qemu-iotests/../../scripts/qemu.py", line 125, in send_fd_scm
+ self._socket_scm_helper)
+QEMUMachineError: socket_scm_helper does not exist
+
----------------------------------------------------------------------
Failures: 045
Failed 1 of 1 tests
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
---
tests/qemu-iotests/check | 2 ++
1 file changed, 2 insertions(+)
diff --git a/tests/qemu-iotests/check b/tests/qemu-iotests/check
index e6b6ff7a04..a2c0bab1cc 100755
--- a/tests/qemu-iotests/check
+++ b/tests/qemu-iotests/check
@@ -597,6 +597,8 @@ fi
if [ -x "$build_iotests/socket_scm_helper" ]
then
export SOCKET_SCM_HELPER="$build_iotests/socket_scm_helper"
+else
+ _init_error "socket_scm_helper not found"
fi
default_machine=$($QEMU_PROG -machine help | sed -n '/(default)/ s/ .*//p')
--
2.15.1
next prev parent reply other threads:[~2017-12-19 16:37 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-12-19 16:37 [Qemu-devel] [PATCH 0/2] qemu-iotests: add missing make rule for socket_scm_helper Philippe Mathieu-Daudé
2017-12-19 16:37 ` [Qemu-devel] [PATCH 1/2] qemu-iotests: add missing rule to build socket_scm_helper Philippe Mathieu-Daudé
2018-01-29 15:02 ` Max Reitz
2017-12-19 16:37 ` Philippe Mathieu-Daudé [this message]
2018-01-29 15:10 ` [Qemu-devel] [PATCH 2/2] qemu-iotests: check socket_scm_helper is available Max Reitz
2017-12-20 9:57 ` [Qemu-devel] [PATCH 0/2] qemu-iotests: add missing make rule for socket_scm_helper Paolo Bonzini
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20171219163713.31383-3-f4bug@amsat.org \
--to=f4bug@amsat.org \
--cc=eblake@redhat.com \
--cc=kwolf@redhat.com \
--cc=mreitz@redhat.com \
--cc=pbonzini@redhat.com \
--cc=psomas@grnet.gr \
--cc=qemu-block@nongnu.org \
--cc=qemu-devel@nongnu.org \
--cc=stefanha@redhat.com \
--cc=xiawenc@linux.vnet.ibm.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).