[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH RFC v6 5/5] qemu-iotests: s390x: fix test 051
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH RFC v6 5/5] qemu-iotests: s390x: fix test 051 |
Date: |
Wed, 11 Feb 2015 17:23:54 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.3 (gnu/linux) |
Max Reitz <address@hidden> writes:
> On 2015-02-11 at 05:37, Xiao Guang Chen wrote:
>> The tests for device type "ide_cd" should only be tested for the pc platform.
>> The default device id of hard disk on the s390 platform differs to that
>> of the x86 platform. A new variable device_id is defined and "virtio0"
>> set for the s390 platform. A x86 platform specific output file is also
>> needed.
>> A new filter was added to filter orphan warnings.
>>
>> Reviewed-by: Max Reitz <address@hidden>
>> Reviewed-by: Michael Mueller <address@hidden>
>> Signed-off-by: Xiao Guang Chen <address@hidden>
>> ---
>> tests/qemu-iotests/051 | 79 +++++---
>> tests/qemu-iotests/051.out | 175 ++++++----------
>> tests/qemu-iotests/051.pc.out | 427
>> +++++++++++++++++++++++++++++++++++++++
>> tests/qemu-iotests/common.filter | 7 +
>> 4 files changed, 547 insertions(+), 141 deletions(-)
>> create mode 100644 tests/qemu-iotests/051.pc.out
>
> [snip]
>
>> +# removes orphan warnings
>> +_filter_orphan()
>> +{
>> + sed -e 's/Warning: Orphaned drive without device:.*$//g'
>
> Well, the problem with this is that it does not remove the newline
> introduced by the warning, so it's still a different output then if
> the warning had not been emitted at all.
If you want to delete the line, use something like
sed -e '/Warning: Orphaned drive without device:/d'
Yes, sed has actually commands other than 's' ;)
> But as I said in my review for v5, as long as it works on x86 and
> s390, I'm fine with it, so my R-b stands.
>
> Max
>
>> +}
>> +
>> # replace occurrences of QEMU_PROG with "qemu"
>> _filter_qemu()
>> {
>> + _filter_orphan | \
>> sed -e "s#\\(^\\|(qemu) \\)$(basename $QEMU_PROG):#\1QEMU_PROG:#" \
>> -e 's#^QEMU [0-9]\+\.[0-9]\+\.[0-9]\+ monitor#QEMU X.Y.Z monitor#'
>> \
>> -e '/main-loop: WARNING: I\/O thread spun for [0-9]\+
>> iterations/d' \
- [Qemu-devel] [PATCH RFC v6 0/5] Update tests/qemu-iotests failing cases for the s390 platform, Xiao Guang Chen, 2015/02/11
- [Qemu-devel] [PATCH RFC v6 1/5] qemu-iotests: qemu machine type support, Xiao Guang Chen, 2015/02/11
- [Qemu-devel] [PATCH RFC v6 3/5] qemu-iotests: s390x: fix test 041, Xiao Guang Chen, 2015/02/11
- [Qemu-devel] [PATCH RFC v6 4/5] qemu-iotests: s390x: fix test 055, Xiao Guang Chen, 2015/02/11
- [Qemu-devel] [PATCH RFC v6 2/5] qemu-iotests: run qemu with -nodefaults and fix 067, 071, 081 and 087, Xiao Guang Chen, 2015/02/11
- [Qemu-devel] [PATCH RFC v6 5/5] qemu-iotests: s390x: fix test 051, Xiao Guang Chen, 2015/02/11