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

Align detect_helper with log message change #3114

Merged
merged 1 commit into from May 15, 2024
Merged

Conversation

mykyta5
Copy link
Contributor

@mykyta5 mykyta5 commented Apr 11, 2024

"unknown func " log message, produced by BPF verifier may change in future to "program of this type cannot use helper ". Corresponding kernel change:
https://lore.kernel.org/all/20240325152210.377548-1-yatsenko@meta.com/

"unknown func " log message may change in future to  "program of this
type cannot use helper ". Corresponding kernel change:
 https://lore.kernel.org/all/20240325152210.377548-1-yatsenko@meta.com/
@jordalgo
Copy link
Contributor

@mykyta5 Thanks for the update! Did this get merged into linus's tree yet or just bpf-next?

@mykyta5
Copy link
Contributor Author

mykyta5 commented Apr 12, 2024

@mykyta5 Thanks for the update! Did this get merged into linus's tree yet or just bpf-next?

just bpf-next.

@jordalgo
Copy link
Contributor

Maybe let's wait until it hits mainline / linus - Not sure if I'm being overly cautious (cc @ajor @viktormalik @danobi)

@viktormalik
Copy link
Contributor

Maybe let's wait until it hits mainline / linus - Not sure if I'm being overly cautious (cc @ajor @viktormalik @danobi)

Yeah, let's wait for the patch to be merged to Linus' tree.

@jordalgo jordalgo added the kernel Issue may require kernel work label Apr 16, 2024
@mykyta5
Copy link
Contributor Author

mykyta5 commented May 15, 2024

@jordalgo, @viktormalik Looks like it was pulled to Linus repo: torvalds/linux@786bf0e

@jordalgo jordalgo merged commit d45aa54 into bpftrace:master May 15, 2024
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kernel Issue may require kernel work
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants