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

Maxim Levitsky posted 2 patches 3 years, 6 months ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20201101161532.259609-1-mlevitsk@redhat.com
Test checkpatch passed
Maintainers: Kevin Wolf <kwolf@redhat.com>, Max Reitz <mreitz@redhat.com>
There is a newer version of this series
tests/qemu-iotests/240        | 228 +++++++++++++++-------------------
tests/qemu-iotests/240.out    |  76 +++++++-----
tests/qemu-iotests/iotests.py |  10 ++
3 files changed, 153 insertions(+), 161 deletions(-)
[PATCH v2 0/2] Assorted fixes to tests that were broken by recent scsi changes
Posted by Maxim Levitsky 3 years, 6 months ago
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(-)

-- 
2.26.2



Re: [PATCH v2 0/2] Assorted fixes to tests that were broken by recent scsi changes
Posted by Paolo Bonzini 3 years, 6 months ago
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>


Re: [PATCH v2 0/2] Assorted fixes to tests that were broken by recent scsi changes
Posted by Christian Borntraeger 3 years, 6 months ago

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