Count malloc size in pages #649
Labels
A-interface
Area: Interface/API
A-policy
Area: Policy
C-bug
Category: Bug
P-low
Priority: Low. A low-priority issue won't be scheduled and assigned. Any help is welcome.
Comments
qinsoon
added
C-bug
qinsoon
added a commit
that referenced
this issue
Nov 7, 2022
This PR changes how our malloc mark sweep accounts for memory. We are currently accounting bytes used, which is wrong. This PR changes that to page-based accounting. This PR addresses the issue #649 for malloc mark sweep (further work is still needed for the issue for our malloc API). * by default, we do not use bulk XOR for malloc mark sweep (which is known to have issues with page accounting) * Allow different malloc page size for library malloc
udesou
added
the
P-low
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
We should count malloc size by pages in malloc mark sweep. And when we provide counted malloc, we should also count in pages (which means we will need to maintain some metadata to track which page is used in MMTk for those malloc results).
We probably will share some code between those two. #572 was trying to turn the current malloc marksweep space to a manual malloc space. We may not directly use code from the pull request.
The text was updated successfully, but these errors were encountered: