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 committed stat #930

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from
Open

fix committed stat #930

wants to merge 1 commit into from

Conversation

maxbachmann
Copy link

@maxbachmann maxbachmann commented Aug 28, 2024

_mi_os_free reduces the stat by the full size without taking uncommitted sections into account.
For this reason we have to add the uncommitted sections to the stat before calling it.

@maxbachmann
Copy link
Author

This should fix #929

@maxbachmann
Copy link
Author

As an alternative fix we could call _mi_os_free_ex and decrease the stat here.

@daanx
Copy link
Collaborator

daanx commented Aug 29, 2024

Ha, interesting -- looks like I decreased the commit while it should have been increased since _mi_os_free decreases by the full amount. Let me think a bit more on this but I think you are right.

_mi_os_free reduces the stat by the full size without taking uncommitted
sections into account.
For this reason we have to add the uncommitted sections to the stat before
calling it.
@gozetor
Copy link

gozetor commented Oct 4, 2024

Hello!
Is there a reason why this PR can't be merged? We also want to use mi_process_info, but the reported commit size is incorrect.

@daanx
Copy link
Collaborator

daanx commented Oct 4, 2024

Ah, yes it is already in the latest dev/dev-slice but was added manually. I'll close this PR. There will be a fresh release v2.1.8 soon I hope.

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