]> git.ipfire.org Git - thirdparty/kernel/stable.git/commit
selftests/bpf: Fix two net related test failures with 64K page size
authorYonghong Song <yonghong.song@linux.dev>
Thu, 12 Jun 2025 03:50:37 +0000 (20:50 -0700)
committerAlexei Starovoitov <ast@kernel.org>
Fri, 13 Jun 2025 02:07:51 +0000 (19:07 -0700)
commit96fcf7e7a71c2b21c002e9ba9362d88f8beac09a
tree3b8719b762b93e35d048e27d32467b975934ac5e
parent4fc012daf9c074772421c904357abf586336b1ca
selftests/bpf: Fix two net related test failures with 64K page size

When running BPF selftests on arm64 with a 64K page size, I encountered
the following two test failures:
  sockmap_basic/sockmap skb_verdict change tail:FAIL
  tc_change_tail:FAIL

With further debugging, I identified the root cause in the following
kernel code within __bpf_skb_change_tail():

    u32 max_len = BPF_SKB_MAX_LEN;
    u32 min_len = __bpf_skb_min_len(skb);
    int ret;

    if (unlikely(flags || new_len > max_len || new_len < min_len))
        return -EINVAL;

With a 4K page size, new_len = 65535 and max_len = 16064, the function
returns -EINVAL. However, With a 64K page size, max_len increases to
261824, allowing execution to proceed further in the function. This is
because BPF_SKB_MAX_LEN scales with the page size and larger page sizes
result in higher max_len values.

Updating the new_len parameter in both tests based on actual kernel
page size resolved both failures.

Signed-off-by: Yonghong Song <yonghong.song@linux.dev>
Link: https://lore.kernel.org/r/20250612035037.2207911-1-yonghong.song@linux.dev
Signed-off-by: Alexei Starovoitov <ast@kernel.org>
tools/testing/selftests/bpf/progs/test_sockmap_change_tail.c
tools/testing/selftests/bpf/progs/test_tc_change_tail.c