Re: [dpdk-dev] [PATCH v2] eal: use c11 atomic built-ins for interrupt status

2020-07-09 Thread Phil Yang
edhat.com; Neil > Horman ; hka...@marvell.com > Subject: Re: [dpdk-dev] [PATCH v2] eal: use c11 atomic built-ins for interrupt > status > > Hi, > > Noticed 2 typos: Hi Stefan, Thanks for your feedback. Will do. Thanks, Phil

Re: [dpdk-dev] [PATCH v2] eal: use c11 atomic built-ins for interrupt status

2020-07-09 Thread Stefan Puiu
Hi, Noticed 2 typos: On Thu, Jul 9, 2020 at 9:46 AM Phil Yang wrote: > > The event status is defined as a volatile variable and shared between > threads. Use c11 atomic built-ins with explicit ordering instead of > rte_atomic ops which enforce unnecessary barriers on aarch64. > > The event statu

[dpdk-dev] [PATCH v2] eal: use c11 atomic built-ins for interrupt status

2020-07-08 Thread Phil Yang
The event status is defined as a volatile variable and shared between threads. Use c11 atomic built-ins with explicit ordering instead of rte_atomic ops which enforce unnecessary barriers on aarch64. The event status has been cleaned up by the compare-and-swap operation when we free the event data