Skip to content

Conversation

DawidWesierski4
Copy link
Collaborator

Check if the dpdk version for MTL is correct
MTL would not work with older DPDK's.

@DawidWesierski4
Copy link
Collaborator Author

FAILED: lib/libmtl.so.p/src_mt_cni.c.o
cc -Ilib/libmtl.so.p -Ilib -I../lib -I. -I.. -Iinclude -I../include -I/usr/local/include -I/usr/include/x86_64-linux-gnu -I/usr/include/libnl3 -I/usr/include/dbus-1.0 -I/usr/lib/x86_64-linux-gnu/dbus-1.0/include -I/usr/include/json-c -fdiagnostics-color=always -D_FILE_OFFSET_BITS=64 -Wall -Winvalid-pch -O3 -D_DEFAULT_SOURCE '-D__MTL_GIT__="d0792734"' '-D__MTL_LIB_BUILD__="25.02.0.DEV"' -DALLOW_EXPERIMENTAL_API -DMTL_HAS_XDP_BACKEND -DMTL_HAS_RDMA_BACKEND -DMTL_HAS_USDT -fPIC -include rte_config.h -march=native -mrtm -Werror -Wall -Wunused-parameter -msse4.2 -DMTL_HAS_AVX2 -DMTL_HAS_AVX512 -DMTL_HAS_AVX512_VBMI2 -MD -MQ lib/libmtl.so.p/src_mt_cni.c.o -MF lib/libmtl.so.p/src_mt_cni.c.o.d -o lib/libmtl.so.p/src_mt_cni.c.o -c ../lib/src/mt_cni.c
../lib/src/mt_cni.c: In function ‘cni_rx_handle’:
../lib/src/mt_cni.c:255:32: error: converting a packed ‘struct rte_ipv4_hdr’ pointer (alignment 1) to a ‘struct mt_ipv4_udp’ pointer (alignment 2) may result in an unaligned pointer value [-Werror=address-of-packed-member]
255 | (struct mt_ipv4_udp*)ipv4_hdr);
| ^~~~~~~~~~~
In file included from /usr/local/include/rte_flow.h:25,
from /usr/local/include/rte_eth_ctrl.h:11,
from /usr/local/include/rte_ethdev.h:1452,
from ../lib/src/mt_main.h:11,
from ../lib/src/mt_cni.h:8,
from ../lib/src/mt_cni.c:5:
/usr/local/include/rte_ip.h:41:8: note: defined here
41 | struct rte_ipv4_hdr {
| ^~~~~~~~~~~~
../lib/src/mt_main.h:119:8: note: defined here
119 | struct mt_ipv4_udp {
| ^~~~

  we need this due to this dpdk API difference 

@DawidWesierski4 DawidWesierski4 merged commit fda7130 into OpenVisualCloud:main Aug 21, 2025
19 of 22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants