Re: [V2 PATCH 2/2] KVM test: Add vhost-net support

2010-10-11 Thread Lucas Meneghel Rodrigues
On Fri, 2010-10-08 at 21:46 -0400, Jason Wang wrote: > Hi Lucas: > > Vhost-net backend is only valid when we use tap, so it should work with > "nic_mode=tap". Maybe we could add a comment above the "vhost=on" to warn the > user to use tap when he need vhost. Ok, fair enough, that's what I've done

Re: [V2 PATCH 2/2] KVM test: Add vhost-net support

2010-10-08 Thread Jason Wang
Hi Lucas: Vhost-net backend is only valid when we use tap, so it should work with "nic_mode=tap". Maybe we could add a comment above the "vhost=on" to warn the user to use tap when he need vhost. - "Lucas Meneghel Rodrigues" wrote: > On Fri, 2010-10-08 at 12:19 -0300, Lucas Meneghel Rodrigu

[PATCH 2/2] KVM test: Add vhost-net support

2010-10-08 Thread Lucas Meneghel Rodrigues
Vhost is a kernel-level backend for virtio. This patch add a nic_param named "vhost" to enable/disable vhost backend. Changes from v2: - Rebase against latest HEAD lmr - This one has problems so I'll hold on about applying this one. Signed-off-by: Jason Wang --- client/tests/kvm/kvm_vm.py

Re: [V2 PATCH 2/2] KVM test: Add vhost-net support

2010-10-08 Thread Lucas Meneghel Rodrigues
On Fri, 2010-10-08 at 12:19 -0300, Lucas Meneghel Rodrigues wrote: > On Sun, 2010-09-26 at 10:28 +0800, Jason Wang wrote: > > Vhost is a kernel-level backend for virtio. This patch add a nic_params > > named > > "vhost" to enable/disable vhost backend. > > Jason, I had rebased your patches. The p

Re: [V2 PATCH 2/2] KVM test: Add vhost-net support

2010-10-08 Thread Lucas Meneghel Rodrigues
On Sun, 2010-09-26 at 10:28 +0800, Jason Wang wrote: > Vhost is a kernel-level backend for virtio. This patch add a nic_params named > "vhost" to enable/disable vhost backend. Jason, I had rebased your patches. The patches themselves look good, but apparently the syntax proposed here is invalid, a

[V2 PATCH 2/2] KVM test: Add vhost-net support

2010-09-25 Thread Jason Wang
Vhost is a kernel-level backend for virtio. This patch add a nic_params named "vhost" to enable/disable vhost backend. Signed-off-by: Jason Wang --- client/tests/kvm/kvm_vm.py |7 +-- client/tests/kvm/tests_base.cfg.sample |2 ++ 2 files changed, 7 insertions(+), 2 deleti

[PATCH 2/2] KVM test: Add vhost-net support

2010-09-25 Thread Jason Wang
Vhost is a kernel-level backend for virtio. This patch add a nic_params named "vhost" to enable/disable vhost backend. Signed-off-by: Jason Wang --- client/tests/kvm/kvm_vm.py |7 +-- client/tests/kvm/tests_base.cfg.sample |2 ++ 2 files changed, 7 insertions(+), 2 deleti