Re: [PATCH] devtools: exclude atomic checks for common cnxk

[Jerin Jacob <jerinjacobk@gmail.com>]

On Thu, Mar 27, 2025 at 4:50 PM <pbhagavatula@marvell.com> wrote:
>
> From: Pavan Nikhilesh <pbhagavatula@marvell.com>
>
> Since driver/common/cnxk has common code that is
> used across multiple platforms, allow use of
> GCC builtin atomic functions.
>
> Signed-off-by: Pavan Nikhilesh <pbhagavatula@marvell.com>

Acked-by: Jerin Jacob <jerinj@marvell.com>

> ---
>  Rest of drivers i.e., drivers/*/cnxk have been migrated to use stdatomics in:
>  https://patchwork.dpdk.org/project/dpdk/patch/20250327111120.1641-1-pbhagavatula@marvell.com/
>
>  devtools/checkpatches.sh | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/devtools/checkpatches.sh b/devtools/checkpatches.sh
> index c9088bb403..2470d4fa83 100755
> --- a/devtools/checkpatches.sh
> +++ b/devtools/checkpatches.sh
> @@ -139,6 +139,7 @@ check_forbidden_additions() { # <patch>
>
>         # refrain from using compiler __atomic_xxx builtins
>         awk -v FOLDERS="lib drivers app examples" \
> +               -v SKIP_FILES='drivers/common/cnxk/' \
>                 -v EXPRESSIONS="__atomic_.*\\\( __ATOMIC_(RELAXED|CONSUME|ACQUIRE|RELEASE|ACQ_REL|SEQ_CST)" \
>                 -v RET_ON_FAIL=1 \
>                 -v MESSAGE='Using __atomic_xxx/__ATOMIC_XXX built-ins, prefer rte_atomic_xxx/rte_memory_order_xxx' \
> --
> 2.43.0
>