[PATCH] devtools: extend check on compiler builtin atomics
David Marchand
david.marchand at redhat.com
Thu Oct 19 09:08:45 CEST 2023
On Tue, Oct 17, 2023 at 1:34 PM Tyler Retzlaff
<roretzla at linux.microsoft.com> wrote:
>
> On Tue, Oct 17, 2023 at 01:31:25PM +0200, David Marchand wrote:
> > rte_memory_order_* should be used when calling the new stdatomic API.
> > Add a check on __ATOMIC_* tokens.
> >
> > Signed-off-by: David Marchand <david.marchand at redhat.com>
> Acked-by: Tyler Retzlaff <roretzla at linux.microsoft.com>
>
[snip]
> > -v RET_ON_FAIL=1 \
> > - -v MESSAGE='Using __atomic_xxx built-ins, prefer rte_atomic_xxx' \
> > + -v MESSAGE='Using __atomic_xxx/__ATOMIC_XXX built-ins, prefer rte_atomic_xxx' \
>
> Not sure if it should say prefer rte_atomic_xxx/rte_memory_order_xxx ?
Looks ok to me, I did this small change while applying.
Applied, thanks.
--
David Marchand
More information about the dev
mailing list