On 26 September 2014 11:37, Alex Bennée <alex.ben...@linaro.org> wrote: > > Peter Maydell <peter.mayd...@linaro.org> writes: > >> 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.mayd...@linaro.org> >> Cc: qemu-sta...@nongnu.org > <snip> > > I've been testing it with my latest Travis patches (- the make check > once patch) and it seems a lot better now: > > Reviewed-by: Alex Bennée <alex.ben...@linaro.org> > Tested-by: Alex Bennée <alex.ben...@linaro.org>
Thanks; applied to master. -- PMM
