-
Notifications
You must be signed in to change notification settings - Fork 8
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
SDL Audio driver is unstable and potentially hangs or crashes #26
Comments
I this still an issue? It was rewritten in XboxDev/nxdk-sdl@d5d814e |
I'm not sure. I don't think it crashes, but it might not be stable either?
That surprises me. Was the problem from https://github.com/XboxDev/nxdk-sdl/pull/27#issuecomment-635409017 resolved eventually? Also, most importantly, XboxDev/nxdk#362 almost definitely still exists, which I consider to be a major issue for stability (it breaks application state across applications / quick-reboot). |
Before merging the SDL audio support, we have known about potential issues in the SDL audio portion:
The exact behavior seems to depend on different factors like chosen playback rate / SDL audio format.
We'll have to stress-test, review and debug the code.
It's also possible that the issues are unrelated to the audio driver - we have not been able to confirm anything specifically. We just had independent reports of crashes in applications using SDL audio.
Any underlying subsystem (SDL logic or xaudio for example) could be responsible.
Unfortunately the SDL audio subsystem has competing backend design philosophies, so it's complicated and hard to compare to other drivers (and there's no good reference driver within SDL itself either). Debugging this might take a while.
I also have to recommend against testing in XQEMU because the QEMU AC97 implementation might cause additional timing or interrupt injection issues.
As a workaround you will have to disable all SDL audio (including SDL mixer) in your application; if you don't do this, there could be potential crashes due to SDL audio.
The text was updated successfully, but these errors were encountered: