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

DOSBox SVN suffers low graphics performance on macOS with output=opengl #220

Closed
kjliew opened this issue Sep 25, 2022 · 2 comments
Closed
Assignees
Milestone

Comments

@kjliew
Copy link

kjliew commented Sep 25, 2022

Homebrew had replaced sdl with sdl12-compat. Vanilla DOSBox SVN r4481 with output=opengl suffers low graphics performance. Tested using MDKDOS and PCPBENCH DOS4GW SVGA benchmarks. The drop in performance is extremely significant that OpenGL seemed Unaccelerated though GL strings returned "Apple M1 OpenGL 2.1 Metal - 76.3"

On modern Linux, sdl12-compat is superb for DOSBox SVN r4481.

MSYS2/mingw-w64 does not seem to endorse sdl12-compat. So this is not tested on Windows 10/11.

Apple M1 macOS Monterey 12.6
SDL2-2.24.0
sdl12-compat 1.2.56

@icculus icculus self-assigned this Sep 25, 2022
@icculus icculus added this to the 1.2.60 milestone Sep 25, 2022
@kjliew
Copy link
Author

kjliew commented Sep 27, 2022

I figured it out that this is not sdl12-compat to blame, but due to DOSBox SVN handling of SDL_ACTIVEEVENT and changing priority on losing focus. The drop in performance reproduced exactly on Windows 11.

On Linux, DOSBox SVN has hackery that filtered out unwanted SDL_ACTIVEEVENT. The hackery was tagged with comment that this was an SDL1.2 bug on X11 but the same hackery did not fix the problem on Windows 11/macOS. There is a more elegant & easy way to deal with unwanted SDL_ACTIVEEVENT that solves this issue on Windows/macOS.

@icculus Since you had marked this issue with a milestone, perhaps you found other related issues that you planned to fix. Otherwise, feel free to close it as I think the ball should be in the court of DOSBox SVN.

@icculus
Copy link
Collaborator

icculus commented Sep 27, 2022

Cool, I hadn't gotten around to debugging this yet, so I appreciate you having done this deeper research!

I'll close this issue now.

@icculus icculus closed this as not planned Won't fix, can't repro, duplicate, stale Sep 27, 2022
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

No branches or pull requests

2 participants