mbox series

[v2,0/2] Assorted fixes to tests that were broken by recent scsi changes

Message ID 20201101161532.259609-1-mlevitsk@redhat.com (mailing list archive)
Headers show
Series Assorted fixes to tests that were broken by recent scsi changes | expand

Message

Maxim Levitsky Nov. 1, 2020, 4:15 p.m. UTC 
While most of the patches in V1 of this series are already merged upstream,
the patch that fixes iotest 240 was broken on s390 and was not accepted.

This is	an updated version of this patch, based on Paulo's suggestion,
that hopefully makes this iotest work on both x86 and s390.

Best regards,
	Maxim Levitsky

Maxim Levitsky (2):
  iotests: add filter_qmp_virtio_scsi function
  iotests: rewrite iotest 240 in python

 tests/qemu-iotests/240        | 228 +++++++++++++++-------------------
 tests/qemu-iotests/240.out    |  76 +++++++-----
 tests/qemu-iotests/iotests.py |  10 ++
 3 files changed, 153 insertions(+), 161 deletions(-)

Comments

Paolo Bonzini Nov. 2, 2020, 8:33 a.m. UTC | #1 
On 01/11/20 17:15, Maxim Levitsky wrote:
> While most of the patches in V1 of this series are already merged upstream,
> the patch that fixes iotest 240 was broken on s390 and was not accepted.
> 
> This is	an updated version of this patch, based on Paulo's suggestion,
> that hopefully makes this iotest work on both x86 and s390.
> 
> Best regards,
> 	Maxim Levitsky
> 
> Maxim Levitsky (2):
>   iotests: add filter_qmp_virtio_scsi function
>   iotests: rewrite iotest 240 in python
> 
>  tests/qemu-iotests/240        | 228 +++++++++++++++-------------------
>  tests/qemu-iotests/240.out    |  76 +++++++-----
>  tests/qemu-iotests/iotests.py |  10 ++
>  3 files changed, 153 insertions(+), 161 deletions(-)
> 

Reviewed-by: Paolo Bonzini <pbonzini@redhat.com>
Christian Borntraeger Nov. 3, 2020, 11:57 a.m. UTC | #2 
On 01.11.20 17:15, Maxim Levitsky wrote:
> While most of the patches in V1 of this series are already merged upstream,
> the patch that fixes iotest 240 was broken on s390 and was not accepted.
> 
> This is	an updated version of this patch, based on Paulo's suggestion,
> that hopefully makes this iotest work on both x86 and s390.
> 
> Best regards,
> 	Maxim Levitsky
> 
> Maxim Levitsky (2):
>   iotests: add filter_qmp_virtio_scsi function
>   iotests: rewrite iotest 240 in python

Both patches on s390x
Tested-by: Christian Borntraeger <borntraeger@de.ibm.com>


Thanks.

> 
>  tests/qemu-iotests/240        | 228 +++++++++++++++-------------------
>  tests/qemu-iotests/240.out    |  76 +++++++-----
>  tests/qemu-iotests/iotests.py |  10 ++
>  3 files changed, 153 insertions(+), 161 deletions(-)
>