summaryrefslogtreecommitdiffstats
path: root/net/xdp
diff options
context:
space:
mode:
authorAndrii Nakryiko <andrii@kernel.org>2022-01-26 11:30:58 -0800
committerAlexei Starovoitov <ast@kernel.org>2022-01-26 20:04:01 -0800
commitff943683f8a6dbf887c16275d0e80c1c5391b7bb (patch)
tree896653a559a199252cd0abe18072c7c8de2bc170 /net/xdp
parente5465a9027e9703d8f5ec9c0387e32fb3a02b58f (diff)
downloadlinux-ff943683f8a6dbf887c16275d0e80c1c5391b7bb.tar.gz
linux-ff943683f8a6dbf887c16275d0e80c1c5391b7bb.tar.bz2
linux-ff943683f8a6dbf887c16275d0e80c1c5391b7bb.zip
selftests/bpf: fix uprobe offset calculation in selftests
Fix how selftests determine relative offset of a function that is uprobed. Previously, there was an assumption that uprobed function is always in the first executable region, which is not always the case (libbpf CI hits this case now). So get_base_addr() approach in isolation doesn't work anymore. So teach get_uprobe_offset() to determine correct memory mapping and calculate uprobe offset correctly. While at it, I merged together two implementations of get_uprobe_offset() helper, moving powerpc64-specific logic inside (had to add extra {} block to avoid unused variable error for insn). Also ensured that uprobed functions are never inlined, but are still static (and thus local to each selftest), by using a no-op asm volatile block internally. I didn't want to keep them global __weak, because some tests use uprobe's ref counter offset (to test USDT-like logic) which is not compatible with non-refcounted uprobe. So it's nicer to have each test uprobe target local to the file and guaranteed to not be inlined or skipped by the compiler (which can happen with static functions, especially if compiling selftests with -O2). Signed-off-by: Andrii Nakryiko <andrii@kernel.org> Link: https://lore.kernel.org/r/20220126193058.3390292-1-andrii@kernel.org Signed-off-by: Alexei Starovoitov <ast@kernel.org>
Diffstat (limited to 'net/xdp')
0 files changed, 0 insertions, 0 deletions