From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:40539) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1YW6qL-0007Pt-Ab for qemu-devel@nongnu.org; Thu, 12 Mar 2015 13:24:22 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1YW6qH-0007IV-Uq for qemu-devel@nongnu.org; Thu, 12 Mar 2015 13:24:21 -0400 Message-ID: <5501CBBF.6010704@redhat.com> Date: Thu, 12 Mar 2015 13:24:15 -0400 From: John Snow MIME-Version: 1.0 References: <1426008582-19022-1-git-send-email-jsnow@redhat.com> In-Reply-To: <1426008582-19022-1-git-send-email-jsnow@redhat.com> Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH 0/2] ahci: test varying sector offsets List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-block@nongnu.org Cc: kwolf@redhat.com, qemu-devel@nongnu.org, stefanha@redhat.com NACK. Patchew caught a problem with how I was generating the QEMU_IMG path. Not important for 2.3, anyway. Will re-spin with a 2.4 subject. --js On 03/10/2015 01:29 PM, John Snow wrote: > This is a re-send of patches 7 & 8 from an earlier series, > "[PATCH v2 0/8] ahci: add more IO tests" which ultimately got bounced > back because I used some glib functions that were too new. > > This is effectively v3 of these two patches: > > - Removed "./" from the execution CLI. Now you can set an absolute or > relative path for QTEST_QEMU_IMG and it will work either way. The default > as generated by the Makefile will be a relative path. > > - Removed the g_spawn_check_exit_status glib call from mkimg(). See the > in-line comments in patch 1/2 for correctness justification. > > John Snow (2): > qtest/ahci: add qcow2 support to ahci-test > qtest/ahci: test different disk sectors > > tests/Makefile | 1 + > tests/ahci-test.c | 84 +++++++++++++++++++++++++++++++++++++-------------- > tests/libqos/ahci.c | 10 +++--- > tests/libqos/ahci.h | 4 +-- > tests/libqos/libqos.c | 41 +++++++++++++++++++++++++ > tests/libqos/libqos.h | 2 ++ > 6 files changed, 113 insertions(+), 29 deletions(-) >