On 03/25/2015 08:55 AM, Stefan Hajnoczi wrote:
On Fri, Mar 13, 2015 at 03:22:01PM -0400, 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.
v2:
- Patchew caught a pathing problem with the qemu-img binary;
the relative path produced by the Makefile does not prepend
"./", so I was relying on the /distro's/ qemu-img by accident.
Fix that by using realpath().
v1:
- 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 | 44 +++++++++++++++++++++++++++
tests/libqos/libqos.h | 2 ++
6 files changed, 116 insertions(+), 29 deletions(-)
Acked-by: Stefan Hajnoczi <stefa...@redhat.com>
As in traditional Linux Acked-by. I've looked briefly at the patches
and am happy.
Great, thanks.
Staged for 2.4.
