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

Superslab bug fix #259

Merged
merged 2 commits into from
Nov 10, 2020
Merged

Superslab bug fix #259

merged 2 commits into from
Nov 10, 2020

Conversation

mjp41
Copy link
Member

@mjp41 mjp41 commented Nov 9, 2020

Fixes the meta-data initialisation for the superslab in the 16MiB case.

This superseeds #258 as the problem is more general.

mjp41 added 2 commits November 9, 2020 21:16
If the Superslab meta-data is larger than an OS page, then the
subsequent pages could be decommited. This removes the skipped
initialisation in that case.
This simulates the pages being switched off with unexpected content, and
thus, when turned on we don't assume they have been unchanged.
@mjp41 mjp41 mentioned this pull request Nov 9, 2020
@mjp41 mjp41 requested a review from davidchisnall November 9, 2020 21:21
@mjp41
Copy link
Member Author

mjp41 commented Nov 9, 2020

@devnexen could you confirm this fixes your issue from #258 ? Thanks

@mjp41 mjp41 merged commit 3ca29d6 into microsoft:master Nov 10, 2020
@mjp41 mjp41 deleted the superslab-bug-fix branch November 10, 2020 09:57
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.

3 participants