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 a bug related to process_info memory usage #1547

Merged
merged 1 commit into from
Feb 18, 2025

Conversation

pguyot
Copy link
Collaborator

@pguyot pguyot commented Feb 17, 2025

When calling repeatedly process_info on another process, memory allocated to create the reply tuple was never freed until the process did a garbage collection. Fix the bug by allocating the response on a disposed heap since this is copied to a message.

These changes are made under both the "Apache 2.0" and the "GNU Lesser General
Public License 2.1 or later" license terms (dual license).

SPDX-License-Identifier: Apache-2.0 OR LGPL-2.1-or-later

When calling repeatedly process_info on another process, memory allocated
to create the reply tuple was never freed until the process did a garbage
collection. Fix the bug by allocating the response on a disposed heap since
this is copied to a message.

Signed-off-by: Paul Guyot <[email protected]>
@bettio bettio merged commit 3dd59d5 into atomvm:release-0.6 Feb 18, 2025
101 of 102 checks passed
@pguyot pguyot deleted the w08/gc-signals branch February 19, 2025 06: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.

2 participants