-
Notifications
You must be signed in to change notification settings - Fork 8
Enable CONFIG_FTRACE_SYSCALLS
#13
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
Conversation
|
Need this as part of fixing javierhonduco/lightswitch#323 (comment) |
|
Hi @javierhonduco, thanks for the PR. The vmlinux.h files with this change will be generated by a CI job. Could you please check that new vmlinux.h fixes your issue and add a commit updating all the vmlinux.h files? We automated the generation process, but not the "update the files and open a PR" part. Feel free to take that on, if you're interested. Thank you! |
|
It feels like we should probably make sure to flag outdated headers at the least. I'd assume everything is fully reproducible so we should be able to just diff what we have vs. what we expect. It's what we do in |
Was going to add this in another PR but makes sense, will add another commit once the other opened PRs are merged. Thanks for the prompt review and generally improving CI. |
Only #15 should have relevance for you, I believe. So you should be good to go now with the update while the remaining ones are in review. |
f19f57b to
bc17bb2
Compare
|
Will update the various vmlinux.h files once they are produced in CI |
To add various useful types including `struct syscall_trace_enter` and `struct syscall_trace_exit`. See https://gitlab.com/redhat/centos-stream/src/kernel/centos-stream-9/-/merge_requests/3717 Test Plan ========= Validated that `struct syscall_trace_{enter,exit}` is included. CI will check that there are no inconsistencies
bc17bb2 to
9ef04c1
Compare
To kernel 6.18 with the following config libbpf/vmlinux.h#13. Test Plan ========= CI
d-e-s-o
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems fine to me, thanks. Will let CI have the last word.
To add various useful types including
struct syscall_trace_enterand
struct syscall_trace_exit.See https://gitlab.com/redhat/centos-stream/src/kernel/centos-stream-9/-/merge_requests/3717
Test Plan
Validated that
struct syscall_trace_{enter,exit}is included https://github.com/javierhonduco/vmlinux.h/actions/runs/20752218340