Skip to content

Commit

Permalink
samples/bpf: Use getppid instead of getpgrp for array map stress
Browse files Browse the repository at this point in the history
When cross-compiling the bpf sample map_perf_test for aarch64, I find that
__NR_getpgrp is undefined. This causes build errors. This syscall is deprecated
and requires defining __ARCH_WANT_SYSCALL_DEPRECATED. To avoid having to define
that, just use a different syscall (getppid) for the array map stress test.

Acked-by: Alexei Starovoitov <ast@kernel.org>
Signed-off-by: Joel Fernandes <joelaf@google.com>
Acked-by: Daniel Borkmann <daniel@iogearbox.net>
Signed-off-by: David S. Miller <davem@davemloft.net>
  • Loading branch information
Joel Fernandes authored and David S. Miller committed Sep 21, 2017
1 parent ae3e24f commit 95ec669
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion samples/bpf/map_perf_test_kern.c
Original file line number Diff line number Diff line change
Expand Up @@ -266,7 +266,7 @@ int stress_hash_map_lookup(struct pt_regs *ctx)
return 0;
}

SEC("kprobe/sys_getpgrp")
SEC("kprobe/sys_getppid")
int stress_array_map_lookup(struct pt_regs *ctx)
{
u32 key = 1, i;
Expand Down
2 changes: 1 addition & 1 deletion samples/bpf/map_perf_test_user.c
Original file line number Diff line number Diff line change
Expand Up @@ -282,7 +282,7 @@ static void test_array_lookup(int cpu)

start_time = time_get_ns();
for (i = 0; i < max_cnt; i++)
syscall(__NR_getpgrp, 0);
syscall(__NR_getppid, 0);
printf("%d:array_lookup %lld lookups per sec\n",
cpu, max_cnt * 1000000000ll * 64 / (time_get_ns() - start_time));
}
Expand Down

0 comments on commit 95ec669

Please sign in to comment.