Hi Miguel, On Wed, 3 Nov 2021 11:30:35 +0100 Miguel Ojeda <miguel.ojeda.sandonis@xxxxxxxxx> wrote: > > On Wed, Nov 3, 2021 at 4:19 AM Stephen Rothwell <sfr@xxxxxxxxxxxxxxxx> wrote: > > > > diff --cc scripts/Makefile.modfinal > > index 7f39599e9fae,c0842e999a75..000000000000 > > --- a/scripts/Makefile.modfinal > > +++ b/scripts/Makefile.modfinal > > @@@ -39,11 -39,12 +39,12 @@@ quiet_cmd_ld_ko_o = LD [M] $ > > > > quiet_cmd_btf_ko = BTF [M] $@ > > cmd_btf_ko = \ > > - if [ -f vmlinux ]; then \ > > - LLVM_OBJCOPY="$(OBJCOPY)" $(PAHOLE) -J $(PAHOLE_FLAGS) --btf_base vmlinux $@; \ > > - $(RESOLVE_BTFIDS) -b vmlinux $@; \ > > - else \ > > + if [ ! -f vmlinux ]; then \ > > printf "Skipping BTF generation for %s due to unavailability of vmlinux\n" $@ 1>&2; \ > > + elif $(srctree)/scripts/is_rust_module.sh $@; then \ > > + printf "Skipping BTF generation for %s because it's a Rust module\n" $@ 1>&2; \ > > + else \ > > - LLVM_OBJCOPY="$(OBJCOPY)" $(PAHOLE) -J --btf_base vmlinux $@; \ > > ++ LLVM_OBJCOPY="$(OBJCOPY)" $(PAHOLE) -J $(PAHOLE_FLAGS) --btf_base vmlinux $@; \ > > fi; > > It looks like the `$(RESOLVE_BTFIDS)` line is gone with this > resolution. The rest looks good. Oops, sorry about that. I have added it back from today. BTW, are you intending to submit the rust tree this merge window? -- Cheers, Stephen Rothwell
Attachment:
pgp2bGBLEVthS.pgp
Description: OpenPGP digital signature