All of lore.kernel.org
 help / color / mirror / Atom feed
From: Fam Zheng <famz@redhat.com>
To: "Philippe Mathieu-Daudé" <f4bug@amsat.org>
Cc: "Peter Maydell" <peter.maydell@linaro.org>,
	qemu-devel@nongnu.org, "Kamil Rytarowski" <kamil@netbsd.org>,
	stefanha@redhat.com, "Cleber Rosa" <crosa@redhat.com>,
	pbonzini@redhat.com, "Alex Bennée" <alex.bennee@linaro.org>
Subject: Re: [Qemu-devel] [PATCH v4 03/10] tests: Add vm test lib
Date: Wed, 30 Aug 2017 18:15:52 +0800	[thread overview]
Message-ID: <20170830101551.GM4208@lemon.lan> (raw)
In-Reply-To: <20170830032949.GC4208@lemon.lan>

On Wed, 08/30 11:29, Fam Zheng wrote:
> On Tue, 08/29 09:15, Philippe Mathieu-Daudé wrote:
> > > +        self._args = [ \
> > > +            "-nodefaults", "-m", "2G",
> > > +            "-cpu", "host",
> > > +            "-netdev", "user,id=vnet,hostfwd=:0.0.0.0:0-:22",
> > 
> > Testing with debian/unstable:
> > 
> > $ make vm-build-netbsd V=1
> > ./tests/vm/netbsd  --debug   --image "tests/vm/netbsd.img" --build-qemu .
> > DEBUG:root:Creating archive /tmp/qemu-vm-PxfXNv/data-3a52c.tar for data dir:
> > .
> > DEBUG:root:QEMU args: -nodefaults -m 2G -cpu host -netdev
> > user,id=vnet,hostfwd=:0.0.0.0:0-:22 -device virtio-net-pci,netdev=vnet -vnc
> > :0,to=20 -serial file:/tmp/qemu-vm-PxfXNv/serial.out -smp 4 -enable-kvm
> > -device VGA -drive
> > file=tests/vm/netbsd.img,snapshot=on,if=none,id=drive0,cache=writeback
> > -device virtio-blk,drive=drive0,bootindex=0 -drive file=/tmp/qemu-vm-PxfXNv/data-3a52c.tar,if=none,id=data-3a52c,cache=writeback,format=raw
> > -device virtio-blk,drive=data-3a52c,serial=data-3a52c,bootindex=1
> > Failed to prepare guest environment
> 
> Can you please look into the stderr of the QEMU command line to see what
> arguments went wrong? (I hope the qemu.py improvement patches on the list can
> give a better error message in such cases.)

I've tested with Fedora 26's qemu-system-x86-2.9.0-5.fc26.x86_64 and got the
failure due to the hostfwd syntax:

qemu-system-x86_64: -netdev user,id=vnet,hostfwd=:0.0.0.0:0-:22: invalid host forwarding rule ':0.0.0.0:0-:22'
qemu-system-x86_64: -netdev user,id=vnet,hostfwd=:0.0.0.0:0-:22: Device 'user' could not be initialized

But it makes sense to use dynamic port allocation to avoid collision.

Since it's about developing QEMU, using cutting edge QEMU features is not that
bad. Let's keep this one.

Fam

  reply	other threads:[~2017-08-30 10:16 UTC|newest]

Thread overview: 33+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-08-28 17:46 [Qemu-devel] [PATCH v4 00/10] tests: Add VM based build tests (for non-x86_64 and/or non-Linux) Fam Zheng
2017-08-28 17:46 ` [Qemu-devel] [PATCH v4 01/10] gitignore: Ignore vm test images Fam Zheng
2017-08-28 20:12   ` Philippe Mathieu-Daudé
2017-08-28 17:46 ` [Qemu-devel] [PATCH v4 02/10] qemu.py: Add "wait()" method Fam Zheng
2017-08-28 17:47 ` [Qemu-devel] [PATCH v4 03/10] tests: Add vm test lib Fam Zheng
2017-08-29 12:06   ` Philippe Mathieu-Daudé
2017-08-29 12:11     ` Daniel P. Berrange
2017-08-30  3:27       ` Fam Zheng
2017-08-29 13:10     ` Philippe Mathieu-Daudé
2017-08-29 13:22       ` Kamil Rytarowski
2017-08-29 13:35         ` Philippe Mathieu-Daudé
2017-08-30 10:16           ` Fam Zheng
2017-08-30 13:14             ` Fam Zheng
2017-08-29 12:15   ` Philippe Mathieu-Daudé
2017-08-30  3:29     ` Fam Zheng
2017-08-30 10:15       ` Fam Zheng [this message]
2017-08-29 17:34   ` Philippe Mathieu-Daudé
2017-08-30  3:34     ` Fam Zheng
2017-09-01 19:29       ` Philippe Mathieu-Daudé
2017-09-02  5:05         ` Fam Zheng
2017-08-28 17:47 ` [Qemu-devel] [PATCH v4 04/10] tests: Add ubuntu.i386 image Fam Zheng
2017-08-28 17:47 ` [Qemu-devel] [PATCH v4 05/10] tests: Add FreeBSD image Fam Zheng
2017-08-28 17:47 ` [Qemu-devel] [PATCH v4 06/10] tests: Add NetBSD image Fam Zheng
2017-08-29 12:09   ` Philippe Mathieu-Daudé
2017-08-29 21:47     ` Philippe Mathieu-Daudé
2017-08-30  3:32       ` Fam Zheng
2017-08-28 17:47 ` [Qemu-devel] [PATCH v4 07/10] tests: Add OpenBSD image Fam Zheng
2017-08-28 17:47 ` [Qemu-devel] [PATCH v4 08/10] Makefile: Add rules to run vm tests Fam Zheng
2017-08-28 20:18   ` Philippe Mathieu-Daudé
2017-08-29 10:50     ` Fam Zheng
2017-08-28 17:47 ` [Qemu-devel] [PATCH v4 09/10] MAINTAINERS: Add tests/vm entry Fam Zheng
2017-08-28 17:47 ` [Qemu-devel] [PATCH v4 10/10] tests: Add README for vm tests Fam Zheng
2017-08-28 17:59 ` [Qemu-devel] [PATCH v4 00/10] tests: Add VM based build tests (for non-x86_64 and/or non-Linux) no-reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20170830101551.GM4208@lemon.lan \
    --to=famz@redhat.com \
    --cc=alex.bennee@linaro.org \
    --cc=crosa@redhat.com \
    --cc=f4bug@amsat.org \
    --cc=kamil@netbsd.org \
    --cc=pbonzini@redhat.com \
    --cc=peter.maydell@linaro.org \
    --cc=qemu-devel@nongnu.org \
    --cc=stefanha@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.