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

xbox threads test core option #380

Merged
merged 3 commits into from
Jun 22, 2023
Merged

xbox threads test core option #380

merged 3 commits into from
Jun 22, 2023

Conversation

grant2258
Copy link

No description provided.

@greenchili2
Copy link

I was able to try a test build that someone was able to compile ( Gabo? ) on the Xbox Series X and this did the trick! Every game with discrete sound that I tried worked. I tried 59 of 176 parent rom sets. A few I had to update the romsets due to the changes in the jump to 0.255.

Suffice it to say "I'm getting it on like Donkey Kong".

@greenchili2
Copy link

I just tried cruis'n USA and it crashes... I did a search and I noticed the SDLMAME_EMSCRIPTEN defines are checked in the VIDEO.CPP file.. as well as some OSD source files related to retroarch. So more investigation may be needed to find a better way to do this. In the meantime if we could try something with the threading turned off in the video code as well. That would be great!

@joepogo
Copy link

joepogo commented Jun 22, 2023

Yea, i havent been able to test yet but i can confirm through greenchili this toggle will fix some of the games, def the discrete sound core games.

Once i get home ill be able to test ibara, mmmpork and the others to see if they load as well.

Thanks so much with everything you have done so far @grant2258 ! This is def moving in the right direction. :0

@grant2258
Copy link
Author

grant2258 commented Jun 22, 2023

well just tested the theory that was presented glad you guys can play a new game. I cant really work in hardware I dont have else I could make more progress on it. Im not sure how compatible mingw is with winrt either from what ive read its not implemented. I could try doing a mingw ucrt build not sure if it will work though would require a few changes as its all setup for mingw32 and mingw64. If i get the time ill look into it. I was just testing the threading theory @barbudreadmon posted. Its good to rule things out in a practical way at the very least at least you have a few new games to play.

@sonninnos
Copy link
Collaborator

sonninnos commented Jun 22, 2023

So is this ready for merge then?

Not really a fan of the current core option placement since it is between display related things, but it can always be moved later, so not that important.

@joepogo
Copy link

joepogo commented Jun 22, 2023

Yea! It's ready I think. Most games are working now! I'm working on a list to post in the issue ticket but I can def say go for it @sonninnos ! :)

@sonninnos
Copy link
Collaborator

Ok, let's go then.

@sonninnos sonninnos merged commit e8c8991 into libretro:master Jun 22, 2023
@grant2258 grant2258 deleted the apply_to_master branch July 14, 2023 12:45
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

Successfully merging this pull request may close these issues.

None yet

4 participants