main-loop.c: Handle SIGINT, SIGHUP and SIGTERM synchronously
Add the termination signals SIGINT, SIGHUP and SIGTERM to the
list of signals which we handle synchronously via a signalfd.
This avoids a race condition where if we took the SIGTERM
in the middle of qemu_shutdown_requested:
    int r = shutdown_requested;
[SIGTERM here...]
    shutdown_requested = 0;
then the setting of the shutdown_requested flag by
termsig_handler() would be lost and QEMU would fail to
shut down. This was causing 'make check' to hang occasionally.
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Reviewed-by: Paolo Bonzini <pbonzini@redhat.com>
Reviewed-by: Alex Bennée <alex.bennee@linaro.org>
Tested-by: Alex Bennée <alex.bennee@linaro.org>
Message-id: 1411660269-11081-1-git-send-email-peter.maydell@linaro.org
Cc: qemu-stable@nongnu.org
			
			
This commit is contained in:
		
							parent
							
								
									6a0fcbdf2d
								
							
						
					
					
						commit
						15124e1420
					
				| 
						 | 
				
			
			@ -84,6 +84,9 @@ static int qemu_signal_init(void)
 | 
			
		|||
    sigaddset(&set, SIGIO);
 | 
			
		||||
    sigaddset(&set, SIGALRM);
 | 
			
		||||
    sigaddset(&set, SIGBUS);
 | 
			
		||||
    sigaddset(&set, SIGINT);
 | 
			
		||||
    sigaddset(&set, SIGHUP);
 | 
			
		||||
    sigaddset(&set, SIGTERM);
 | 
			
		||||
    pthread_sigmask(SIG_BLOCK, &set, NULL);
 | 
			
		||||
 | 
			
		||||
    sigdelset(&set, SIG_IPI);
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
		Loading…
	
		Reference in New Issue