[dpdk-dev] [PATCH 1/3] eventdev: fix race condition on timer list counter

Phil Yang Phil.Yang at arm.com
Thu Jul 2 05:28:29 CEST 2020


> -----Original Message-----
> From: Jerin Jacob <jerinjacobk at gmail.com>
> Sent: Wednesday, July 1, 2020 7:22 PM
> To: Phil Yang <Phil.Yang at arm.com>
> Cc: Honnappa Nagarahalli <Honnappa.Nagarahalli at arm.com>; Carrillo, Erik G
> <erik.g.carrillo at intel.com>; dev at dpdk.org; drc at linux.vnet.ibm.com;
> Ruifeng Wang <Ruifeng.Wang at arm.com>; Dharmik Thakkar
> <Dharmik.Thakkar at arm.com>; nd <nd at arm.com>; stable at dpdk.org
> Subject: Re: [dpdk-dev] [PATCH 1/3] eventdev: fix race condition on timer list
> counter
> 
> On Mon, Jun 22, 2020 at 3:18 PM Phil Yang <Phil.Yang at arm.com> wrote:
> >
> > > -----Original Message-----
> > > From: Honnappa Nagarahalli <Honnappa.Nagarahalli at arm.com>
> > > Sent: Friday, June 19, 2020 2:26 AM
> > > To: Carrillo, Erik G <erik.g.carrillo at intel.com>; Phil Yang
> > > <Phil.Yang at arm.com>; dev at dpdk.org
> > > Cc: drc at linux.vnet.ibm.com; Ruifeng Wang <Ruifeng.Wang at arm.com>;
> > > Dharmik Thakkar <Dharmik.Thakkar at arm.com>; nd <nd at arm.com>;
> > > stable at dpdk.org; Honnappa Nagarahalli
> <Honnappa.Nagarahalli at arm.com>;
> > > nd <nd at arm.com>
> > > Subject: RE: [PATCH 1/3] eventdev: fix race condition on timer list counter
> > >
> 
> > > Since this commit will be back ported, we should prefer to use rte_atomic
> > > APIs for this commit. Otherwise, we will have a mix of rte_atomic and C11
> > > APIs.
> > > My suggestion is to fix this bug using rte_atomic so that backported code
> will
> > > have only rte_atomic APIs. Add another commit (if required) in this series
> to
> > > make the bug fix use C11 APIs (this commit will not be backported).
> >
> > Agree.
> > I will change this patch to the rte_atomic version in the next version.
> 
> Hi Phil,
> 
> Could you send the next version? I would like to take this series for
> RC1(next-eventdev tree)

Thanks, Jerin.
I will upstream the new patch series soon.


> 
> 
> >
> > Thanks,
> > Phil
> >


More information about the dev mailing list