* [PATCH bpf-next] selftests/bpf: fix attach point for non-x86 arches in test_progs/lsm
@ 2022-08-15 12:24 Artem Savkov
2022-08-15 18:46 ` Daniel Borkmann
0 siblings, 1 reply; 4+ messages in thread
From: Artem Savkov @ 2022-08-15 12:24 UTC (permalink / raw)
To: Alexei Starovoitov, Daniel Borkmann, Andrii Nakryiko, bpf, netdev
Cc: linux-kernel, Artem Savkov
Use SYS_PREFIX macro from bpf_misc.h instead of hard-coded '__x64_'
prefix for sys_setdomainname attach point in lsm test.
Signed-off-by: Artem Savkov <asavkov@redhat.com>
---
tools/testing/selftests/bpf/progs/lsm.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/tools/testing/selftests/bpf/progs/lsm.c b/tools/testing/selftests/bpf/progs/lsm.c
index 33694ef8acfa..d8d8af623bc2 100644
--- a/tools/testing/selftests/bpf/progs/lsm.c
+++ b/tools/testing/selftests/bpf/progs/lsm.c
@@ -4,6 +4,7 @@
* Copyright 2020 Google LLC.
*/
+#include "bpf_misc.h"
#include "vmlinux.h"
#include <bpf/bpf_helpers.h>
#include <bpf/bpf_tracing.h>
@@ -160,7 +161,7 @@ int BPF_PROG(test_task_free, struct task_struct *task)
int copy_test = 0;
-SEC("fentry.s/__x64_sys_setdomainname")
+SEC("fentry.s/" SYS_PREFIX "sys_setdomainname")
int BPF_PROG(test_sys_setdomainname, struct pt_regs *regs)
{
void *ptr = (void *)PT_REGS_PARM1(regs);
--
2.37.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH bpf-next] selftests/bpf: fix attach point for non-x86 arches in test_progs/lsm
2022-08-15 12:24 [PATCH bpf-next] selftests/bpf: fix attach point for non-x86 arches in test_progs/lsm Artem Savkov
@ 2022-08-15 18:46 ` Daniel Borkmann
2022-08-16 5:52 ` [PATCH bpf-next v2] " Artem Savkov
0 siblings, 1 reply; 4+ messages in thread
From: Daniel Borkmann @ 2022-08-15 18:46 UTC (permalink / raw)
To: Artem Savkov, Alexei Starovoitov, Andrii Nakryiko, bpf, netdev
Cc: linux-kernel, Daniel Müller
On 8/15/22 2:24 PM, Artem Savkov wrote:
> Use SYS_PREFIX macro from bpf_misc.h instead of hard-coded '__x64_'
> prefix for sys_setdomainname attach point in lsm test.
>
> Signed-off-by: Artem Savkov <asavkov@redhat.com>
> ---
> tools/testing/selftests/bpf/progs/lsm.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/tools/testing/selftests/bpf/progs/lsm.c b/tools/testing/selftests/bpf/progs/lsm.c
> index 33694ef8acfa..d8d8af623bc2 100644
> --- a/tools/testing/selftests/bpf/progs/lsm.c
> +++ b/tools/testing/selftests/bpf/progs/lsm.c
> @@ -4,6 +4,7 @@
> * Copyright 2020 Google LLC.
> */
>
> +#include "bpf_misc.h"
> #include "vmlinux.h"
> #include <bpf/bpf_helpers.h>
> #include <bpf/bpf_tracing.h>
> @@ -160,7 +161,7 @@ int BPF_PROG(test_task_free, struct task_struct *task)
>
> int copy_test = 0;
>
> -SEC("fentry.s/__x64_sys_setdomainname")
> +SEC("fentry.s/" SYS_PREFIX "sys_setdomainname")
> int BPF_PROG(test_sys_setdomainname, struct pt_regs *regs)
> {
> void *ptr = (void *)PT_REGS_PARM1(regs);
>
Good catch! Could you also update the comment in tools/testing/selftests/bpf/DENYLIST.s390x +46 :
[...]
test_lsm # failed to find kernel BTF type ID of '__x64_sys_setdomainname': -3 (?)
[...]
It should likely say sth like `attach fentry unexpected error: -524 (trampoline)`.
Thanks,
Daniel
^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCH bpf-next v2] selftests/bpf: fix attach point for non-x86 arches in test_progs/lsm
2022-08-15 18:46 ` Daniel Borkmann
@ 2022-08-16 5:52 ` Artem Savkov
2022-08-16 20:40 ` patchwork-bot+netdevbpf
0 siblings, 1 reply; 4+ messages in thread
From: Artem Savkov @ 2022-08-16 5:52 UTC (permalink / raw)
To: Alexei Starovoitov, Daniel Borkmann, Andrii Nakryiko, bpf, netdev
Cc: linux-kernel, Artem Savkov
Use SYS_PREFIX macro from bpf_misc.h instead of hard-coded '__x64_'
prefix for sys_setdomainname attach point in lsm test.
Signed-off-by: Artem Savkov <asavkov@redhat.com>
---
tools/testing/selftests/bpf/DENYLIST.s390x | 2 +-
tools/testing/selftests/bpf/progs/lsm.c | 3 ++-
2 files changed, 3 insertions(+), 2 deletions(-)
diff --git a/tools/testing/selftests/bpf/DENYLIST.s390x b/tools/testing/selftests/bpf/DENYLIST.s390x
index e33cab34d22f..9d8de15e725e 100644
--- a/tools/testing/selftests/bpf/DENYLIST.s390x
+++ b/tools/testing/selftests/bpf/DENYLIST.s390x
@@ -43,7 +43,7 @@ test_bpffs # bpffs test failed 255
test_bprm_opts # failed to auto-attach program 'secure_exec': -524 (trampoline)
test_ima # failed to auto-attach program 'ima': -524 (trampoline)
test_local_storage # failed to auto-attach program 'unlink_hook': -524 (trampoline)
-test_lsm # failed to find kernel BTF type ID of '__x64_sys_setdomainname': -3 (?)
+test_lsm # attach unexpected error: -524 (trampoline)
test_overhead # attach_fentry unexpected error: -524 (trampoline)
test_profiler # unknown func bpf_probe_read_str#45 (overlapping)
timer # failed to auto-attach program 'test1': -524 (trampoline)
diff --git a/tools/testing/selftests/bpf/progs/lsm.c b/tools/testing/selftests/bpf/progs/lsm.c
index 33694ef8acfa..d8d8af623bc2 100644
--- a/tools/testing/selftests/bpf/progs/lsm.c
+++ b/tools/testing/selftests/bpf/progs/lsm.c
@@ -4,6 +4,7 @@
* Copyright 2020 Google LLC.
*/
+#include "bpf_misc.h"
#include "vmlinux.h"
#include <bpf/bpf_helpers.h>
#include <bpf/bpf_tracing.h>
@@ -160,7 +161,7 @@ int BPF_PROG(test_task_free, struct task_struct *task)
int copy_test = 0;
-SEC("fentry.s/__x64_sys_setdomainname")
+SEC("fentry.s/" SYS_PREFIX "sys_setdomainname")
int BPF_PROG(test_sys_setdomainname, struct pt_regs *regs)
{
void *ptr = (void *)PT_REGS_PARM1(regs);
--
2.37.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH bpf-next v2] selftests/bpf: fix attach point for non-x86 arches in test_progs/lsm
2022-08-16 5:52 ` [PATCH bpf-next v2] " Artem Savkov
@ 2022-08-16 20:40 ` patchwork-bot+netdevbpf
0 siblings, 0 replies; 4+ messages in thread
From: patchwork-bot+netdevbpf @ 2022-08-16 20:40 UTC (permalink / raw)
To: Artem Savkov; +Cc: ast, daniel, andrii, bpf, netdev, linux-kernel
Hello:
This patch was applied to bpf/bpf-next.git (master)
by Andrii Nakryiko <andrii@kernel.org>:
On Tue, 16 Aug 2022 07:52:31 +0200 you wrote:
> Use SYS_PREFIX macro from bpf_misc.h instead of hard-coded '__x64_'
> prefix for sys_setdomainname attach point in lsm test.
>
> Signed-off-by: Artem Savkov <asavkov@redhat.com>
> ---
> tools/testing/selftests/bpf/DENYLIST.s390x | 2 +-
> tools/testing/selftests/bpf/progs/lsm.c | 3 ++-
> 2 files changed, 3 insertions(+), 2 deletions(-)
Here is the summary with links:
- [bpf-next,v2] selftests/bpf: fix attach point for non-x86 arches in test_progs/lsm
https://git.kernel.org/bpf/bpf-next/c/807662cac66a
You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2022-08-16 20:40 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-08-15 12:24 [PATCH bpf-next] selftests/bpf: fix attach point for non-x86 arches in test_progs/lsm Artem Savkov
2022-08-15 18:46 ` Daniel Borkmann
2022-08-16 5:52 ` [PATCH bpf-next v2] " Artem Savkov
2022-08-16 20:40 ` patchwork-bot+netdevbpf
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).