FBA Core slowdown after a while since 1.7.4

kikeminchas · 28 September 2018 14:04

Hi,

Since update to 1.7.4 the FBA core starts to slowdown after a while. It starts full speed but after some minutes it starts to slowdown progressively until it becomes unplayable. This doesn’t happen with other cores (Snes9x / Mupen64Plus…) I have tried NeoGeo and CPS2 games which used to run fine, even with some run ahead frames in my system (Nvidia Shield). Any idea? Could somebody else reproduce the issue? Thanks

hunterk · 28 September 2018 14:19

FBA slowing down with runahead is a known thing, but we’re not altogether sure why. Seems like it could be a memory leak with the savestate process.

kikeminchas · 28 September 2018 14:35

Ok, thanks for answering. It is true that I started playing with run ahead since this is my default for FBA core. When it starts to slowdown I deactivate run ahead and it goes back to normal speed but after some minutes slowdown starts again. I will try again with run ahead off since the beginning to make sure slow down does not happen in this case.

Tatsuya79 · 28 September 2018 19:48

We know why, Dwedit fixed that in recent RetroArch nightlies.
It shouldn’t happen anymore now.

Also FBA has been fixed to allow run ahead 2 instances which alleviates many sound problems.

hunterk · 28 September 2018 20:11

I just tried on Android with a nightly RA (I tried 32-bit and 64-bit) and a fresh FBA pull and it still wouldn’t do second-instance. I didn’t wait to see if the perf degraded anymore.

Tatsuya79 · 28 September 2018 20:49

Perhaps the temp path for the 2nd instance has some problem on android.

hunterk · 28 September 2018 20:55

ah, yeah, good call. does it need a read/write location? I can double-check that.

Tatsuya79 · 28 September 2018 21:46

Whatever you can do of that:

github.com

libretro/RetroArch/blob/master/runahead/secondary_core.c#L71


#else
DWORD pathLength = GetTempPathW(0, NULL) + 1;
wchar_t *wideStr = (wchar_t*)malloc(pathLength * sizeof(wchar_t));
wideStr[pathLength - 1] = 0;
GetTempPathW(pathLength, wideStr);


path = utf16_to_utf8_string_alloc(wideStr);
free(wideStr);
#endif
#else
path = "/tmp";
if (getenv("TMPDIR"))
   path = getenv("TMPDIR");


path = strcpy_alloc_force(path);
#endif
return path;
}


static bool write_file_with_random_name(char **tempDllPath,
   const char *retroarchTempPath, const void* data, ssize_t dataSize)

kikeminchas · 29 September 2018 09:14

Thanks hunterk and tatsuya for the answers and work on this. Just to know, I have uodated FBA core and slowdown is still there. I guess we would also need to install nightly or wait to next RA stable release to actually be able to get the fix right?

hunterk · 29 September 2018 12:50

yes, you need to update your RetroArch executable, as well.

PhantomImage · 22 December 2018 19:48

Did you ever come up with a solution for Android being unable to create a second instance?