Re: [dpdk-dev] [PATCH v3] mbuf: use C11 atomic built-ins for refcnt operations

2020-07-09 Thread Phil Yang
> > > > > > > Hi Phil, > > > > > > On Thu, Jul 09, 2020 at 06:10:42PM +0800, Phil Yang wrote: > > > > Use C11 atomic built-ins with explicit ordering instead of > > > > rte_atomic ops which enforce unnecessary barriers on aarch64. > > > > > > > > Signed-off-by: Phil Yang > > > > Reviewed-by: R

Re: [dpdk-dev] [PATCH v3] mbuf: use C11 atomic built-ins for refcnt operations

2020-07-09 Thread Honnappa Nagarahalli
> > > > Hi Phil, > > > > On Thu, Jul 09, 2020 at 06:10:42PM +0800, Phil Yang wrote: > > > Use C11 atomic built-ins with explicit ordering instead of > > > rte_atomic ops which enforce unnecessary barriers on aarch64. > > > > > > Signed-off-by: Phil Yang > > > Reviewed-by: Ruifeng Wang > > > ---

Re: [dpdk-dev] [PATCH v3] mbuf: use C11 atomic built-ins for refcnt operations

2020-07-09 Thread Phil Yang
Hi Oliver, > -Original Message- > From: Olivier Matz > Sent: Thursday, July 9, 2020 7:04 PM > To: Phil Yang > Cc: dev@dpdk.org; step...@networkplumber.org; > david.march...@redhat.com; d...@linux.vnet.ibm.com; Honnappa > Nagarahalli ; Ruifeng Wang > ; nd > Subject: Re: [PATCH v3] mbuf:

Re: [dpdk-dev] [PATCH v3] mbuf: use C11 atomic built-ins for refcnt operations

2020-07-09 Thread Olivier Matz
Hi Phil, On Thu, Jul 09, 2020 at 06:10:42PM +0800, Phil Yang wrote: > Use C11 atomic built-ins with explicit ordering instead of rte_atomic > ops which enforce unnecessary barriers on aarch64. > > Signed-off-by: Phil Yang > Reviewed-by: Ruifeng Wang > --- > v3: > 1.Fix ABI breakage. > 2.Simplif

[dpdk-dev] [PATCH v3] mbuf: use C11 atomic built-ins for refcnt operations

2020-07-09 Thread Phil Yang
Use C11 atomic built-ins with explicit ordering instead of rte_atomic ops which enforce unnecessary barriers on aarch64. Signed-off-by: Phil Yang Reviewed-by: Ruifeng Wang --- v3: 1.Fix ABI breakage. 2.Simplify data type cast. v2: Fix ABI issue: revert the rte_mbuf_ext_shared_info struct refcnt