-
Notifications
You must be signed in to change notification settings - Fork 716
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
btf: move wire type definitions into sys #1218
Labels
good first issue
Good for newcomers
Comments
I see bunch of helper functions and methods in |
We could move those into a separate file in |
OmarTariq612
added a commit
to OmarTariq612/ebpf
that referenced
this issue
Jan 10, 2024
…es.go structs. This is guaranteed to match the kernel struct defenitions.
OmarTariq612
added a commit
to OmarTariq612/ebpf
that referenced
this issue
Jan 10, 2024
…es.go structs. This is guaranteed to match the kernel struct defenitions. Signed-off-by: OmarTariq612 <[email protected]>
What is the status of this? the PR is in draft for a year |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
btf_types.go contains a bunch of hand written structs that map to types defined in the kernel's UAPI. We should replace these with auto generated types in sys package and use those from btf. This guarantees that we match the kernel 1:1 and also ensures that there is no implicit padding present.
The changes need to be made here:
ebpf/internal/cmd/gentypes/main.go
Lines 157 to 208 in 0acd95c
The text was updated successfully, but these errors were encountered: