Max replied that ignoring the UTF-8 error for 130 patch reported by
checkpatch.pl is fine.
Could you please apply the series if there are no further objections.
thanks
On 07/03/2015 03:28 PM, Bo Tu wrote:
v11.
1. Add Reviewed-by of Sascha
2. Refine code change in common.config in order to be easier to read and be
shorter
3. Add more comments in patch description
4. Combine the fix for 041 and 055 in one patch since they address the same
issue
5. Remove the fix for 051 since it fails now
6. checkpatch.pl reports invaid UTF-8 error for 130 patch, because 130.out
contains
some non-text data.
v10.
1. Add Reviewed-by statements for test 049
2. Removed the backslash in qemu-option.c
3. Please apply the series if there are no further objections
v9.
1.Fix issue of line over 80 characters for test 049
2.Add Reviewed-by statements for test 051,130
v8.
1.Modify error message in qemu-option.c when image size is invalid
2.Remove Reviewed-by statements if any functional changes in a new patch version
for test 049,051,130
3.Change patch subject for test 130
4.Add id definition for a drive which will work for all platforms in test 130
5.Disable virtio-scsi-pci for non-PCI systems in test 051
v7.
1. Add a pc specific output file for test 130.
2. A new variable device_id is defined in test 130 to support multiplatform.
3. Update the output file for test 051 based on it's current output.
4. change util/qemu-option.c and test case 049, generate error message
when image size is a negtive value or exceeds the maximum of uint64
v6.
1. Change the filter name from _filter_s390 to _filter_orphan.
2. Update the output file for tese case 081 because no default floopy and
cd-rom.
v5:
1. Add a pc specific output file for test 051.
2. Add a filter to test case 051 to filter s390 specific warnings.
3. Check whether the machine type is pc or not rather than check whether the
machine type
is s390.
4. When using a machine specific reference file if the default machine has an
alias then
use the alias as the output file name otherwise use the default machine name as
the output
file name.
v4:
1. Generate all patches based on the latest master branch.
2. Rearrange patches
v3:
1. Fix a typo in v2.
v2:
1. Drop the patches for test 039 for it has been fixed in upstream.
2. Integrate patches for test 071, 067 and 087.
3. Keep the other patches.
v1:
1. updated the test suite to be default-machine-type-aware, from the previous
platform-aware
2. created a new patch "qemu-iotests: run qemu with -nodefaults" to counterpart
the impact from the commit:
c88930a6866e74953e931ae749781e98e486e5c8
qemu-char: Permit only a single "stdio" character device
When more than one is used, the terminal settings aren't restored
correctly on exit. Fixable. However, such usage makes no sense,
because the users race for input, so outlaw it instead.
If you want to connect multiple things to stdio, use the mux
chardev.
3. updated all the checking of platform name to the current machine name
Bo Tu (5):
qemu-iotests: qemu machine type support
qemu-iotests: disable default qemu devices for cross-platform
compatibility
qemu-iotests: s390x: fix test 041 and 055
qemu-iotests: s390x: fix test 049, reject negative sizes in QemuOpts
qemu-iotests: s390x: fix test 130
tests/qemu-iotests/041 | 6 +
tests/qemu-iotests/049.out | 10 +-
tests/qemu-iotests/055 | 9 ++
tests/qemu-iotests/067 | 8 +-
tests/qemu-iotests/067.out | 266 +--------------------------------------
tests/qemu-iotests/071.out | 4 -
tests/qemu-iotests/081.out | 2 -
tests/qemu-iotests/087.out | 12 --
tests/qemu-iotests/130 | 8 +-
tests/qemu-iotests/130.out | 4 +-
tests/qemu-iotests/check | 5 +
tests/qemu-iotests/common | 1 +
tests/qemu-iotests/common.config | 11 +-
tests/qemu-iotests/common.qemu | 2 +-
tests/qemu-iotests/iotests.py | 1 +
util/qemu-option.c | 5 +
16 files changed, 53 insertions(+), 301 deletions(-)