RE: [PATCH v3 3/7] dma/idxd: replace rte atomics with GCC builtin atomics

2023-05-25 Thread Morten Brørup
> From: Bruce Richardson [mailto:bruce.richard...@intel.com] > Sent: Thursday, 25 May 2023 10.42 > > On Wed, May 24, 2023 at 10:09:04PM +0200, David Marchand wrote: > > Hello Bruce, Kevin, > > > > Review please. > > > > > > On Thu, Mar 23, 2023 at 11:54 PM Tyler Retzlaff > > wrote: > > > > > > Re

Re: [PATCH v3 3/7] dma/idxd: replace rte atomics with GCC builtin atomics

2023-05-25 Thread Kevin Laatz
On 23/03/2023 22:53, Tyler Retzlaff wrote: Replace the use of rte_atomic.h types and functions, instead use GCC supplied C++11 memory model builtins. Signed-off-by: Tyler Retzlaff --- drivers/dma/idxd/idxd_internal.h | 3 +-- drivers/dma/idxd/idxd_pci.c | 8 +--- 2 files changed, 6

Re: [PATCH v3 3/7] dma/idxd: replace rte atomics with GCC builtin atomics

2023-05-25 Thread Bruce Richardson
On Wed, May 24, 2023 at 10:09:04PM +0200, David Marchand wrote: > Hello Bruce, Kevin, > > Review please. > > > On Thu, Mar 23, 2023 at 11:54 PM Tyler Retzlaff > wrote: > > > > Replace the use of rte_atomic.h types and functions, instead use GCC > > supplied C++11 memory model builtins. > > > >

Re: [PATCH v3 3/7] dma/idxd: replace rte atomics with GCC builtin atomics

2023-05-24 Thread David Marchand
Hello Bruce, Kevin, Review please. On Thu, Mar 23, 2023 at 11:54 PM Tyler Retzlaff wrote: > > Replace the use of rte_atomic.h types and functions, instead use GCC > supplied C++11 memory model builtins. > > Signed-off-by: Tyler Retzlaff > --- > drivers/dma/idxd/idxd_internal.h | 3 +-- > driv

[PATCH v3 3/7] dma/idxd: replace rte atomics with GCC builtin atomics

2023-03-23 Thread Tyler Retzlaff
Replace the use of rte_atomic.h types and functions, instead use GCC supplied C++11 memory model builtins. Signed-off-by: Tyler Retzlaff --- drivers/dma/idxd/idxd_internal.h | 3 +-- drivers/dma/idxd/idxd_pci.c | 8 +--- 2 files changed, 6 insertions(+), 5 deletions(-) diff --git a/dri