* [PATCH bpf-next] tools headers: Remove s390 ptrace.h in check-headers.sh
@ 2023-04-07 2:21 Tiezhu Yang
2023-04-07 10:27 ` Jiri Olsa
0 siblings, 1 reply; 4+ messages in thread
From: Tiezhu Yang @ 2023-04-07 2:21 UTC (permalink / raw)
To: Alexei Starovoitov, Daniel Borkmann, Andrii Nakryiko,
Arnaldo Carvalho de Melo
Cc: bpf, linux-perf-users, linux-kernel, loongson-kernel
After commit 1f265d2aea0d ("selftests/bpf: Remove not used headers"),
tools/arch/s390/include/uapi/asm/ptrace.h has been removed, so remove
it in check-headers.sh too, otherwise we can see the following build
warning:
diff: tools/arch/s390/include/uapi/asm/ptrace.h: No such file or directory
Reported-by: kernel test robot <lkp@intel.com>
Link: https://lore.kernel.org/oe-kbuild-all/202304050029.38NdbQPf-lkp@intel.com/
Fixes: 1f265d2aea0d ("selftests/bpf: Remove not used headers")
Signed-off-by: Tiezhu Yang <yangtiezhu@loongson.cn>
---
commit 1f265d2aea0d ("selftests/bpf: Remove not used headers") is in
bpf-next tree, so I prefer this patch can be applied to bpf-next tree.
| 1 -
1 file changed, 1 deletion(-)
--git a/tools/perf/check-headers.sh b/tools/perf/check-headers.sh
index eacca9a..e4a8b53 100755
--- a/tools/perf/check-headers.sh
+++ b/tools/perf/check-headers.sh
@@ -52,7 +52,6 @@ arch/x86/include/uapi/asm/vmx.h
arch/powerpc/include/uapi/asm/kvm.h
arch/s390/include/uapi/asm/kvm.h
arch/s390/include/uapi/asm/kvm_perf.h
-arch/s390/include/uapi/asm/ptrace.h
arch/s390/include/uapi/asm/sie.h
arch/arm/include/uapi/asm/kvm.h
arch/arm64/include/uapi/asm/kvm.h
--
2.1.0
^ permalink raw reply related [flat|nested] 4+ messages in thread* Re: [PATCH bpf-next] tools headers: Remove s390 ptrace.h in check-headers.sh
2023-04-07 2:21 [PATCH bpf-next] tools headers: Remove s390 ptrace.h in check-headers.sh Tiezhu Yang
@ 2023-04-07 10:27 ` Jiri Olsa
2023-04-11 17:35 ` Daniel Borkmann
0 siblings, 1 reply; 4+ messages in thread
From: Jiri Olsa @ 2023-04-07 10:27 UTC (permalink / raw)
To: Tiezhu Yang
Cc: Alexei Starovoitov, Daniel Borkmann, Andrii Nakryiko,
Arnaldo Carvalho de Melo, bpf, linux-perf-users, linux-kernel,
loongson-kernel
On Fri, Apr 07, 2023 at 10:21:30AM +0800, Tiezhu Yang wrote:
> After commit 1f265d2aea0d ("selftests/bpf: Remove not used headers"),
> tools/arch/s390/include/uapi/asm/ptrace.h has been removed, so remove
> it in check-headers.sh too, otherwise we can see the following build
> warning:
>
> diff: tools/arch/s390/include/uapi/asm/ptrace.h: No such file or directory
>
> Reported-by: kernel test robot <lkp@intel.com>
> Link: https://lore.kernel.org/oe-kbuild-all/202304050029.38NdbQPf-lkp@intel.com/
> Fixes: 1f265d2aea0d ("selftests/bpf: Remove not used headers")
> Signed-off-by: Tiezhu Yang <yangtiezhu@loongson.cn>
not sure this should go through Arnaldo's tree instead,
either way is fine with me
Acked-by: Jiri Olsa <jolsa@kernel.org>
jirka
> ---
>
> commit 1f265d2aea0d ("selftests/bpf: Remove not used headers") is in
> bpf-next tree, so I prefer this patch can be applied to bpf-next tree.
>
> tools/perf/check-headers.sh | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/tools/perf/check-headers.sh b/tools/perf/check-headers.sh
> index eacca9a..e4a8b53 100755
> --- a/tools/perf/check-headers.sh
> +++ b/tools/perf/check-headers.sh
> @@ -52,7 +52,6 @@ arch/x86/include/uapi/asm/vmx.h
> arch/powerpc/include/uapi/asm/kvm.h
> arch/s390/include/uapi/asm/kvm.h
> arch/s390/include/uapi/asm/kvm_perf.h
> -arch/s390/include/uapi/asm/ptrace.h
> arch/s390/include/uapi/asm/sie.h
> arch/arm/include/uapi/asm/kvm.h
> arch/arm64/include/uapi/asm/kvm.h
> --
> 2.1.0
>
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [PATCH bpf-next] tools headers: Remove s390 ptrace.h in check-headers.sh
2023-04-07 10:27 ` Jiri Olsa
@ 2023-04-11 17:35 ` Daniel Borkmann
2023-04-12 13:33 ` Arnaldo Carvalho de Melo
0 siblings, 1 reply; 4+ messages in thread
From: Daniel Borkmann @ 2023-04-11 17:35 UTC (permalink / raw)
To: Jiri Olsa, Tiezhu Yang
Cc: Alexei Starovoitov, Andrii Nakryiko, Arnaldo Carvalho de Melo,
bpf, linux-perf-users, linux-kernel, loongson-kernel
On 4/7/23 12:27 PM, Jiri Olsa wrote:
> On Fri, Apr 07, 2023 at 10:21:30AM +0800, Tiezhu Yang wrote:
>> After commit 1f265d2aea0d ("selftests/bpf: Remove not used headers"),
>> tools/arch/s390/include/uapi/asm/ptrace.h has been removed, so remove
>> it in check-headers.sh too, otherwise we can see the following build
>> warning:
>>
>> diff: tools/arch/s390/include/uapi/asm/ptrace.h: No such file or directory
>>
>> Reported-by: kernel test robot <lkp@intel.com>
>> Link: https://lore.kernel.org/oe-kbuild-all/202304050029.38NdbQPf-lkp@intel.com/
>> Fixes: 1f265d2aea0d ("selftests/bpf: Remove not used headers")
>> Signed-off-by: Tiezhu Yang <yangtiezhu@loongson.cn>
>
> not sure this should go through Arnaldo's tree instead,
> either way is fine with me
Agree, perf tree is better given the script is located there, so:
Acked-by: Daniel Borkmann <daniel@iogearbox.net>
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [PATCH bpf-next] tools headers: Remove s390 ptrace.h in check-headers.sh
2023-04-11 17:35 ` Daniel Borkmann
@ 2023-04-12 13:33 ` Arnaldo Carvalho de Melo
0 siblings, 0 replies; 4+ messages in thread
From: Arnaldo Carvalho de Melo @ 2023-04-12 13:33 UTC (permalink / raw)
To: Daniel Borkmann
Cc: Jiri Olsa, Tiezhu Yang, Alexei Starovoitov, Andrii Nakryiko, bpf,
linux-perf-users, linux-kernel, loongson-kernel
Em Tue, Apr 11, 2023 at 07:35:02PM +0200, Daniel Borkmann escreveu:
> On 4/7/23 12:27 PM, Jiri Olsa wrote:
> > On Fri, Apr 07, 2023 at 10:21:30AM +0800, Tiezhu Yang wrote:
> > > After commit 1f265d2aea0d ("selftests/bpf: Remove not used headers"),
> > > tools/arch/s390/include/uapi/asm/ptrace.h has been removed, so remove
> > > it in check-headers.sh too, otherwise we can see the following build
> > > warning:
> > >
> > > diff: tools/arch/s390/include/uapi/asm/ptrace.h: No such file or directory
> > >
> > > Reported-by: kernel test robot <lkp@intel.com>
> > > Link: https://lore.kernel.org/oe-kbuild-all/202304050029.38NdbQPf-lkp@intel.com/
> > > Fixes: 1f265d2aea0d ("selftests/bpf: Remove not used headers")
> > > Signed-off-by: Tiezhu Yang <yangtiezhu@loongson.cn>
> >
> > not sure this should go through Arnaldo's tree instead,
> > either way is fine with me
>
> Agree, perf tree is better given the script is located there, so:
>
> Acked-by: Daniel Borkmann <daniel@iogearbox.net>
Thanks, applied.
- Arnaldo
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2023-04-12 13:34 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-04-07 2:21 [PATCH bpf-next] tools headers: Remove s390 ptrace.h in check-headers.sh Tiezhu Yang
2023-04-07 10:27 ` Jiri Olsa
2023-04-11 17:35 ` Daniel Borkmann
2023-04-12 13:33 ` Arnaldo Carvalho de Melo
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.