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 incorrect maximum ROM size #1103

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

Conversation

Rosalie241
Copy link
Contributor

@Rosalie241 Rosalie241 commented Dec 16, 2024

This allows ROMs larger than 64MiB to work in mupen64plus.

Tested with the following ROMs:

You can find documentation about the console limitations here:

Fixes #997
Fixes #1101

@Rosalie241 Rosalie241 force-pushed the fix-romsize branch 3 times, most recently from 79cf5ea to c22375f Compare December 16, 2024 12:53
@Jj0YzL5nvJ
Copy link
Contributor

This is much better than the previous attempt #685

Looking at your latest modifications, there seems to be a lot more "limits" to change... I can't be of much help in that regard.
Add this ROM to your tests, in my case it always soft freeze (my ParaLLEl RDP/RSP forks are too old =P) on the same part... I don't know if it works with Ares or some other emulator. The ideal would be to test it with EverDrive or something similar.

@loganmc10
Copy link
Member

I would also make sure to check that Paper Mario still works (and that it can save). I remember it did something where it wrote outside the cart address space, but maybe I'm remembering wrong

@GhostlyDark
Copy link
Contributor

I would also make sure to check that Paper Mario still works (and that it can save). I remember it did something where it wrote outside the cart address space, but maybe I'm remembering wrong

Just tested, saving works fine.

@GhostlyDark
Copy link
Contributor

This is much better than the previous attempt #685

Looking at your latest modifications, there seems to be a lot more "limits" to change... I can't be of much help in that regard. Add this ROM to your tests, in my case it always soft freeze (my ParaLLEl RDP/RSP forks are too old =P) on the same part... I don't know if it works with Ares or some other emulator. The ideal would be to test it with EverDrive or something similar.

It does work with Angrylion (fine) and Parallel RDP (glitches), but only with Parallel RSP. Other RSP plugins just freeze on boot.

image

image

@Jj0YzL5nvJ
Copy link
Contributor

It pass with this build: https://github.com/Rosalie241/RMG/actions/runs/12344405888

But the same build fails with the CI cores: https://github.com/mupen64plus/mupen64plus-core/actions?query=branch%3Afix-romsize

That ROM seems to be extremely sensitive to compiler optimizations...

@Rosalie241
Copy link
Contributor Author

It pass with this build: https://github.com/Rosalie241/RMG/actions/runs/12344405888

But the same build fails with the CI cores: https://github.com/mupen64plus/mupen64plus-core/actions?query=branch%3Afix-romsize

That ROM seems to be extremely sensitive to compiler optimizations...

It plays back fully for you without issues? on Fedora it hangs eventually for me when built locally sadly, though now that you mention compiler optimizations, I've had issues with that before while working on improving FPU accuracy, which is extremely frustrating.

@Jj0YzL5nvJ
Copy link
Contributor

It plays back fully for you without issues?

Only on Windows and Wine with this configuration...

@Rosalie241
Copy link
Contributor Author

Rosalie241 commented Jan 9, 2025

It plays back fully for you without issues?

Only on Windows and Wine with this configuration...

Thank you, I can confirm it plays fully with paraLLEl RSP and angrylion-rdp-plus on Windows

image

I can also confirm it works on Linux:

image

I can also confirm saving in Paper Mario works fine still.

@Rosalie241
Copy link
Contributor Author

Thanks to the help of @GhostlyDark, we managed to find a solution to the unreliable behavior, it was a mistake I made.

@Jj0YzL5nvJ can you test it again with the latest changes? It works for GhostlyDark and me fine now.

Copy link
Contributor

@Jj0YzL5nvJ Jj0YzL5nvJ left a comment

Choose a reason for hiding this comment

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

Excellent, runs perfect. Even with my deprecated ParaLLEl RDP/RSP forks =P

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.

The 64MB rom size limit should be 4GiB Cartridges above 64 MB in size?
4 participants