From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:41692) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1XwXiU-0002lI-6w for qemu-devel@nongnu.org; Thu, 04 Dec 2014 09:49:20 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1XwXiO-0002Pf-1T for qemu-devel@nongnu.org; Thu, 04 Dec 2014 09:49:14 -0500 Received: from mx1.redhat.com ([209.132.183.28]:34108) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1XwXiN-0002PZ-ON for qemu-devel@nongnu.org; Thu, 04 Dec 2014 09:49:07 -0500 From: Max Reitz Date: Thu, 4 Dec 2014 15:48:59 +0100 Message-Id: <1417704542-18337-1-git-send-email-mreitz@redhat.com> Subject: [Qemu-devel] [PATCH v2 0/3] iotests: Fix test 039 List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: Kevin Wolf , Max Reitz , Markus Armbruster , Stefan Hajnoczi , =?UTF-8?q?Michael=20M=C3=BCller?= Test 039 used to fail because qemu-io -c abort may generate core dumps even with ulimit -c 0 (and the output then contains "(core dumped)"). Fix this by adding a new qemu-io command "sigraise" which invokes raise(). Using this command to raise SIGKILL for example does not result in a core dump, but it still badly crashes qemu-io (as desired). I am sending this series because we need all tests to work before adding the check-block target to "make check" (which we will hopefully do soon). v2: - Rewrote patch 1: Overloading "abort" may be somehow technically justifyable, but there is no point in grasping at straws when it is much easier to just introduce a new command. Therefore, in this version, "abort" is no longer overloaded and instead a new command, "sigraise", is added (I thought long and hard about that name; I thought "raise" to sound too unspecific, so this is what I came up with) [Markus] - Patch 2: Fixed commit message (s/if it is aborted/when it is killed/) [Markus] - Patch 3: %s/abort -S 9/sigraise 9/ 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/3:[down] 'qemu-io: Add sigraise command' ^^^^ wrong, should be: [0081] [FC] 002/3:[----] [--] 'iotests: Filter for "Killed" in qemu-io output' 003/3:[0006] [FC] 'iotests: Fix test 039' Max Reitz (3): qemu-io: Add sigraise command iotests: Filter for "Killed" in qemu-io output iotests: Fix test 039 qemu-io-cmds.c | 46 ++++++++++++++++++++++++++++++++++++++++ tests/qemu-iotests/039 | 12 ++++++----- tests/qemu-iotests/039.out | 6 +++--- tests/qemu-iotests/common.filter | 2 +- 4 files changed, 57 insertions(+), 9 deletions(-) -- 1.9.3