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

More accurate PS3 memory consumption #12747

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

elad335
Copy link
Contributor

@elad335 elad335 commented Oct 2, 2022

Take the following into the total reported and observed PS3 memory amount:

  • Executable memory.
  • PRX memory. (even HLE'd modules)
  • libio thread, sceNpMatching2Init2 thread and calling malloc with a more than half a megabyte allocation size.
  • cellAudio initialization and port creation, add out-of-memory checks there as well. (SHAREDMEMORY error)

Makes Brink fail to execute some bugged and unused audio code that used to crash it on rpcs3 because now the memory allocations required for it fail and the whole section is skipped.
Fixes #11945
Draft because needs proper hw tests.

@Juan12233
Copy link

when are you going to merge this?

@elad335
Copy link
Contributor Author

elad335 commented Oct 26, 2022

Probably not anytime soon due to lack of hw testing capabilities.

@Darkhost1999
Copy link
Contributor

Are we able to assist in the HW tests?

@Juan12233
Copy link

Yes

@elad335
Copy link
Contributor Author

elad335 commented Oct 28, 2022

No.

@Asinin3
Copy link
Contributor

Asinin3 commented Oct 29, 2022

By Hardware tests, Elad means with a devkit not with a regular PlayStation 3. And the person who has a devkit our devs remote into for testing is currently moving house so it isn't online.

@Darkhost1999

This comment was marked as resolved.

@elad335 elad335 force-pushed the ps3-mem branch 2 times, most recently from 22a9502 to 1d9ab8e Compare August 10, 2023 09:40
@elad335 elad335 marked this pull request as ready for review August 10, 2023 09:41
@Darkhost1999

This comment was marked as resolved.

@Darkhost1999
Copy link
Contributor

Games are booting again and Max Payne 3 is back to the same compatibility as master.

@zarif98
Copy link

zarif98 commented Feb 4, 2024

Was this ever tested on the dev kit? This seems like a good set of changes.

@Darkhost1999
Copy link
Contributor

Draft because needs proper hw tests.

I see the PR is out of draft. How did the hardware tests go?
The PR needs rebased and conflicts resolved.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Brink [BLES01304] | VM: Access violation reading location 0x0 (unmapped memory)
6 participants