[PATCH v3 0/7] replace rte atomics with GCC builtin atomics
Tyler Retzlaff
roretzla at linux.microsoft.com
Fri Mar 24 20:22:46 CET 2023
On Fri, Mar 24, 2023 at 08:09:50AM +0100, Morten Brørup wrote:
> > From: Tyler Retzlaff [mailto:roretzla at linux.microsoft.com]
> > Sent: Thursday, 23 March 2023 23.54
> >
> > Replace the use of rte_atomic.h types and functions, instead use GCC
> > supplied C++11 memory model builtins.
> >
> > This series covers the libraries and drivers that are built on Windows.
> >
> > The code has be converted to use the __atomic builtins but there are
> > additional during conversion i notice that there may be some issues
> > that need to be addressed.
> >
> > I'll comment in the patches where my concerns are so the maintainers
> > may comment.
> >
> > v3:
> > * style, don't use c99 comments
> >
> > v2:
> > * comment code where optimizations may be possible now that memory
> > order can be specified.
> > * comment code where operations should potentially be atomic so that
> > maintainers can review.
> > * change a couple of variables labeled as counters to be unsigned.
> >
>
> I didn't see the v3 when ack'ing the v2, so in case v2 is quickly skipped by maintainers...
yeah, my fault. i hammed up the comment style used and needed to quickly
submit v3 to satisfy checkpatches.
thanks!
>
> Series-acked-by: Morten Brørup <mb at smartsharesystems.com>
More information about the dev
mailing list