From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from eggs.gnu.org ([2001:4830:134:3::10]:44863)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <mreitz@redhat.com>) id 1ZtLlF-00045H-6r
	for qemu-devel@nongnu.org; Mon, 02 Nov 2015 15:31:26 -0500
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <mreitz@redhat.com>) id 1ZtLlE-0002lL-6Y
	for qemu-devel@nongnu.org; Mon, 02 Nov 2015 15:31:25 -0500
References: <cover.1446232490.git.jcody@redhat.com>
From: Max Reitz <mreitz@redhat.com>
Message-ID: <5637C813.2050902@redhat.com>
Date: Mon, 2 Nov 2015 21:31:15 +0100
MIME-Version: 1.0
In-Reply-To: <cover.1446232490.git.jcody@redhat.com>
Content-Type: multipart/signed; micalg=pgp-sha256;
	protocol="application/pgp-signature";
	boundary="73F2T0Jdi7eNxQKnKILG1HNKBG2vK2EJE"
Subject: Re: [Qemu-devel] [PATCH v3 0/2] qemu-iotests: fix cleanup of
	background processes
List-Id: <qemu-devel.nongnu.org>
List-Unsubscribe: <https://lists.nongnu.org/mailman/options/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.nongnu.org/archive/html/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <https://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: Jeff Cody <jcody@redhat.com>, qemu-devel@nongnu.org
Cc: kwolf@redhat.com, jsnow@redhat.com, qemu-block@nongnu.org

This is an OpenPGP/MIME signed message (RFC 4880 and 3156)
--73F2T0Jdi7eNxQKnKILG1HNKBG2vK2EJE
Content-Type: text/plain; charset=iso-8859-15
Content-Transfer-Encoding: quoted-printable

On 30.10.2015 20:25, Jeff Cody wrote:
> Changes from v2:
>=20
> * Pulled patch 2 into this series
>=20
> * Patch 1: Moved non-empty test conditionals inside
>            pid file existance check (thanks Max)
>=20
>            Added a fix in for patch 058, for its
>            self-launched qemu-nbd instance (thanks Max)
>=20
> * Patch 2: Only print the valgrind logs if the exit error
>            matches (thanks Max)=20
>=20
> Changes from v1:
>=20
> * use 'read' instead of 'cat' (thanks Eric)
> * quote variable in variable test (thanks Eric)
>=20
> Jeff Cody (2):
>   qemu-iotests: fix cleanup of background processes
>   qemu-iotests: fix -valgrind option for check
>=20
>  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


--73F2T0Jdi7eNxQKnKILG1HNKBG2vK2EJE
Content-Type: application/pgp-signature; name="signature.asc"
Content-Description: OpenPGP digital signature
Content-Disposition: attachment; filename="signature.asc"

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2

iQEcBAEBCAAGBQJWN8gTAAoJEDuxQgLoOKytOegH/2/FrCTWBAvNY5JD4JHDtRyS
U5YbIOXsWEhPoc0lZF5xaB/qelrFpHAEvnPF/IE9JXMRVFVETGdw/CH25Ydriev+
RSbplennXW2wKtLJoMKiBzwn+b9on5ApmELcbu+cOwIjm8uiqaPJBkmxI5pYw19R
DS1kmt+G7+ZHe3e0AJ88IDIvp039pHfQh4nUA0x5MLHfgHV2RNyww7RoZA+zAREu
PVcHIrZAWIYrbevOUiZULz5XIkSnoGIXFmGBT+WIxR/acrw/3qsEtUkNhwHlqJKk
gHPfAbhHfnxBlTIp00tuQ+BVp09/qwbEPry1N4JUPm2gshJNoY0wEgFLWXZHuqo=
=DuvN
-----END PGP SIGNATURE-----

--73F2T0Jdi7eNxQKnKILG1HNKBG2vK2EJE--