Commit 3a029e1f authored by Jean-Philippe Brucker's avatar Jean-Philippe Brucker Committed by Daniel Borkmann

selftests/bpf: Fix build of task_pt_regs test for arm64

struct pt_regs is not exported to userspace on all archs. arm64 and s390
export "user_pt_regs" instead, which causes build failure at the moment:

  progs/test_task_pt_regs.c:8:16: error: variable has incomplete type 'struct pt_regs'
  struct pt_regs current_regs = {};

Instead of using pt_regs from ptrace.h, use the larger kernel struct
from vmlinux.h directly. Since the test runner task_pt_regs.c does not
have access to the kernel struct definition, copy it into a char array.

Fixes: 576d47bb ("bpf: selftests: Add bpf_task_pt_regs() selftest")
Suggested-by: default avatarAndrii Nakryiko <andrii.nakryiko@gmail.com>
Signed-off-by: default avatarJean-Philippe Brucker <jean-philippe@linaro.org>
Signed-off-by: default avatarDaniel Borkmann <daniel@iogearbox.net>
Tested-by: default avatarIlya Leoshkevich <iii@linux.ibm.com>
Acked-by: default avatarIlya Leoshkevich <iii@linux.ibm.com>
Link: https://lore.kernel.org/bpf/20210906163635.302307-1-jean-philippe@linaro.org
parent 49ca6153
// SPDX-License-Identifier: GPL-2.0 // SPDX-License-Identifier: GPL-2.0
#define _GNU_SOURCE #define _GNU_SOURCE
#include <test_progs.h> #include <test_progs.h>
#include <linux/ptrace.h>
#include "test_task_pt_regs.skel.h" #include "test_task_pt_regs.skel.h"
void test_task_pt_regs(void) void test_task_pt_regs(void)
......
// SPDX-License-Identifier: GPL-2.0 // SPDX-License-Identifier: GPL-2.0
#include <linux/ptrace.h> #include "vmlinux.h"
#include <linux/bpf.h>
#include <bpf/bpf_helpers.h> #include <bpf/bpf_helpers.h>
#include <bpf/bpf_tracing.h> #include <bpf/bpf_tracing.h>
struct pt_regs current_regs = {}; #define PT_REGS_SIZE sizeof(struct pt_regs)
struct pt_regs ctx_regs = {};
/*
* The kernel struct pt_regs isn't exported in its entirety to userspace.
* Pass it as an array to task_pt_regs.c
*/
char current_regs[PT_REGS_SIZE] = {};
char ctx_regs[PT_REGS_SIZE] = {};
int uprobe_res = 0; int uprobe_res = 0;
SEC("uprobe/trigger_func") SEC("uprobe/trigger_func")
...@@ -17,8 +22,10 @@ int handle_uprobe(struct pt_regs *ctx) ...@@ -17,8 +22,10 @@ int handle_uprobe(struct pt_regs *ctx)
current = bpf_get_current_task_btf(); current = bpf_get_current_task_btf();
regs = (struct pt_regs *) bpf_task_pt_regs(current); regs = (struct pt_regs *) bpf_task_pt_regs(current);
__builtin_memcpy(&current_regs, regs, sizeof(*regs)); if (bpf_probe_read_kernel(current_regs, PT_REGS_SIZE, regs))
__builtin_memcpy(&ctx_regs, ctx, sizeof(*ctx)); return 0;
if (bpf_probe_read_kernel(ctx_regs, PT_REGS_SIZE, ctx))
return 0;
/* Prove that uprobe was run */ /* Prove that uprobe was run */
uprobe_res = 1; uprobe_res = 1;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment