-
Notifications
You must be signed in to change notification settings - Fork 190
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
[WIP] ebpf: remove bcc macros and types #728
Conversation
rootfs
commented
Jun 8, 2023
•
edited
Loading
edited
- remove all bcc macro and function
- pass bcc compile
- pass non-bcc compile
- integrated with collector
Signed-off-by: Huamin Chen <[email protected]>
Signed-off-by: Huamin Chen <[email protected]>
Signed-off-by: Huamin Chen <[email protected]>
Signed-off-by: Huamin Chen <[email protected]>
exit 1 | ||
fi | ||
if [ ! -f ${DIR}/../_output/bpf/vmlinux.h ]; then | ||
echo "downloading vmlinux.h" |
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.
This looks similar to what libebpf does.
btw, can we build this in one kernel/LinuxOS and run in a different kernel/LinuxOS?
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.
we build for each vmlinux.h
variations, and potential for each cpu platforms (arm, x86, and s390x)
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.
Can you please also add some documentation of this change?
Maybe in doc/dev
?
Good point! Will do |
turns out the bcc elf pkg hasn't got any refresh for a long time. Even the pre-compiled module can be loaded, maps, tables, and probes are still missing. Stop working on pre-compile for now. Will revisit it when situations change. |
I suggest giving the cilium library a shot - it'll definitely make our work a lot easier! |