<snip>

> 
> +Cc Konstantin and Honnappa for guidance
> 
> 05/11/2020 09:55, Jiawen Wu:
> > On Thursday, November 5, 2020 9:55 AM, Jiawen Wu wrote:
> > > On Thursday, November 5, 2020 1:24 AM, Ferruh Yigit wrote:
> > > > On 11/3/2020 11:08 PM, Thomas Monjalon wrote:
> > > > > When pulling in the main branch, I see some checkpatches
> > > > > warnings (in order of criticality):
> > > > >       Using rte_smp_[r/w]mb
> > > > >       Using rte_panic/rte_exit
> > > > >       Using compiler attribute directly
> > > > >
> > > > > Please could you fix them (at least first two) before the second 
> > > > > series?
Thomas, IMO, these should result in errors in checkpatch, not just warnings. Do 
you see any issues?

> [...]
> >
> > I saw ' rte_smp_*mb ' is in the deprecation notices, but there is no
> > function can be an alternative.
> > I would like to use 'rte_atomic_thread_fence(__ATOMIC_ACQUIRE)' to
> > replace 'rte_smp_rmb()', is it the correct usage?
> 
> 
> 

Reply via email to