Re: [dpdk-dev] [PATCH v4 4/4] eal/atomic: add wrapper for c11 atomics

2020-05-14 Thread Mattias Rönnblom
On 2020-05-14 10:34, Morten Brørup wrote: > + Added people from the related discussion regarding the ARM roadmap > [https://protect2.fireeye.com/v1/url?k=10efdd7b-4e4f1ed2-10ef9de0-86959e472243-b772fef31e4ae6af&q=1&e=e3b0051e-bb23-4a30-84c7-7e5e80f83325&u=https%3A%2F%2Fmails.dpdk.org%2Farchives%2F

Re: [dpdk-dev] [PATCH v4 4/4] eal/atomic: add wrapper for c11 atomics

2020-05-14 Thread Honnappa Nagarahalli
> Subject: Re: [PATCH v4 4/4] eal/atomic: add wrapper for c11 atomics > > On 2020-05-14 10:34, Morten Brørup wrote: > > + Added people from the related discussion regarding the ARM roadmap > [https://protect2.fireeye.com/v1/url?k=10efdd7b-4e4f1ed2-10ef9de0- > 86959e472243-b772fef31e4ae6af&q=1&e=

Re: [dpdk-dev] [PATCH v4 4/4] eal/atomic: add wrapper for c11 atomics

2020-05-14 Thread Morten Brørup
+ Added people from the related discussion regarding the ARM roadmap [https://mails.dpdk.org/archives/dev/2020-April/162580.html]. > From: Mattias Rönnblom [mailto:mattias.ronnb...@ericsson.com] > Sent: Wednesday, May 13, 2020 10:17 PM > > On 2020-05-13 21:40, Honnappa Nagarahalli wrote: > > >

Re: [dpdk-dev] [PATCH v4 4/4] eal/atomic: add wrapper for c11 atomics

2020-05-13 Thread Mattias Rönnblom
On 2020-05-13 21:40, Honnappa Nagarahalli wrote: > > Subject: Re: [PATCH v4 4/4] eal/atomic: add wrapper for c11 atomics On Tue, May 12, 2020 at 4:03 pm, Phil Yang wrote: parameter. Signed-off-by: Phil Yang >>

Re: [dpdk-dev] [PATCH v4 4/4] eal/atomic: add wrapper for c11 atomics

2020-05-13 Thread Mattias Rönnblom
On 2020-05-12 20:20, Stephen Hemminger wrote: > On Tue, May 12, 2020 at 4:03 pm, Phil Yang wrote: >> parameter. Signed-off-by: Phil Yang > > > > > What is the purpose of having rte_atomic at all? > Is this level of indirection really helping? > To allow a different imple

Re: [dpdk-dev] [PATCH v4 4/4] eal/atomic: add wrapper for c11 atomics

2020-05-13 Thread Mattias Rönnblom
On 2020-05-13 10:57, Morten Brørup wrote: >> From: Honnappa Nagarahalli [mailto:honnappa.nagaraha...@arm.com] >> Sent: Tuesday, May 12, 2020 9:24 PM >> >> >> >> Subject: Re: [PATCH v4 4/4] eal/atomic: add wrapper for c11 atomics >> >> On Tue, May 12, 2020 at 4:03 pm, Phil Yang

Re: [dpdk-dev] [PATCH v4 4/4] eal/atomic: add wrapper for c11 atomics

2020-05-13 Thread Honnappa Nagarahalli
> >> > >> Subject: Re: [PATCH v4 4/4] eal/atomic: add wrapper for c11 atomics > >> > >> On Tue, May 12, 2020 at 4:03 pm, Phil Yang > >> wrote: > >> > >> parameter. Signed-off-by: Phil Yang > >> > >> > >> What is the purpose of having rte_atomi

Re: [dpdk-dev] [PATCH v4 4/4] eal/atomic: add wrapper for c11 atomics

2020-05-13 Thread Honnappa Nagarahalli
> > Subject: RE: [PATCH v4 4/4] eal/atomic: add wrapper for c11 atomics > > > > > From: Phil Yang [mailto:phil.y...@arm.com] > > > Sent: Tuesday, May 12, 2020 10:03 AM > > > > > > Wraps up compiler c11 atomic built-ins with explicit memory ordering > > > parameter. > > > > > > Signed-off-by: Phil

Re: [dpdk-dev] [PATCH v4 4/4] eal/atomic: add wrapper for c11 atomics

2020-05-13 Thread Honnappa Nagarahalli
> > Subject: Re: [PATCH v4 4/4] eal/atomic: add wrapper for c11 atomics > > > > On Tue, May 12, 2020 at 4:03 pm, Phil Yang > > wrote: > > > > parameter. Signed-off-by: Phil Yang > > > > > > What is the purpose of having rte_atomic at all? > > I

Re: [dpdk-dev] [PATCH v4 4/4] eal/atomic: add wrapper for c11 atomics

2020-05-13 Thread Honnappa Nagarahalli
> > > > > > Subject: Re: [PATCH v4 4/4] eal/atomic: add wrapper for c11 atomics > > > > On Tue, May 12, 2020 at 4:03 pm, Phil Yang > wrote: > > > > parameter. Signed-off-by: Phil Yang > > > > > > What is the purpose of having rte_atomic at all?

Re: [dpdk-dev] [PATCH v4 4/4] eal/atomic: add wrapper for c11 atomics

2020-05-13 Thread Ananyev, Konstantin
> > > Subject: Re: [PATCH v4 4/4] eal/atomic: add wrapper for c11 atomics > > On Tue, May 12, 2020 at 4:03 pm, Phil Yang wrote: > > parameter. Signed-off-by: Phil Yang > > > What is the purpose of having rte_atomic at all? > Is this level

Re: [dpdk-dev] [PATCH v4 4/4] eal/atomic: add wrapper for c11 atomics

2020-05-13 Thread Phil Yang
> -Original Message- > From: Morten Brørup > Sent: Tuesday, May 12, 2020 7:18 PM > To: Phil Yang ; tho...@monjalon.net; dev@dpdk.org > Cc: bruce.richard...@intel.com; ferruh.yi...@intel.com; > hemant.agra...@nxp.com; Honnappa Nagarahalli > ; jer...@marvell.com; > ktray...@redhat.com; konst

Re: [dpdk-dev] [PATCH v4 4/4] eal/atomic: add wrapper for c11 atomics

2020-05-13 Thread Morten Brørup
> From: Honnappa Nagarahalli [mailto:honnappa.nagaraha...@arm.com] > Sent: Tuesday, May 12, 2020 9:24 PM > > > > Subject: Re: [PATCH v4 4/4] eal/atomic: add wrapper for c11 atomics > > On Tue, May 12, 2020 at 4:03 pm, Phil Yang > wrote: > > parameter. Signed-off-by:

Re: [dpdk-dev] [PATCH v4 4/4] eal/atomic: add wrapper for c11 atomics

2020-05-12 Thread Honnappa Nagarahalli
Subject: Re: [PATCH v4 4/4] eal/atomic: add wrapper for c11 atomics On Tue, May 12, 2020 at 4:03 pm, Phil Yang wrote: parameter. Signed-off-by: Phil Yang What is the purpose of having rte_atomic at all? Is this level of indirection really

Re: [dpdk-dev] [PATCH v4 4/4] eal/atomic: add wrapper for c11 atomics

2020-05-12 Thread Stephen Hemminger
On Tue, May 12, 2020 at 4:03 pm, Phil Yang wrote: parameter. Signed-off-by: Phil Yang > What is the purpose of having rte_atomic at all? Is this level of indirection really helping?

Re: [dpdk-dev] [PATCH v4 4/4] eal/atomic: add wrapper for c11 atomics

2020-05-12 Thread Morten Brørup
> From: Phil Yang [mailto:phil.y...@arm.com] > Sent: Tuesday, May 12, 2020 10:03 AM > > Wraps up compiler c11 atomic built-ins with explicit memory ordering > parameter. > > Signed-off-by: Phil Yang > --- > lib/librte_eal/include/generic/rte_atomic_c11.h | 139 > > lib/