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

Fix value reported by get_maxrss on macos #344

Merged
merged 2 commits into from
Nov 12, 2024
Merged

Fix value reported by get_maxrss on macos #344

merged 2 commits into from
Nov 12, 2024

Conversation

maximecb
Copy link
Contributor

The value of ru_maxrss is in bytes on macos, which contradicts what the manpage says at:
https://developer.apple.com/library/archive/documentation/System/Conceptual/ManPages_iPhoneOS/man2/getrusage.2.html

Apparently, the manpage is wrong, but this information has never been corrected. I can't seem to get the Apple bug reporting software to work and so am unable to report this issue to Apple :(

The value of ru_maxrss is in bytes on macos, which contradicts
what the manpage says at:
https://developer.apple.com/library/archive/documentation/System/Conceptual/ManPages_iPhoneOS/man2/getrusage.2.html

Apparently, the manppage is wrong, but this information has never
been corrected. I can't seem to get the Apple bug reporting
software to work and so am unable to report this issue to Apple.
@maximecb maximecb requested a review from rwstauner November 12, 2024 19:45
harness/harness-common.rb Outdated Show resolved Hide resolved
Copy link
Contributor

@rwstauner rwstauner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Makes the MaxRSS a lot closer to the RSS 👍

Co-authored-by: Takashi Kokubun <[email protected]>
@maximecb maximecb enabled auto-merge (squash) November 12, 2024 20:11
@maximecb maximecb merged commit a892dad into main Nov 12, 2024
4 checks passed
@maximecb maximecb deleted the maxrss_macos branch November 12, 2024 20:21
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.

3 participants