Re: FW: compilation|FAILURE| pw(141419) sid(32237) job(PER_PATCH_BUILD12332)[v2,6/6] eal: provide option to use compiler memcpy instead of RTE

Re: FW: compilation|FAILURE| pw(141419) sid(32237) job(PER_PATCH_BUILD12332)[v2,6/6] eal: provide option to use compiler memcpy instead of RTE

[Mattias Rönnblom]

On 2024-06-20 10:11, Mattias Rönnblom wrote:
> 
> 
> -----Original Message-----
> From: sys_stv@intel.com <sys_stv@intel.com>
> Sent: Thursday, 20 June 2024 09:55
> To: test-report@dpdk.org; Mattias Rönnblom <mattias.ronnblom@ericsson.com>
> Subject: compilation|FAILURE| pw(141419) sid(32237) job(PER_PATCH_BUILD12332)[v2,6/6] eal: provide option to use compiler memcpy instead of RTE
> 
> 
> Test-Label: Intel-compilation
> Test-Status: FAILURE
> http://dpdk.org/patch/141419
> 
> _Compilation issues_
> 
> Submitter: Mattias Rönnblom <mattias.ronnblom@ericsson.com>
> Date: 2024-06-20 07:24:52
> Reply_mail: <20240620072452.420029-7-mattias.ronnblom@ericsson.com>
> 
> DPDK git baseline: Repo:dpdk, CommitID: 4a44d97f0a52a76258c6a6cb6a713f4380a8ab1f
> 
> 
> Meson Build Summary: 23 Builds Done, 22 Successful, 1 Failures, 0 Blocked
> 
> +-------------------+------------+--------------+------------+------------+-----------+----------+------------+
> | os                | gcc-static | clang-static | icc-static | gcc-shared | gcc-debug | document | gcc-16byte |
> +-------------------+------------+--------------+------------+------------+-----------+----------+------------+
> | OpenAnolis8.8-64  | pass       |              |            |            |           |          |            |
> | FreeBSD14-64      | pass       | pass         |            | pass       | pass      |          |            |
> | RHEL94-64         | pass       | pass         |            | pass       | pass      |          |            |
> | SUSE15-64         | pass       | pass         |            |            |           |          |            |
> | CBL-Mariner2.0-64 | pass       |              |            |            |           |          |            |
> | UB2404-32         | fail       |              |            |            |           |          |            |
> | RHEL93-64         | pass       |              |            |            |           |          |            |
> | UB2404-64         | pass       | pass         |            |            |           | pass     | pass       |
> | RHEL94-64Rt       | pass       |              |            |            |           |          |            |
> | UB2204-64         | pass       |              |            |            |           |          |            |
> | FC40-64           | pass       | pass         |            |            |           |          |            |
> | UB2404-64Rt       | pass       |              |            |            |           |          |            |
> +-------------------+------------+--------------+------------+------------+-----------+----------+------------+
> 
> Comments:
> Because of DPDK bug (https://bugs.dpdk.org/show_bug.cgi?id=928),
> All the dpdk-next-* branch add `Ddisable_drivers=event/cnxk` option when build with ICC complier.
> 
> Test environment and configuration as below:
> 
> 
> OS: OpenAnolis8.8-64
>          Kernel Version: 5.10.134-13.an8.x86_64
>          GCC Version: gcc (GCC) 8.5.0 20210514 (Anolis 8.5.0-10.0.3)
>          Clang Version: 13.0.1 (Anolis 13.0.1-2.0.2.module+an8.7.0+10996+1588f068)
>          x86_64-native-linuxapp-gcc
> 
> OS: FreeBSD14-64
>          Kernel Version: 14.0-RELEASE
>          GCC Version: gcc (FreeBSD Ports Collection) 12.2.0
>          Clang Version: 16.0.6 (https://github.com/llvm/llvm-project.git llvmorg-16.0.6-0-g7cbf1a259152)
>          x86_64-native-bsdapp-gcc
>          x86_64-native-bsdapp-clang
>          x86_64-native-bsdapp-gcc+shared
>          x86_64-native-bsdapp-gcc+debug
> 
> OS: RHEL94-64
>          Kernel Version: 5.14.0-427.13.1.el9_4.x86_64
>          GCC Version: gcc (GCC) 11.4.1 20231218 (Red Hat 11.4.1-3)
>          Clang Version: 17.0.6 (Red Hat, Inc. 17.0.6-5.el9)
>          x86_64-native-linuxapp-gcc
>          x86_64-native-linuxapp-clang
>          x86_64-native-linuxapp-gcc+shared
>          x86_64-native-linuxapp-gcc+debug
> 
> OS: SUSE15-64
>          Kernel Version: 5.14.21-150500.53-default
>          GCC Version: gcc (SUSE Linux) 7.5.0
>          Clang Version: 15.0.7
>          x86_64-native-linuxapp-clang
>          x86_64-native-linuxapp-gcc
> 
> OS: CBL-Mariner2.0-64
>          Kernel Version: 5.15.55.1_2e9a4f9+
>          GCC Version: gcc (GCC) 11.2.0
>          Clang Version: NA
>          x86_64-native-linuxapp-gcc
> 
> OS: UB2404-32
>          Kernel Version: 6.8.0-31-generic
>          GCC Version: gcc (Ubuntu 13.2.0-23ubuntu4) 13.2.0
>          Clang Version: NA
>          i686-native-linuxapp-gcc
> 
> OS: RHEL93-64
>          Kernel Version: 5.14.0-362.8.1.el9_3.x86_64
>          GCC Version: gcc (GCC) 11.4.1 20231218 (Red Hat 11.4.1-3)
>          Clang Version: 17.0.6 (Red Hat, Inc. 17.0.6-5.el9)
>          x86_64-native-linuxapp-gcc
> 
> OS: UB2404-64
>          Kernel Version: 6.8.0-31-generic
>          GCC Version: gcc (Ubuntu 13.2.0-23ubuntu4) 13.2.0
>          Clang Version: 18.1.3 (1)
>          x86_64-native-linuxapp-gcc+16byte
>          x86_64-native-linuxapp-gcc
>          x86_64-native-linuxapp-clang
>          x86_64-native-linuxapp-doc
> 
> OS: RHEL94-64Rt
>          Kernel Version: 5.14.0-427.13.1.el9_4.x86_64+rt
>          GCC Version: gcc (GCC) 11.4.1 20231218 (Red Hat 11.4.1-3)
>          Clang Version: 17.0.6 (Red Hat, Inc. 17.0.6-5.el9)
>          x86_64-native-linuxapp-gcc
> 
> OS: UB2204-64
>          Kernel Version: 5.15.0-94-generic
>          GCC Version: gcc (Ubuntu 11.4.0-1ubuntu1~22.04) 11.4.0
>          Clang Version: 14.0.0-1ubuntu1.1
>          x86_64-native-linuxapp-gcc
> 
> OS: FC40-64
>          Kernel Version: 6.8.5-301.fc40.x86_64
>          GCC Version: gcc (GCC) 14.0.1 20240411 (Red Hat 14.0.1-0)
>          Clang Version: 18.1.1 (Fedora 18.1.1-1.fc40)
>          x86_64-native-linuxapp-gcc
>          x86_64-native-linuxapp-clang
> 
> OS: UB2404-64Rt
>          Kernel Version: 6.8.0-rt8
>          GCC Version: gcc (Ubuntu 13.2.0-23ubuntu4) 13.2.0
>          Clang Version: 18.1.3 (1)
>          x86_64-native-linuxapp-gcc
> 
> 
> 
> *Build Failed #1:
> OS: UB2404-32
> Target: i686-native-linuxapp-gcc
> FAILED: drivers/libtmp_rte_net_fm10k.a.p/net_fm10k_fm10k_rxtx_vec.c.o
> gcc -Idrivers/libtmp_rte_net_fm10k.a.p -Idrivers -I../drivers -Idrivers/net/fm10k -I../drivers/net/fm10k -Idrivers/net/fm10k/base -I../drivers/net/fm10k/base -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 -Idrivers/bus/vdev -I../drivers/bus/vdev -fdiagnostics-color=always -D_FILE_OFFSET_BITS=64 -Wall -Winvalid-pch -Wextra -Werror -std=c11 -O3 -include rte_config.h -Wcast-qual -Wdeprecated -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-packed-member -Wno-packed-not-aligned -Wno-missing-field-initializers -Wno-zero-length-bounds -Wno-pointer-to-int-cast -D_GNU_SOURCE -m32 -fPIC -march=native -mrtm -DALLOW_EXPERIMENTAL_API -DALLOW_INTERNAL_API -Wno-format-truncation -DRTE_LOG_DEFAULT_LOGTYPE=pmd.net.fm10k -MD -MQ drivers/libtmp_rte_net_fm10k.a.p/net_fm10k_fm10k_rxtx_vec.c.o -MF drivers/libtmp_rte_net_fm10k.a.p/net_fm10k_fm10k_rxtx_vec.c.o.d -o drivers/libtmp_rte_net_fm10k.a.p/net_fm10k_fm10k_rxtx_vec.c.o -c ../drivers/net/fm10k/fm10k_rxtx_vec.c
> ../drivers/net/fm10k/fm10k_rxtx_vec.c: In function ‘fm10k_desc_to_olflags_v’:
> ../drivers/net/fm10k/fm10k_rxtx_vec.c:132:21: error: implicit declaration of function ‘_mm_cvtsi128_si64’; did you mean ‘_mm_cvtsi128_si16’? [-Werror=implicit-function-declaration]
>    132 |         vol.dword = _mm_cvtsi128_si64(vtag1);

 From what I can tell, _mm_cvtsi128_si64() is only available on 64-bit 
x86. I fail to understand how this code could ever compile on 32-bit.

A somewhat unrelated question: why are there no maintainers listed for 
many of the Intel drivers?

>        |                     ^~~~~~~~~~~~~~~~~
>        |                     _mm_cvtsi128_si16
> ../drivers/net/fm10k/fm10k_rxtx_vec.c:132:21: error: nested extern declaration of ‘_mm_cvtsi128_si64’ [-Werror=nested-externs]
> cc1: all warnings being treated as errors [1133/2279] Compiling C object drivers/libtmp_rte_net_gve.a.p/net_gve_gve_version.c.o
> [1134/2279] Compiling C object drivers/libtmp_rte_net_fm10k.a.p/net_fm10k_fm10k_ethdev.c.o
> [1135/2279] Compiling C object drivers/libtmp_rte_net_gve.a.p/net_gve_gve_rx_dqo.c.o
> [1136/2279] Compiling C object drivers/libtmp_rte_net_gve.a.p/net_gve_gve_tx_dqo.c.o
> [1137/2279] Compiling C object drivers/libtmp_rte_net_failsafe.a.p/net_failsafe_failsafe_ops.c.o
> [1138/2279] Compiling C object drivers/libtmp_rte_net_gve.a.p/net_gve_base_gve_adminq.c.o
> [1139/2279] Compiling C object drivers/libtmp_rte_net_gve.a.p/net_gve_gve_rx.c.o
> [1140/2279] Compiling C object drivers/libtmp_rte_net_gve.a.p/net_gve_gve_tx.c.o
> [1141/2279] Compiling C object drivers/libtmp_rte_net_gve.a.p/net_gve_gve_ethdev.c.o
> ninja: build stopped
> 
> 
> DPDK STV team

Re: FW: compilation|FAILURE| pw(141419) sid(32237) job(PER_PATCH_BUILD12332)[v2,6/6] eal: provide option to use compiler memcpy instead of RTE

[Bruce Richardson]

On Thu, Jun 20, 2024 at 10:20:42AM +0200, Mattias Rönnblom wrote:
> On 2024-06-20 10:11, Mattias Rönnblom wrote:
> > 
> > 
> > -----Original Message----- From: sys_stv@intel.com <sys_stv@intel.com>
> > Sent: Thursday, 20 June 2024 09:55 To: test-report@dpdk.org; Mattias
> > Rönnblom <mattias.ronnblom@ericsson.com> Subject: compilation|FAILURE|
> > pw(141419) sid(32237) job(PER_PATCH_BUILD12332)[v2,6/6] eal: provide
> > option to use compiler memcpy instead of RTE
> > 
> > 

<snip>

> > *Build Failed #1: OS: UB2404-32 Target: i686-native-linuxapp-gcc
> > FAILED: drivers/libtmp_rte_net_fm10k.a.p/net_fm10k_fm10k_rxtx_vec.c.o
> > gcc -Idrivers/libtmp_rte_net_fm10k.a.p -Idrivers -I../drivers
> > -Idrivers/net/fm10k -I../drivers/net/fm10k -Idrivers/net/fm10k/base
> > -I../drivers/net/fm10k/base -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 -Idrivers/bus/vdev
> > -I../drivers/bus/vdev -fdiagnostics-color=always -D_FILE_OFFSET_BITS=64
> > -Wall -Winvalid-pch -Wextra -Werror -std=c11 -O3 -include rte_config.h
> > -Wcast-qual -Wdeprecated -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-packed-member -Wno-packed-not-aligned
> > -Wno-missing-field-initializers -Wno-zero-length-bounds
> > -Wno-pointer-to-int-cast -D_GNU_SOURCE -m32 -fPIC -march=native -mrtm
> > -DALLOW_EXPERIMENTAL_API -DALLOW_INTERNAL_API -Wno-format-truncation
> > -DRTE_LOG_DEFAULT_LOGTYPE=pmd.net.fm10k -MD -MQ
> > drivers/libtmp_rte_net_fm10k.a.p/net_fm10k_fm10k_rxtx_vec.c.o -MF
> > drivers/libtmp_rte_net_fm10k.a.p/net_fm10k_fm10k_rxtx_vec.c.o.d -o
> > drivers/libtmp_rte_net_fm10k.a.p/net_fm10k_fm10k_rxtx_vec.c.o -c
> > ../drivers/net/fm10k/fm10k_rxtx_vec.c
> > ../drivers/net/fm10k/fm10k_rxtx_vec.c: In function
> > ‘fm10k_desc_to_olflags_v’:
> > ../drivers/net/fm10k/fm10k_rxtx_vec.c:132:21: error: implicit
> > declaration of function ‘_mm_cvtsi128_si64’; did you mean
> > ‘_mm_cvtsi128_si16’? [-Werror=implicit-function-declaration] 132 |
> > vol.dword = _mm_cvtsi128_si64(vtag1);
> 
> From what I can tell, _mm_cvtsi128_si64() is only available on 64-bit
> x86. I fail to understand how this code could ever compile on 32-bit.
> 
> A somewhat unrelated question: why are there no maintainers listed for
> many of the Intel drivers?
> 

I can certainly answer this last question :-) A number of the DPDK team in PRC
who were our driver maintainers are no longer working on DPDK, and so
removed themselves from the maintainers file. Those of us based in Ireland
and India are ramping up on the drivers over time and should step up
officially as maintainers - especially for the most active drivers - in the
near future. The drivers are not so much unmaintained, as that we don't
have a single "best" name to put against them just yet.

/Bruce