[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [PATCH v3 0/2] qemu-iotests: fix cleanup of background
From: |
Max Reitz |
Subject: |
Re: [Qemu-block] [PATCH v3 0/2] qemu-iotests: fix cleanup of background processes |
Date: |
Mon, 2 Nov 2015 21:31:15 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.3.0 |
On 30.10.2015 20:25, Jeff Cody wrote:
> Changes from v2:
>
> * Pulled patch 2 into this series
>
> * Patch 1: Moved non-empty test conditionals inside
> pid file existance check (thanks Max)
>
> Added a fix in for patch 058, for its
> self-launched qemu-nbd instance (thanks Max)
>
> * Patch 2: Only print the valgrind logs if the exit error
> matches (thanks Max)
>
> Changes from v1:
>
> * use 'read' instead of 'cat' (thanks Eric)
> * quote variable in variable test (thanks Eric)
>
> Jeff Cody (2):
> qemu-iotests: fix cleanup of background processes
> qemu-iotests: fix -valgrind option for check
>
> tests/qemu-iotests/039.out | 30 +++++++++++++++++++++++++-----
> tests/qemu-iotests/058 | 12 ++++++++----
> tests/qemu-iotests/061.out | 12 ++++++++++--
> tests/qemu-iotests/137.out | 6 +++++-
> tests/qemu-iotests/common | 9 ++-------
> tests/qemu-iotests/common.config | 32 +++++++++++++++++++++++++++++---
> tests/qemu-iotests/common.qemu | 18 ++++++++++++------
> tests/qemu-iotests/common.rc | 18 +++++-------------
> 8 files changed, 96 insertions(+), 41 deletions(-)
Thanks, applied to my block tree with the three occurrences of ! -n in
patch 1 changed to -z:
https://github.com/XanClic/qemu/commits/block
Max
signature.asc
Description: OpenPGP digital signature
- Re: [Qemu-block] [PATCH v3 0/2] qemu-iotests: fix cleanup of background processes,
Max Reitz <=