On 26 Jul 2026, at 17:33, Eli Britstein wrote:
> From: Ariel Levkovich <[email protected]>
>
> Add a new option to build ovs with doca by specifying '--with-doca' in the
> configure line.
>
> This flag must be used along with '--with-dpdk', otherwise the configure
> step will fail.
>
> An example:
>
> ./configure --prefix=/usr --localstatedir=/var --sysconfdir=/etc \
> --with-dpdk=static --with-doca
>
> The link type follows the selected dpdk one.
>
> Co-authored-by: Salem Sol <[email protected]>
> Signed-off-by: Salem Sol <[email protected]>
> Co-authored-by: Eli Britstein <[email protected]>
> Signed-off-by: Eli Britstein <[email protected]>
> Signed-off-by: Ariel Levkovich <[email protected]>
Hi Eli,
Found one more real problem with this patch, see below.
//Eelco
[...]
> diff --git a/.ci/doca-build.sh b/.ci/doca-build.sh
> new file mode 100755
> index 000000000..70c791bf7
> --- /dev/null
> +++ b/.ci/doca-build.sh
> @@ -0,0 +1,54 @@
> +#!/bin/bash
> +
> +set -o errexit
> +set -x
> +
> +CFLAGS_FOR_OVS="-g -O2"
> +EXTRA_OPTS="--enable-Werror"
> +JOBS=${JOBS:-"-j4"}
> +
> +DOCA_LINK="${DOCA_LINK:-static}"
> +
> +DPDK_INSTALL_DIR="$(pwd)/dpdk-dir"
> +DPDK_LIB="${DPDK_INSTALL_DIR}/lib/x86_64-linux-gnu"
> +DOCA_PKGCONFIG=$(find /opt/mellanox/doca -name pkgconfig -type d \
> + 2>/dev/null | head -1)
> +
> +if [ ! -f "${DPDK_INSTALL_DIR}/cached-version" ]; then
> + echo "Could not find DPDK in ${DPDK_INSTALL_DIR}"
> + exit 1
> +fi
> +
> +echo "Found cached DPDK $(cat ${DPDK_INSTALL_DIR}/cached-version)" \
> + "build in ${DPDK_INSTALL_DIR}"
> +
> +PKG_CONFIG_PATH="${DPDK_LIB}/pkgconfig:${DOCA_PKGCONFIG}:${PKG_CONFIG_PATH}"
> +export PKG_CONFIG_PATH
> +export PATH="${DPDK_INSTALL_DIR}/bin:${PATH}"
> +
> +if [ "$DOCA_LINK" = "shared" ]; then
> + DOCA_LIB="${DOCA_PKGCONFIG%/pkgconfig}"
> + export LD_LIBRARY_PATH="${DPDK_LIB}:${DOCA_LIB}:${LD_LIBRARY_PATH}"
> +fi
> +
> +sudo ldconfig
> +EXTRA_OPTS="$EXTRA_OPTS --with-dpdk=$DOCA_LINK --with-doca"
> +
> +./boot.sh
> +./configure CFLAGS="${CFLAGS_FOR_OVS}" $EXTRA_OPTS
> +make ${JOBS} check TESTSUITEFLAGS="${JOBS} RECHECK=yes"
Should be:
make ${JOBS} check TESTSUITEFLAGS="${JOBS}" RECHECK=yes
[...]
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
