On 04.12.2018 14:49, Christian Borntraeger wrote:
>
>
> On 04.12.2018 14:46, Christian Borntraeger wrote:
>> FWIW, this testcase fails with current qemu master on s390:
>>
>> QEMU --
>> "/home/cborntra/REPOS/qemu/build/tests/qemu-iotests/../../s390x-softmmu/qemu-system-s390x"
>> -nodefaults -machine accel=qtest
>> QEMU_IMG --
>> "/home/cborntra/REPOS/qemu/build/tests/qemu-iotests/../../qemu-img"
>> QEMU_IO --
>> "/home/cborntra/REPOS/qemu/build/tests/qemu-iotests/../../qemu-io" --cache
>> writeback -f qcow2
>> QEMU_NBD --
>> "/home/cborntra/REPOS/qemu/build/tests/qemu-iotests/../../qemu-nbd"
>> IMGFMT -- qcow2 (compat=1.1)
>> IMGPROTO -- file
>> PLATFORM -- Linux/s390x s38lp08 4.19.0+
>> TEST_DIR -- /home/cborntra/REPOS/qemu/build/tests/qemu-iotests/scratch
>> SOCKET_SCM_HELPER --
>> /home/cborntra/REPOS/qemu/build/tests/qemu-iotests/socket_scm_helper
>> 235
>> [failed, exit status 1] - output mismatch (see 235.out.bad)
>> --- /home/cborntra/REPOS/qemu/tests/qemu-iotests/235.out 2018-12-04
>> 14:44:27.913714608 +0100
>> +++ /home/cborntra/REPOS/qemu/build/tests/qemu-iotests/235.out.bad
>> 2018-12-04 14:44:51.512958864 +0100
>> @@ -1,3 +1,14 @@
>> -{"return": {}}
>> -{"return": {}}
>> -{"return": {}}
>> +Traceback (most recent call last):
>> + File "235", line 54, in <module>
>> + vm.launch()
>> + File
>> "/home/cborntra/REPOS/qemu/tests/qemu-iotests/../../scripts/qemu.py", line
>> 295, in launch
>> + self._launch()
>> + File
>> "/home/cborntra/REPOS/qemu/tests/qemu-iotests/../../scripts/qemu.py", line
>> 321, in _launch
>> + self._post_launch()
>> + File
>> "/home/cborntra/REPOS/qemu/tests/qemu-iotests/../../scripts/qemu.py", line
>> 266, in _post_launch
>> + self._qmp.accept()
>> + File
>> "/home/cborntra/REPOS/qemu/tests/qemu-iotests/../../scripts/qmp/qmp.py",
>> line 155, in accept
>> + self.__sock, _ = self.__sock.accept()
>> + File "/usr/lib64/python2.7/socket.py", line 206, in accept
>> + sock, addr = self._sock.accept()
>> +socket.timeout: timed out
>> On 03.12.2018 17:58, Kevin Wolf wrote:
>>> From: Vladimir Sementsov-Ogievskiy <vsement...@virtuozzo.com>
>>>
>>> This test is broken without previous commit fixing dead-lock in mirror.
>>>
>>> Signed-off-by: Vladimir Sementsov-Ogievskiy <vsement...@virtuozzo.com>
>>> Signed-off-by: Max Reitz <mre...@redhat.com>
>>> Acked-by: Vladimir Sementsov-Ogievskiy <vsement...@virtuozzo.com>
>>> Signed-off-by: Kevin Wolf <kw...@redhat.com>
>>> ---
>>> tests/qemu-iotests/235 | 76 ++++++++++++++++++++++++++++++++++++++
>>> tests/qemu-iotests/235.out | 3 ++
>>> tests/qemu-iotests/group | 1 +
>>> 3 files changed, 80 insertions(+)
>>> create mode 100755 tests/qemu-iotests/235
>>> create mode 100644 tests/qemu-iotests/235.out
>>>
>>> diff --git a/tests/qemu-iotests/235 b/tests/qemu-iotests/235
>>> new file mode 100755
>>> index 0000000000..da044ed34e
>>> --- /dev/null
>>> +++ b/tests/qemu-iotests/235
[...]
>>> +# prepare source image
>>> +qemu_img_create('-f', iotests.imgfmt, '-o', 'preallocation=metadata', disk,
>>> + str(size))
>>> +
>>> +vm = QEMUMachine(iotests.qemu_prog)
>>> +vm.add_args('-machine', 'pc,accel=kvm')
This (pc) clearly does not work on other architectures.
In addition to that, I also need to add -no-shutdown on s390 (see 068 for a
similar case)
This hack makes it work for me.
diff --git a/tests/qemu-iotests/235 b/tests/qemu-iotests/235
index da044ed34e..05aa641a74 100755
--- a/tests/qemu-iotests/235
+++ b/tests/qemu-iotests/235
@@ -49,7 +49,8 @@ qemu_img_create('-f', iotests.imgfmt, '-o',
'preallocation=metadata', disk,
str(size))
vm = QEMUMachine(iotests.qemu_prog)
-vm.add_args('-machine', 'pc,accel=kvm')
+vm.add_args('-machine', 'accel=kvm')
+vm.add_args('-no-shutdown')
vm.add_args('-drive', 'id=src,file=' + disk)
vm.launch()
