diff mbox series

[v4] vdpa: mlx5: fix vdpa/vhost dependencies

Message ID 20201128213905.27409-1-rdunlap@infradead.org
State New
Headers show
Series [v4] vdpa: mlx5: fix vdpa/vhost dependencies | expand

Commit Message

Randy Dunlap Nov. 28, 2020, 9:39 p.m. UTC
drivers/vdpa/mlx5/ uses vhost_iotlb*() interfaces, so select
VHOST_IOTLB to make them be built.

However, if VHOST_IOTLB is the only VHOST symbol that is
set/enabled, the object file still won't be built because
drivers/Makefile won't descend into drivers/vhost/ to build it,
so make drivers/Makefile build the needed binary whenever
VHOST_IOTLB is set, like it does for VHOST_RING.

Fixes these build errors:
ERROR: modpost: "vhost_iotlb_itree_next" [drivers/vdpa/mlx5/mlx5_vdpa.ko] undefined!
ERROR: modpost: "vhost_iotlb_itree_first" [drivers/vdpa/mlx5/mlx5_vdpa.ko] undefined!

Fixes: 29064bfdabd5 ("vdpa/mlx5: Add support library for mlx5 VDPA implementation")
Fixes: aff90770e54c ("vdpa/mlx5: Fix dependency on MLX5_CORE")
Reported-by: kernel test robot <lkp@intel.com>
Signed-off-by: Randy Dunlap <rdunlap@infradead.org>
Cc: Eli Cohen <eli@mellanox.com>
Cc: Parav Pandit <parav@mellanox.com>
Cc: "Michael S. Tsirkin" <mst@redhat.com>
Cc: Jason Wang <jasowang@redhat.com>
Cc: virtualization@lists.linux-foundation.org
Cc: Saeed Mahameed <saeedm@nvidia.com>
Cc: Leon Romanovsky <leonro@nvidia.com>
Cc: netdev@vger.kernel.org
---
v2: change from select to depends on VHOST (Saeed)
v3: change to depends on VHOST_IOTLB (Jason)
v4: use select VHOST_IOTLB (Michael); also add to drivers/Makefile

 drivers/Makefile     |    1 +
 drivers/vdpa/Kconfig |    1 +
 2 files changed, 2 insertions(+)

Comments

Jason Wang Nov. 30, 2020, 2:12 a.m. UTC | #1
On 2020/11/29 上午5:39, Randy Dunlap wrote:
> drivers/vdpa/mlx5/ uses vhost_iotlb*() interfaces, so select

> VHOST_IOTLB to make them be built.

>

> However, if VHOST_IOTLB is the only VHOST symbol that is

> set/enabled, the object file still won't be built because

> drivers/Makefile won't descend into drivers/vhost/ to build it,

> so make drivers/Makefile build the needed binary whenever

> VHOST_IOTLB is set, like it does for VHOST_RING.

>

> Fixes these build errors:

> ERROR: modpost: "vhost_iotlb_itree_next" [drivers/vdpa/mlx5/mlx5_vdpa.ko] undefined!

> ERROR: modpost: "vhost_iotlb_itree_first" [drivers/vdpa/mlx5/mlx5_vdpa.ko] undefined!

>

> Fixes: 29064bfdabd5 ("vdpa/mlx5: Add support library for mlx5 VDPA implementation")

> Fixes: aff90770e54c ("vdpa/mlx5: Fix dependency on MLX5_CORE")

> Reported-by: kernel test robot <lkp@intel.com>

> Signed-off-by: Randy Dunlap <rdunlap@infradead.org>

> Cc: Eli Cohen <eli@mellanox.com>

> Cc: Parav Pandit <parav@mellanox.com>

> Cc: "Michael S. Tsirkin" <mst@redhat.com>

> Cc: Jason Wang <jasowang@redhat.com>

> Cc: virtualization@lists.linux-foundation.org

> Cc: Saeed Mahameed <saeedm@nvidia.com>

> Cc: Leon Romanovsky <leonro@nvidia.com>

> Cc: netdev@vger.kernel.org

> ---

> v2: change from select to depends on VHOST (Saeed)

> v3: change to depends on VHOST_IOTLB (Jason)

> v4: use select VHOST_IOTLB (Michael); also add to drivers/Makefile

>

>   drivers/Makefile     |    1 +

>   drivers/vdpa/Kconfig |    1 +

>   2 files changed, 2 insertions(+)

>

> --- linux-next-20201127.orig/drivers/vdpa/Kconfig

> +++ linux-next-20201127/drivers/vdpa/Kconfig

> @@ -32,6 +32,7 @@ config IFCVF

>   

>   config MLX5_VDPA

>   	bool

> +	select VHOST_IOTLB

>   	help

>   	  Support library for Mellanox VDPA drivers. Provides code that is

>   	  common for all types of VDPA drivers. The following drivers are planned:

> --- linux-next-20201127.orig/drivers/Makefile

> +++ linux-next-20201127/drivers/Makefile

> @@ -143,6 +143,7 @@ obj-$(CONFIG_OF)		+= of/

>   obj-$(CONFIG_SSB)		+= ssb/

>   obj-$(CONFIG_BCMA)		+= bcma/

>   obj-$(CONFIG_VHOST_RING)	+= vhost/

> +obj-$(CONFIG_VHOST_IOTLB)	+= vhost/

>   obj-$(CONFIG_VHOST)		+= vhost/

>   obj-$(CONFIG_VLYNQ)		+= vlynq/

>   obj-$(CONFIG_GREYBUS)		+= greybus/

>


Acked-by: Jason Wang <jasowang@redhat.com>


Thanks
diff mbox series

Patch

--- linux-next-20201127.orig/drivers/vdpa/Kconfig
+++ linux-next-20201127/drivers/vdpa/Kconfig
@@ -32,6 +32,7 @@  config IFCVF
 
 config MLX5_VDPA
 	bool
+	select VHOST_IOTLB
 	help
 	  Support library for Mellanox VDPA drivers. Provides code that is
 	  common for all types of VDPA drivers. The following drivers are planned:
--- linux-next-20201127.orig/drivers/Makefile
+++ linux-next-20201127/drivers/Makefile
@@ -143,6 +143,7 @@  obj-$(CONFIG_OF)		+= of/
 obj-$(CONFIG_SSB)		+= ssb/
 obj-$(CONFIG_BCMA)		+= bcma/
 obj-$(CONFIG_VHOST_RING)	+= vhost/
+obj-$(CONFIG_VHOST_IOTLB)	+= vhost/
 obj-$(CONFIG_VHOST)		+= vhost/
 obj-$(CONFIG_VLYNQ)		+= vlynq/
 obj-$(CONFIG_GREYBUS)		+= greybus/