> -----Original Message-----
> From: David Marchand <[email protected]>
> Sent: Wednesday, July 15, 2020 8:29 PM
> To: Phil Yang <[email protected]>
> Cc: Olivier Matz <[email protected]>; dev <[email protected]>; Stephen
> Hemminger <[email protected]>; David Christensen
> <[email protected]>; Honnappa Nagarahalli
> <[email protected]>; Ruifeng Wang
> <[email protected]>; nd <[email protected]>; Dodji Seketeli
> <[email protected]>; Aaron Conole <[email protected]>
> 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 <[email protected]> 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 <[email protected]>
> > Reviewed-by: Ruifeng Wang <[email protected]>
> > ---
> > 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 had tested with test-meson-builds in my env and it didn't give any error 
message.  The reference version is v20.05.
$  DPDK_ABI_REF_DIR=$PWD/reference  DPDK_ABI_REF_VERSION=v20.05 
./devtools/test-meson-builds.sh

It seems to be a problem with my test environment.
I will fix this problem as soon as possible.


> 
> It seems the robot had a hiccup as I can't see a report in the test-report ml.
> 
> 
> --
> David Marchand

Reply via email to