* [Qemu-devel] [PATCH v4 0/2] qemu-iotests: Add dependency to qemu-block tools
@ 2019-03-07 15:10 Philippe Mathieu-Daudé
2019-03-07 15:10 ` [Qemu-devel] [PATCH v4 1/2] ahci-test: Add dependency to qemu-img tool Philippe Mathieu-Daudé
` (2 more replies)
0 siblings, 3 replies; 5+ messages in thread
From: Philippe Mathieu-Daudé @ 2019-03-07 15:10 UTC (permalink / raw)
To: qemu-block, qemu-devel
Cc: Max Reitz, John Snow, Eric Blake, Kevin Wolf,
Philippe Mathieu-Daudé
Hi Kevin, Max, John.
Two trivial fixes I encountered while trying to run QEMU test suite on
a distribution which doesn't has the qemu-tools distrib-wide installed,
see https://lists.gnu.org/archive/html/qemu-devel/2018-06/msg08415.html:
$ make vm-build-ubuntu.i386
Traceback (most recent call last):
File "source/qemu/tests/vm/basevm.py", line 236, in main
return vm.build_image(args.image)
File "tests/vm/ubuntu.i386", line 67, in build_image
subprocess.check_call(["qemu-img", "resize", img_tmp, "50G"])
OSError: [Errno 2] No such file or directory
tests/vm/Makefile.include:23: recipe for target 'tests/vm/ubuntu.i386.img' failed
make: *** [tests/vm/ubuntu.i386.img] Error 2
All patches are reviewed.
Since v3:
- dropped RFC patches
- split generic tests/ patches (sent as another series)
Since v2:
- Added R-b tags
- Addressed Eric's comments (in code and commit descriptions)
Since v1:
- various qemu-block fixes and notes.
Regards,
Phil.
v3: https://lists.gnu.org/archive/html/qemu-devel/2019-02/msg00952.html
v2: https://lists.gnu.org/archive/html/qemu-devel/2019-01/msg07513.html
v1: https://lists.gnu.org/archive/html/qemu-devel/2019-01/msg06676.html
Philippe Mathieu-Daudé (2):
ahci-test: Add dependency to qemu-img tool
qemu-iotests: Add dependency to qemu-nbd tool
tests/Makefile.include | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
--
2.20.1
^ permalink raw reply [flat|nested] 5+ messages in thread
* [Qemu-devel] [PATCH v4 1/2] ahci-test: Add dependency to qemu-img tool
2019-03-07 15:10 [Qemu-devel] [PATCH v4 0/2] qemu-iotests: Add dependency to qemu-block tools Philippe Mathieu-Daudé
@ 2019-03-07 15:10 ` Philippe Mathieu-Daudé
2019-03-07 15:10 ` [Qemu-devel] [PATCH v4 2/2] qemu-iotests: Add dependency to qemu-nbd tool Philippe Mathieu-Daudé
2019-03-07 16:55 ` [Qemu-devel] [PATCH v4 0/2] qemu-iotests: Add dependency to qemu-block tools Kevin Wolf
2 siblings, 0 replies; 5+ messages in thread
From: Philippe Mathieu-Daudé @ 2019-03-07 15:10 UTC (permalink / raw)
To: qemu-block, qemu-devel
Cc: Max Reitz, John Snow, Eric Blake, Kevin Wolf,
Philippe Mathieu-Daudé
Since the ahci-test uses qemu-img, add a dependency to build it
before using it.
This fixes:
$ gmake check-qtest V=1
QTEST_QEMU_BINARY=x86_64-softmmu/qemu-system-x86_64 QTEST_QEMU_IMG=qemu-img tests/ahci-test
Failed to execute child process "/tmp/qemu-test.19tMRF/qemu-img" (No such file or directory)
ERROR:tests/libqos/libqos.c:192:mkimg: assertion failed: (ret && !err)
Reviewed-by: John Snow <jsnow@redhat.com>
Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
---
tests/Makefile.include | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tests/Makefile.include b/tests/Makefile.include
index 97e1cb90a3..b6bd0a5ed8 100644
--- a/tests/Makefile.include
+++ b/tests/Makefile.include
@@ -758,7 +758,7 @@ tests/prom-env-test$(EXESUF): tests/prom-env-test.o $(libqos-obj-y)
tests/rtas-test$(EXESUF): tests/rtas-test.o $(libqos-spapr-obj-y)
tests/fdc-test$(EXESUF): tests/fdc-test.o
tests/ide-test$(EXESUF): tests/ide-test.o $(libqos-pc-obj-y)
-tests/ahci-test$(EXESUF): tests/ahci-test.o $(libqos-pc-obj-y)
+tests/ahci-test$(EXESUF): tests/ahci-test.o $(libqos-pc-obj-y) qemu-img$(EXESUF)
tests/ipmi-kcs-test$(EXESUF): tests/ipmi-kcs-test.o
tests/ipmi-bt-test$(EXESUF): tests/ipmi-bt-test.o
tests/hd-geo-test$(EXESUF): tests/hd-geo-test.o
--
2.20.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [Qemu-devel] [PATCH v4 2/2] qemu-iotests: Add dependency to qemu-nbd tool
2019-03-07 15:10 [Qemu-devel] [PATCH v4 0/2] qemu-iotests: Add dependency to qemu-block tools Philippe Mathieu-Daudé
2019-03-07 15:10 ` [Qemu-devel] [PATCH v4 1/2] ahci-test: Add dependency to qemu-img tool Philippe Mathieu-Daudé
@ 2019-03-07 15:10 ` Philippe Mathieu-Daudé
2019-03-07 16:17 ` Eric Blake
2019-03-07 16:55 ` [Qemu-devel] [PATCH v4 0/2] qemu-iotests: Add dependency to qemu-block tools Kevin Wolf
2 siblings, 1 reply; 5+ messages in thread
From: Philippe Mathieu-Daudé @ 2019-03-07 15:10 UTC (permalink / raw)
To: qemu-block, qemu-devel
Cc: Max Reitz, John Snow, Eric Blake, Kevin Wolf,
Philippe Mathieu-Daudé
Since a9660664fde, some iotests use qemu-nbd.
Add a dependency to build it before using it.
This fixes:
$ make check-block
GEN qemu-img-cmds.h
CC qemu-img.o
LINK qemu-img
CC qemu-io.o
LINK qemu-io
CC tests/qemu-iotests/socket_scm_helper.o
LINK tests/qemu-iotests/socket_scm_helper
tests/qemu-iotests-quick.sh
check: qemu-nbd not found
make: *** [tests/Makefile.include:1059: check-tests/qemu-iotests-quick.sh] Error 1
Reviewed-by: Eric Blake <eblake@redhat.com>
Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
---
tests/Makefile.include | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tests/Makefile.include b/tests/Makefile.include
index b6bd0a5ed8..45a4302103 100644
--- a/tests/Makefile.include
+++ b/tests/Makefile.include
@@ -1104,7 +1104,7 @@ clean-tcg: $(CLEAN_TCG_TARGET_RULES)
QEMU_IOTESTS_HELPERS-$(call land,$(CONFIG_SOFTMMU),$(CONFIG_LINUX)) = tests/qemu-iotests/socket_scm_helper$(EXESUF)
.PHONY: check-tests/qemu-iotests-quick.sh
-check-tests/qemu-iotests-quick.sh: tests/qemu-iotests-quick.sh qemu-img$(EXESUF) qemu-io$(EXESUF) $(QEMU_IOTESTS_HELPERS-y)
+check-tests/qemu-iotests-quick.sh: tests/qemu-iotests-quick.sh qemu-img$(EXESUF) qemu-io$(EXESUF) qemu-nbd$(EXESUF) $(QEMU_IOTESTS_HELPERS-y)
$<
.PHONY: $(patsubst %, check-%, $(check-qapi-schema-y))
--
2.20.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [Qemu-devel] [PATCH v4 2/2] qemu-iotests: Add dependency to qemu-nbd tool
2019-03-07 15:10 ` [Qemu-devel] [PATCH v4 2/2] qemu-iotests: Add dependency to qemu-nbd tool Philippe Mathieu-Daudé
@ 2019-03-07 16:17 ` Eric Blake
0 siblings, 0 replies; 5+ messages in thread
From: Eric Blake @ 2019-03-07 16:17 UTC (permalink / raw)
To: Philippe Mathieu-Daudé, qemu-block, qemu-devel
Cc: Max Reitz, John Snow, Kevin Wolf
[-- Attachment #1: Type: text/plain, Size: 1170 bytes --]
On 3/7/19 9:10 AM, Philippe Mathieu-Daudé wrote:
> Since a9660664fde, some iotests use qemu-nbd.
> Add a dependency to build it before using it.
> This fixes:
>
> $ make check-block
> GEN qemu-img-cmds.h
> CC qemu-img.o
> LINK qemu-img
> CC qemu-io.o
> LINK qemu-io
> CC tests/qemu-iotests/socket_scm_helper.o
> LINK tests/qemu-iotests/socket_scm_helper
> tests/qemu-iotests-quick.sh
> check: qemu-nbd not found
> make: *** [tests/Makefile.include:1059: check-tests/qemu-iotests-quick.sh] Error 1
>
> Reviewed-by: Eric Blake <eblake@redhat.com>
> Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
> ---
> tests/Makefile.include | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
I still have an NBD pull request coming up before soft freeze. Should I
be planning to take this patch (as it is NBD-related) or the whole
series (even though the first is more of a stretch), or is there a
better tree for this to be staged through?
--
Eric Blake, Principal Software Engineer
Red Hat, Inc. +1-919-301-3226
Virtualization: qemu.org | libvirt.org
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [Qemu-devel] [PATCH v4 0/2] qemu-iotests: Add dependency to qemu-block tools
2019-03-07 15:10 [Qemu-devel] [PATCH v4 0/2] qemu-iotests: Add dependency to qemu-block tools Philippe Mathieu-Daudé
2019-03-07 15:10 ` [Qemu-devel] [PATCH v4 1/2] ahci-test: Add dependency to qemu-img tool Philippe Mathieu-Daudé
2019-03-07 15:10 ` [Qemu-devel] [PATCH v4 2/2] qemu-iotests: Add dependency to qemu-nbd tool Philippe Mathieu-Daudé
@ 2019-03-07 16:55 ` Kevin Wolf
2 siblings, 0 replies; 5+ messages in thread
From: Kevin Wolf @ 2019-03-07 16:55 UTC (permalink / raw)
To: Philippe Mathieu-Daudé
Cc: qemu-block, qemu-devel, Max Reitz, John Snow, Eric Blake
Am 07.03.2019 um 16:10 hat Philippe Mathieu-Daudé geschrieben:
> Hi Kevin, Max, John.
>
> Two trivial fixes I encountered while trying to run QEMU test suite on
> a distribution which doesn't has the qemu-tools distrib-wide installed,
> see https://lists.gnu.org/archive/html/qemu-devel/2018-06/msg08415.html:
>
> $ make vm-build-ubuntu.i386
> Traceback (most recent call last):
> File "source/qemu/tests/vm/basevm.py", line 236, in main
> return vm.build_image(args.image)
> File "tests/vm/ubuntu.i386", line 67, in build_image
> subprocess.check_call(["qemu-img", "resize", img_tmp, "50G"])
> OSError: [Errno 2] No such file or directory
> tests/vm/Makefile.include:23: recipe for target 'tests/vm/ubuntu.i386.img' failed
> make: *** [tests/vm/ubuntu.i386.img] Error 2
>
> All patches are reviewed.
Thanks, applied to the block branch.
Kevin
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2019-03-07 16:56 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-03-07 15:10 [Qemu-devel] [PATCH v4 0/2] qemu-iotests: Add dependency to qemu-block tools Philippe Mathieu-Daudé
2019-03-07 15:10 ` [Qemu-devel] [PATCH v4 1/2] ahci-test: Add dependency to qemu-img tool Philippe Mathieu-Daudé
2019-03-07 15:10 ` [Qemu-devel] [PATCH v4 2/2] qemu-iotests: Add dependency to qemu-nbd tool Philippe Mathieu-Daudé
2019-03-07 16:17 ` Eric Blake
2019-03-07 16:55 ` [Qemu-devel] [PATCH v4 0/2] qemu-iotests: Add dependency to qemu-block tools Kevin Wolf
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.