Project

General

Profile

Emulator Issues #8630

SDL: Dolphin ignores SIGTERM and SIGINT

Added by flacs almost 6 years ago.

Status:
Fixed
Priority:
Normal
Assignee:
Category:
Controls
% Done:

0%

Operating system:
N/A
Issue type:
Bug
Milestone:
Regression:
Yes
Relates to usability:
No
Relates to performance:
No
Easy:
Yes
Relates to maintainability:
No
Regression start:
Fixed in:

Description

What's the problem? Describe what went wrong in few words.
Dolphin does not terminate on SIGTERM or SIGINT.

What did you expect to happen instead?
Dolphin should terminate.

What steps will reproduce the problem?
1. Start Dolphin.
2. Run "killall dolphin-emu" or press Ctrl+C.
3. Nothing happens.

Which versions of Dolphin did you test on?
master (4.0-6571, 97382ec90a8c960d12836aa5b979e70da40bb9f5)

What are your PC specifications?
Linux with SDL 2.0.3

Is there any other relevant information?
This issue is caused by SDL. SDL_INIT_JOYSTICK implies SDL_INIT_EVENTS, which installs signal handlers for SIGTERM and SIGINT and translates them into SDL_QUIT events. http://hg.libsdl.org/SDL/rev/0979c3e31211 (to be released in SDL 2.0.4) provides a way to prevent this:

SDL_SetHint(SDL_HINT_NO_SIGNAL_HANDLERS, "1");


Related issues

Blocked by Emulator - Emulator Issues #7486: SDL/Linux: segfault on second game launch if haptic compatible controller plugged in.Fixed

History

#1 Updated by flacs almost 6 years ago

  • Status changed from New to Work started

#3 Updated by flacs almost 6 years ago

  • Status changed from Work started to Fix pending

#4 Updated by Buddybenj almost 6 years ago

  • Status changed from Fix pending to Fixed

#5 Updated by PEmu almost 6 years ago

PR #2530 was reverted in #2565. Should this be still be marked as Fixed?

#6 Updated by JosJuice almost 6 years ago

  • Status changed from Fixed to Accepted

#7 Updated by phire almost 6 years ago

  • Status changed from Accepted to Work started

I should be removing SDL in a few days anyway, that will fix this issue.

#8 Updated by phire almost 6 years ago

  • Status changed from Work started to Fix pending

#9 Updated by phire almost 6 years ago

  • Status changed from Fix pending to Fixed

Also available in: Atom PDF