[dpdk-dev] [PATCH] vhost: fix meson build issues

[dpdk-dev] [PATCH] vhost: fix meson build issues

[Tomasz Duszynski]

This patch addresses following meson build issues:

1) Since rte_vdpa.h includes rte_pci.h it introduces pci
   dependency thus deps array should be updated accordingly.

2) Since vhost.h includes rte_vdpa.h vdpa.c should be added to
   the sources list. Otherwise we end up with linker errors
   caused by undefined references.

Fixes: 34b30b2e7e42 ("vhost: add apis for datapath configuration")
Cc: zhihong.wang@intel.com

Signed-off-by: Tomasz Duszynski <tdu@semihalf.com>
---
 lib/librte_vhost/meson.build | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/lib/librte_vhost/meson.build b/lib/librte_vhost/meson.build
index 9e8c0e7..8d14877 100644
--- a/lib/librte_vhost/meson.build
+++ b/lib/librte_vhost/meson.build
@@ -10,6 +10,6 @@ endif
 version = 4
 allow_experimental_apis = true
 sources = files('fd_man.c', 'iotlb.c', 'socket.c', 'vhost.c', 'vhost_user.c',
-		'virtio_net.c')
+		'virtio_net.c', 'vdpa.c')
 headers = files('rte_vhost.h')
-deps += ['ethdev']
+deps += ['ethdev', 'pci']
--
2.7.4

Re: [dpdk-dev] [PATCH] vhost: fix meson build issues

[Bruce Richardson]

On Thu, Apr 05, 2018 at 10:08:16AM +0200, Tomasz Duszynski wrote:
> This patch addresses following meson build issues:
> 
> 1) Since rte_vdpa.h includes rte_pci.h it introduces pci
>    dependency thus deps array should be updated accordingly.
> 
> 2) Since vhost.h includes rte_vdpa.h vdpa.c should be added to
>    the sources list. Otherwise we end up with linker errors
>    caused by undefined references.
> 
> Fixes: 34b30b2e7e42 ("vhost: add apis for datapath configuration")
> Cc: zhihong.wang@intel.com
> 
> Signed-off-by: Tomasz Duszynski <tdu@semihalf.com>
> ---

Confirmed that meson build on dpdk-next-virtio tree is broken and that this
fixes it.

Acked-by: Bruce Richardson <bruce.richardson@intel.com>

Re: [dpdk-dev] [PATCH] vhost: fix meson build issues

[Maxime Coquelin]

On 04/05/2018 10:08 AM, Tomasz Duszynski wrote:
> This patch addresses following meson build issues:
> 
> 1) Since rte_vdpa.h includes rte_pci.h it introduces pci
>     dependency thus deps array should be updated accordingly.
> 
> 2) Since vhost.h includes rte_vdpa.h vdpa.c should be added to
>     the sources list. Otherwise we end up with linker errors
>     caused by undefined references.
> 
> Fixes: 34b30b2e7e42 ("vhost: add apis for datapath configuration")
> Cc: zhihong.wang@intel.com
> 
> Signed-off-by: Tomasz Duszynski <tdu@semihalf.com>
> ---
>   lib/librte_vhost/meson.build | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
> 

Acked-by: Maxime Coquelin <maxime.coquelin@redhat.com>

Thanks,
Maxime

Re: [dpdk-dev] [PATCH] vhost: fix meson build issues

[Maxime Coquelin]

On 04/05/2018 10:08 AM, Tomasz Duszynski wrote:
> This patch addresses following meson build issues:
> 
> 1) Since rte_vdpa.h includes rte_pci.h it introduces pci
>     dependency thus deps array should be updated accordingly.
> 
> 2) Since vhost.h includes rte_vdpa.h vdpa.c should be added to
>     the sources list. Otherwise we end up with linker errors
>     caused by undefined references.
> 
> Fixes: 34b30b2e7e42 ("vhost: add apis for datapath configuration")
> Cc: zhihong.wang@intel.com
> 
> Signed-off-by: Tomasz Duszynski <tdu@semihalf.com>
> ---
>   lib/librte_vhost/meson.build | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
> 

Applied to dpdk-next-virtio/master.

Thanks,
Maxime

Re: [dpdk-dev] [PATCH] vhost: fix meson build issues

[Ferruh Yigit]

On 4/5/2018 9:03 PM, Maxime Coquelin wrote:
> 
> 
> On 04/05/2018 10:08 AM, Tomasz Duszynski wrote:
>> This patch addresses following meson build issues:
>>
>> 1) Since rte_vdpa.h includes rte_pci.h it introduces pci
>>     dependency thus deps array should be updated accordingly.
>>
>> 2) Since vhost.h includes rte_vdpa.h vdpa.c should be added to
>>     the sources list. Otherwise we end up with linker errors
>>     caused by undefined references.
>>
>> Fixes: 34b30b2e7e42 ("vhost: add apis for datapath configuration")
>> Cc: zhihong.wang@intel.com
>>
>> Signed-off-by: Tomasz Duszynski <tdu@semihalf.com>
>> ---
>>   lib/librte_vhost/meson.build | 4 ++--
>>   1 file changed, 2 insertions(+), 2 deletions(-)
>>
> 
> Applied to dpdk-next-virtio/master.

Squashed into relevant commit in next-net, thanks.