From: Max Reitz <mreitz@redhat.com>
To: qemu-block@nongnu.org
Cc: Kevin Wolf <kwolf@redhat.com>,
Maxim Levitsky <mlevitsk@redhat.com>,
qemu-devel@nongnu.org, Max Reitz <mreitz@redhat.com>
Subject: [PATCH v2 00/21] iotests: Allow ./check -o data_file
Date: Tue, 15 Oct 2019 16:27:08 +0200 [thread overview]
Message-ID: <20191015142729.18123-1-mreitz@redhat.com> (raw)
Hi,
The cover letter from v1 (explaining the motivation behind this series
and the general structure) is here:
https://lists.nongnu.org/archive/html/qemu-block/2019-09/msg01323.html
For v2, I’ve tried to address Maxim’s comments:
- Patch 1 through 3: New
- Patch 4: Only print feature bits instead of blacklisting stuff that we
don’t need
- Patch 5:
- Fix typo
- Add comment why 098 needs compat=1.1
- Patch 16: Use _check_test_img
- Patch 17: Use the new _filter_json_filename
- Patch 18: Rethink the incompatible feature filter approach: Instead of
filtering out the data_file bit, just check whether the
dirty bit is present (because that is all we want to know)
- Patch 19: Use the new _filter_json_filename
- Patch 20: Rebase conflicts due to the changes to patch 5
- Patch 21:
- Add and use _get_data_file
- Add a comment how the data_file_filter in _filter_qemu_img_map works
git-backport-diff against v1:
Key:
[----] : patches are identical
[####] : number of functional differences between upstream/downstream patch
[down] : patch is downstream-only
The flags [FC] indicate (F)unctional and (C)ontextual differences, respectively
001/21:[down] 'iotests/qcow2.py: Add dump-header-exts'
002/21:[down] 'iotests/qcow2.py: Split feature fields into bits'
003/21:[down] 'iotests: Add _filter_json_filename'
004/21:[0060] [FC] 'iotests: Filter refcount_order in 036'
005/21:[0003] [FC] 'iotests: Replace IMGOPTS by _unsupported_imgopts'
006/21:[----] [--] 'iotests: Drop compat=1.1 in 050'
007/21:[----] [--] 'iotests: Let _make_test_img parse its parameters'
008/21:[----] [--] 'iotests: Add -o and --no-opts to _make_test_img'
009/21:[----] [--] 'iotests: Inject space into -ocompat=0.10 in 051'
010/21:[----] [--] 'iotests: Replace IMGOPTS= by -o'
011/21:[----] [--] 'iotests: Replace IMGOPTS='' by --no-opts'
012/21:[----] [--] 'iotests: Drop IMGOPTS use in 267'
013/21:[----] [--] 'iotests: Avoid qemu-img create'
014/21:[----] [--] 'iotests: Use _rm_test_img for deleting test images'
015/21:[----] [--] 'iotests: Avoid cp/mv of test images'
016/21:[0004] [FC] 'iotests: Make 091 work with data_file'
017/21:[0004] [FC] 'iotests: Make 110 work with data_file'
018/21:[0002] [FC] 'iotests: Make 137 work with data_file'
019/21:[0004] [FC] 'iotests: Make 198 work with data_file'
020/21:[0002] [FC] 'iotests: Disable data_file where it cannot be used'
021/21:[0034] [FC] 'iotests: Allow check -o data_file'
Max Reitz (21):
iotests/qcow2.py: Add dump-header-exts
iotests/qcow2.py: Split feature fields into bits
iotests: Add _filter_json_filename
iotests: Filter refcount_order in 036
iotests: Replace IMGOPTS by _unsupported_imgopts
iotests: Drop compat=1.1 in 050
iotests: Let _make_test_img parse its parameters
iotests: Add -o and --no-opts to _make_test_img
iotests: Inject space into -ocompat=0.10 in 051
iotests: Replace IMGOPTS= by -o
iotests: Replace IMGOPTS='' by --no-opts
iotests: Drop IMGOPTS use in 267
iotests: Avoid qemu-img create
iotests: Use _rm_test_img for deleting test images
iotests: Avoid cp/mv of test images
iotests: Make 091 work with data_file
iotests: Make 110 work with data_file
iotests: Make 137 work with data_file
iotests: Make 198 work with data_file
iotests: Disable data_file where it cannot be used
iotests: Allow check -o data_file
tests/qemu-iotests/005 | 2 +-
tests/qemu-iotests/007 | 5 ++-
tests/qemu-iotests/014 | 2 +
tests/qemu-iotests/015 | 5 ++-
tests/qemu-iotests/019 | 6 +--
tests/qemu-iotests/020 | 6 +--
tests/qemu-iotests/024 | 10 ++---
tests/qemu-iotests/026 | 5 ++-
tests/qemu-iotests/028 | 2 +-
tests/qemu-iotests/029 | 7 ++--
tests/qemu-iotests/031 | 9 ++--
tests/qemu-iotests/031.out | 36 ++++++++--------
tests/qemu-iotests/036 | 15 ++++---
tests/qemu-iotests/036.out | 66 ++++-------------------------
tests/qemu-iotests/039 | 27 +++++-------
tests/qemu-iotests/039.out | 22 +++++-----
tests/qemu-iotests/043 | 4 +-
tests/qemu-iotests/046 | 2 +
tests/qemu-iotests/048 | 4 +-
tests/qemu-iotests/050 | 8 +---
tests/qemu-iotests/051 | 7 ++--
tests/qemu-iotests/053 | 4 +-
tests/qemu-iotests/058 | 7 ++--
tests/qemu-iotests/059 | 20 ++++-----
tests/qemu-iotests/060 | 12 +++---
tests/qemu-iotests/060.out | 20 ++++-----
tests/qemu-iotests/061 | 61 ++++++++++++++-------------
tests/qemu-iotests/061.out | 72 ++++++++++++++++----------------
tests/qemu-iotests/062 | 3 +-
tests/qemu-iotests/063 | 18 ++++----
tests/qemu-iotests/063.out | 3 +-
tests/qemu-iotests/066 | 3 +-
tests/qemu-iotests/067 | 6 ++-
tests/qemu-iotests/068 | 4 +-
tests/qemu-iotests/069 | 2 +-
tests/qemu-iotests/071 | 7 +++-
tests/qemu-iotests/073 | 2 +
tests/qemu-iotests/074 | 4 +-
tests/qemu-iotests/079 | 3 +-
tests/qemu-iotests/080 | 7 ++--
tests/qemu-iotests/081 | 6 +--
tests/qemu-iotests/085 | 18 ++++----
tests/qemu-iotests/085.out | 8 ++--
tests/qemu-iotests/088 | 2 +-
tests/qemu-iotests/090 | 2 +
tests/qemu-iotests/091 | 2 +-
tests/qemu-iotests/091.out | 2 -
tests/qemu-iotests/092 | 2 +-
tests/qemu-iotests/094 | 4 +-
tests/qemu-iotests/095 | 5 ++-
tests/qemu-iotests/098 | 6 ++-
tests/qemu-iotests/099 | 10 +++--
tests/qemu-iotests/103 | 5 ++-
tests/qemu-iotests/106 | 2 +-
tests/qemu-iotests/108 | 8 ++--
tests/qemu-iotests/109 | 4 +-
tests/qemu-iotests/110 | 11 +++--
tests/qemu-iotests/110.out | 4 +-
tests/qemu-iotests/111 | 3 +-
tests/qemu-iotests/112 | 37 ++++++++--------
tests/qemu-iotests/114 | 2 +
tests/qemu-iotests/115 | 3 +-
tests/qemu-iotests/121 | 9 ++--
tests/qemu-iotests/122 | 6 ++-
tests/qemu-iotests/123 | 4 +-
tests/qemu-iotests/125 | 2 +-
tests/qemu-iotests/137 | 17 +++++---
tests/qemu-iotests/137.out | 6 +--
tests/qemu-iotests/138 | 5 ++-
tests/qemu-iotests/141 | 4 +-
tests/qemu-iotests/142 | 2 +-
tests/qemu-iotests/144 | 4 +-
tests/qemu-iotests/153 | 12 ++----
tests/qemu-iotests/156 | 10 ++++-
tests/qemu-iotests/159 | 2 +-
tests/qemu-iotests/160 | 3 +-
tests/qemu-iotests/161 | 4 +-
tests/qemu-iotests/170 | 2 +-
tests/qemu-iotests/172 | 6 +--
tests/qemu-iotests/173 | 3 +-
tests/qemu-iotests/174 | 2 +-
tests/qemu-iotests/175 | 2 +-
tests/qemu-iotests/176 | 7 +++-
tests/qemu-iotests/178 | 6 +--
tests/qemu-iotests/182 | 2 +-
tests/qemu-iotests/183 | 2 +-
tests/qemu-iotests/185 | 4 +-
tests/qemu-iotests/187 | 6 +--
tests/qemu-iotests/190 | 4 +-
tests/qemu-iotests/191 | 11 ++---
tests/qemu-iotests/195 | 2 +-
tests/qemu-iotests/197 | 6 +--
tests/qemu-iotests/198 | 6 ++-
tests/qemu-iotests/198.out | 4 +-
tests/qemu-iotests/200 | 7 ++--
tests/qemu-iotests/201 | 6 +--
tests/qemu-iotests/214 | 3 +-
tests/qemu-iotests/215 | 6 +--
tests/qemu-iotests/217 | 3 +-
tests/qemu-iotests/220 | 5 ++-
tests/qemu-iotests/225 | 2 +-
tests/qemu-iotests/229 | 3 +-
tests/qemu-iotests/232 | 4 +-
tests/qemu-iotests/243 | 10 +++--
tests/qemu-iotests/244 | 15 ++++---
tests/qemu-iotests/247 | 4 +-
tests/qemu-iotests/249 | 4 +-
tests/qemu-iotests/250 | 5 ++-
tests/qemu-iotests/252 | 2 +-
tests/qemu-iotests/265 | 2 +-
tests/qemu-iotests/267 | 17 +++++---
tests/qemu-iotests/common.filter | 47 ++++++++++++++++++++-
tests/qemu-iotests/common.rc | 63 ++++++++++++++++++++++++----
tests/qemu-iotests/qcow2.py | 23 ++++++++--
114 files changed, 599 insertions(+), 451 deletions(-)
--
2.21.0
next reply other threads:[~2019-10-15 14:32 UTC|newest]
Thread overview: 48+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-10-15 14:27 Max Reitz [this message]
2019-10-15 14:27 ` [PATCH v2 01/21] iotests/qcow2.py: Add dump-header-exts Max Reitz
2019-11-06 15:37 ` Maxim Levitsky
2019-10-15 14:27 ` [PATCH v2 02/21] iotests/qcow2.py: Split feature fields into bits Max Reitz
2019-11-06 15:37 ` Maxim Levitsky
2019-10-15 14:27 ` [PATCH v2 03/21] iotests: Add _filter_json_filename Max Reitz
2019-11-06 15:44 ` Maxim Levitsky
2019-11-07 8:59 ` Max Reitz
2019-11-07 10:01 ` Maxim Levitsky
2019-10-15 14:27 ` [PATCH v2 04/21] iotests: Filter refcount_order in 036 Max Reitz
2019-11-06 15:45 ` Maxim Levitsky
2019-10-15 14:27 ` [PATCH v2 05/21] iotests: Replace IMGOPTS by _unsupported_imgopts Max Reitz
2019-11-06 15:45 ` Maxim Levitsky
2019-11-07 9:08 ` Max Reitz
2019-11-07 9:56 ` Maxim Levitsky
2019-10-15 14:27 ` [PATCH v2 06/21] iotests: Drop compat=1.1 in 050 Max Reitz
2019-10-15 14:27 ` [PATCH v2 07/21] iotests: Let _make_test_img parse its parameters Max Reitz
2019-10-15 14:27 ` [PATCH v2 08/21] iotests: Add -o and --no-opts to _make_test_img Max Reitz
2019-10-15 14:27 ` [PATCH v2 09/21] iotests: Inject space into -ocompat=0.10 in 051 Max Reitz
2019-10-15 14:27 ` [PATCH v2 10/21] iotests: Replace IMGOPTS= by -o Max Reitz
2019-11-06 15:47 ` Maxim Levitsky
2019-11-07 9:20 ` Max Reitz
2019-11-07 9:52 ` Maxim Levitsky
2019-10-15 14:27 ` [PATCH v2 11/21] iotests: Replace IMGOPTS='' by --no-opts Max Reitz
2019-10-15 14:27 ` [PATCH v2 12/21] iotests: Drop IMGOPTS use in 267 Max Reitz
2019-11-06 15:50 ` Maxim Levitsky
2019-10-15 14:27 ` [PATCH v2 13/21] iotests: Avoid qemu-img create Max Reitz
2019-10-15 14:27 ` [PATCH v2 14/21] iotests: Use _rm_test_img for deleting test images Max Reitz
2019-11-06 15:47 ` Maxim Levitsky
2019-10-15 14:27 ` [PATCH v2 15/21] iotests: Avoid cp/mv of " Max Reitz
2019-10-15 14:27 ` [PATCH v2 16/21] iotests: Make 091 work with data_file Max Reitz
2019-11-06 15:50 ` Maxim Levitsky
2019-10-15 14:27 ` [PATCH v2 17/21] iotests: Make 110 " Max Reitz
2019-11-06 15:50 ` Maxim Levitsky
2019-10-15 14:27 ` [PATCH v2 18/21] iotests: Make 137 " Max Reitz
2019-11-06 15:51 ` Maxim Levitsky
2019-10-15 14:27 ` [PATCH v2 19/21] iotests: Make 198 " Max Reitz
2019-11-06 15:51 ` Maxim Levitsky
2019-10-15 14:27 ` [PATCH v2 20/21] iotests: Disable data_file where it cannot be used Max Reitz
2019-11-06 15:52 ` Maxim Levitsky
2019-11-07 11:36 ` Max Reitz
2019-11-07 15:19 ` Maxim Levitsky
2019-11-07 16:55 ` Max Reitz
2019-10-15 14:27 ` [PATCH v2 21/21] iotests: Allow check -o data_file Max Reitz
2019-11-06 15:52 ` Maxim Levitsky
2019-10-16 0:19 ` [PATCH v2 00/21] iotests: Allow ./check " no-reply
2019-10-16 7:18 ` Max Reitz
2019-11-06 15:52 ` Maxim Levitsky
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=20191015142729.18123-1-mreitz@redhat.com \
--to=mreitz@redhat.com \
--cc=kwolf@redhat.com \
--cc=mlevitsk@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).