[PATCH v2 4/6] distributor: adapt for EAL optional atomics API changes
Morten Brørup
mb at smartsharesystems.com
Mon Aug 14 10:07:59 CEST 2023
> From: Tyler Retzlaff [mailto:roretzla at linux.microsoft.com]
> Sent: Friday, 11 August 2023 19.32
>
> Adapt distributor for EAL optional atomics API changes
>
> Signed-off-by: Tyler Retzlaff <roretzla at linux.microsoft.com>
> ---
> lib/distributor/distributor_private.h | 2 +-
> lib/distributor/rte_distributor_single.c | 44 ++++++++++++++++---------------
> -
> 2 files changed, 23 insertions(+), 23 deletions(-)
>
> diff --git a/lib/distributor/distributor_private.h
> b/lib/distributor/distributor_private.h
> index 7101f63..ffbdae5 100644
> --- a/lib/distributor/distributor_private.h
> +++ b/lib/distributor/distributor_private.h
> @@ -52,7 +52,7 @@
> * Only 64-bits of the memory is actually used though.
> */
> union rte_distributor_buffer_single {
> - volatile int64_t bufptr64;
> + volatile int64_t __rte_atomic bufptr64;
As mentioned in my review to the 2/6 patch, I think __rte_atomic should come before the type, like this:
> + volatile __rte_atomic int64_t bufptr64;
> char pad[RTE_CACHE_LINE_SIZE*3];
> } __rte_cache_aligned;
The rest is simple search-replace; easy to review, so...
Reviewed-by: Morten Brørup <mb at smartsharesystems.com>
More information about the dev
mailing list