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 un-cleared VO bits for contiguous monotone PR #1071

Merged
merged 1 commit into from
Jan 16, 2024

Conversation

wks
Copy link
Collaborator

@wks wks commented Jan 15, 2024

Fixed a bug that when using a generational plan, if mutators allocated less than one chunk of memory into the nursery between GCs, some VO bits will not be cleared.

This PR also ensures the field MonotonePageResourceSync::current_chunk grows monotonically during GC when the space is contiguous.

Fixes: #1070

Fixed a bug that when using a generational plan, if mutators allocated
less than one chunk of memory into the nursery between GCs, some VO bits
will not be cleared.

This PR also ensures the field `MonotonePageResourceSync::current_chunk`
grows monotonically during GC when the space is contiguous.
@wks wks requested a review from qinsoon January 15, 2024 07:32
@qinsoon qinsoon added the PR-testing Run binding tests for the pull request (deprecated: use PR-extended-testing instead) label Jan 15, 2024
Copy link
Member

@qinsoon qinsoon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@qinsoon
Copy link
Member

qinsoon commented Jan 15, 2024

The JikesRVM test failed, due to an issue for downloading classpath from its FTP server. It should be unrelated with the change. I will see if we can work around it. I also noticed a random failure for Ruby tests (see run attempt 3). It is probably unrelated as well.

@qinsoon qinsoon added this pull request to the merge queue Jan 16, 2024
Merged via the queue into mmtk:master with commit 2b132cf Jan 16, 2024
26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR-testing Run binding tests for the pull request (deprecated: use PR-extended-testing instead)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Panic with 'VO bit already set' in GenImmix
2 participants