mbox series

[v2,0/6] use rte atomic thread fence

Message ID 1707979859-3984-1-git-send-email-roretzla@linux.microsoft.com (mailing list archive)
Headers
Series use rte atomic thread fence |

Message

Tyler Retzlaff Feb. 15, 2024, 6:50 a.m. UTC 
Replace use of __atomic_thread_fence with rte_atomic_thread_fence.

Notes:

  The rest of lib/lpm will be converted to rte_atomic in a separate
  series (to be submitted soon).

  There are existing checkpatches checks that catch use of both
  __atomic_thread_fence and __rte_atomic_thread_fence in new
  submissions.

v2:
    * change series to use rte_atomic_thread_fence instead of
      __rte_atomic_thread_fence (internal)
    * also change __atomic_thread_fence in lib/lpm

Tyler Retzlaff (6):
  distributor: use rte atomic thread fence
  eal: use rte atomic thread fence
  hash: use rte atomic thread fence
  ring: use rte atomic thread fence
  stack: use rte atomic thread fence
  lpm: use rte atomic thread fence

 lib/distributor/rte_distributor.c |  2 +-
 lib/eal/common/eal_common_trace.c |  2 +-
 lib/eal/include/rte_mcslock.h     |  4 ++--
 lib/hash/rte_cuckoo_hash.c        | 10 +++++-----
 lib/lpm/rte_lpm.c                 |  4 ++--
 lib/ring/rte_ring_c11_pvt.h       |  4 ++--
 lib/stack/rte_stack_lf_c11.h      |  2 +-
 7 files changed, 14 insertions(+), 14 deletions(-)

Comments

fengchengwen Feb. 18, 2024, 3:23 a.m. UTC | #1 
Series-acked-by: Chengwen Feng <fengchengwen@huawei.com>

On 2024/2/15 14:50, Tyler Retzlaff wrote:
> Replace use of __atomic_thread_fence with rte_atomic_thread_fence.
> 
> Notes:
> 
>   The rest of lib/lpm will be converted to rte_atomic in a separate
>   series (to be submitted soon).
> 
>   There are existing checkpatches checks that catch use of both
>   __atomic_thread_fence and __rte_atomic_thread_fence in new
>   submissions.
> 
> v2:
>     * change series to use rte_atomic_thread_fence instead of
>       __rte_atomic_thread_fence (internal)
>     * also change __atomic_thread_fence in lib/lpm
> 
> Tyler Retzlaff (6):
>   distributor: use rte atomic thread fence
>   eal: use rte atomic thread fence
>   hash: use rte atomic thread fence
>   ring: use rte atomic thread fence
>   stack: use rte atomic thread fence
>   lpm: use rte atomic thread fence
> 
>  lib/distributor/rte_distributor.c |  2 +-
>  lib/eal/common/eal_common_trace.c |  2 +-
>  lib/eal/include/rte_mcslock.h     |  4 ++--
>  lib/hash/rte_cuckoo_hash.c        | 10 +++++-----
>  lib/lpm/rte_lpm.c                 |  4 ++--
>  lib/ring/rte_ring_c11_pvt.h       |  4 ++--
>  lib/stack/rte_stack_lf_c11.h      |  2 +-
>  7 files changed, 14 insertions(+), 14 deletions(-)
>
Thomas Monjalon Feb. 18, 2024, 12:18 p.m. UTC | #2 
> > Replace use of __atomic_thread_fence with rte_atomic_thread_fence.
> > 
> > Notes:
> > 
> >   The rest of lib/lpm will be converted to rte_atomic in a separate
> >   series (to be submitted soon).
> > 
> >   There are existing checkpatches checks that catch use of both
> >   __atomic_thread_fence and __rte_atomic_thread_fence in new
> >   submissions.
> > 
> > v2:
> >     * change series to use rte_atomic_thread_fence instead of
> >       __rte_atomic_thread_fence (internal)
> >     * also change __atomic_thread_fence in lib/lpm
> > 
> > Tyler Retzlaff (6):
> >   distributor: use rte atomic thread fence
> >   eal: use rte atomic thread fence
> >   hash: use rte atomic thread fence
> >   ring: use rte atomic thread fence
> >   stack: use rte atomic thread fence
> >   lpm: use rte atomic thread fence
> 
> Series-acked-by: Chengwen Feng <fengchengwen@huawei.com>

Acked-by: Thomas Monjalon <thomas@monjalon.net>

Squashed and applied, thanks.