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

bpf2go: emit structs.HostLayout into generated structs #1686

Open
lmb opened this issue Feb 17, 2025 · 0 comments
Open

bpf2go: emit structs.HostLayout into generated structs #1686

lmb opened this issue Feb 17, 2025 · 0 comments
Labels
enhancement New feature or request

Comments

@lmb
Copy link
Collaborator

lmb commented Feb 17, 2025

We recently made a change to make sure our syscall wrappers continue to work on future versions of go, see #1681. Timo points out that we should do the same for bpf2go:

          Looks good, thanks! We should probably also emit these in bpf2go types.

Originally posted by @ti-mo in #1681 (review)

I think it might make sense to just update GoFormatter to always include HostLayout in structs instead of the hack I did for gentypes.

@lmb lmb added the enhancement New feature or request label Feb 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant