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

Disable hypervisor by default on macOS #6585

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

gdkchan
Copy link
Member

@gdkchan gdkchan commented Mar 31, 2024

Thanks to #6057 and #6356, most performance and compatibility issues with JIT have been fixed. Since some games freezes randomly when using hypervisor, I believe using JIT is a better default on macOS now. This only changes the default setting, people that already has it enabled or disabled will keep their current setting.

It would be nice to get some confirmation that games generally running better on JIT than hypervisor, or in other words, that this is a good change, I'm doing mainly based on reports of people that had better results with JIT.

@github-actions github-actions bot added the gui Related to Ryujinx.Ui label Mar 31, 2024
@ryujinx-mako ryujinx-mako bot requested review from AcK77, emmauss, TSRBerry and a team March 31, 2024 11:45
@MutantAura
Copy link
Collaborator

Stability wise JIT is much superior but there are still some fairly significant performance delta's to HV.
Tested on base M2 w/ 10 GPU cores

Legends Arceus (Town entrance)
HV: 42FPS
JIT: 34FPS (-19%)

Breath of the Wild (Tested by adding the hv-fix back)
HV: 48FPS
JIT: 42FPS (-12%)

Persona 5 Royal (Station Square)
HV: 75FPS
JIT: 64FPS (-15%)

Tears of the Kingdom (Lookout landing)
HV: 33FPS
JIT: 23FPS (-31%)

Super Mario Oddssey
HV: 101FPS
JIT: 90FPS (-11%)

With that said, the first two games are pretty much unplayable via HV due to freezes so take those results as a best case scenario. The performance hit globally is still quite large, in ToTK in particular it's massive, but the decision I think will swing on what people value more?

@TSRBerry
Copy link
Member

TSRBerry commented May 2, 2024

Should we ask for opinions before we merge this?

@tgcordell
Copy link

tgcordell commented May 3, 2024

A bunch of memory PRs were merged since the last benchmark, so I wanted to test again

Game - TOTK, lookout landing
Ryujinx - 1.1.1293
M1 Pro w/ 16GB ram
HV - 35 fps
JIT - 28 fps (-20%)

IMO - The bigger thing holding back the Ryujinx experience on Mac is the hitching due to shader recompilation, called out in #5713 - but I know thats waiting on a MoltenVK enhancement.

@gdkchan
Copy link
Member Author

gdkchan commented May 16, 2024

Should we ask for opinions before we merge this?

Some people said that games freeze with hypervisor disabled, other said it fixes those issues, so I can only assume it only works for some people based on that, or maybe it is a red herring and doesn't fix the problem at all. So I dunno what is the better default. At least I think the crashes that some games have with hypervisor enabled are consistently fixed.

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

Successfully merging this pull request may close these issues.

None yet

6 participants