[dpdk-dev] [PATCH v7 3/4] bus/vdev: move to vdev bus to drivers/bus
De Lara Guarch, Pablo
pablo.de.lara.guarch at intel.com
Mon Oct 23 16:34:35 CEST 2017
Hi Jianfeng,
> -----Original Message-----
> From: Tan, Jianfeng
> Sent: Friday, October 13, 2017 12:52 PM
> To: dev at dpdk.org
> Cc: jblunck at infradead.org; Richardson, Bruce
> <bruce.richardson at intel.com>; Ananyev, Konstantin
> <konstantin.ananyev at intel.com>; De Lara Guarch, Pablo
> <pablo.de.lara.guarch at intel.com>; thomas at monjalon.net;
> yliu at fridaylinux.org; maxime.coquelin at redhat.com;
> mtetsuyah at gmail.com; Yigit, Ferruh <ferruh.yigit at intel.com>; Tan,
> Jianfeng <jianfeng.tan at intel.com>
> Subject: [PATCH v7 3/4] bus/vdev: move to vdev bus to drivers/bus
>
> Move the vdev bus from lib/librte_eal to drivers/bus.
>
> As the crypto vdev helper function refers to data structure in rte_vdev.h, so
> we move those helper function into drivers/bus too.
I am seeing some errors with this patch, compiling the test app:
test/test/test_eventdev.c:54:10: error: implicit declaration of function 'rte_vdev_init'
[-Werror=implicit-function-declaration]
return rte_vdev_init("event_skeleton", NULL);
^~~~~~~~~~~~~
test/test/test_eventdev.c:54:3: error: nested extern declaration of 'rte_vdev_init'
[-Werror=nested-externs]
return rte_vdev_init("event_skeleton", NULL);
^~~~~~
Also, I have a couple of comments more below.
...
> +++ b/drivers/bus/vdev/rte_bus_vdev_version.map
> @@ -0,0 +1,8 @@
> +DPDK_17.11 {
> + global:
> +
> + rte_vdev_init;
> + rte_vdev_register;
> + rte_vdev_uninit;
> + rte_vdev_unregister;
These functions should be removed from rte_eal_version.map,
for Linux and BSD, right?
Also, should this be documented in release notes?
More information about the dev
mailing list