Skip to content

Remove unnecessary app heap memory clean operations to reduce process RSS #1608

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

Merged
merged 1 commit into from
Oct 18, 2022

Conversation

shengyun-zhou
Copy link
Contributor

With hardware boundary checking enabled, the app heap memory comes from os_mmap(). Clearing the whole heap in the memory allocator causes process RSS to reach maximum app heap size immediately and wastes lots of memory, so we should remove unnecessary memory clean operations in the memory allocator.

@wenyongh
Copy link
Contributor

LGTM

@wenyongh wenyongh merged commit 84b1a6c into bytecodealliance:main Oct 18, 2022
@shengyun-zhou shengyun-zhou deleted the app-heap-rss-fix branch October 19, 2022 13:21
vickiegpt pushed a commit to vickiegpt/wamr-aot-gc-checkpoint-restore that referenced this pull request May 27, 2024
… RSS (bytecodealliance#1608)

With hardware boundary checking enabled, the app heap memory comes from `os_mmap()`.
Clearing the whole heap in the memory allocator causes process RSS to reach maximum
app heap size immediately and wastes lots of memory, so we had better remove the
unnecessary memory clean operations in the memory allocator.
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.

2 participants