From: David Marchand <david.marchand@redhat.com>
To: dev@dpdk.org
Cc: bruce.richardson@intel.com, Aaron Conole <aconole@redhat.com>
Subject: [PATCH v2 9/9] ci: run reference binaries against current ABI
Date: Thu, 23 Jul 2026 14:55:58 +0200 [thread overview]
Message-ID: <20260723125559.3087579-10-david.marchand@redhat.com> (raw)
In-Reply-To: <20260723125559.3087579-1-david.marchand@redhat.com>
Let's check if a testpmd and unit tests from a reference build can run
with the current release libraries/drivers.
This gives another level of ABI checking, like catching regressions on
experimental or internal ABI symbols that get promoted to stable.
We may need to skip unit tests that were added since the reference.
So add a ABI_SKIP_TESTS environment variable in preparation.
Signed-off-by: David Marchand <david.marchand@redhat.com>
---
Changes since v1:
- dropped hack, testing this patch can't be done in the CI now,
- emptied the excluded tests list,
- added test-null.sh,
---
.ci/linux-build.sh | 33 ++++++++++++++++++++++++++++++---
.github/workflows/build.yml | 1 +
2 files changed, 31 insertions(+), 3 deletions(-)
diff --git a/.ci/linux-build.sh b/.ci/linux-build.sh
index a5e602ed83..032f3726e8 100755
--- a/.ci/linux-build.sh
+++ b/.ci/linux-build.sh
@@ -206,23 +206,33 @@ if [ "$ABI_CHECKS" = "true" ]; then
# don't try to link apps.
REF_OPTS="$REF_OPTS -Dcheck_includes=false"
REF_OPTS="$REF_OPTS -Ddeveloper_mode=disabled"
- REF_OPTS="$REF_OPTS -Ddisable_apps=*"
+ REF_OPTS="$REF_OPTS -Denable_apps=test,test-pmd"
REF_OPTS="$REF_OPTS -Denable_docs=false"
REF_OPTS="$REF_OPTS -Dexamples="
- REF_OPTS="$REF_OPTS -Dtests=false"
refsrcdir=$(readlink -f $(pwd)/../dpdk-$REF_GIT_TAG)
git clone --single-branch -b "$REF_GIT_TAG" $REF_GIT_REPO $refsrcdir
meson setup $REF_OPTS $refsrcdir $refsrcdir/build
ninja -C $refsrcdir/build
DESTDIR=$(pwd)/reference meson install -C $refsrcdir/build
find reference/usr/local -name '*.a' -delete
- rm -rf reference/usr/local/bin
+ rm -rf reference/usr/local/bin/*
+ cp $refsrcdir/build/app/dpdk-test reference/usr/local/bin/dpdk-test
+ cp $refsrcdir/build/app/dpdk-testpmd reference/usr/local/bin/dpdk-testpmd
rm -rf reference/usr/local/share
echo $REF_GIT_TAG > reference/VERSION
fi
DESTDIR=$(pwd)/install meson install -C build
devtools/check-abi.sh reference install ${ABI_CHECKS_WARN_ONLY:-}
+
+ failed=
+ configure_coredump
+ mv -f build/app/dpdk-testpmd build/app/dpdk-testpmd.ori
+ cp reference/usr/local/bin/dpdk-testpmd build/app/dpdk-testpmd
+ devtools/test-null.sh || failed="true"
+ mv -f build/app/dpdk-testpmd.ori build/app/dpdk-testpmd
+ catch_coredump
+ [ "$failed" != "true" ]
fi
if [ "$RUN_TESTS" = "true" ]; then
@@ -233,6 +243,23 @@ if [ "$RUN_TESTS" = "true" ]; then
catch_ubsan DPDK:fast-tests build/meson-logs/testlog.txt
check_traces
[ "$failed" != "true" ]
+
+ if [ "$ABI_CHECKS" = "true" ]; then
+ failed=
+ configure_coredump
+ mv -f build/app/dpdk-test build/app/dpdk-test.ori
+ cp reference/usr/local/bin/dpdk-test build/app/dpdk-test
+ for t in ${ABI_SKIP_TESTS}; do
+ DPDK_TEST_SKIP=${DPDK_TEST_SKIP+$DPDK_TEST_SKIP,}$t
+ done
+ sudo env DPDK_TEST_SKIP="$DPDK_TEST_SKIP" \
+ meson test -C build --suite fast-tests -t 3 --no-stdsplit --print-errorlogs || failed="true"
+ catch_coredump
+ catch_ubsan DPDK:fast-tests build/meson-logs/testlog.txt
+ check_traces
+ mv -f build/app/dpdk-test.ori build/app/dpdk-test
+ [ "$failed" != "true" ]
+ fi
fi
# Test examples compilation with an installed dpdk
diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
index c63a9a3982..5ee55f4d27 100644
--- a/.github/workflows/build.yml
+++ b/.github/workflows/build.yml
@@ -39,6 +39,7 @@ jobs:
env:
AARCH64: ${{ matrix.config.cross == 'aarch64' }}
ABI_CHECKS: ${{ contains(matrix.config.checks, 'abi') }}
+ ABI_SKIP_TESTS: none
ASAN: ${{ contains(matrix.config.checks, 'asan') }}
BUILD_32BIT: ${{ matrix.config.cross == 'i386' }}
BUILD_DEBUG: ${{ contains(matrix.config.checks, 'debug') }}
--
2.54.0
prev parent reply other threads:[~2026-07-23 12:57 UTC|newest]
Thread overview: 36+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-07-17 9:29 [PATCH 00/10] Limit usage of internal API in tests David Marchand
2026-07-17 9:29 ` [PATCH 01/10] bbdev: fix stats aggregation from queues David Marchand
2026-07-17 17:22 ` Chautru, Nicolas
2026-07-18 7:47 ` David Marchand
2026-07-17 9:29 ` [PATCH 02/10] bbdev: add per-queue statistics API David Marchand
2026-07-17 9:29 ` [PATCH 03/10] hash: fix GFNI stubs export David Marchand
2026-07-17 9:38 ` Bruce Richardson
2026-07-17 9:55 ` Konstantin Ananyev
2026-07-17 9:29 ` [PATCH 04/10] test: uninline helper for forking David Marchand
2026-07-17 9:39 ` Bruce Richardson
2026-07-17 9:30 ` [PATCH 05/10] test/bonding: get MAC address with public API David Marchand
2026-07-17 9:30 ` [PATCH 06/10] test/devargs: check driver presence " David Marchand
2026-07-17 9:55 ` Bruce Richardson
2026-07-17 10:08 ` David Marchand
2026-07-17 10:14 ` Bruce Richardson
2026-07-17 12:34 ` David Marchand
2026-07-17 12:56 ` Thomas Monjalon
2026-07-17 9:30 ` [PATCH 07/10] test/vdev: find device " David Marchand
2026-07-17 9:30 ` [PATCH 08/10] test: limit internal API usage David Marchand
2026-07-17 9:30 ` [PATCH 09/10] ci: make ABI reference generation faster David Marchand
2026-07-17 9:30 ` [PATCH 10/10] ci: run reference unit tests against ABI David Marchand
2026-07-17 9:35 ` [PATCH 00/10] Limit usage of internal API in tests David Marchand
2026-07-23 12:55 ` [PATCH v2 0/9] " David Marchand
2026-07-23 12:55 ` [PATCH v2 1/9] bbdev: add per-queue statistics API David Marchand
2026-07-23 12:55 ` [PATCH v2 2/9] hash: fix GFNI stubs export David Marchand
2026-07-23 12:55 ` [PATCH v2 3/9] test: uninline helper for forking David Marchand
2026-07-23 12:55 ` [PATCH v2 4/9] test/bonding: get MAC address with public API David Marchand
2026-07-23 13:30 ` Bruce Richardson
2026-07-23 12:55 ` [PATCH v2 5/9] test/devargs: rely on net null driver David Marchand
2026-07-23 13:31 ` Bruce Richardson
2026-07-23 12:55 ` [PATCH v2 6/9] test/vdev: find device with public API David Marchand
2026-07-23 12:55 ` [PATCH v2 7/9] test: limit internal API usage David Marchand
2026-07-23 13:33 ` Bruce Richardson
2026-07-23 15:04 ` Morten Brørup
2026-07-23 12:55 ` [PATCH v2 8/9] ci: make ABI reference generation faster David Marchand
2026-07-23 12:55 ` David Marchand [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20260723125559.3087579-10-david.marchand@redhat.com \
--to=david.marchand@redhat.com \
--cc=aconole@redhat.com \
--cc=bruce.richardson@intel.com \
--cc=dev@dpdk.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox