> -----Original Message----- > From: Harman Kalra <hka...@marvell.com> > Sent: Wednesday, July 8, 2020 7:41 PM > To: Phil Yang <phil.y...@arm.com> > Cc: dev@dpdk.org; d...@linux.vnet.ibm.com; Honnappa Nagarahalli > <honnappa.nagaraha...@arm.com>; Ruifeng Wang > <ruifeng.w...@arm.com>; nd <n...@arm.com> > 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 <phil.y...@arm.com> > > Reviewed-by: Ruifeng Wang <ruifeng.w...@arm.com> > > --- > > 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 <hka...@marvell.com> > <snip>
