* [dpdk-dev] [PATCH v1] bbdev: remove experimental tag from API @ 2021-08-31 16:25 Nicolas Chautru 2021-08-31 16:25 ` Nicolas Chautru 2021-08-31 16:38 ` [dpdk-dev] " David Marchand 0 siblings, 2 replies; 12+ messages in thread From: Nicolas Chautru @ 2021-08-31 16:25 UTC (permalink / raw) To: dev, gakhil Cc: thomas, trix, hemant.agrawal, mingshan.zhang, arun.joshi, david.marchand, mdr, Nicolas Chautru Sending again this change to move bbdev to stable API. This was previously suggested last year https://patches.dpdk.org/project/dpdk/patch/1593213242-157394-2-git-send-email-nicolas.chautru@intel.com/ but there was request from community to wait another year to confirm formally this api is mature. There has been no change for the last few years and we are receiving emails from bots as these experimental tags are expiring now. Nicolas Chautru (1): bbdev: remove experimental tag from API doc/guides/rel_notes/release_21_11.rst | 2 ++ lib/bbdev/rte_bbdev.h | 32 -------------------------------- lib/bbdev/rte_bbdev_op.h | 9 --------- lib/bbdev/rte_bbdev_pmd.h | 7 ------- lib/bbdev/version.map | 2 +- 5 files changed, 3 insertions(+), 49 deletions(-) -- 1.8.3.1 ^ permalink raw reply [flat|nested] 12+ messages in thread
* [dpdk-dev] [PATCH v1] bbdev: remove experimental tag from API 2021-08-31 16:25 [dpdk-dev] [PATCH v1] bbdev: remove experimental tag from API Nicolas Chautru @ 2021-08-31 16:25 ` Nicolas Chautru 2021-10-08 19:29 ` [dpdk-dev] [EXT] " Akhil Goyal 2021-08-31 16:38 ` [dpdk-dev] " David Marchand 1 sibling, 1 reply; 12+ messages in thread From: Nicolas Chautru @ 2021-08-31 16:25 UTC (permalink / raw) To: dev, gakhil Cc: thomas, trix, hemant.agrawal, mingshan.zhang, arun.joshi, david.marchand, mdr, Nicolas Chautru This promotes the bbdev interface to stable. Overdue for some time as bbdev interface has been stable. Signed-off-by: Nicolas Chautru <nicolas.chautru@intel.com> --- doc/guides/rel_notes/release_21_11.rst | 2 ++ lib/bbdev/rte_bbdev.h | 32 -------------------------------- lib/bbdev/rte_bbdev_op.h | 9 --------- lib/bbdev/rte_bbdev_pmd.h | 7 ------- lib/bbdev/version.map | 2 +- 5 files changed, 3 insertions(+), 49 deletions(-) diff --git a/doc/guides/rel_notes/release_21_11.rst b/doc/guides/rel_notes/release_21_11.rst index d707a55..2b1edb5 100644 --- a/doc/guides/rel_notes/release_21_11.rst +++ b/doc/guides/rel_notes/release_21_11.rst @@ -83,6 +83,8 @@ API Changes This section is a comment. Do not overwrite or remove it. Also, make sure to start the actual text at the margin. ======================================================= +* bbdev: the experimental tag is dropped from the bbdev library, and its + interfaces are considered stable as of DPDK 21.11. ABI Changes diff --git a/lib/bbdev/rte_bbdev.h b/lib/bbdev/rte_bbdev.h index 7017124..bff08a8 100644 --- a/lib/bbdev/rte_bbdev.h +++ b/lib/bbdev/rte_bbdev.h @@ -10,10 +10,6 @@ * * Wireless base band device abstraction APIs. * - * @warning - * @b EXPERIMENTAL: - * All functions in this file may be changed or removed without prior notice. - * * This API allows an application to discover, configure and use a device to * process operations. An asynchronous API (enqueue, followed by later dequeue) * is used for processing operations. @@ -55,7 +51,6 @@ enum rte_bbdev_state { * @return * The total number of usable devices. */ -__rte_experimental uint16_t rte_bbdev_count(void); @@ -68,7 +63,6 @@ enum rte_bbdev_state { * @return * true if device ID is valid and device is attached, false otherwise. */ -__rte_experimental bool rte_bbdev_is_valid(uint16_t dev_id); @@ -82,7 +76,6 @@ enum rte_bbdev_state { * - The next device, or * - RTE_BBDEV_MAX_DEVS if none found */ -__rte_experimental uint16_t rte_bbdev_find_next(uint16_t dev_id); @@ -112,7 +105,6 @@ enum rte_bbdev_state { * - -EBUSY if the identified device has already started * - -ENOMEM if unable to allocate memory */ -__rte_experimental int rte_bbdev_setup_queues(uint16_t dev_id, uint16_t num_queues, int socket_id); @@ -130,7 +122,6 @@ enum rte_bbdev_state { * - -EBUSY if the identified device has already started * - -ENOTSUP if the interrupts are not supported by the device */ -__rte_experimental int rte_bbdev_intr_enable(uint16_t dev_id); @@ -160,7 +151,6 @@ struct rte_bbdev_queue_conf { * - EINVAL if the identified queue size or priority are invalid * - EBUSY if the identified queue or its device have already started */ -__rte_experimental int rte_bbdev_queue_configure(uint16_t dev_id, uint16_t queue_id, const struct rte_bbdev_queue_conf *conf); @@ -176,7 +166,6 @@ struct rte_bbdev_queue_conf { * - 0 on success * - negative value on failure - as returned from PMD driver */ -__rte_experimental int rte_bbdev_start(uint16_t dev_id); @@ -190,7 +179,6 @@ struct rte_bbdev_queue_conf { * @return * - 0 on success */ -__rte_experimental int rte_bbdev_stop(uint16_t dev_id); @@ -204,7 +192,6 @@ struct rte_bbdev_queue_conf { * @return * - 0 on success */ -__rte_experimental int rte_bbdev_close(uint16_t dev_id); @@ -222,7 +209,6 @@ struct rte_bbdev_queue_conf { * - 0 on success * - negative value on failure - as returned from PMD driver */ -__rte_experimental int rte_bbdev_queue_start(uint16_t dev_id, uint16_t queue_id); @@ -238,7 +224,6 @@ struct rte_bbdev_queue_conf { * - 0 on success * - negative value on failure - as returned from PMD driver */ -__rte_experimental int rte_bbdev_queue_stop(uint16_t dev_id, uint16_t queue_id); @@ -272,7 +257,6 @@ struct rte_bbdev_stats { * - 0 on success * - EINVAL if invalid parameter pointer is provided */ -__rte_experimental int rte_bbdev_stats_get(uint16_t dev_id, struct rte_bbdev_stats *stats); @@ -284,7 +268,6 @@ struct rte_bbdev_stats { * @return * - 0 on success */ -__rte_experimental int rte_bbdev_stats_reset(uint16_t dev_id); @@ -347,7 +330,6 @@ struct rte_bbdev_info { * - 0 on success * - EINVAL if invalid parameter pointer is provided */ -__rte_experimental int rte_bbdev_info_get(uint16_t dev_id, struct rte_bbdev_info *dev_info); @@ -374,7 +356,6 @@ struct rte_bbdev_queue_info { * - 0 on success * - EINVAL if invalid parameter pointer is provided */ -__rte_experimental int rte_bbdev_queue_info_get(uint16_t dev_id, uint16_t queue_id, struct rte_bbdev_queue_info *queue_info); @@ -490,7 +471,6 @@ struct __rte_cache_aligned rte_bbdev { * The number of operations actually enqueued (this is the number of processed * entries in the @p ops array). */ -__rte_experimental static inline uint16_t rte_bbdev_enqueue_enc_ops(uint16_t dev_id, uint16_t queue_id, struct rte_bbdev_enc_op **ops, uint16_t num_ops) @@ -521,7 +501,6 @@ struct __rte_cache_aligned rte_bbdev { * The number of operations actually enqueued (this is the number of processed * entries in the @p ops array). */ -__rte_experimental static inline uint16_t rte_bbdev_enqueue_dec_ops(uint16_t dev_id, uint16_t queue_id, struct rte_bbdev_dec_op **ops, uint16_t num_ops) @@ -552,7 +531,6 @@ struct __rte_cache_aligned rte_bbdev { * The number of operations actually enqueued (this is the number of processed * entries in the @p ops array). */ -__rte_experimental static inline uint16_t rte_bbdev_enqueue_ldpc_enc_ops(uint16_t dev_id, uint16_t queue_id, struct rte_bbdev_enc_op **ops, uint16_t num_ops) @@ -583,7 +561,6 @@ struct __rte_cache_aligned rte_bbdev { * The number of operations actually enqueued (this is the number of processed * entries in the @p ops array). */ -__rte_experimental static inline uint16_t rte_bbdev_enqueue_ldpc_dec_ops(uint16_t dev_id, uint16_t queue_id, struct rte_bbdev_dec_op **ops, uint16_t num_ops) @@ -616,7 +593,6 @@ struct __rte_cache_aligned rte_bbdev { * The number of operations actually dequeued (this is the number of entries * copied into the @p ops array). */ -__rte_experimental static inline uint16_t rte_bbdev_dequeue_enc_ops(uint16_t dev_id, uint16_t queue_id, struct rte_bbdev_enc_op **ops, uint16_t num_ops) @@ -649,7 +625,6 @@ struct __rte_cache_aligned rte_bbdev { * copied into the @p ops array). */ -__rte_experimental static inline uint16_t rte_bbdev_dequeue_dec_ops(uint16_t dev_id, uint16_t queue_id, struct rte_bbdev_dec_op **ops, uint16_t num_ops) @@ -681,7 +656,6 @@ struct __rte_cache_aligned rte_bbdev { * The number of operations actually dequeued (this is the number of entries * copied into the @p ops array). */ -__rte_experimental static inline uint16_t rte_bbdev_dequeue_ldpc_enc_ops(uint16_t dev_id, uint16_t queue_id, struct rte_bbdev_enc_op **ops, uint16_t num_ops) @@ -712,7 +686,6 @@ struct __rte_cache_aligned rte_bbdev { * The number of operations actually dequeued (this is the number of entries * copied into the @p ops array). */ -__rte_experimental static inline uint16_t rte_bbdev_dequeue_ldpc_dec_ops(uint16_t dev_id, uint16_t queue_id, struct rte_bbdev_dec_op **ops, uint16_t num_ops) @@ -764,7 +737,6 @@ typedef void (*rte_bbdev_cb_fn)(uint16_t dev_id, * @return * Zero on success, negative value on failure. */ -__rte_experimental int rte_bbdev_callback_register(uint16_t dev_id, enum rte_bbdev_event_type event, rte_bbdev_cb_fn cb_fn, void *cb_arg); @@ -788,7 +760,6 @@ typedef void (*rte_bbdev_cb_fn)(uint16_t dev_id, * - EINVAL if invalid parameter pointer is provided * - EAGAIN if the provided callback pointer does not exist */ -__rte_experimental int rte_bbdev_callback_unregister(uint16_t dev_id, enum rte_bbdev_event_type event, rte_bbdev_cb_fn cb_fn, void *cb_arg); @@ -809,7 +780,6 @@ typedef void (*rte_bbdev_cb_fn)(uint16_t dev_id, * - 0 on success * - negative value on failure - as returned from PMD driver */ -__rte_experimental int rte_bbdev_queue_intr_enable(uint16_t dev_id, uint16_t queue_id); @@ -826,7 +796,6 @@ typedef void (*rte_bbdev_cb_fn)(uint16_t dev_id, * - 0 on success * - negative value on failure - as returned from PMD driver */ -__rte_experimental int rte_bbdev_queue_intr_disable(uint16_t dev_id, uint16_t queue_id); @@ -854,7 +823,6 @@ typedef void (*rte_bbdev_cb_fn)(uint16_t dev_id, * - ENOTSUP if interrupts are not supported by the identified device * - negative value on failure - as returned from PMD driver */ -__rte_experimental int rte_bbdev_queue_intr_ctl(uint16_t dev_id, uint16_t queue_id, int epfd, int op, void *data); diff --git a/lib/bbdev/rte_bbdev_op.h b/lib/bbdev/rte_bbdev_op.h index f946842..ef3ecdb 100644 --- a/lib/bbdev/rte_bbdev_op.h +++ b/lib/bbdev/rte_bbdev_op.h @@ -9,9 +9,6 @@ * @file rte_bbdev_op.h * * Defines wireless base band layer 1 operations and capabilities - * - * @warning - * @b EXPERIMENTAL: this API may change without prior notice */ #ifdef __cplusplus @@ -815,7 +812,6 @@ struct rte_bbdev_op_pool_private { * Operation type as string or NULL if op_type is invalid * */ -__rte_experimental const char* rte_bbdev_op_type_str(enum rte_bbdev_op_type op_type); @@ -839,7 +835,6 @@ struct rte_bbdev_op_pool_private { * - Pointer to a mempool on success, * - NULL pointer on failure. */ -__rte_experimental struct rte_mempool * rte_bbdev_op_pool_create(const char *name, enum rte_bbdev_op_type type, unsigned int num_elements, unsigned int cache_size, @@ -859,7 +854,6 @@ struct rte_mempool * * - 0 on success * - EINVAL if invalid mempool is provided */ -__rte_experimental static inline int rte_bbdev_enc_op_alloc_bulk(struct rte_mempool *mempool, struct rte_bbdev_enc_op **ops, uint16_t num_ops) @@ -896,7 +890,6 @@ struct rte_mempool * * - 0 on success * - EINVAL if invalid mempool is provided */ -__rte_experimental static inline int rte_bbdev_dec_op_alloc_bulk(struct rte_mempool *mempool, struct rte_bbdev_dec_op **ops, uint16_t num_ops) @@ -929,7 +922,6 @@ struct rte_mempool * * @param num_ops * Number of structures */ -__rte_experimental static inline void rte_bbdev_dec_op_free_bulk(struct rte_bbdev_dec_op **ops, unsigned int num_ops) { @@ -947,7 +939,6 @@ struct rte_mempool * * @param num_ops * Number of structures */ -__rte_experimental static inline void rte_bbdev_enc_op_free_bulk(struct rte_bbdev_enc_op **ops, unsigned int num_ops) { diff --git a/lib/bbdev/rte_bbdev_pmd.h b/lib/bbdev/rte_bbdev_pmd.h index 237e336..dd0e359 100644 --- a/lib/bbdev/rte_bbdev_pmd.h +++ b/lib/bbdev/rte_bbdev_pmd.h @@ -10,9 +10,6 @@ * * Wireless base band driver-facing APIs. * - * @warning - * @b EXPERIMENTAL: this API may change without prior notice - * * This API provides the mechanism for device drivers to register with the * bbdev interface. User applications should not use this API. */ @@ -43,7 +40,6 @@ * @return * - Slot in the rte_bbdev array for a new device; */ -__rte_experimental struct rte_bbdev * rte_bbdev_allocate(const char *name); @@ -56,7 +52,6 @@ struct rte_bbdev * * @return * - 0 on success, negative on error */ -__rte_experimental int rte_bbdev_release(struct rte_bbdev *bbdev); @@ -71,7 +66,6 @@ struct rte_bbdev * * - NULL otherwise * */ -__rte_experimental struct rte_bbdev * rte_bbdev_get_named_dev(const char *name); @@ -190,7 +184,6 @@ struct rte_bbdev_ops { * @param ret_param * To pass data back to user application. */ -__rte_experimental void rte_bbdev_pmd_callback_process(struct rte_bbdev *dev, enum rte_bbdev_event_type event, void *ret_param); diff --git a/lib/bbdev/version.map b/lib/bbdev/version.map index 3624eb1..cce3f3c 100644 --- a/lib/bbdev/version.map +++ b/lib/bbdev/version.map @@ -1,4 +1,4 @@ -EXPERIMENTAL { +DPDK_22 { global: rte_bbdev_allocate; -- 1.8.3.1 ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [dpdk-dev] [EXT] [PATCH v1] bbdev: remove experimental tag from API 2021-08-31 16:25 ` Nicolas Chautru @ 2021-10-08 19:29 ` Akhil Goyal 2021-10-31 19:17 ` Akhil Goyal 0 siblings, 1 reply; 12+ messages in thread From: Akhil Goyal @ 2021-10-08 19:29 UTC (permalink / raw) To: Nicolas Chautru, dev, thomas, hemant.agrawal, nipun.gupta Cc: trix, mingshan.zhang, arun.joshi, david.marchand, mdr, Jerin Jacob Kollanukkaran > This promotes the bbdev interface to stable. > Overdue for some time as bbdev interface has been stable. > > Signed-off-by: Nicolas Chautru <nicolas.chautru@intel.com> > --- Acked-by: Akhil Goyal <gakhil@marvell.com> If anybody has any objections please raise or else this patch Will be merged in RC1. ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [dpdk-dev] [EXT] [PATCH v1] bbdev: remove experimental tag from API 2021-10-08 19:29 ` [dpdk-dev] [EXT] " Akhil Goyal @ 2021-10-31 19:17 ` Akhil Goyal 2021-11-06 8:58 ` Thomas Monjalon 0 siblings, 1 reply; 12+ messages in thread From: Akhil Goyal @ 2021-10-31 19:17 UTC (permalink / raw) To: Akhil Goyal, Nicolas Chautru, dev, thomas, hemant.agrawal, nipun.gupta Cc: trix, mingshan.zhang, arun.joshi, david.marchand, mdr, Jerin Jacob Kollanukkaran > > > This promotes the bbdev interface to stable. > > Overdue for some time as bbdev interface has been stable. > > > > Signed-off-by: Nicolas Chautru <nicolas.chautru@intel.com> > > --- > Acked-by: Akhil Goyal <gakhil@marvell.com> > > If anybody has any objections please raise or else this patch > Will be merged in RC1. No response received. Applied to dpdk-next-crypto ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [dpdk-dev] [EXT] [PATCH v1] bbdev: remove experimental tag from API 2021-10-31 19:17 ` Akhil Goyal @ 2021-11-06 8:58 ` Thomas Monjalon 0 siblings, 0 replies; 12+ messages in thread From: Thomas Monjalon @ 2021-11-06 8:58 UTC (permalink / raw) To: Akhil Goyal, Nicolas Chautru, dev, hemant.agrawal, nipun.gupta Cc: trix, mingshan.zhang, arun.joshi, david.marchand, mdr, Jerin Jacob Kollanukkaran 31/10/2021 20:17, Akhil Goyal: > > > > > This promotes the bbdev interface to stable. > > > Overdue for some time as bbdev interface has been stable. > > > > > > Signed-off-by: Nicolas Chautru <nicolas.chautru@intel.com> > > > --- > > Acked-by: Akhil Goyal <gakhil@marvell.com> > > > > If anybody has any objections please raise or else this patch > > Will be merged in RC1. > > No response received. > Applied to dpdk-next-crypto The file MAINTAINERS should be updated as well. I will fix it. ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [dpdk-dev] [PATCH v1] bbdev: remove experimental tag from API 2021-08-31 16:25 [dpdk-dev] [PATCH v1] bbdev: remove experimental tag from API Nicolas Chautru 2021-08-31 16:25 ` Nicolas Chautru @ 2021-08-31 16:38 ` David Marchand 2021-09-01 3:52 ` Hemant Agrawal 1 sibling, 1 reply; 12+ messages in thread From: David Marchand @ 2021-08-31 16:38 UTC (permalink / raw) To: Nicolas Chautru Cc: dev, Akhil Goyal, Thomas Monjalon, Tom Rix, Hemant Agrawal, mingshan.zhang, arun.joshi, Ray Kinsella, Maxime Coquelin On Tue, Aug 31, 2021 at 6:27 PM Nicolas Chautru <nicolas.chautru@intel.com> wrote: > > This was previously suggested last year https://patches.dpdk.org/project/dpdk/patch/1593213242-157394-2-git-send-email-nicolas.chautru@intel.com/ > but there was request from community to wait another year to confirm formally this api is mature. The request was to wait for a new vendor (i.e. non Intel) to implement a bbdev driver for their hw. NXP proposed a driver for this class: https://patchwork.dpdk.org/project/dpdk/list/?series=16642&state=%2A&archive=both What is the status of this driver? I see no update on the bbdev API, which seems good, but a confirmation is needed. -- David Marchand ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [dpdk-dev] [PATCH v1] bbdev: remove experimental tag from API 2021-08-31 16:38 ` [dpdk-dev] " David Marchand @ 2021-09-01 3:52 ` Hemant Agrawal 2021-09-01 11:14 ` Kinsella, Ray 2021-09-01 14:51 ` Chautru, Nicolas 0 siblings, 2 replies; 12+ messages in thread From: Hemant Agrawal @ 2021-09-01 3:52 UTC (permalink / raw) To: David Marchand, Nicolas Chautru Cc: dev, Akhil Goyal, Thomas Monjalon, Tom Rix, mingshan.zhang, arun.joshi, Ray Kinsella, Maxime Coquelin > > On Tue, Aug 31, 2021 at 6:27 PM Nicolas Chautru > <nicolas.chautru@intel.com> wrote: > > > > This was previously suggested last year > > > https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fpatc > > hes.dpdk.org%2Fproject%2Fdpdk%2Fpatch%2F1593213242-157394-2-git- > send-e > > mail- > nicolas.chautru%40intel.com%2F&data=04%7C01%7Chemant.agrawal% > > > 40nxp.com%7Cfcdfb339ec284057db1508d96c9dd048%7C686ea1d3bc2b4c6f > a92cd99 > > > c5c301635%7C0%7C0%7C637660247324212288%7CUnknown%7CTWFpbGZ > sb3d8eyJWIjo > > > iMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1 > 000& > > ;sdata=%2FYwaT29I4XlUxedcdCeyvM5z1R5QeNGTLLPY3XQBKjU%3D& > ;reserved=0 > > but there was request from community to wait another year to confirm > formally this api is mature. > > The request was to wait for a new vendor (i.e. non Intel) to implement a > bbdev driver for their hw. > > NXP proposed a driver for this class: > https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fpatch > work.dpdk.org%2Fproject%2Fdpdk%2Flist%2F%3Fseries%3D16642%26state > %3D%252A%26archive%3Dboth&data=04%7C01%7Chemant.agrawal% > 40nxp.com%7Cfcdfb339ec284057db1508d96c9dd048%7C686ea1d3bc2b4c6f > a92cd99c5c301635%7C0%7C0%7C637660247324212288%7CUnknown%7CT > WFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLC > JXVCI6Mn0%3D%7C1000&sdata=iorsQ7wAXyU2%2FaqeV5CrDp9asl8ztZ > nSrbDuJpCv9Gk%3D&reserved=0 > > What is the status of this driver? [Hemant] We are in process of resending the patches in next few days. > I see no update on the bbdev API, which seems good, but a confirmation is > needed. > > > -- > David Marchand ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [dpdk-dev] [PATCH v1] bbdev: remove experimental tag from API 2021-09-01 3:52 ` Hemant Agrawal @ 2021-09-01 11:14 ` Kinsella, Ray 2021-09-01 14:54 ` Chautru, Nicolas 2021-09-01 14:51 ` Chautru, Nicolas 1 sibling, 1 reply; 12+ messages in thread From: Kinsella, Ray @ 2021-09-01 11:14 UTC (permalink / raw) To: Hemant Agrawal, David Marchand, Nicolas Chautru Cc: dev, Akhil Goyal, Thomas Monjalon, Tom Rix, mingshan.zhang, arun.joshi, Maxime Coquelin On 01/09/2021 04:52, Hemant Agrawal wrote: >> >> On Tue, Aug 31, 2021 at 6:27 PM Nicolas Chautru >> <nicolas.chautru@intel.com> wrote: >>> >>> This was previously suggested last year >>> >> https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fpatc >>> hes.dpdk.org%2Fproject%2Fdpdk%2Fpatch%2F1593213242-157394-2-git- >> send-e >>> mail- >> nicolas.chautru%40intel.com%2F&data=04%7C01%7Chemant.agrawal% >>> >> 40nxp.com%7Cfcdfb339ec284057db1508d96c9dd048%7C686ea1d3bc2b4c6f >> a92cd99 >>> >> c5c301635%7C0%7C0%7C637660247324212288%7CUnknown%7CTWFpbGZ >> sb3d8eyJWIjo >>> >> iMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1 >> 000& >>> ;sdata=%2FYwaT29I4XlUxedcdCeyvM5z1R5QeNGTLLPY3XQBKjU%3D& >> ;reserved=0 >>> but there was request from community to wait another year to confirm >> formally this api is mature. >> >> The request was to wait for a new vendor (i.e. non Intel) to implement a >> bbdev driver for their hw. >> >> NXP proposed a driver for this class: >> https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fpatch >> work.dpdk.org%2Fproject%2Fdpdk%2Flist%2F%3Fseries%3D16642%26state >> %3D%252A%26archive%3Dboth&data=04%7C01%7Chemant.agrawal% >> 40nxp.com%7Cfcdfb339ec284057db1508d96c9dd048%7C686ea1d3bc2b4c6f >> a92cd99c5c301635%7C0%7C0%7C637660247324212288%7CUnknown%7CT >> WFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLC >> JXVCI6Mn0%3D%7C1000&sdata=iorsQ7wAXyU2%2FaqeV5CrDp9asl8ztZ >> nSrbDuJpCv9Gk%3D&reserved=0 >> >> What is the status of this driver? > > [Hemant] We are in process of resending the patches in next few days. Ok - just so we are clear - the net-net of that, is that the bbdev APIs will be 'experimental' for another year, right? > >> I see no update on the bbdev API, which seems good, but a confirmation is >> needed. >> >> >> -- >> David Marchand > ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [dpdk-dev] [PATCH v1] bbdev: remove experimental tag from API 2021-09-01 11:14 ` Kinsella, Ray @ 2021-09-01 14:54 ` Chautru, Nicolas 2021-09-13 19:44 ` Maxime Coquelin 0 siblings, 1 reply; 12+ messages in thread From: Chautru, Nicolas @ 2021-09-01 14:54 UTC (permalink / raw) To: Kinsella, Ray, Hemant Agrawal, David Marchand Cc: dev, Akhil Goyal, Thomas Monjalon, Tom Rix, Zhang, Mingshan, Joshi, Arun, Maxime Coquelin > -----Original Message----- > From: Kinsella, Ray <mdr@ashroe.eu> > Sent: Wednesday, September 1, 2021 4:15 AM > To: Hemant Agrawal <hemant.agrawal@nxp.com>; David Marchand > <david.marchand@redhat.com>; Chautru, Nicolas > <nicolas.chautru@intel.com> > Cc: dev <dev@dpdk.org>; Akhil Goyal <gakhil@marvell.com>; Thomas > Monjalon <thomas@monjalon.net>; Tom Rix <trix@redhat.com>; Zhang, > Mingshan <mingshan.zhang@intel.com>; Joshi, Arun > <arun.joshi@intel.com>; Maxime Coquelin <maxime.coquelin@redhat.com> > Subject: Re: [PATCH v1] bbdev: remove experimental tag from API > > > > On 01/09/2021 04:52, Hemant Agrawal wrote: > >> > >> On Tue, Aug 31, 2021 at 6:27 PM Nicolas Chautru > >> <nicolas.chautru@intel.com> wrote: > >>> > >>> This was previously suggested last year > >>> > >> > https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fpat > >> c > >>> hes.dpdk.org%2Fproject%2Fdpdk%2Fpatch%2F1593213242-157394-2- > git- > >> send-e > >>> mail- > >> > nicolas.chautru%40intel.com%2F&data=04%7C01%7Chemant.agrawal% > >>> > >> > 40nxp.com%7Cfcdfb339ec284057db1508d96c9dd048%7C686ea1d3bc2b4c6f > >> a92cd99 > >>> > >> c5c301635%7C0%7C0%7C637660247324212288%7CUnknown%7CTWFpbGZ > >> sb3d8eyJWIjo > >>> > >> > iMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1 > >> 000& > >>> > ;sdata=%2FYwaT29I4XlUxedcdCeyvM5z1R5QeNGTLLPY3XQBKjU%3D& > >> ;reserved=0 > >>> but there was request from community to wait another year to confirm > >> formally this api is mature. > >> > >> The request was to wait for a new vendor (i.e. non Intel) to > >> implement a bbdev driver for their hw. > >> > >> NXP proposed a driver for this class: > >> > https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fpat > >> ch > work.dpdk.org%2Fproject%2Fdpdk%2Flist%2F%3Fseries%3D16642%26state > >> > %3D%252A%26archive%3Dboth&data=04%7C01%7Chemant.agrawal% > >> > 40nxp.com%7Cfcdfb339ec284057db1508d96c9dd048%7C686ea1d3bc2b4c6f > >> a92cd99c5c301635%7C0%7C0%7C637660247324212288%7CUnknown%7CT > >> > WFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLC > >> > JXVCI6Mn0%3D%7C1000&sdata=iorsQ7wAXyU2%2FaqeV5CrDp9asl8ztZ > >> nSrbDuJpCv9Gk%3D&reserved=0 > >> > >> What is the status of this driver? > > > > [Hemant] We are in process of resending the patches in next few days. > > Ok - just so we are clear - the net-net of that, is that the bbdev APIs will be > 'experimental' > for another year, right? There is little reason to defer further I believe. Henant please confirm you are not looking at any ABI change based on discussion for the last few months. > > > > >> I see no update on the bbdev API, which seems good, but a > >> confirmation is needed. Indeed that's all we need. Confirmation no change is require for new PMD. > >> > >> > >> -- > >> David Marchand > > ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [dpdk-dev] [PATCH v1] bbdev: remove experimental tag from API 2021-09-01 14:54 ` Chautru, Nicolas @ 2021-09-13 19:44 ` Maxime Coquelin 2021-09-14 3:13 ` Hemant Agrawal 0 siblings, 1 reply; 12+ messages in thread From: Maxime Coquelin @ 2021-09-13 19:44 UTC (permalink / raw) To: Chautru, Nicolas, Kinsella, Ray, Hemant Agrawal, David Marchand Cc: dev, Akhil Goyal, Thomas Monjalon, Tom Rix, Zhang, Mingshan, Joshi, Arun On 9/1/21 4:54 PM, Chautru, Nicolas wrote: > > >> -----Original Message----- >> From: Kinsella, Ray <mdr@ashroe.eu> >> Sent: Wednesday, September 1, 2021 4:15 AM >> To: Hemant Agrawal <hemant.agrawal@nxp.com>; David Marchand >> <david.marchand@redhat.com>; Chautru, Nicolas >> <nicolas.chautru@intel.com> >> Cc: dev <dev@dpdk.org>; Akhil Goyal <gakhil@marvell.com>; Thomas >> Monjalon <thomas@monjalon.net>; Tom Rix <trix@redhat.com>; Zhang, >> Mingshan <mingshan.zhang@intel.com>; Joshi, Arun >> <arun.joshi@intel.com>; Maxime Coquelin <maxime.coquelin@redhat.com> >> Subject: Re: [PATCH v1] bbdev: remove experimental tag from API >> >> >> >> On 01/09/2021 04:52, Hemant Agrawal wrote: >>>> >>>> On Tue, Aug 31, 2021 at 6:27 PM Nicolas Chautru >>>> <nicolas.chautru@intel.com> wrote: >>>>> >>>>> This was previously suggested last year >>>>> >>>> >> https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fpat >>>> c >>>>> hes.dpdk.org%2Fproject%2Fdpdk%2Fpatch%2F1593213242-157394-2- >> git- >>>> send-e >>>>> mail- >>>> >> nicolas.chautru%40intel.com%2F&data=04%7C01%7Chemant.agrawal% >>>>> >>>> >> 40nxp.com%7Cfcdfb339ec284057db1508d96c9dd048%7C686ea1d3bc2b4c6f >>>> a92cd99 >>>>> >>>> c5c301635%7C0%7C0%7C637660247324212288%7CUnknown%7CTWFpbGZ >>>> sb3d8eyJWIjo >>>>> >>>> >> iMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1 >>>> 000& >>>>> >> ;sdata=%2FYwaT29I4XlUxedcdCeyvM5z1R5QeNGTLLPY3XQBKjU%3D& >>>> ;reserved=0 >>>>> but there was request from community to wait another year to confirm >>>> formally this api is mature. >>>> >>>> The request was to wait for a new vendor (i.e. non Intel) to >>>> implement a bbdev driver for their hw. >>>> >>>> NXP proposed a driver for this class: >>>> >> https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fpat >>>> ch >> work.dpdk.org%2Fproject%2Fdpdk%2Flist%2F%3Fseries%3D16642%26state >>>> >> %3D%252A%26archive%3Dboth&data=04%7C01%7Chemant.agrawal% >>>> >> 40nxp.com%7Cfcdfb339ec284057db1508d96c9dd048%7C686ea1d3bc2b4c6f >>>> a92cd99c5c301635%7C0%7C0%7C637660247324212288%7CUnknown%7CT >>>> >> WFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLC >>>> >> JXVCI6Mn0%3D%7C1000&sdata=iorsQ7wAXyU2%2FaqeV5CrDp9asl8ztZ >>>> nSrbDuJpCv9Gk%3D&reserved=0 >>>> >>>> What is the status of this driver? >>> >>> [Hemant] We are in process of resending the patches in next few days. >> >> Ok - just so we are clear - the net-net of that, is that the bbdev APIs will be >> 'experimental' >> for another year, right? > > There is little reason to defer further I believe. Henant please confirm you are not looking at any ABI change based on discussion for the last few months. > >> >>> >>>> I see no update on the bbdev API, which seems good, but a >>>> confirmation is needed. > > Indeed that's all we need. Confirmation no change is require for new PMD. Hemant, could you confirm this is good for you? Thanks, Maxime >>>> >>>> >>>> -- >>>> David Marchand >>> ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [dpdk-dev] [PATCH v1] bbdev: remove experimental tag from API 2021-09-13 19:44 ` Maxime Coquelin @ 2021-09-14 3:13 ` Hemant Agrawal 0 siblings, 0 replies; 12+ messages in thread From: Hemant Agrawal @ 2021-09-14 3:13 UTC (permalink / raw) To: Maxime Coquelin, Chautru, Nicolas, Kinsella, Ray, David Marchand Cc: dev, Akhil Goyal, Thomas Monjalon, Tom Rix, Zhang, Mingshan, Joshi, Arun HI Maxime, > > On 9/1/21 4:54 PM, Chautru, Nicolas wrote: > > > > > >> -----Original Message----- > >> From: Kinsella, Ray <mdr@ashroe.eu> > >> Sent: Wednesday, September 1, 2021 4:15 AM > >> To: Hemant Agrawal <hemant.agrawal@nxp.com>; David Marchand > >> <david.marchand@redhat.com>; Chautru, Nicolas > >> <nicolas.chautru@intel.com> > >> Cc: dev <dev@dpdk.org>; Akhil Goyal <gakhil@marvell.com>; Thomas > >> Monjalon <thomas@monjalon.net>; Tom Rix <trix@redhat.com>; Zhang, > >> Mingshan <mingshan.zhang@intel.com>; Joshi, Arun > >> <arun.joshi@intel.com>; Maxime Coquelin > <maxime.coquelin@redhat.com> > >> Subject: Re: [PATCH v1] bbdev: remove experimental tag from API > >> > >> > >> > >> On 01/09/2021 04:52, Hemant Agrawal wrote: > >>>> > >>>> On Tue, Aug 31, 2021 at 6:27 PM Nicolas Chautru > >>>> <nicolas.chautru@intel.com> wrote: > >>>>> > >>>>> This was previously suggested last year > >>>>> > >>>> > >> https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fpat > >>>> c > >>>>> hes.dpdk.org%2Fproject%2Fdpdk%2Fpatch%2F1593213242-157394-2- > >> git- > >>>> send-e > >>>>> mail- > >>>> > >> > nicolas.chautru%40intel.com%2F&data=04%7C01%7Chemant.agrawal% > >>>>> > >>>> > >> > 40nxp.com%7Cfcdfb339ec284057db1508d96c9dd048%7C686ea1d3bc2b4c6f > >>>> a92cd99 > >>>>> > >>>> > c5c301635%7C0%7C0%7C637660247324212288%7CUnknown%7CTWFpbGZ > >>>> sb3d8eyJWIjo > >>>>> > >>>> > >> > iMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1 > >>>> 000& > >>>>> > >> ;sdata=%2FYwaT29I4XlUxedcdCeyvM5z1R5QeNGTLLPY3XQBKjU%3D& > >>>> ;reserved=0 > >>>>> but there was request from community to wait another year to > >>>>> confirm > >>>> formally this api is mature. > >>>> > >>>> The request was to wait for a new vendor (i.e. non Intel) to > >>>> implement a bbdev driver for their hw. > >>>> > >>>> NXP proposed a driver for this class: > >>>> > >> https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fpat > >>>> ch > >> > work.dpdk.org%2Fproject%2Fdpdk%2Flist%2F%3Fseries%3D16642%26state > >>>> > >> %3D%252A%26archive%3Dboth&data=04%7C01%7Chemant.agrawal% > >>>> > >> > 40nxp.com%7Cfcdfb339ec284057db1508d96c9dd048%7C686ea1d3bc2b4c6f > >>>> > a92cd99c5c301635%7C0%7C0%7C637660247324212288%7CUnknown%7CT > >>>> > >> > WFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLC > >>>> > >> > JXVCI6Mn0%3D%7C1000&sdata=iorsQ7wAXyU2%2FaqeV5CrDp9asl8ztZ > >>>> nSrbDuJpCv9Gk%3D&reserved=0 > >>>> > >>>> What is the status of this driver? > >>> > >>> [Hemant] We are in process of resending the patches in next few days. > >> > >> Ok - just so we are clear - the net-net of that, is that the bbdev > >> APIs will be 'experimental' > >> for another year, right? > > > > There is little reason to defer further I believe. Henant please confirm you > are not looking at any ABI change based on discussion for the last few > months. > > > >> > >>> > >>>> I see no update on the bbdev API, which seems good, but a > >>>> confirmation is needed. > > > > Indeed that's all we need. Confirmation no change is require for new PMD. > > Hemant, could you confirm this is good for you? [Hemant] I am ok to remove experimental from these APIs. However, can that wait towards later part for this release. NXP will be sending few changes in next few days for the APIs. Regards, Hemant > > Thanks, > Maxime > > >>>> > >>>> > >>>> -- > >>>> David Marchand > >>> ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [dpdk-dev] [PATCH v1] bbdev: remove experimental tag from API 2021-09-01 3:52 ` Hemant Agrawal 2021-09-01 11:14 ` Kinsella, Ray @ 2021-09-01 14:51 ` Chautru, Nicolas 1 sibling, 0 replies; 12+ messages in thread From: Chautru, Nicolas @ 2021-09-01 14:51 UTC (permalink / raw) To: Hemant Agrawal, David Marchand Cc: dev, Akhil Goyal, Thomas Monjalon, Tom Rix, Zhang, Mingshan, Joshi, Arun, Ray Kinsella, Maxime Coquelin > -----Original Message----- > From: Hemant Agrawal <hemant.agrawal@nxp.com> > Sent: Tuesday, August 31, 2021 8:53 PM > To: David Marchand <david.marchand@redhat.com>; Chautru, Nicolas > <nicolas.chautru@intel.com> > Cc: dev <dev@dpdk.org>; Akhil Goyal <gakhil@marvell.com>; Thomas > Monjalon <thomas@monjalon.net>; Tom Rix <trix@redhat.com>; Zhang, > Mingshan <mingshan.zhang@intel.com>; Joshi, Arun > <arun.joshi@intel.com>; Ray Kinsella <mdr@ashroe.eu>; Maxime Coquelin > <maxime.coquelin@redhat.com> > Subject: RE: [PATCH v1] bbdev: remove experimental tag from API > > > > > On Tue, Aug 31, 2021 at 6:27 PM Nicolas Chautru > > <nicolas.chautru@intel.com> wrote: > > > > > > This was previously suggested last year > > > > > > https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fpatc > > > hes.dpdk.org%2Fproject%2Fdpdk%2Fpatch%2F1593213242-157394-2-git- > > send-e > > > mail- > > > nicolas.chautru%40intel.com%2F&data=04%7C01%7Chemant.agrawal% > > > > > 40nxp.com%7Cfcdfb339ec284057db1508d96c9dd048%7C686ea1d3bc2b4c6f > > a92cd99 > > > > > c5c301635%7C0%7C0%7C637660247324212288%7CUnknown%7CTWFpbGZ > > sb3d8eyJWIjo > > > > > iMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1 > > 000& > > > > ;sdata=%2FYwaT29I4XlUxedcdCeyvM5z1R5QeNGTLLPY3XQBKjU%3D& > > ;reserved=0 > > > but there was request from community to wait another year to confirm > > formally this api is mature. > > > > The request was to wait for a new vendor (i.e. non Intel) to implement > > a bbdev driver for their hw. > > > > NXP proposed a driver for this class: > > > https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fpatc > > h > work.dpdk.org%2Fproject%2Fdpdk%2Flist%2F%3Fseries%3D16642%26state > > %3D%252A%26archive%3Dboth&data=04%7C01%7Chemant.agrawal% > > 40nxp.com%7Cfcdfb339ec284057db1508d96c9dd048%7C686ea1d3bc2b4c6f > > a92cd99c5c301635%7C0%7C0%7C637660247324212288%7CUnknown%7CT > > > WFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLC > > > JXVCI6Mn0%3D%7C1000&sdata=iorsQ7wAXyU2%2FaqeV5CrDp9asl8ztZ > > nSrbDuJpCv9Gk%3D&reserved=0 > > > > What is the status of this driver? > > [Hemant] We are in process of resending the patches in next few days. Hi Hemant, You are not suggesting any change to the actual API and exposed bbdev symbols right? Based on this please ack the patch to remove the experimental patch. Thanks Nic > > > I see no update on the bbdev API, which seems good, but a confirmation > > is needed. > > > > > > -- > > David Marchand ^ permalink raw reply [flat|nested] 12+ messages in thread
end of thread, other threads:[~2021-11-06 8:58 UTC | newest] Thread overview: 12+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2021-08-31 16:25 [dpdk-dev] [PATCH v1] bbdev: remove experimental tag from API Nicolas Chautru 2021-08-31 16:25 ` Nicolas Chautru 2021-10-08 19:29 ` [dpdk-dev] [EXT] " Akhil Goyal 2021-10-31 19:17 ` Akhil Goyal 2021-11-06 8:58 ` Thomas Monjalon 2021-08-31 16:38 ` [dpdk-dev] " David Marchand 2021-09-01 3:52 ` Hemant Agrawal 2021-09-01 11:14 ` Kinsella, Ray 2021-09-01 14:54 ` Chautru, Nicolas 2021-09-13 19:44 ` Maxime Coquelin 2021-09-14 3:13 ` Hemant Agrawal 2021-09-01 14:51 ` Chautru, Nicolas
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).