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

vm: Avoid unnecessary preallocations #12443

Closed
wants to merge 1 commit into from
Closed

Conversation

elad335
Copy link
Contributor

@elad335 elad335 commented Aug 2, 2022

It's no secret RPCS3 is a bit of a memory hog, with 16GB of RAM while in-game and a few Chrome tabs open (not even including background 3rd party processes) you can run out of RAM and use the system pagefile (extremely slow memory compared to RAM located on disk). Luckily there is a simple change to reduce more than 0.5GB already: by avoiding preallocating the whole VM regions at once, using blocks of 32MB and allocating them as needed instead.

@Nekotekina
Copy link
Member

This could be solved more cleverly by utilizing copy-on-write zero pages. On systems with overcommit support this problem doesn't exist in the first place.

@Megamouse Megamouse added the Optimization Optimizes existing code label Aug 2, 2022
@elad335
Copy link
Contributor Author

elad335 commented Aug 12, 2022

Superseded by #12466

@elad335
Copy link
Contributor Author

elad335 commented Sep 17, 2022

Just testing.

@elad335 elad335 reopened this Sep 17, 2022
@elad335 elad335 marked this pull request as draft September 17, 2022 15:37
@elad335
Copy link
Contributor Author

elad335 commented Sep 21, 2022

Untested.

@elad335 elad335 closed this Sep 21, 2022
@elad335 elad335 deleted the memory-hog branch May 6, 2024 14:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Optimization Optimizes existing code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants