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

Disc swapping is not working properly in Skies of Arcadia #189

Closed
bslenul opened this issue Feb 26, 2021 · 11 comments
Closed

Disc swapping is not working properly in Skies of Arcadia #189

bslenul opened this issue Feb 26, 2021 · 11 comments
Labels
bug Something isn't working

Comments

@bslenul
Copy link
Contributor

bslenul commented Feb 26, 2021

Platform / OS / Hardware: PC, Windows 10 64bit

Github hash: bbcfc83

Hardware: i5-4670K, GTX 970

Description of the Issue

Disc swapping seems to be broken with Skies of Arcadia (bbcfc83 being the current commit when typing this), sometimes after inserting the new disc it just freezes on the "Checking disc data..." screen, sometimes it sends you to BIOS directly after this screen and sometimes (but it's pretty rare) it boots just fine. It seems to consistently boot fine with interpreter or HLE BIOS.

Debugging Steps Tested

  • Boot Skies of Arcadia disc 2.
  • Start a new game.
  • Eject disc when asked.
  • Insert disc 1.
  • It will say "Checking disc data..." then either it will be stuck or it will send you to BIOS. If you're lucky it will boot just fine.
@bslenul bslenul changed the title Disc swapping is currently broken Disc swapping is currently broken with some games (Skies of Arcadia, Shenmue, maybe others?) Feb 26, 2021
@flyinghead
Copy link
Owner

I'm not sure there is any regression here. Games use different methods for switching disk.

Shenmue exits to BIOS as soon as the disk tray is open. This can be seen here: https://youtu.be/bpIu5iNnbDo?t=17060.

Skies of Arcadia disk switching sometimes work, sometimes doesn't (hanging on "Checking disk data" or rebooting to BIOS). This can probably be improved.

The most critical one is D2 because proper disk switching is required in order to continue the game. Other multidisk games are fine if saving to VMU and restarting the game with the next disk. (Correct me if I'm wrong here.)

@bslenul
Copy link
Contributor Author

bslenul commented Feb 27, 2021

Shenmue exits to BIOS as soon as the disk tray is open. This can be seen here: https://youtu.be/bpIu5iNnbDo?t=17060.

Oooh so it happens even on real hardware, I didn't remember that, my bad!

But yeah Skies of Arcadia made me super confused, I thought I bisected properly with the libretro core, then when I tried it again few hours later it didn't work anymore... I made more tests earlier today and yeah, noticed it was random, I was about to post a video showing its weird behavior but you're already aware of it.

At first I thought it was a regression since I didn't see any issue opened about it but it probably isn't, I edited the issue so it's now a SoA issue only.

D2 swap is currently broken on the libretro core however, I'll see if the changes made in 3bc508d can work on it then I'll open an issue on libretro repo.

Anyway, thanks for replying and for the clarifications! Sorry again for the false alarm about Shenmue :x

@bslenul bslenul changed the title Disc swapping is currently broken with some games (Skies of Arcadia, Shenmue, maybe others?) Disc swapping is not working properly in Skies of Arcadia Feb 27, 2021
@flyinghead
Copy link
Owner

The commit you mention will certainly fix the D2 problem on the core.

After some more testing with SoA, disk switching doesn't seem to work at all with a real BIOS and works randomly with HLE BIOS.

@bslenul
Copy link
Contributor Author

bslenul commented Feb 27, 2021

I got it to boot a few times with real BIOS but it's pretty rare, with HLE I just tried like ~10 times in a raw, got it to boot 9 times, 1 time the "Checking disc data..." disapearred but the game was stuck.

@flyinghead flyinghead added the bug Something isn't working label Feb 27, 2021
vkedwardli pushed a commit to vkedwardli/flycast that referenced this issue Jan 23, 2023
vkedwardli added a commit to vkedwardli/flycast that referenced this issue Jul 3, 2023
deps: update vulkan to 1.3.250.1, fix 404 on macOS
flyinghead added a commit that referenced this issue Jan 24, 2024
Add a 1 s delay when inserting a disk before the drive is ready.
Fixes disk swapping for Skies of Arcadia, Shenmue II (except HLE), Alone
in the Dark, Dancing Blade, First Kiss Story II and Kaen Seibo.
Issue #1046
Issue #189
Fixes BIOS menu disk swapping.
@flyinghead
Copy link
Owner

This should be fixed in the latest dev build.

@bslenul
Copy link
Contributor Author

bslenul commented Jan 24, 2024

Just tested with Skies of Arcadia (booting from disc 2 then either selecting "new game" or loading a disc 1 save) and the game is stuck on the "please insert disc 1 and close the disc door" screen after swapping discs. Same behavior on both standalone and the Libretro core (both 10d3ffd).

The swap still works fine when using the HLE BIOS.

@flyinghead
Copy link
Owner

Right, a last minute change broke this game. More work needed.

@flyinghead
Copy link
Owner

And thank you for your lightspeed fast report. I could pinpoint the issue in no time while it's still hot.

flyinghead added a commit that referenced this issue Jan 25, 2024
Status is Busy when disk swapping is in progress so libGDR_GetDiscType
always returns NoDisk/Open. Rely on the scheduler instead so that the
correct type is returned when disk swapping is finished.
Issue #1046
Issue #189
@flyinghead
Copy link
Owner

It should now be fixed by 116dc0e

@bslenul
Copy link
Contributor Author

bslenul commented Jan 25, 2024

Skies of Arcadia works great now 👍 However it doesn't seem to work with Resident Evil Code Veronica and Alone in the Dark, both games are telling me I inserted the incorrect disc.

edit: To be clear this is a regression from cbd01fb for these 2 games, not from 116dc0e.

D2 still swaps fine and Shenmue II now works 👍

Should I close this so we can continue in #1046? Instead of having 2 similar issues :p

@flyinghead
Copy link
Owner

I tested Alone in the Dark before my commit and just tested again and it works for me.

Anyway, yes it's probably better to continue in #1046 now that Skies of Arcadia works.

@bslenul bslenul closed this as completed Jan 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants