]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
selftests/bpf: Use pid_t consistently in test_progs.c
authorTony Ambardar <tony.ambardar@gmail.com>
Tue, 23 Jul 2024 05:54:28 +0000 (22:54 -0700)
committerAndrii Nakryiko <andrii@kernel.org>
Mon, 29 Jul 2024 22:05:07 +0000 (15:05 -0700)
Use pid_t rather than __pid_t when allocating memory for 'worker_pids' in
'struct test_env', as this is its declared type and also avoids compile
errors seen building against musl libc on mipsel64:

  test_progs.c:1738:49: error: '__pid_t' undeclared (first use in this function); did you mean 'pid_t'?
   1738 |                 env.worker_pids = calloc(sizeof(__pid_t), env.workers);
        |                                                 ^~~~~~~
        |                                                 pid_t
  test_progs.c:1738:49: note: each undeclared identifier is reported only once for each function it appears in

Fixes: 91b2c0afd00c ("selftests/bpf: Add parallelism to test_progs")
Signed-off-by: Tony Ambardar <tony.ambardar@gmail.com>
Signed-off-by: Andrii Nakryiko <andrii@kernel.org>
Acked-by: Geliang Tang <geliang@kernel.org>
Link: https://lore.kernel.org/bpf/c6447da51a94babc1931711a43e2ceecb135c93d.1721713597.git.tony.ambardar@gmail.com
tools/testing/selftests/bpf/test_progs.c

index 89ff704e9dad5eabfe35047616f109f6f133a27e..60c5ec0f6abf6b65b567a4abc5a0429f5526ab55 100644 (file)
@@ -1731,7 +1731,7 @@ int main(int argc, char **argv)
        /* launch workers if requested */
        env.worker_id = -1; /* main process */
        if (env.workers) {
-               env.worker_pids = calloc(sizeof(__pid_t), env.workers);
+               env.worker_pids = calloc(sizeof(pid_t), env.workers);
                env.worker_socks = calloc(sizeof(int), env.workers);
                if (env.debug)
                        fprintf(stdout, "Launching %d workers.\n", env.workers);