David Marchand <david.march...@redhat.com> writes: > Subject: Re: [PATCH v4 1/2] mbuf: use C11 atomic built-ins for refcnt > operations > > On Thu, Jul 9, 2020 at 5:59 PM Phil Yang <phil.y...@arm.com> 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 <phil.y...@arm.com> > > Reviewed-by: Ruifeng Wang <ruifeng.w...@arm.com> > > --- > > v4: > > 1. Add union for refcnt_atomic and refcnt in rte_mbuf_ext_shared_info > > to avoid ABI breakage. (Olivier) > > 2. Add notice of refcnt_atomic deprecation. (Honnappa) > > v4 does not pass the checks (in both my env, and Travis). > https://travis-ci.com/github/ovsrobot/dpdk/jobs/359393389#L2405
I think we need an exception in 'libabigail.abignore' for this change. Is that OK with you? Thanks, Phil > > It seems the robot had a hiccup as I can't see a report in the test-report ml. > > > -- > David Marchand
