[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-stable] [Qemu-devel] [PATCH] vl: Delay initialization of memor
From: |
Eduardo Habkost |
Subject: |
Re: [Qemu-stable] [Qemu-devel] [PATCH] vl: Delay initialization of memory backends |
Date: |
Fri, 2 Sep 2016 15:10:55 -0300 |
User-agent: |
Mutt/1.6.2 (2016-07-01) |
On Fri, Sep 02, 2016 at 04:56:49PM +0200, Paolo Bonzini wrote:
> On 02/09/2016 16:04, Eduardo Habkost wrote:
[...]
> > Now, why exactly vhost-user-test needs accel=tcg and can't use
> > accel=qtest?
>
> Because it uses iPXE to exercise the NIC.
Is it used to actually exercise the NIC, or just to initialize
it?
Original commit says "pc-bios/pxe-virtio.rom is used to
instantiate a virtio pcicontroller". I don't see any test code
that receives any data from iPXE: nothing is read from log_fd,
the VHOST_USER_SET_VRING_KICK/VHOST_USER_SET_VRING_CALL FDs are
simply ignored. The only memory writes/reads are on the first
memory page.
I suggest simply using libqos. If somebody really wants rx/tx
tests using vhost-user, they can be written using libqos (just
like virtio-net-test.c).
--
Eduardo
- Re: [Qemu-stable] [Qemu-devel] [PATCH] vl: Delay initialization of memory backends, Eduardo Habkost, 2016/09/01
- Re: [Qemu-stable] [Qemu-devel] [PATCH] vl: Delay initialization of memory backends, Paolo Bonzini, 2016/09/01
- Re: [Qemu-stable] [Qemu-devel] [PATCH] vl: Delay initialization of memory backends, Eduardo Habkost, 2016/09/01
- Re: [Qemu-stable] [Qemu-devel] [PATCH] vl: Delay initialization of memory backends, Eduardo Habkost, 2016/09/01
- Re: [Qemu-stable] [Qemu-devel] [PATCH] vl: Delay initialization of memory backends, Bandan Das, 2016/09/01
- Re: [Qemu-stable] [Qemu-devel] [PATCH] vl: Delay initialization of memory backends, Paolo Bonzini, 2016/09/02
- Re: [Qemu-stable] [Qemu-devel] [PATCH] vl: Delay initialization of memory backends, Eduardo Habkost, 2016/09/02
- Re: [Qemu-stable] [Qemu-devel] [PATCH] vl: Delay initialization of memory backends, Paolo Bonzini, 2016/09/02
- Re: [Qemu-stable] [Qemu-devel] [PATCH] vl: Delay initialization of memory backends,
Eduardo Habkost <=
- Re: [Qemu-stable] [Qemu-devel] [PATCH] vl: Delay initialization of memory backends, Paolo Bonzini, 2016/09/05
Re: [Qemu-stable] [Qemu-devel] [PATCH] vl: Delay initialization of memory backends, Michal Privoznik, 2016/09/01
Re: [Qemu-stable] [Qemu-devel] [PATCH] vl: Delay initialization of memory backends, Markus Armbruster, 2016/09/02