register signal handler after initializing SDL.
PATCH 1/1 register signal handler after initializing SDL.
SDL library initialization mangles signal handlers, so QEMU should register them after initializing SDL. This was the case before and code even have a comment about that. Fix it to be so again.
Signed-off-by Gleb Natapov <[email protected]>
Reviewed-by Peter Maydell <[email protected]>
vl.c +3 -3
@@ -3148,9 +3148,6 @@ int main(int argc, char **argv, char **envp)cpu_synchronize_all_post_init (); -/* must be after terminal init, SDL library changes signal handlers */ -os_setup_signal_handling (); -set_numa_modes (); current_machine = machine;
@@ -3206,6 +3203,9 @@ int main(int argc, char **argv, char **envp)break ; } +/* must be after terminal init, SDL library changes signal handlers */ +os_setup_signal_handling (); +#ifdef CONFIG_VNC /* init remote displays */ if (vnc_display) {
--
Gleb.
Aurelien Jarno
AppliedQuoted 6 lines · Gleb Natapov
SDL library initialization mangles signal handlers, so QEMU should register them after initializing SDL. This was the case before and code even have a comment about that. Fix it to be so again.
Signed-off-by Gleb Natapov <[email protected]>Reviewed-by Peter Maydell <[email protected]>
Thanks, applied.
Quoted 28 lines
vl.c +3
-4@@ -3148,9 +3148,6 @@ int main(int argc, char **argv, char **envp)cpu_synchronize_all_post_init (); -/* must be after terminal init, SDL library changes signal handlers */ -os_setup_signal_handling (); -set_numa_modes (); current_machine = machine;@@ -3206,6 +3203,9 @@ int main(int argc, char **argv, char **envp)break ; } +/* must be after terminal init, SDL library changes signal handlers */ +os_setup_signal_handling (); +#ifdef CONFIG_VNC /* init remote displays */ if (vnc_display) { --Gleb.
Signature · 2 lines
Aurelien Jarno GPG: 1024D/F1BCDB73 [email protected] http://www.aurel32.net