[dpdk-dev] [PATCH v8 3/3] ring: introduce new header file to support C11 memory model

Thomas Monjalon thomas at monjalon.net
Fri Jan 19 00:52:29 CET 2018


17/01/2018 10:09, Thomas Monjalon:
> 17/01/2018 09:47, Jia He:
> > 
> > Hi Thomas
> > 
> > On 1/17/2018 4:24 PM, Thomas Monjalon Wrote:
> > > 17/01/2018 05:03, Jia He:
> > >> To support C11 memory model barrier, 2 options are suggested by Jerin:
> > >> 1. use rte_smp_rmb
> > >> 2. use load_acquire/store_release(refer to [1]).
> > >> CONFIG_RTE_RING_USE_C11_MEM_MODEL is provided, and by default it is "n"
> > >> on any architectures so far.
> > > In previous patches, it was enabled for ARM.
> > > You decided to not enable it at all?
> > Sorry, maybe I misunderstand your previous mail.
> >  >This config option should be added in the common file (as disabled).
> > Do you mean  CONFIG_RTE_RING_USE_C11_MEM_MODEL=n in comm_base and
> > "y" in armv8 config?
> 
> Yes, exactly

Please, could you send a v9?
Thanks


More information about the dev mailing list