[dpdk-dev] [PATCH] mempool: fix mlx driver loading

[dpdk-dev] [PATCH] mempool: fix mlx driver loading

[Thomas Monjalon]

The function rte_mempool_obj_iter used in mlx drivers
was not exported. So the driver loading was failing:

EAL: open shared lib librte_pmd_mlx4.so
EAL: x86_64-native-linuxapp-gcc/lib/librte_pmd_mlx4.so:
	undefined symbol: rte_mempool_obj_iter

Fixes: 9d41beed24b0 ("lib: provide initial versioning")

Signed-off-by: Thomas Monjalon <thomas.monjalon@6wind.com>
---
 doc/guides/rel_notes/release_2_2.rst       | 5 +++++
 lib/librte_mempool/rte_mempool_version.map | 1 +
 2 files changed, 6 insertions(+)

diff --git a/doc/guides/rel_notes/release_2_2.rst b/doc/guides/rel_notes/release_2_2.rst
index 511d7a0..551ff7d 100644
--- a/doc/guides/rel_notes/release_2_2.rst
+++ b/doc/guides/rel_notes/release_2_2.rst
@@ -197,6 +197,11 @@ Drivers
   Fixed the issue of not freeing memzone in the call to free the memory for
   adminq DMA.
 
+* **mlx: Fixed driver loading.**
+
+  The mlx drivers were unable to load when built as a shared library,
+  due to a missing symbol in mempool library.
+
 * **vhost: Fixed Qemu shutdown.**
 
   Fixed issue with libvirt ``virsh destroy`` not killing the VM.
diff --git a/lib/librte_mempool/rte_mempool_version.map b/lib/librte_mempool/rte_mempool_version.map
index a9e6098..17151e0 100644
--- a/lib/librte_mempool/rte_mempool_version.map
+++ b/lib/librte_mempool/rte_mempool_version.map
@@ -9,6 +9,7 @@ DPDK_2.0 {
 	rte_mempool_dump;
 	rte_mempool_list_dump;
 	rte_mempool_lookup;
+	rte_mempool_obj_iter;
 	rte_mempool_walk;
 	rte_mempool_xmem_create;
 	rte_mempool_xmem_size;
-- 
2.5.2

Re: [dpdk-dev] [PATCH] mempool: fix mlx driver loading

[Thomas Monjalon]

2015-12-04 18:07, Thomas Monjalon:
> The function rte_mempool_obj_iter used in mlx drivers
> was not exported. So the driver loading was failing:
> 
> EAL: open shared lib librte_pmd_mlx4.so
> EAL: x86_64-native-linuxapp-gcc/lib/librte_pmd_mlx4.so:
> 	undefined symbol: rte_mempool_obj_iter
> 
> Fixes: 9d41beed24b0 ("lib: provide initial versioning")
> 
> Signed-off-by: Thomas Monjalon <thomas.monjalon@6wind.com>

Applied