> -----Original Message----- > From: Harman Kalra <[email protected]> > Sent: Wednesday, July 8, 2020 7:41 PM > To: Phil Yang <[email protected]> > Cc: [email protected]; [email protected]; Honnappa Nagarahalli > <[email protected]>; Ruifeng Wang > <[email protected]>; nd <[email protected]> > Subject: Re: [dpdk-dev] [PATCH 2/2] eal: use c11 atomics for interrupt status > > On Thu, Jun 11, 2020 at 06:24:25PM +0800, Phil Yang wrote: > > The event status is defined as a volatile variable and shared > > between threads. Use c11 atomics with explicit ordering instead > > of rte_atomic ops which enforce unnecessary barriers on aarch64. > > > > Signed-off-by: Phil Yang <[email protected]> > > Reviewed-by: Ruifeng Wang <[email protected]> > > --- > > Patches looks fine to me with commit message changes suggested > by Honnappa, i.e. using C11 atomic built-ins. > Since first patch of the series will not be backported, so I > think both patches can be combined into one.
OK. Thanks. I will update in the next version. Thanks, Phil > > Reviewed-by: Harman Kalra <[email protected]> > <snip>
