David Marchand <david.march...@redhat.com> writes:
> Subject: Re: [PATCH v2 0/4] use C11 atomic builtins for libs > > On Wed, Sep 16, 2020 at 10:24 AM Phil Yang <phil.y...@arm.com> wrote: > > > > Since rte_atomicXX APIs are not allowed to be used[1][2], use C11 atomic > > builtins instead in eal, bbdev, power, and ethdev libs. > > > > [1] > http://code.dpdk.org/dpdk/latest/source/doc/guides/rel_notes/deprecatio > n.rst#L87 > > [2] > http://code.dpdk.org/dpdk/latest/source/devtools/checkpatches.sh#L80 > > > > v2: > > Fix Clang int-conversion warning. > > > > v1: > > Initial version. > > > > Phil Yang (4): > > eal: use C11 atomic builtins for already initialized check > > bbdev: use C11 atomic builtins for device processing counter > > power: use C11 atomic builtins for power in use state update > > ethdev: use C11 atomic builtins for link status update > > Thanks Phil. > Just a small comment on the first patch, the rest lgtm. > > Nicolas, David H., could you have a look at (resp.) patch 2, 3? Thanks for your comments. I will address them in the next version. Thanks, Phil > > > -- > David Marchand