On Fri, 31 Oct 2025 12:43:09 +0100 Thomas Monjalon <[email protected]> wrote:
> When running on limited platforms like GitHub Actions, > the functional unit test "memcpy_autotest" > may hit a timeout, especially when running with UBSan. > > This change skips testing some alignment offsets (from 0 to 31). > It will test only every 3-byte offsets for the source buffer, > and every 2-byte offsets for the destination buffer. > So it is supposed to be 6x faster with a reasonably smaller coverage. > > Signed-off-by: Thomas Monjalon <[email protected]> > --- > app/test/test_memcpy.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/app/test/test_memcpy.c b/app/test/test_memcpy.c > index 802dc4631b..ea9e76e9ad 100644 > --- a/app/test/test_memcpy.c > +++ b/app/test/test_memcpy.c > @@ -105,8 +105,8 @@ func_test(void) > unsigned int off_src, off_dst, i; > int ret; > > - for (off_src = 0; off_src < ALIGNMENT_UNIT; off_src++) { > - for (off_dst = 0; off_dst < ALIGNMENT_UNIT; off_dst++) { > + for (off_src = 0; off_src < ALIGNMENT_UNIT; off_src += 3) { > + for (off_dst = 0; off_dst < ALIGNMENT_UNIT; off_dst += 2) { > for (i = 0; i < RTE_DIM(buf_sizes); i++) { > ret = test_single_memcpy(off_src, off_dst, > buf_sizes[i]); This was fixed by later patch. commit bc15681021d5588072f0635ccb9937d127c64b3d Author: Stephen Hemminger <[email protected]> Date: Thu Feb 26 08:48:43 2026 -0800 test/memcpy: reduce alignment offset coverage The memcpy test sweeps all 32x32 src/dst alignment offset pairs which causes it to timeout on slow emulated 32-bit build environments [1]. Replace with a curated set of 7 offsets {0, 1, 7, 15, 16, 17, 31} that cover the interesting alignment boundaries. This reduces the iterations from 38912 to 1862 while covering the same code paths. [1] https://build.opensuse.org/package/live_build_log/home:bluca:dpdk/dpdk/Debian_Testing/i586 Reported-by: Luca Boccassi <[email protected]> Signed-off-by: Stephen Hemminger <[email protected]>
