All of lore.kernel.org
 help / color / mirror / Atom feed
From: Yonghong Song <yhs@fb.com>
To: Vitaly Chikunov <vt@altlinux.org>,
	Arnaldo Carvalho de Melo <arnaldo.melo@gmail.com>
Cc: Arnaldo Carvalho de Melo <acme@kernel.org>,
	dwarves@vger.kernel.org, bpf <bpf@vger.kernel.org>,
	Martin Reboredo <yakoyoku@gmail.com>
Subject: Re: pahole v1.24: FAILED: load BTF from vmlinux: Invalid argument
Date: Fri, 26 Aug 2022 09:47:59 -0700	[thread overview]
Message-ID: <00cb99bd-bd17-3ff6-9008-92861518aff8@fb.com> (raw)
In-Reply-To: <20220826025944.hd7htqqwljhse6ht@altlinux.org>



On 8/25/22 7:59 PM, Vitaly Chikunov wrote:
> On Fri, Aug 26, 2022 at 05:52:20AM +0300, Vitaly Chikunov wrote:
>> Arnaldo,
>>
>> On Thu, Aug 25, 2022 at 08:16:20PM +0300, Vitaly Chikunov wrote:
>>> On Thu, Aug 25, 2022 at 01:47:59PM -0300, Arnaldo Carvalho de Melo wrote:
>>>> On Thu, Aug 25, 2022, 1:35 PM Vitaly Chikunov <vt@altlinux.org> wrote:
>>>>>
>>>>> I also noticed that after upgrading pahole to v1.24 kernel build (tested on
>>>>> v5.18.19, v5.15.63, sorry for not testing on mainline) fails with:
>>>>>
>>>>>      BTFIDS  vmlinux
>>>>>    + ./tools/bpf/resolve_btfids/resolve_btfids vmlinux
>>>>>    FAILED: load BTF from vmlinux: Invalid argument
>>>>>
>>>>> Perhaps, .tmp_vmlinux.btf is generated incorrectly? Downgrading dwarves to
>>>>> v1.23 resolves the issue.
>>>>>
>>>>
>>>> Can you try this, from Martin Reboredo (Archlinux):
>>>>
>>>> Can you try a build of the kernel or the by passing the
>>>> --skip_encoding_btf_enum64 to scripts/pahole-flags.sh?
>>>>
>>>> Here's a patch for either in tree scripts/pahole-flags.sh or
>>>> /usr/lib/modules/5.19.3-arch1-1/build/scripts/pahole-flags.sh
>>>
>>> This patch helped and kernel builds successfully after applying it.
>>> (Didn't notice this suggestion in release discussion thread.)
>>
>> Even thought it now compiles with this patch, it does not boot
>> afterwards (in virtme-like env), witch such console messages:
> 
> I'm talking here about 5.15.62. Yes, proposed patch does not apply there
> (since there is no `scripts/pahole-flags.sh`), but I updated
> `scripts/link-vmlinux.sh` with the similar `if` to append
> `--skip_encoding_btf_enum64` which lets then compilation pass.

Right, pahole v1.24 supports enum64 to correctly encode
some big 64bit values in BTF. But enum64 is only supported
in recent kernels. For old kernels, --skip_encoding_btf_enum64
is the way to workaround the issue.

> 
> Thanks,
> 
>>
>>    [    0.767649] Run /init as init process
>>    [    0.770858] BPF:[593] ENUM perf_event_task_context
>>    [    0.771262] BPF:size=4 vlen=4
>>    [    0.771511] BPF:
>>    [    0.771680] BPF:Invalid btf_info kind_flag
>>    [    0.772016] BPF:
>>    [    0.772016]
>>    [    0.772288] failed to validate module [9pnet] BTF: -22
>>    init_module '9pnet.ko' error -1
>>    [    0.785515] 9p: Unknown symbol p9_client_getattr_dotl (err -2)
>>    [    0.786005] 9p: Unknown symbol p9_client_wstat (err -2)
>>    [    0.786438] 9p: Unknown symbol p9_client_open (err -2)
>>    [    0.786863] 9p: Unknown symbol p9_client_rename (err -2)
>>    [    0.787307] 9p: Unknown symbol p9_client_remove (err -2)
>>    [    0.787749] 9p: Unknown symbol p9_client_renameat (err -2)
[...]

  parent reply	other threads:[~2022-08-26 16:51 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-08-25 16:35 pahole v1.24: FAILED: load BTF from vmlinux: Invalid argument Vitaly Chikunov
     [not found] ` <CA+JHD904e2TPpz1ybsaaqD+qMTDcueXu4nVcmotEPhxNfGN+Gw@mail.gmail.com>
2022-08-25 17:16   ` Vitaly Chikunov
2022-08-26  2:52     ` Vitaly Chikunov
2022-08-26  2:59       ` Vitaly Chikunov
2022-08-26 13:52         ` Jiri Olsa
2022-08-26 16:37           ` Arnaldo Carvalho de Melo
2022-08-27 17:33             ` Vitaly Chikunov
2022-08-27 18:33               ` Arnaldo Carvalho de Melo
2022-08-28  1:41                 ` Vitaly Chikunov
2022-08-26 16:51           ` Yonghong Song
2022-08-26 17:01             ` Jiri Olsa
2022-08-26 18:53               ` Yonghong Song
2022-08-26 20:19                 ` Jiri Olsa
2022-08-27  1:06                   ` Vitaly Chikunov
2022-08-26 16:47         ` Yonghong Song [this message]
2022-08-26 16:52           ` Arnaldo Carvalho de Melo

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=00cb99bd-bd17-3ff6-9008-92861518aff8@fb.com \
    --to=yhs@fb.com \
    --cc=acme@kernel.org \
    --cc=arnaldo.melo@gmail.com \
    --cc=bpf@vger.kernel.org \
    --cc=dwarves@vger.kernel.org \
    --cc=vt@altlinux.org \
    --cc=yakoyoku@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.