Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ci: arm64 tests fail #1460

Open
lmb opened this issue May 10, 2024 · 4 comments
Open

ci: arm64 tests fail #1460

lmb opened this issue May 10, 2024 · 4 comments
Labels
bug Something isn't working

Comments

@lmb
Copy link
Collaborator

lmb commented May 10, 2024

Describe the bug

Tests on arm64 fail since the new kernel doesn't have the necessary config for kprobe_multi.

=== Failed
=== FAIL: link TestKprobeMulti (0.00s)
    kprobe_multi_test.go:16: Feature 'bpf_link_kprobe_multi' isn't supported even though kernel is newer than v5.18

=== FAIL: link TestKprobeMultiErrors (0.00s)
    kprobe_multi_test.go:59: Feature 'bpf_link_kprobe_multi' isn't supported even though kernel is newer than v5.18

=== FAIL: link TestKprobeMultiCookie (0.00s)
    kprobe_multi_test.go:79: Feature 'bpf_link_kprobe_multi' isn't supported even though kernel is newer than v5.18

=== FAIL: link TestKprobeMultiProgramCall (0.00s)
    kprobe_multi_test.go:94: Feature 'bpf_link_kprobe_multi' isn't supported even though kernel is newer than v5.18

=== FAIL: link TestHaveBPFLinkKprobeMulti (0.00s)
    feature.go:30: Feature 'bpf_link_kprobe_multi' isn't supported even though kernel is newer than v5.18

How to reproduce

Run the arm64 ci task.

Version information

main

@lmb lmb added the bug Something isn't working label May 10, 2024
lmb added a commit to lmb/ebpf that referenced this issue May 10, 2024
Seems like the new CI arm64 kernels doesn't have the necessary config
to support kprobe_multi. Ignore the error until we figure out what
to do.

Updates cilium#1460

Signed-off-by: Lorenz Bauer <[email protected]>
lmb added a commit to lmb/ebpf that referenced this issue May 10, 2024
Seems like the new CI arm64 kernels doesn't have the necessary config
to support kprobe_multi. Ignore the error until we figure out what
to do.

Updates cilium#1460

Signed-off-by: Lorenz Bauer <[email protected]>
lmb added a commit to lmb/ebpf that referenced this issue May 10, 2024
Seems like the new CI arm64 kernels doesn't have the necessary config
to support kprobe_multi. Ignore the error until we figure out what
to do.

Updates cilium#1460

Signed-off-by: Lorenz Bauer <[email protected]>
lmb added a commit to lmb/ebpf that referenced this issue May 10, 2024
Seems like the new CI arm64 kernels doesn't have the necessary config
to support kprobe_multi. Ignore the error until we figure out what
to do.

Updates cilium#1460

Signed-off-by: Lorenz Bauer <[email protected]>
lmb added a commit that referenced this issue May 10, 2024
Seems like the new CI arm64 kernels doesn't have the necessary config
to support kprobe_multi. Ignore the error until we figure out what
to do.

Updates #1460

Signed-off-by: Lorenz Bauer <[email protected]>
@lmb
Copy link
Collaborator Author

lmb commented May 10, 2024

Seems pretty clear that CONFIG_FPROBE is needed: https://elixir.bootlin.com/linux/latest/source/kernel/trace/bpf_trace.c#L3090 Further debugging in #1462

@alexellis
Copy link

Hi, I added everything you asked for and redeployed to all CNCF servers:

#1245 (comment)

If you could run grep /boot/config-6.9.10 you should see CONFIG_FPROBE etc as per above.

@lmb
Copy link
Collaborator Author

lmb commented May 10, 2024

Here is the equivalent of the grep you mentioned: https://github.com/cilium/ebpf/actions/runs/9031025998/job/24816484288?pr=1462#step:5:275

Maybe you are missing some of the dependencies? https://cateee.net/lkddb/web-lkddb/FPROBE.html

@alexellis
Copy link

I've added these dependencies, but I'm not sure if this is supported for 64-bit Arm, because olddefconfig removes them when I run it. This typically means a dependent CONFIG_ is missing or the feature is not available.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants