From: "Daniel P. Berrange" <berrange@redhat.com>
To: qemu-devel@nongnu.org
Cc: qemu-block@nongnu.org, Max Reitz <mreitz@redhat.com>,
"Daniel P. Berrange" <berrange@redhat.com>
Subject: [Qemu-devel] [PATCH 0/6] Improve I/O tests coverage of LUKS
Date: Mon, 19 Sep 2016 18:35:38 +0100 [thread overview]
Message-ID: <1474306544-24708-1-git-send-email-berrange@redhat.com> (raw)
The main goal of this series is to get the I/O tests passing
100% with LUKS when run with './check -luks'.
In testing this I noticed that test timings are cached
across runs with different block backends which is very
misleading.
It adds a few more combinations to the LUKS/dmcrypt
interoperability test.
To make LUKS testing not quite as slow, we drop the
PBKDF iteration count down to a very small value.
This doesn't remove all overhead, as formatting the
volume will always measure PBKDF timing over a 1
second interval.
Finally I've been trying to get I/O tests working
under travis and hitting a tonne of non-deterministic
failures, even with qcow2 format. I've put one easy
fix at the end, but there are more needed. If anyone
fancies testing travis more, the config change I was
experimenting with is
diff --git a/.travis.yml b/.travis.yml
index f30b10e..7b5fe92 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -33,6 +33,7 @@ addons:
- libvte-2.90-dev
- sparse
- uuid-dev
+ - bc
# The channel name "irc.oftc.net#qemu" is encrypted against qemu/qemu
# to prevent IRC notifications from forks. This was created using:
@@ -64,7 +65,7 @@ before_install:
before_script:
- ./configure ${CONFIG}
script:
- - make -j3 && ${TEST_CMD}
+ - make -j3 && eval ${TEST_CMD}
matrix:
include:
# gprof/gcov are GCC features
@@ -86,6 +87,16 @@ matrix:
- env: CONFIG="--with-coroutine=gthread"
TEST_CMD=""
compiler: gcc
+ - env: CONFIG="--target-list=x86_64-softmmu"
+ TEST_CMD="cd tests/qemu-iotests && ./check -luks"
+ compiler: gcc
+ sudo: required
+ - env: CONFIG="--target-list=x86_64-softmmu"
+ TEST_CMD="cd tests/qemu-iotests && ./check -raw"
+ compiler: gcc
+ - env: CONFIG="--target-list=x86_64-softmmu"
+ TEST_CMD="cd tests/qemu-iotests && ./check -qcow2"
+ compiler: gcc
- env: CONFIG=""
os: osx
compiler: clang
All three tests - luks, raw and qcow2 - will randomly fail very
often when run on public travis infra IME :-(
Daniel P. Berrange (6):
iotests: throw away test timings if args change
iotests: fix remainining tests to work with LUKS
iotests: reduce PBKDF iterations when testing LUKS
iotests: add more LUKS hash combination tests
iotests: chown LUKS device before qemu-io launches
iotests: use _filter_qemu with test 140
tests/qemu-iotests/033 | 16 +-
tests/qemu-iotests/120 | 25 +-
tests/qemu-iotests/140 | 17 +-
tests/qemu-iotests/145 | 18 +-
tests/qemu-iotests/149 | 26 +-
tests/qemu-iotests/149.out | 1002 ++++++++++++++++++++++++++++----------
tests/qemu-iotests/157 | 17 +-
tests/qemu-iotests/157.out | 16 +-
tests/qemu-iotests/check | 21 +-
tests/qemu-iotests/common.filter | 3 +-
tests/qemu-iotests/common.rc | 3 +
11 files changed, 865 insertions(+), 299 deletions(-)
--
2.7.4
next reply other threads:[~2016-09-19 17:36 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-09-19 17:35 Daniel P. Berrange [this message]
2016-09-19 17:35 ` [Qemu-devel] [PATCH 1/6] iotests: throw away test timings if args change Daniel P. Berrange
2016-09-19 19:53 ` Eric Blake
2016-09-20 9:36 ` Daniel P. Berrange
2016-09-20 14:06 ` Eric Blake
2016-09-20 14:15 ` Daniel P. Berrange
2016-09-20 14:38 ` Eric Blake
2016-09-20 14:39 ` Daniel P. Berrange
2016-10-31 14:28 ` [Qemu-devel] [Qemu-block] " Kevin Wolf
2016-10-31 14:34 ` Daniel P. Berrange
2016-09-19 17:35 ` [Qemu-devel] [PATCH 2/6] iotests: fix remainining tests to work with LUKS Daniel P. Berrange
2016-09-19 17:35 ` [Qemu-devel] [PATCH 3/6] iotests: reduce PBKDF iterations when testing LUKS Daniel P. Berrange
2016-09-19 17:35 ` [Qemu-devel] [PATCH 4/6] iotests: add more LUKS hash combination tests Daniel P. Berrange
2016-09-19 17:35 ` [Qemu-devel] [PATCH 5/6] iotests: chown LUKS device before qemu-io launches Daniel P. Berrange
2016-09-19 17:35 ` [Qemu-devel] [PATCH 6/6] iotests: use _filter_qemu with test 140 Daniel P. Berrange
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=1474306544-24708-1-git-send-email-berrange@redhat.com \
--to=berrange@redhat.com \
--cc=mreitz@redhat.com \
--cc=qemu-block@nongnu.org \
--cc=qemu-devel@nongnu.org \
/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).