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

Add API to create maps at runtime #935

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

alessandrod
Copy link
Collaborator

No description provided.

This is in preparation of unifying map defs
Now both BtfMap and LegacyMap hold MapDef
Copy link

netlify bot commented Apr 21, 2024

Deploy Preview for aya-rs-docs ready!

Name Link
🔨 Latest commit 6eb38cf
🔍 Latest deploy log https://app.netlify.com/sites/aya-rs-docs/deploys/6624f5ded75e7c0008e4a777
😎 Deploy Preview https://deploy-preview-935--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 aya This is about aya (userspace) aya-obj Relating to the aya-obj crate labels Apr 21, 2024
Maps can be created at runtime, in which case they don't have any ELF
info. Split the ELF bits into ElfMapData.

This is in preparation of adding MapData::create() to create maps at
runtime.
@ajwerner
Copy link
Member

I'd be happy to review this when it's ready (if you're seeking review). I was just thinking about implementing this sort of functionality.

@astoycos
Copy link
Member

+1 super interested in this API!

Copy link

mergify bot commented May 3, 2024

@alessandrod, this pull request is now in conflict and requires a rebase.

@mergify mergify bot added the needs-rebase label May 3, 2024
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) aya-obj Relating to the aya-obj crate needs-rebase
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants