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

DRZ80 not working #1701

Open
wn2000 opened this issue Mar 31, 2024 · 8 comments
Open

DRZ80 not working #1701

wn2000 opened this issue Mar 31, 2024 · 8 comments

Comments

@wn2000
Copy link

wn2000 commented Mar 31, 2024

Originally posted by @wn2000 in #766 (comment)

@mahoneyt944
Copy link
Collaborator

Are you testing this with the current core? This core now has core options to force enable these modes.

@mahoneyt944
Copy link
Collaborator

If you force enable one of these modes, check the info log that it replaced the CPU

log_cb(RETRO_LOG_INFO, LOGPRE "Replaced Z80\n");

@wn2000
Copy link
Author

wn2000 commented Mar 31, 2024

To be clear there are two issues:

  1. The bug on these lines prevents the CPU replacement (as it's comparing a pointer address against an enum):

if (type==CPU_Z80 && Machine->drv->cpu[i].cpu_flags&CPU_AUDIO_CPU)

  1. After fixing this bug (so the log correctly shows "Replaced Z80"), some sound effects are missing (tested with sdfight before and after the fix).
    So there's some issue with the original PR that added the DrZ80 support.

@mahoneyt944
Copy link
Collaborator

It's possible the replacement CPU isn't fully compatible as the original. I'd assume this would be a case by case type of thing. Are the replacement cpus working at all after your fix?... to confirm the replaced cpu is operational at least even if buggy?

The core option allows you to replace a main z80 CPU, z80 sound CPU, or both....so there maybe specific combo that works better for some games than others.

@wn2000
Copy link
Author

wn2000 commented Apr 1, 2024

Yeah so it seems DrZ80 doesn't work at all. Tried 88 games, with Cyclone+DrZ80 it freezes at the ROM OK screen; with Cyclone only it works fine.
But Cyclone also has issues. AvP would black screen if Cyclone is enabled.

So, maybe not worth it after all.

@arcadez2003
Copy link
Collaborator

arcadez2003 commented Apr 1, 2024

Yeah as i commented at the time they were first supported here the cyclone cpu cores are buggy for a fair few games
but boost up the performance of many others hence we turned off the cyclone's by default and make them a toggle
option via the menus.

i suppose back in the day a list would have been made as to which games should be played using it and which should
not but no one can be bothered with such undertakings nowadays :)

@wn2000
Copy link
Author

wn2000 commented Apr 1, 2024

Looks like cyclone received some updates recently:

https://github.com/notaz/cyclone68000/commits/master/

@grant2258
Copy link
Contributor

This was a strange one. I deleted the mame2003 plus folder and done a fresh git clone compiled it and 88 games works. I think the git might be locking or some other odd behavior on the retropie I installed it is old but its the latest on their website. Try deleting your mame source directory and grab it again then recompile. Ive included the retropie bin 4.8 I compiled if you using that.

mame2003_plus_libretro.so.zip

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