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

Pahole #515

Closed
chadbrewbaker opened this issue Oct 12, 2021 · 2 comments
Closed

Pahole #515

chadbrewbaker opened this issue Oct 12, 2021 · 2 comments
Labels
• misc Related to other topics (e.g. CI).

Comments

@chadbrewbaker
Copy link

Pahole is in a separate repo. https://git.kernel.org/pub/scm/devel/pahole/pahole.git/

It is a utility for shrinking struct layouts https://lwn.net/Articles/335942/

Could we add it as a repo? Not only so the infrastructure is in place for more kernel test tooling, but also because the lack of pahole support is something Rust needs. Eventually pahole features should probably be upstreamed to Rust mainline and llvm mainline.

@alex
Copy link
Member

alex commented Oct 12, 2021

I don't understand what the request is here.

I'm not sure this is required for Rust in any event -- rustc automatically orders struct fields to minimize wasted padding bytes.

@ojeda ojeda added prio: low • misc Related to other topics (e.g. CI). labels Oct 25, 2021
@ojeda
Copy link
Member

ojeda commented Oct 25, 2021

Yes, please clarify the request.

@ojeda ojeda closed this as completed Oct 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
• misc Related to other topics (e.g. CI).
Development

No branches or pull requests

3 participants