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

aya: tidy bpf syscalls and feature support functions into submodules #961

Closed
wants to merge 1 commit into from

Conversation

tyrone-wu
Copy link
Contributor

@tyrone-wu tyrone-wu commented May 31, 2024

This PR organizes the bpf syscall related functions and feature support functions in aya/src/sys/bpf.rs into submodules:

aya/src/sys/
  ├─ btf.rs
  ├─ link.rs
  ├─ map.rs
  ├─ object.rs
  ├─ program.rs
  ├─ utils.rs
  ├─ feature_probe.rs
  └─ ...

This is to help make development slightly less daunting to parse and expand on. For example, #959 would fit in utils.rs, and additional feature detection stuff (like #957) would go in feature_probe.rs.

This also updates all the imports that involve these submodules.

Still kind of a working progress with organizing the feature probe stuff.

Copy link

netlify bot commented May 31, 2024

Deploy Preview for aya-rs-docs ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 0c1d9f2
🔍 Latest deploy log https://app.netlify.com/sites/aya-rs-docs/deploys/665b8650d412630008e0ed9e
😎 Deploy Preview https://deploy-preview-961--aya-rs-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@mergify mergify bot added the aya This is about aya (userspace) label May 31, 2024
This organizes the bpf syscalls and feature probe functions in
aya/src/sys/bpf.rs into submodules: btf, link, map, object, program,
utils, and feature_probe. This is to help make development slightly
less daunting to parse and expand on.

The submodules, except for feature_probe, are re-exported so that
importing them remains unchanged (i.e. crate::sys::foo). The feature
probe submodule isn't re-exported (uses crate::sys::feature_probe::foo)
so that there's some separation between bpf syscall stuff and feature
probing.
@tyrone-wu
Copy link
Contributor Author

Actually on second thought, I don't think this is necessary. There's some things im looking at in aya-obj anyway.

@tyrone-wu tyrone-wu deleted the aya/tidy-sys-funcs branch July 17, 2024 17:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
aya This is about aya (userspace)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant