Skip to content

Adding stock gc heap resizing heuristics to MMTk#57147

Merged
oscardssmith merged 2 commits intoJuliaLang:masterfrom
udesou:adding-stock-gc-trigger
Jan 27, 2025
Merged

Adding stock gc heap resizing heuristics to MMTk#57147
oscardssmith merged 2 commits intoJuliaLang:masterfrom
udesou:adding-stock-gc-trigger

Conversation

@udesou
Copy link
Copy Markdown
Contributor

@udesou udesou commented Jan 23, 2025

This PR sets MMTk to use the stock GC heuristics for heap resizing by default. It also updates the mmtk-julia version to incorporate the changes that support the new gc triggering heuristics (see mmtk/mmtk-julia#222).

@udesou udesou added the GC Garbage collector label Jan 23, 2025
@udesou udesou requested a review from d-netto January 23, 2025 23:16
@udesou udesou force-pushed the adding-stock-gc-trigger branch 2 times, most recently from 1dd799e to a9f4be1 Compare January 24, 2025 03:29
@d-netto
Copy link
Copy Markdown
Member

d-netto commented Jan 24, 2025

I suspect you did a bunch of changes in the MMTk binding or MMTk core to incorporate the heuristic changes.

If you did, could you please also link the relevant PRs here?

@udesou
Copy link
Copy Markdown
Contributor Author

udesou commented Jan 24, 2025

Linked in the description. I might start pinging you in those PRs as well from now on if that’s okay. That way you’ll get more familiar with the binding code and it will be super helpful to have another set of eyes reviewing them (for the master branch of the binding at least).

@udesou udesou force-pushed the adding-stock-gc-trigger branch 2 times, most recently from 0c816c5 to 2194778 Compare January 24, 2025 19:38
@d-netto d-netto force-pushed the adding-stock-gc-trigger branch from 2194778 to 6bed715 Compare January 25, 2025 21:50
@d-netto d-netto force-pushed the adding-stock-gc-trigger branch from 6bed715 to 34b290d Compare January 27, 2025 14:45
@oscardssmith oscardssmith merged commit 47a6ad0 into JuliaLang:master Jan 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

GC Garbage collector

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants