* [PATCH] meson: always include contrib/libvhost-user
@ 2020-11-06 21:03 Stefan Hajnoczi
2020-11-07 8:53 ` Michael S. Tsirkin
2020-11-08 15:54 ` Philippe Mathieu-Daudé
0 siblings, 2 replies; 4+ messages in thread
From: Stefan Hajnoczi @ 2020-11-06 21:03 UTC (permalink / raw)
To: qemu-devel
Cc: Philippe Mathieu-Daudé, Stefan Hajnoczi, Michael S . Tsirkin
libvhost-user is needed when CONFIG_LINUX is set. The CONFIG_VHOST_USER
check in meson.build is incorrect.
In fact, no explicit check is needed since this dependency is not built
by default. If something declares a dependency on libvhost-user then it
will be built, otherwise it won't be built (i.e. on non-Linux hosts).
This fixes ./configure --disable-vhost-user && make.
Fixes: bc15e44cb2191bbb2318878acdf5038134e56394 ("configure: introduce --enable-vhost-user-blk-server")
Reported-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Reported-by: Michael S. Tsirkin <mst@redhat.com>
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
---
meson.build | 6 +-----
1 file changed, 1 insertion(+), 5 deletions(-)
diff --git a/meson.build b/meson.build
index f5175010df..b473620321 100644
--- a/meson.build
+++ b/meson.build
@@ -1450,11 +1450,7 @@ trace_events_subdirs += [
'util',
]
-vhost_user = not_found
-if 'CONFIG_VHOST_USER' in config_host
- subdir('contrib/libvhost-user')
-endif
-
+subdir('contrib/libvhost-user')
subdir('qapi')
subdir('qobject')
subdir('stubs')
--
2.28.0
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] meson: always include contrib/libvhost-user
2020-11-06 21:03 [PATCH] meson: always include contrib/libvhost-user Stefan Hajnoczi
@ 2020-11-07 8:53 ` Michael S. Tsirkin
2020-11-08 15:54 ` Philippe Mathieu-Daudé
1 sibling, 0 replies; 4+ messages in thread
From: Michael S. Tsirkin @ 2020-11-07 8:53 UTC (permalink / raw)
To: Stefan Hajnoczi; +Cc: Philippe Mathieu-Daudé, qemu-devel
On Fri, Nov 06, 2020 at 09:03:40PM +0000, Stefan Hajnoczi wrote:
> libvhost-user is needed when CONFIG_LINUX is set. The CONFIG_VHOST_USER
> check in meson.build is incorrect.
>
> In fact, no explicit check is needed since this dependency is not built
> by default. If something declares a dependency on libvhost-user then it
> will be built, otherwise it won't be built (i.e. on non-Linux hosts).
>
> This fixes ./configure --disable-vhost-user && make.
>
> Fixes: bc15e44cb2191bbb2318878acdf5038134e56394 ("configure: introduce --enable-vhost-user-blk-server")
> Reported-by: Philippe Mathieu-Daudé <philmd@redhat.com>
> Reported-by: Michael S. Tsirkin <mst@redhat.com>
> Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
Applied, thanks!
It's probably time to move libvhost-user out of contrib.
> ---
> meson.build | 6 +-----
> 1 file changed, 1 insertion(+), 5 deletions(-)
>
> diff --git a/meson.build b/meson.build
> index f5175010df..b473620321 100644
> --- a/meson.build
> +++ b/meson.build
> @@ -1450,11 +1450,7 @@ trace_events_subdirs += [
> 'util',
> ]
>
> -vhost_user = not_found
> -if 'CONFIG_VHOST_USER' in config_host
> - subdir('contrib/libvhost-user')
> -endif
> -
> +subdir('contrib/libvhost-user')
> subdir('qapi')
> subdir('qobject')
> subdir('stubs')
> --
> 2.28.0
>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] meson: always include contrib/libvhost-user
2020-11-06 21:03 [PATCH] meson: always include contrib/libvhost-user Stefan Hajnoczi
2020-11-07 8:53 ` Michael S. Tsirkin
@ 2020-11-08 15:54 ` Philippe Mathieu-Daudé
2020-11-09 9:01 ` Thomas Huth
1 sibling, 1 reply; 4+ messages in thread
From: Philippe Mathieu-Daudé @ 2020-11-08 15:54 UTC (permalink / raw)
To: Stefan Hajnoczi, qemu-devel; +Cc: Michael S . Tsirkin
On 11/6/20 10:03 PM, Stefan Hajnoczi wrote:
> libvhost-user is needed when CONFIG_LINUX is set. The CONFIG_VHOST_USER
> check in meson.build is incorrect.
>
> In fact, no explicit check is needed since this dependency is not built
> by default. If something declares a dependency on libvhost-user then it
> will be built, otherwise it won't be built (i.e. on non-Linux hosts).
>
> This fixes ./configure --disable-vhost-user && make.
>
> Fixes: bc15e44cb2191bbb2318878acdf5038134e56394 ("configure: introduce --enable-vhost-user-blk-server")
> Reported-by: Philippe Mathieu-Daudé <philmd@redhat.com>
> Reported-by: Michael S. Tsirkin <mst@redhat.com>
> Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
> ---
> meson.build | 6 +-----
> 1 file changed, 1 insertion(+), 5 deletions(-)
>
> diff --git a/meson.build b/meson.build
> index f5175010df..b473620321 100644
> --- a/meson.build
> +++ b/meson.build
> @@ -1450,11 +1450,7 @@ trace_events_subdirs += [
> 'util',
> ]
>
> -vhost_user = not_found
> -if 'CONFIG_VHOST_USER' in config_host
> - subdir('contrib/libvhost-user')
> -endif
> -
> +subdir('contrib/libvhost-user')
I haven't reviewed what this code does, but testing it I see:
$ ../configure --disable-vhost-user
vhost-user support: YES
which doesn't look correct...
> subdir('qapi')
> subdir('qobject')
> subdir('stubs')
>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] meson: always include contrib/libvhost-user
2020-11-08 15:54 ` Philippe Mathieu-Daudé
@ 2020-11-09 9:01 ` Thomas Huth
0 siblings, 0 replies; 4+ messages in thread
From: Thomas Huth @ 2020-11-09 9:01 UTC (permalink / raw)
To: Philippe Mathieu-Daudé, Stefan Hajnoczi, qemu-devel
Cc: Michael S . Tsirkin
On 08/11/2020 16.54, Philippe Mathieu-Daudé wrote:
> On 11/6/20 10:03 PM, Stefan Hajnoczi wrote:
>> libvhost-user is needed when CONFIG_LINUX is set. The CONFIG_VHOST_USER
>> check in meson.build is incorrect.
>>
>> In fact, no explicit check is needed since this dependency is not built
>> by default. If something declares a dependency on libvhost-user then it
>> will be built, otherwise it won't be built (i.e. on non-Linux hosts).
>>
>> This fixes ./configure --disable-vhost-user && make.
>>
>> Fixes: bc15e44cb2191bbb2318878acdf5038134e56394 ("configure: introduce --enable-vhost-user-blk-server")
>> Reported-by: Philippe Mathieu-Daudé <philmd@redhat.com>
>> Reported-by: Michael S. Tsirkin <mst@redhat.com>
>> Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
>> ---
>> meson.build | 6 +-----
>> 1 file changed, 1 insertion(+), 5 deletions(-)
>>
>> diff --git a/meson.build b/meson.build
>> index f5175010df..b473620321 100644
>> --- a/meson.build
>> +++ b/meson.build
>> @@ -1450,11 +1450,7 @@ trace_events_subdirs += [
>> 'util',
>> ]
>>
>> -vhost_user = not_found
>> -if 'CONFIG_VHOST_USER' in config_host
>> - subdir('contrib/libvhost-user')
>> -endif
>> -
>> +subdir('contrib/libvhost-user')
>
> I haven't reviewed what this code does, but testing it I see:
>
> $ ../configure --disable-vhost-user
>
> vhost-user support: YES
>
> which doesn't look correct...
I think it's a bug in meson.build: It says "vhost-user" but then uses the
CONFIG_VHOST_KERNEL switch instead. I just sent a patch to fix this.
Thomas
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2020-11-09 9:03 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-11-06 21:03 [PATCH] meson: always include contrib/libvhost-user Stefan Hajnoczi
2020-11-07 8:53 ` Michael S. Tsirkin
2020-11-08 15:54 ` Philippe Mathieu-Daudé
2020-11-09 9:01 ` Thomas Huth
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.