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

Noticeable Lag While Saving #477

Closed
JMC47 opened this issue Jun 26, 2019 · 8 comments
Closed

Noticeable Lag While Saving #477

JMC47 opened this issue Jun 26, 2019 · 8 comments

Comments

@JMC47
Copy link

JMC47 commented Jun 26, 2019

It appears that melonDS does not asynchronously write savefiles, as whenever games like Mario Kart DS save a ghost file, my framerate drops tremendously. The issue disappears if I move the game/save to a solidstate harddrive.

Other emulators have asynchronous flushing to file to prevent issues like this.

@Arisotura
Copy link
Member

good point

@nathanielcwm
Copy link

It's noticeable in Ace Attorney as well

@Zettymaster
Copy link
Contributor

Zettymaster commented Aug 16, 2019 via email

@RSDuck
Copy link
Member

RSDuck commented Aug 25, 2019

for some games like 4th gen Pokemon games you have to keep in mind that it taxes the CPU more than usually during saving, because it does encryption and what not on the save file. For some reason that's not the case for 5th gen Pokemon games, I guess they've optimised it at that point (it's also far quicker).

@Liger0
Copy link

Liger0 commented Apr 22, 2020

It also slows down on 5th gen, although less.

@ichee
Copy link

ichee commented Nov 12, 2020

62e3f41 seems to fix this issue, though I don't have all reported games to test with.

Castlevania Dawn of Sorrow was affected as well, but not after this commit.

@RSDuck
Copy link
Member

RSDuck commented Nov 12, 2020

62e3f41 still didn't really fix it (as the IO operations are still completely synchronous), though I'm happy to hear that this change atleast alleviated the problem.

@RSDuck
Copy link
Member

RSDuck commented Jan 25, 2021

fixed with #925

@RSDuck RSDuck closed this as completed Jan 25, 2021
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

No branches or pull requests

7 participants