| Message ID | 20200923140459.3029213-1-jolsa@kernel.org |
|---|---|
| State | Changes Requested |
| Delegated to: | BPF Maintainers |
| Headers | show |
| Series | [PATCHv2,bpf-next,1/2] bpf: Use --no-fail option if CONFIG_BPF is not enabled | expand |
On Wed, Sep 23, 2020 at 7:06 AM Jiri Olsa <jolsa@kernel.org> wrote: > > Currently all the resolve_btfids 'users' are under CONFIG_BPF > code, so if we have CONFIG_BPF disabled, resolve_btfids will > fail, because there's no data to resolve. > > Disabling resolve_btfids if there's CONFIG_BPF disabled, > so we won't fail such builds. > > Suggested-by: Andrii Nakryiko <andriin@fb.com> > Signed-off-by: Jiri Olsa <jolsa@kernel.org> > --- > v2 changes: > - disable resolve_btfids completely when CONFIG_BPF is not defined Acked-by: Andrii Nakryiko <andriin@fb.com> [...]
On Wed, Sep 23, 2020 at 09:09:17AM -0700, Andrii Nakryiko wrote: > On Wed, Sep 23, 2020 at 7:06 AM Jiri Olsa <jolsa@kernel.org> wrote: damn I'm blind this week.. will send v3 with proper subject :-\ jirka > > > > Currently all the resolve_btfids 'users' are under CONFIG_BPF > > code, so if we have CONFIG_BPF disabled, resolve_btfids will > > fail, because there's no data to resolve. > > > > Disabling resolve_btfids if there's CONFIG_BPF disabled, > > so we won't fail such builds. > > > > Suggested-by: Andrii Nakryiko <andriin@fb.com> > > Signed-off-by: Jiri Olsa <jolsa@kernel.org> > > --- > > v2 changes: > > - disable resolve_btfids completely when CONFIG_BPF is not defined > > Acked-by: Andrii Nakryiko <andriin@fb.com> > > [...] >
diff --git a/Makefile b/Makefile index c4470a4e131f..b05682eea2c0 100644 --- a/Makefile +++ b/Makefile @@ -1084,13 +1084,15 @@ ifdef CONFIG_STACK_VALIDATION endif endif +ifdef CONFIG_BPF ifdef CONFIG_DEBUG_INFO_BTF ifeq ($(has_libelf),1) resolve_btfids_target := tools/bpf/resolve_btfids FORCE else ERROR_RESOLVE_BTFIDS := 1 endif -endif +endif # CONFIG_DEBUG_INFO_BTF +endif # CONFIG_BPF PHONY += prepare0 diff --git a/scripts/link-vmlinux.sh b/scripts/link-vmlinux.sh index e6e2d9e5ff48..dbde59d343b1 100755 --- a/scripts/link-vmlinux.sh +++ b/scripts/link-vmlinux.sh @@ -341,9 +341,9 @@ fi vmlinux_link vmlinux "${kallsymso}" ${btf_vmlinux_bin_o} # fill in BTF IDs -if [ -n "${CONFIG_DEBUG_INFO_BTF}" ]; then -info BTFIDS vmlinux -${RESOLVE_BTFIDS} vmlinux +if [ -n "${CONFIG_DEBUG_INFO_BTF}" -a -n "${CONFIG_BPF}" ]; then + info BTFIDS vmlinux + ${RESOLVE_BTFIDS} vmlinux fi if [ -n "${CONFIG_BUILDTIME_TABLE_SORT}" ]; then
Currently all the resolve_btfids 'users' are under CONFIG_BPF code, so if we have CONFIG_BPF disabled, resolve_btfids will fail, because there's no data to resolve. Disabling resolve_btfids if there's CONFIG_BPF disabled, so we won't fail such builds. Suggested-by: Andrii Nakryiko <andriin@fb.com> Signed-off-by: Jiri Olsa <jolsa@kernel.org> --- v2 changes: - disable resolve_btfids completely when CONFIG_BPF is not defined Makefile | 4 +++- scripts/link-vmlinux.sh | 6 +++--- 2 files changed, 6 insertions(+), 4 deletions(-)