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

[WIP] ebpf: remove bcc macros and types #728

Closed
wants to merge 5 commits into from

Conversation

rootfs
Copy link
Contributor

@rootfs rootfs commented Jun 8, 2023

  • remove all bcc macro and function
  • pass bcc compile
  • pass non-bcc compile
  • integrated with collector

@rootfs
Copy link
Contributor Author

rootfs commented Jun 8, 2023

#716

Signed-off-by: Huamin Chen <[email protected]>
exit 1
fi
if [ ! -f ${DIR}/../_output/bpf/vmlinux.h ]; then
echo "downloading vmlinux.h"
Copy link
Collaborator

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?

Copy link
Contributor Author

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)

Copy link
Collaborator

@marceloamaral marceloamaral left a 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?

@rootfs
Copy link
Contributor Author

rootfs commented Jun 12, 2023

Can you please also add some documentation of this change? Maybe in doc/dev?

Good point! Will do

@rootfs
Copy link
Contributor Author

rootfs commented Jun 12, 2023

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.

@rootfs rootfs closed this Jun 12, 2023
@marceloamaral
Copy link
Collaborator

I suggest giving the cilium library a shot - it'll definitely make our work a lot easier!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants