From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:41096) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZXD9F-0006kz-7f for qemu-devel@nongnu.org; Wed, 02 Sep 2015 14:52:42 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ZXD9E-0006n1-BX for qemu-devel@nongnu.org; Wed, 02 Sep 2015 14:52:41 -0400 From: Max Reitz Date: Wed, 2 Sep 2015 20:52:24 +0200 Message-Id: <1441219948-13242-1-git-send-email-mreitz@redhat.com> Subject: [Qemu-devel] [PATCH v2 0/4] iotests: Emit signal-kill messages List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-block@nongnu.org Cc: Kevin Wolf , Jeff Cody , qemu-devel@nongnu.org, Max Reitz Currently, if a qemu-related command (qemu, qemu-io, qemu-img, qemu-nbd) is invoked in an iotest, receives a signal and is subsequently killed (e.g. a segmentation fault), this is not logged in the test output. The first patch in this series makes the bash tests no longer suppress that line, and the second patch adds a similar notification for the python tests. Patch 3 tries to fix some handling of spaces in command filenames, and apparently actually succeeds in doing so (all Python tests work, most bash tests work; if they fail, it's the test's fault). However, it does not fix handling of spaces in arguments: This is because we probably don't have to worry about that anyway, and because it would be pretty difficult to fix. v2: - Patch 1: Added, pulled in from v4 of my "blockdev: BlockBackend and media" series (needed in patch 2) - Patch 2: Added, required because patch 3 now fixes argument handling for the Python iotests - Patch 3: - Fix handling of spaces in command filenames [Jeff] - Fix argument handling for Python iotests (accidental fallout from the above) git-backport-diff against v1: Key: [----] : patches are identical [####] : number of functional differences between upstream/downstream patch [down] : patch is downstream-only The flags [FC] indicate (F)unctional and (C)ontextual differences, respectively 001/4:[down] 'iotests: More options for VM.add_drive()' 002/4:[down] 'iotests: Respect -nodefaults in tests 41 and 55' 003/4:[0047] [FC] 'iotests: Do not suppress segfaults in bash tests' 004/4:[----] [--] 'iotests: Warn if python subprocess is killed' Max Reitz (4): iotests: More options for VM.add_drive() iotests: Respect -nodefaults in tests 41 and 55 iotests: Do not suppress segfaults in bash tests iotests: Warn if python subprocess is killed tests/qemu-iotests/039 | 19 +++++---------- tests/qemu-iotests/039.out | 6 ++--- tests/qemu-iotests/041 | 12 +++++++--- tests/qemu-iotests/055 | 10 +++++--- tests/qemu-iotests/061 | 6 +++-- tests/qemu-iotests/061.out | 2 ++ tests/qemu-iotests/check | 8 +++---- tests/qemu-iotests/common.config | 29 +++++++++++++++++++---- tests/qemu-iotests/common.rc | 12 +++++++++- tests/qemu-iotests/iotests.py | 51 ++++++++++++++++++++++++++++++---------- 10 files changed, 110 insertions(+), 45 deletions(-) -- 2.5.1