[Qemu-devel] [PATCH v11 0/5] Update tests/qemu-iotests failing cases for the s390 platform

[Bo Tu]

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(-)

-- 
2.3.0