Re: [PATCH 00/33] add Marvell cn20k SOC support for mempool and net

[Jerin Jacob <jerinjacobk@gmail.com>]

On Tue, Sep 10, 2024 at 2:54 PM Nithin Dabilpuram
<ndabilpuram@marvell.com> wrote:
>
> This series adds support for Marvell cn20k SOC for mempool and
> net PMD's.
>
> This series also adds few net/cnxk PMD updates to expose IPsec
> features supported by HW that are very custom in nature and
> some enhancements for cn10k.


# Please update release note for mempool cn20k driver support
# Please update release note for ethdev cn20k driver support
# Please split non cn20k driver patches to separate series.
# Please fix the following build issue


total: 0 errors, 0 warnings
Applying: net/cnxk: add PMD APIs to submit CPT instruction


ccache clang -Idrivers/libtmp_rte_net_cnxk.a.p -Idrivers -I../drivers
-Idrivers/net/cnxk -I../drivers/net/cnxk -Ilib/ethdev -I../lib/ethdev
-I. -I.. -Iconfig -I../config -Ilib/eal/include -I../lib/eal/include
-Ilib/eal/linux/include -I..
/lib/eal/linux/include -Ilib/eal/x86/include -I../lib/eal/x86/include
-Ilib/eal/common -I../lib/eal/common -Ilib/eal -I../lib/eal
-Ilib/kvargs -I../lib/kvargs -Ilib/log -I../lib/log -Ilib/metrics
-I../lib/metrics -Ilib/telemetry -I../lib
/telemetry -Ilib/net -I../lib/net -Ilib/mbuf -I../lib/mbuf
-Ilib/mempool -I../lib/mempool -Ilib/ring -I../lib/ring -Ilib/meter
-I../lib/meter -Idrivers/bus/pci -I../drivers/bus/pci
-I../drivers/bus/pci/linux -Ilib/pci -I../lib/pci -Idriv
ers/bus/vdev -I../drivers/bus/vdev -Ilib/cryptodev -I../lib/cryptodev
-Ilib/rcu -I../lib/rcu -Ilib/eventdev -I../lib/eventdev -Ilib/hash
-I../lib/hash -Ilib/timer -I../lib/timer -Ilib/dmadev -I../lib/dmadev
-Ilib/security -I../lib/securi
ty -Idrivers/common/cnxk -I../drivers/common/cnxk
-Idrivers/mempool/cnxk -I../drivers/mempool/cnxk
-fdiagnostics-color=always -D_FILE_OFFSET_BITS=64 -Wall -Winvalid-pch
-Wextra -Werror -std=c11 -O2 -g -include rte_config.h -Wcast-qual -W
deprecated -Wformat -Wformat-nonliteral -Wformat-security
-Wmissing-declarations -Wmissing-prototypes -Wnested-externs
-Wold-style-definition -Wpointer-arith -Wsign-compare
-Wstrict-prototypes -Wundef -Wwrite-strings -Wno-address-of-pack
ed-member -Wno-missing-field-initializers -D_GNU_SOURCE -fPIC
-march=native -mrtm -DALLOW_EXPERIMENTAL_API -DALLOW_INTERNAL_API
-Wno-format-truncation -flax-vector-conversions -Wno-strict-aliasing
-Wno-asm-operand-widths -DRTE_LOG_DEFAUL
T_LOGTYPE=pmd.net.cnxk -MD -MQ
drivers/libtmp_rte_net_cnxk.a.p/net_cnxk_cn20k_ethdev.c.o -MF
drivers/libtmp_rte_net_cnxk.a.p/net_cnxk_cn20k_ethdev.c.o.d -o
drivers/libtmp_rte_net_cnxk.a.p/net_cnxk_cn20k_ethdev.c.o -c
../drivers/net/cnxk/
cn20k_ethdev.c
In file included from ../drivers/net/cnxk/cn20k_ethdev.c:4:
In file included from ../drivers/net/cnxk/cn20k_ethdev.h:8:
../drivers/net/cnxk/cnxk_ethdev.h:516:8: error: address argument to
atomic operation must be a pointer to _Atomic type ('int32_t *' (aka
'int *') invalid)
  516 |         val = rte_atomic_fetch_sub_explicit(fc_sw, nb_inst,
__ATOMIC_RELAXED) - nb_inst;
      |               ^                             ~~~~~
../lib/eal/include/rte_stdatomic.h:95:2: note: expanded from macro
'rte_atomic_fetch_sub_explicit'
   95 |         atomic_fetch_sub_explicit(ptr, val, memorder)
      |         ^                         ~~~
/usr/lib/clang/18/include/stdatomic.h:154:35: note: expanded from
macro 'atomic_fetch_sub_explicit'
  154 | #define atomic_fetch_sub_explicit __c11_atomic_fetch_sub
      |                                   ^
In file included from ../drivers/net/cnxk/cn20k_ethdev.c:4:
In file included from ../drivers/net/cnxk/cn20k_ethdev.h:8:
../drivers/net/cnxk/cnxk_ethdev.h:520:30: error: address argument to
atomic operation must be a pointer to _Atomic type ('uint64_t *' (aka
'unsigned long *') invalid)
  520 |         newval = (int64_t)nb_desc -
rte_atomic_load_explicit(fc, __ATOMIC_RELAXED);
      |                                     ^                        ~~
../lib/eal/include/rte_stdatomic.h:73:2: note: expanded from macro
'rte_atomic_load_explicit'
   73 |         atomic_load_explicit(ptr, memorder)
      |         ^                    ~~~
/usr/lib/clang/18/include/stdatomic.h:139:30: note: expanded from
macro 'atomic_load_explicit'
  139 | #define atomic_load_explicit __c11_atomic_load
      |                              ^
In file included from ../drivers/net/cnxk/cn20k_ethdev.c:4:
In file included from ../drivers/net/cnxk/cn20k_ethdev.h:8:
../drivers/net/cnxk/cnxk_ethdev.h:523:7: error: address argument to
atomic operation must be a pointer to _Atomic type ('int32_t *' (aka
'int *') invalid)
  523 |         if
(!rte_atomic_compare_exchange_strong_explicit(fc_sw, &val, newval,
__ATOMIC_RELEASE,
      |              ^                                           ~~~~~
../lib/eal/include/rte_stdatomic.h:83:2: note: expanded from macro
'rte_atomic_compare_exchange_strong_explicit'
   83 |         atomic_compare_exchange_strong_explicit(ptr, expected,
desired, \
      |         ^                                       ~~~
/usr/lib/clang/18/include/stdatomic.h:145:49: note: expanded from
macro 'atomic_compare_exchange_strong_explicit'
  145 | #define atomic_compare_exchange_strong_explicit
__c11_atomic_compare_exchange_strong
      |                                                 ^
3 errors generated.

[for-main]dell[dpdk-next-net-mrvl] $ clang -v
clang version 18.1.8
Target: x86_64-pc-linux-gnu
Thread model: posix
InstalledDir: /usr/bin
Found candidate GCC installation: /usr/bin/../lib/gcc/x86_64-pc-linux-gnu/14.2.1
Found candidate GCC installation:
/usr/bin/../lib64/gcc/x86_64-pc-linux-gnu/14.2.1
Selected GCC installation: /usr/bin/../lib64/gcc/x86_64-pc-linux-gnu/14.2.1
Candidate multilib: .;@m64
Candidate multilib: 32;@m32
Selected multilib: .;@m64