diff options
| author | Alan Maguire <alan.maguire@oracle.com> | 2025-09-11 17:30:56 +0100 |
|---|---|---|
| committer | Daniel Borkmann <daniel@iogearbox.net> | 2025-09-15 20:39:24 +0200 |
| commit | 3ae4c527080ce81b889ffc2780e077770b95ae88 (patch) | |
| tree | a3e122290086991e91cb42d8b945b71d8adae82f /tools/testing/selftests/bpf/prog_tests | |
| parent | Merge branch 'remove-use-of-current-cgns-in-bpf_cgroup_from_id' (diff) | |
| download | linux-3ae4c527080ce81b889ffc2780e077770b95ae88.tar.gz linux-3ae4c527080ce81b889ffc2780e077770b95ae88.zip | |
selftests/bpf: More open-coded gettid syscall cleanup
Commit 0e2fb011a0ba ("selftests/bpf: Clean up open-coded gettid syscall
invocations") addressed the issue that older libc may not have a gettid()
function call wrapper for the associated syscall.
A few more instances have crept into tests, use sys_gettid() instead, and
poison raw gettid() usage to avoid future issues.
Signed-off-by: Alan Maguire <alan.maguire@oracle.com>
Signed-off-by: Daniel Borkmann <daniel@iogearbox.net>
Acked-by: Daniel Borkmann <daniel@iogearbox.net>
Link: https://lore.kernel.org/bpf/20250911163056.543071-1-alan.maguire@oracle.com
Diffstat (limited to 'tools/testing/selftests/bpf/prog_tests')
4 files changed, 4 insertions, 4 deletions
diff --git a/tools/testing/selftests/bpf/prog_tests/cgroup_xattr.c b/tools/testing/selftests/bpf/prog_tests/cgroup_xattr.c index e0dd966e4a3e..5ad904e9d15d 100644 --- a/tools/testing/selftests/bpf/prog_tests/cgroup_xattr.c +++ b/tools/testing/selftests/bpf/prog_tests/cgroup_xattr.c @@ -44,7 +44,7 @@ static void test_read_cgroup_xattr(void) if (!ASSERT_OK_PTR(skel, "read_cgroupfs_xattr__open_and_load")) goto out; - skel->bss->target_pid = gettid(); + skel->bss->target_pid = sys_gettid(); if (!ASSERT_OK(read_cgroupfs_xattr__attach(skel), "read_cgroupfs_xattr__attach")) goto out; diff --git a/tools/testing/selftests/bpf/prog_tests/kernel_flag.c b/tools/testing/selftests/bpf/prog_tests/kernel_flag.c index a133354ac9bc..97b00c7efe94 100644 --- a/tools/testing/selftests/bpf/prog_tests/kernel_flag.c +++ b/tools/testing/selftests/bpf/prog_tests/kernel_flag.c @@ -16,7 +16,7 @@ void test_kernel_flag(void) if (!ASSERT_OK_PTR(lsm_skel, "lsm_skel")) return; - lsm_skel->bss->monitored_tid = gettid(); + lsm_skel->bss->monitored_tid = sys_gettid(); ret = test_kernel_flag__attach(lsm_skel); if (!ASSERT_OK(ret, "test_kernel_flag__attach")) diff --git a/tools/testing/selftests/bpf/prog_tests/task_local_data.h b/tools/testing/selftests/bpf/prog_tests/task_local_data.h index a408d10c3688..2de38776a2d4 100644 --- a/tools/testing/selftests/bpf/prog_tests/task_local_data.h +++ b/tools/testing/selftests/bpf/prog_tests/task_local_data.h @@ -158,7 +158,7 @@ static int __tld_init_data_p(int map_fd) void *data_alloc = NULL; int err, tid_fd = -1; - tid_fd = syscall(SYS_pidfd_open, gettid(), O_EXCL); + tid_fd = syscall(SYS_pidfd_open, sys_gettid(), O_EXCL); if (tid_fd < 0) { err = -errno; goto out; diff --git a/tools/testing/selftests/bpf/prog_tests/test_task_local_data.c b/tools/testing/selftests/bpf/prog_tests/test_task_local_data.c index 3b5cd2cd89c7..9fd6306b455c 100644 --- a/tools/testing/selftests/bpf/prog_tests/test_task_local_data.c +++ b/tools/testing/selftests/bpf/prog_tests/test_task_local_data.c @@ -63,7 +63,7 @@ void *test_task_local_data_basic_thread(void *arg) if (!ASSERT_OK_PTR(value2, "tld_get_data")) goto out; - tid = gettid(); + tid = sys_gettid(); *value0 = tid + 0; *value1 = tid + 1; |