[dpdk-dev] [PATCH v2] eventdev: promote adapter functions as stable
Rao, Nikhil
nikhil.rao at intel.com
Mon Apr 22 13:32:56 CEST 2019
> -----Original Message-----
> From: jerinj at marvell.com [mailto:jerinj at marvell.com]
> Sent: Friday, April 19, 2019 5:41 PM
> To: dev at dpdk.org; Thomas Monjalon <thomas at monjalon.net>; Jerin Jacob
> <jerinj at marvell.com>; Mattias Rönnblom
> <mattias.ronnblom at ericsson.com>; Van Haaren, Harry
> <harry.van.haaren at intel.com>; Gujjar, Abhinandan S
> <abhinandan.gujjar at intel.com>; Rao, Nikhil <nikhil.rao at intel.com>;
> Carrillo, Erik G <erik.g.carrillo at intel.com>
> Cc: pbhagavatula at marvell.com; hemant.agrawal at nxp.com;
> nipun.gupta at nxp.com; Ma, Liang J <liang.j.ma at intel.com>
> Subject: [dpdk-dev] [PATCH v2] eventdev: promote adapter functions as
> stable
>
> From: Jerin Jacob <jerinj at marvell.com>
>
> Promote the adapter functions and rte_event_port_unlinks_in_progress()
> as stable as it's been added for a while now and multiple drivers and test
> application like test-eventdev has been tested using the adapter APIs.
>
> Cc: nikhil.rao at intel.com
> Cc: erik.g.carrillo at intel.com
> Cc: abhinandan.gujjar at intel.com
> Cc: pbhagavatula at marvell.com
> Cc: hemant.agrawal at nxp.com
> Cc: nipun.gupta at nxp.com
> Cc: harry.van.haaren at intel.com
> Cc: mattias.ronnblom at ericsson.com
> Cc: liang.j.ma at intel.com
>
> Signed-off-by: Jerin Jacob <jerinj at marvell.com>
> ---
>
> allow experimental api still set to true for following drivers.
>
> drivers/event/sw - due to usage of rte_service_may_be_active() API
> drivers/event/dpaa[2] - due to usage of rte_mem_iova2virt() API
>
>
> v2:
> - Remove @warning EXPERIMENTAL from API(Erik)
>
> ---
> MAINTAINERS | 8 +-
> app/test-eventdev/Makefile | 1 -
> app/test-eventdev/meson.build | 1 -
> drivers/event/dsw/Makefile | 1 -
> drivers/event/dsw/meson.build | 1 -
> drivers/event/octeontx/Makefile | 1 -
> drivers/event/octeontx/meson.build | 1 -
> examples/eventdev_pipeline/Makefile | 2 -
> examples/eventdev_pipeline/meson.build | 1 -
> .../rte_event_crypto_adapter.c | 22 ++---
> .../rte_event_crypto_adapter.h | 73 +++-------------
> .../rte_event_eth_rx_adapter.h | 50 +----------
> .../rte_event_eth_tx_adapter.c | 22 ++---
> .../rte_event_eth_tx_adapter.h | 76 ++++-------------
> lib/librte_eventdev/rte_event_timer_adapter.c | 20 ++---
> lib/librte_eventdev/rte_event_timer_adapter.h | 83 +++----------------
> lib/librte_eventdev/rte_eventdev.c | 8 +-
> lib/librte_eventdev/rte_eventdev.h | 14 +---
> lib/librte_eventdev/rte_eventdev_version.map | 4 +-
> 19 files changed, 85 insertions(+), 304 deletions(-)
>
> /**
> - * @warning
> - * @b EXPERIMENTAL: this API may change without prior notice
> - *
> * Register callback to process Rx packets, this is supported for
> * SW based packet transfers.
> * @see rte_event_eth_rx_cb_fn
> @@ -501,7 +453,7 @@ int
> rte_event_eth_rx_adapter_service_id_get(uint8_t id, uint32_t *service_id);
> * - 0: Success
> * - <0: Error code on failure.
> */
> -int __rte_experimental
> +int
> rte_event_eth_rx_adapter_cb_register(uint8_t id,
> uint16_t eth_dev_id,
> rte_event_eth_rx_adapter_cb_fn cb_fn,
Sorry for the delay in reply, I would like to retain the rte_event_eth_rx_adapter_cb_register() function as experimental.
I will be posting a patch for this function to accept events instead of mbufs.
Since Thomas just applied the patch, should I be posting a new patch that reverts the lines above ?
Nikhil
More information about the dev
mailing list