mbox series

[0/6] skip ENOTSUPP for BPF selftests

Message ID cover.1720075006.git.tanggeliang@kylinos.cn (mailing list archive)
Headers show
Series skip ENOTSUPP for BPF selftests | expand

Message

Geliang Tang July 4, 2024, 6:48 a.m. UTC
From: Geliang Tang <tanggeliang@kylinos.cn>

BPF selftests seem to have not been fully tested on Loongarch platforms.
There are so many "ENOTSUPP" (-524) errors when running BPF selftests on
them since lacking BPF trampoline on Loongarch.

For these "ENOTSUPP" tests, it's better to skip them, instead of reporting
some "ENOTSUPP" errors. This patchset skips ENOTSUPP in ASSERT_OK/
ASSERT_OK_PTR/ASSERT_GE helpers to fix them. This is useful for running BPF
selftests for other architectures too.

Geliang Tang (6):
  selftests/bpf: Define ENOTSUPP in testing_helpers.h
  selftests/bpf: Skip ENOTSUPP in ASSERT_OK
  selftests/bpf: Use ASSERT_OK to skip ENOTSUPP
  selftests/bpf: Null checks for link in bpf_tcp_ca
  selftests/bpf: Skip ENOTSUPP in ASSERT_OK_PTR
  selftests/bpf: Skip ENOTSUPP in ASSERT_GE

 .../selftests/bpf/prog_tests/bpf_tcp_ca.c     | 20 +++++++++-------
 .../testing/selftests/bpf/prog_tests/d_path.c |  2 +-
 .../selftests/bpf/prog_tests/lsm_cgroup.c     | 10 +-------
 .../selftests/bpf/prog_tests/module_attach.c  |  2 +-
 .../selftests/bpf/prog_tests/ringbuf.c        |  2 +-
 .../selftests/bpf/prog_tests/sock_addr.c      |  4 ----
 .../selftests/bpf/prog_tests/test_bprm_opts.c |  2 +-
 .../selftests/bpf/prog_tests/test_ima.c       |  2 +-
 .../selftests/bpf/prog_tests/trace_ext.c      |  2 +-
 tools/testing/selftests/bpf/test_maps.c       |  4 ----
 tools/testing/selftests/bpf/test_progs.h      | 24 ++++++++++++++-----
 tools/testing/selftests/bpf/test_verifier.c   |  4 ----
 tools/testing/selftests/bpf/testing_helpers.h |  4 ++++
 13 files changed, 41 insertions(+), 41 deletions(-)

Comments

Geliang Tang July 5, 2024, 2:40 a.m. UTC | #1
On Thu, 2024-07-04 at 14:48 +0800, Geliang Tang wrote:
> From: Geliang Tang <tanggeliang@kylinos.cn>
> 
> BPF selftests seem to have not been fully tested on Loongarch
> platforms.
> There are so many "ENOTSUPP" (-524) errors when running BPF selftests
> on
> them since lacking BPF trampoline on Loongarch.
> 
> For these "ENOTSUPP" tests, it's better to skip them, instead of
> reporting
> some "ENOTSUPP" errors. This patchset skips ENOTSUPP in ASSERT_OK/
> ASSERT_OK_PTR/ASSERT_GE helpers to fix them. This is useful for
> running BPF
> selftests for other architectures too.

Superseded. v2 is just sent out.

Thanks,
-Geliang

> 
> Geliang Tang (6):
>   selftests/bpf: Define ENOTSUPP in testing_helpers.h
>   selftests/bpf: Skip ENOTSUPP in ASSERT_OK
>   selftests/bpf: Use ASSERT_OK to skip ENOTSUPP
>   selftests/bpf: Null checks for link in bpf_tcp_ca
>   selftests/bpf: Skip ENOTSUPP in ASSERT_OK_PTR
>   selftests/bpf: Skip ENOTSUPP in ASSERT_GE
> 
>  .../selftests/bpf/prog_tests/bpf_tcp_ca.c     | 20 +++++++++-------
>  .../testing/selftests/bpf/prog_tests/d_path.c |  2 +-
>  .../selftests/bpf/prog_tests/lsm_cgroup.c     | 10 +-------
>  .../selftests/bpf/prog_tests/module_attach.c  |  2 +-
>  .../selftests/bpf/prog_tests/ringbuf.c        |  2 +-
>  .../selftests/bpf/prog_tests/sock_addr.c      |  4 ----
>  .../selftests/bpf/prog_tests/test_bprm_opts.c |  2 +-
>  .../selftests/bpf/prog_tests/test_ima.c       |  2 +-
>  .../selftests/bpf/prog_tests/trace_ext.c      |  2 +-
>  tools/testing/selftests/bpf/test_maps.c       |  4 ----
>  tools/testing/selftests/bpf/test_progs.h      | 24 ++++++++++++++---
> --
>  tools/testing/selftests/bpf/test_verifier.c   |  4 ----
>  tools/testing/selftests/bpf/testing_helpers.h |  4 ++++
>  13 files changed, 41 insertions(+), 41 deletions(-)
>