[Qemu-devel] [PATCH] tests: re-enable vhost-user-test

2015-10-15 Thread marcandre . lureau
From: Marc-André Lureau Commit 7fe34ca9c2e actually disabled vhost-user-test altogether, since CONFIG_VHOST_NET is a per-target config variable. CONFIG_VHOST_NET_USED is the host config, actually unused since 1322629b4f2. But it was set whenever vhost_net was enabled. tests/vhost-user-test is a

[Qemu-devel] [PATCH] tests: re-enable vhost-user-test

2015-10-15 Thread marcandre . lureau
From: Marc-André Lureau Commit 7fe34ca9c2e actually disabled vhost-user-test altogether, since CONFIG_VHOST_NET is a per-target config variable. tests/vhost-user-test is already x86/64 softmmu specific test, in order to enable it correctly, kvm & vhost-net are also conditions. To check that, set

Re: [Qemu-devel] [PATCH] tests: re-enable vhost-user-test

2015-10-15 Thread Marc-André Lureau
nack, CONFIG_VHOST_NET_USED is actually used, my bad. I'll send another patch On Thu, Oct 15, 2015 at 2:07 PM, wrote: > From: Marc-André Lureau > > Commit 7fe34ca9c2e actually disabled vhost-user-test altogether, > since CONFIG_VHOST_NET is a per-target config variable. > > CONFIG_VHOST_NET_USE

Re: [Qemu-devel] [PATCH] tests: re-enable vhost-user-test

2015-10-24 Thread Michael S. Tsirkin
On Thu, Oct 15, 2015 at 04:39:25PM +0200, marcandre.lur...@redhat.com wrote: > From: Marc-André Lureau > > Commit 7fe34ca9c2e actually disabled vhost-user-test altogether, > since CONFIG_VHOST_NET is a per-target config variable. > > tests/vhost-user-test is already x86/64 softmmu specific test,

Re: [Qemu-devel] [PATCH] tests: re-enable vhost-user-test

2015-10-26 Thread Marc-André Lureau
Hi On Sat, Oct 24, 2015 at 9:44 PM, Michael S. Tsirkin wrote: > On Thu, Oct 15, 2015 at 04:39:25PM +0200, marcandre.lur...@redhat.com wrote: >> From: Marc-André Lureau >> >> Commit 7fe34ca9c2e actually disabled vhost-user-test altogether, >> since CONFIG_VHOST_NET is a per-target config variable

Re: [Qemu-devel] [PATCH] tests: re-enable vhost-user-test

2015-10-26 Thread Michael S. Tsirkin
On Mon, Oct 26, 2015 at 12:25:38PM +0100, Marc-André Lureau wrote: > Hi > > On Sat, Oct 24, 2015 at 9:44 PM, Michael S. Tsirkin wrote: > > On Thu, Oct 15, 2015 at 04:39:25PM +0200, marcandre.lur...@redhat.com wrote: > >> From: Marc-André Lureau > >> > >> Commit 7fe34ca9c2e actually disabled vhos

Re: [Qemu-devel] [PATCH] tests: re-enable vhost-user-test

2015-10-26 Thread Marc-André Lureau
HI On Mon, Oct 26, 2015 at 12:38 PM, Michael S. Tsirkin wrote: > Like this then? > > if test target_name == "i386" -o target_name == "x86_64" > then > echo "CONFIG_VHOST_NET_TEST_$target_name=y" >> $config_host_mak > fi > > > ifeq ($(CONFIG_VHOST_NET_i386),y) > check-qtest-i386-y += tests

Re: [Qemu-devel] [PATCH] tests: re-enable vhost-user-test

2015-10-26 Thread Michael S. Tsirkin
On Mon, Oct 26, 2015 at 01:09:56PM +0100, Marc-André Lureau wrote: > HI > > On Mon, Oct 26, 2015 at 12:38 PM, Michael S. Tsirkin wrote: > > Like this then? > > > > if test target_name == "i386" -o target_name == "x86_64" > > then > > echo "CONFIG_VHOST_NET_TEST_$target_name=y" >> $config_