On 10/26/23 02:31, Tyler Retzlaff wrote:
Replace the use of gcc builtin __atomic_xxx intrinsics with corresponding rte_atomic_xxx optional stdatomic API Signed-off-by: Tyler Retzlaff <roret...@linux.microsoft.com> --- lib/bbdev/rte_bbdev.c | 6 +++--- lib/bbdev/rte_bbdev.h | 2 +- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/lib/bbdev/rte_bbdev.c b/lib/bbdev/rte_bbdev.c index 155323e..cfebea0 100644 --- a/lib/bbdev/rte_bbdev.c +++ b/lib/bbdev/rte_bbdev.c @@ -208,7 +208,7 @@ struct rte_bbdev * return NULL; }- __atomic_fetch_add(&bbdev->data->process_cnt, 1, __ATOMIC_RELAXED);+ rte_atomic_fetch_add_explicit(&bbdev->data->process_cnt, 1, rte_memory_order_relaxed); bbdev->data->dev_id = dev_id; bbdev->state = RTE_BBDEV_INITIALIZED;@@ -250,8 +250,8 @@ struct rte_bbdev *}/* clear shared BBDev Data if no process is using the device anymore */- if (__atomic_fetch_sub(&bbdev->data->process_cnt, 1, - __ATOMIC_RELAXED) - 1 == 0) + if (rte_atomic_fetch_sub_explicit(&bbdev->data->process_cnt, 1, + rte_memory_order_relaxed) - 1 == 0) memset(bbdev->data, 0, sizeof(*bbdev->data));memset(bbdev, 0, sizeof(*bbdev));diff --git a/lib/bbdev/rte_bbdev.h b/lib/bbdev/rte_bbdev.h index d12e2e7..e1aee08 100644 --- a/lib/bbdev/rte_bbdev.h +++ b/lib/bbdev/rte_bbdev.h @@ -482,7 +482,7 @@ struct rte_bbdev_data { uint16_t dev_id; /**< Device ID */ int socket_id; /**< NUMA socket that device is on */ bool started; /**< Device run-time state */ - uint16_t process_cnt; /** Counter of processes using the device */ + RTE_ATOMIC(uint16_t) process_cnt; /** Counter of processes using the device */ };/* Forward declarations */
Reviewed-by: Maxime Coquelin <maxime.coque...@redhat.com> Thanks, Maxime
