[dpdk-dev] [PATCH] event/octeontx2: use c11 atomics for statistics

Jerin Jacob jerinjacobk at gmail.com
Sat Apr 4 18:39:01 CEST 2020


On Mon, Mar 30, 2020 at 11:56 AM Phil Yang <Phil.Yang at arm.com> wrote:

> >       if (flags & OTX2_TIM_ENA_STATS)
> > -             rte_atomic64_add(&tim_ring->arm_cnt, index);
> > +             __atomic_fetch_add(&tim_ring->arm_cnt, index,
> > __ATOMIC_RELAXED);
> >
> >       return index;
> >  }
> > @@ -130,7 +130,8 @@ tim_timer_arm_tmo_brst(const struct
> > rte_event_timer_adapter *adptr,
> >                       break;
> >       }
> >       if (flags & OTX2_TIM_ENA_STATS)
> > -             rte_atomic64_add(&tim_ring->arm_cnt, set_timers);
> > +             __atomic_fetch_add(&tim_ring->arm_cnt, set_timers,
> > +                                __ATOMIC_RELAXED);
> >
> >       return set_timers;
> >  }
> > --
> > 2.17.1
>
> It goods good to me.
>
> Reviewed-by: Phil Yang <phil.yang at arm.com>

Applied to dpdk-next-eventdev/master. Thanks.


>
> Thanks,
> Phil


More information about the dev mailing list