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

Allow configurable and platform-specific page sizes #2721

Merged
merged 1 commit into from
Dec 11, 2022

Conversation

nia-e
Copy link
Contributor

@nia-e nia-e commented Dec 9, 2022

This fixes #2644 by setting platform-default page sizes along with a command line flag to override size to a specific value (e.g. in the case of aarch64 Linux on M1 silicon). There's still some code cleanup to be done and tests need to be added but I'm opening this for now.

@nia-e nia-e changed the title WIP: Allow configurable and platform-specific page sizes Allow configurable and platform-specific page sizes Dec 11, 2022
src/intptrcast.rs Outdated Show resolved Hide resolved
src/machine.rs Outdated Show resolved Hide resolved
src/machine.rs Outdated Show resolved Hide resolved
README.md Show resolved Hide resolved
src/machine.rs Outdated Show resolved Hide resolved
@RalfJung
Copy link
Member

Looks good. :) Just a formatting nit, and please squash the commits.

@nia-e
Copy link
Contributor Author

nia-e commented Dec 11, 2022

Done! Thanks for the feedback and sorry for the hiccups

src/machine.rs Show resolved Hide resolved
@RalfJung
Copy link
Member

@bors r+

@bors
Copy link
Contributor

bors commented Dec 11, 2022

📌 Commit 22628ff has been approved by RalfJung

It is now in the queue for this repository.

@bors
Copy link
Contributor

bors commented Dec 11, 2022

⌛ Testing commit 22628ff with merge 0805372...

@bors
Copy link
Contributor

bors commented Dec 11, 2022

☀️ Test successful - checks-actions
Approved by: RalfJung
Pushing 0805372 to master...

@bors bors merged commit 0805372 into rust-lang:master Dec 11, 2022
@nia-e nia-e deleted the fix-pagesize branch December 11, 2022 16:51
bors added a commit to rust-lang-ci/rust that referenced this pull request Dec 25, 2022
RalfJung pushed a commit to RalfJung/miri that referenced this pull request Dec 28, 2022
spikespaz pushed a commit to spikespaz/dotwalk-rs that referenced this pull request Aug 29, 2024
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.

Miri's hardcoded PAGE_SIZE is sometimes inaccurate
4 participants