Re: [PATCH v2] eal: introduce atomics abstraction

2023-04-03 Thread Tyler Retzlaff
On Mon, Apr 03, 2023 at 11:11:35PM +0200, Mattias Rönnblom wrote: > On 2023-02-08 22:43, Tyler Retzlaff wrote: > >Introduce atomics abstraction that permits optional use of standard C11 > >atomics when meson is provided the new enable_stdatomics=true option. > > > > Terminology nitpicking: I don't

Re: [PATCH v2] eal: introduce atomics abstraction

2023-04-03 Thread Mattias Rönnblom
On 2023-02-09 18:38, Tyler Retzlaff wrote: On Thu, Feb 09, 2023 at 09:04:16AM +, Bruce Richardson wrote: On Wed, Feb 08, 2023 at 01:43:38PM -0800, Tyler Retzlaff wrote: Introduce atomics abstraction that permits optional use of standard C11 atomics when meson is provided the new enable_stda

RE: [PATCH v2] eal: introduce atomics abstraction

2023-04-03 Thread Honnappa Nagarahalli
> -Original Message- > From: Mattias Rönnblom > Sent: Monday, April 3, 2023 4:12 PM > To: Tyler Retzlaff ; dev@dpdk.org > Cc: david.march...@redhat.com; tho...@monjalon.net; > m...@smartsharesystems.com; Honnappa Nagarahalli > ; bruce.richard...@intel.com > Sub

Re: [PATCH v2] eal: introduce atomics abstraction

2023-04-03 Thread Mattias Rönnblom
On 2023-02-09 09:05, Morten Brørup wrote: From: Tyler Retzlaff [mailto:roret...@linux.microsoft.com] Sent: Wednesday, 8 February 2023 22.44 Introduce atomics abstraction that permits optional use of standard C11 atomics when meson is provided the new enable_stdatomics=true option. Signed-off-by

Re: [PATCH v2] eal: introduce atomics abstraction

2023-04-03 Thread Mattias Rönnblom
On 2023-02-08 22:43, Tyler Retzlaff wrote: Introduce atomics abstraction that permits optional use of standard C11 atomics when meson is provided the new enable_stdatomics=true option. Terminology nitpicking: I don't think these functions provide any abstraction at all. They are just wrappers

Re: [PATCH v2] eal: introduce atomics abstraction

2023-02-09 Thread Tyler Retzlaff
On Thu, Feb 09, 2023 at 10:13:41PM +, Ferruh Yigit wrote: > On 2/9/2023 5:40 PM, Tyler Retzlaff wrote: > > On Thu, Feb 09, 2023 at 12:53:41PM +, Ferruh Yigit wrote: > >> On 2/9/2023 9:04 AM, Bruce Richardson wrote: > >>> On Wed, Feb 08, 2023 at 01:43:38PM -0800, Tyler Retzlaff wrote: >

Re: [PATCH v2] eal: introduce atomics abstraction

2023-02-09 Thread Ferruh Yigit
On 2/9/2023 5:40 PM, Tyler Retzlaff wrote: > On Thu, Feb 09, 2023 at 12:53:41PM +, Ferruh Yigit wrote: >> On 2/9/2023 9:04 AM, Bruce Richardson wrote: >>> On Wed, Feb 08, 2023 at 01:43:38PM -0800, Tyler Retzlaff wrote: Introduce atomics abstraction that permits optional use of standard C11

Re: [PATCH v2] eal: introduce atomics abstraction

2023-02-09 Thread Tyler Retzlaff
On Thu, Feb 09, 2023 at 08:19:14PM +0100, Morten Brørup wrote: > > From: Tyler Retzlaff [mailto:roret...@linux.microsoft.com] > > Sent: Thursday, 9 February 2023 19.15 > > > > On Thu, Feb 09, 2023 at 09:05:46AM +0100, Morten Brørup wrote: > > > > From: Tyler Retzlaff [mailto:roret...@linux.microso

RE: [PATCH v2] eal: introduce atomics abstraction

2023-02-09 Thread Morten Brørup
> From: Tyler Retzlaff [mailto:roret...@linux.microsoft.com] > Sent: Thursday, 9 February 2023 19.15 > > On Thu, Feb 09, 2023 at 09:05:46AM +0100, Morten Brørup wrote: > > > From: Tyler Retzlaff [mailto:roret...@linux.microsoft.com] > > > Sent: Wednesday, 8 February 2023 22.44 > > > > > > Introduc

Re: [PATCH v2] eal: introduce atomics abstraction

2023-02-09 Thread Tyler Retzlaff
On Thu, Feb 09, 2023 at 09:05:46AM +0100, Morten Brørup wrote: > > From: Tyler Retzlaff [mailto:roret...@linux.microsoft.com] > > Sent: Wednesday, 8 February 2023 22.44 > > > > Introduce atomics abstraction that permits optional use of standard C11 > > atomics when meson is provided the new enable

Re: [PATCH v2] eal: introduce atomics abstraction

2023-02-09 Thread Tyler Retzlaff
On Thu, Feb 09, 2023 at 12:53:41PM +, Ferruh Yigit wrote: > On 2/9/2023 9:04 AM, Bruce Richardson wrote: > > On Wed, Feb 08, 2023 at 01:43:38PM -0800, Tyler Retzlaff wrote: > >> Introduce atomics abstraction that permits optional use of standard C11 > >> atomics when meson is provided the new e

Re: [PATCH v2] eal: introduce atomics abstraction

2023-02-09 Thread Tyler Retzlaff
On Thu, Feb 09, 2023 at 09:04:16AM +, Bruce Richardson wrote: > On Wed, Feb 08, 2023 at 01:43:38PM -0800, Tyler Retzlaff wrote: > > Introduce atomics abstraction that permits optional use of standard C11 > > atomics when meson is provided the new enable_stdatomics=true option. > > > > Signed-o

Re: [PATCH v2] eal: introduce atomics abstraction

2023-02-09 Thread Ferruh Yigit
On 2/9/2023 9:04 AM, Bruce Richardson wrote: > On Wed, Feb 08, 2023 at 01:43:38PM -0800, Tyler Retzlaff wrote: >> Introduce atomics abstraction that permits optional use of standard C11 >> atomics when meson is provided the new enable_stdatomics=true option. >> >> Signed-off-by: Tyler Retzlaff >>

Re: [PATCH v2] eal: introduce atomics abstraction

2023-02-09 Thread Bruce Richardson
On Wed, Feb 08, 2023 at 01:43:38PM -0800, Tyler Retzlaff wrote: > Introduce atomics abstraction that permits optional use of standard C11 > atomics when meson is provided the new enable_stdatomics=true option. > > Signed-off-by: Tyler Retzlaff > --- > config/meson.build | 11

RE: [PATCH v2] eal: introduce atomics abstraction

2023-02-09 Thread Morten Brørup
> From: Tyler Retzlaff [mailto:roret...@linux.microsoft.com] > Sent: Wednesday, 8 February 2023 22.44 > > Introduce atomics abstraction that permits optional use of standard C11 > atomics when meson is provided the new enable_stdatomics=true option. > > Signed-off-by: Tyler Retzlaff > --- Looks