On 27 October 2014 14:39, Paolo Bonzini <pbonz...@redhat.com> wrote: > > > On 10/27/2014 03:18 PM, Peter Maydell wrote: >> On 26 October 2014 09:32, Jan Kiszka <jan.kis...@web.de> wrote: >>> From: Jan Kiszka <jan.kis...@siemens.com> >>> >>> This reverts commit 15124e142034d21341ec9f1a304a1dc5a6c25681. It breaks >>> debuggability of qemu. >>> >>> Signed-off-by: Jan Kiszka <jan.kis...@siemens.com> >>> --- >>> >>> Feel free to apply this before or after "Make qemu_shutdown_requested >>> signal-safe". >>> >>> main-loop.c | 3 --- >>> 1 file changed, 3 deletions(-) >>> >>> diff --git a/main-loop.c b/main-loop.c >>> index d2e64f1..53393a4 100644 >>> --- a/main-loop.c >>> +++ b/main-loop.c >>> @@ -84,9 +84,6 @@ static int qemu_signal_init(void) >>> sigaddset(&set, SIGIO); >>> sigaddset(&set, SIGALRM); >>> sigaddset(&set, SIGBUS); >>> - sigaddset(&set, SIGINT); >>> - sigaddset(&set, SIGHUP); >>> - sigaddset(&set, SIGTERM); >> >> I'm planning to apply this patch but with the following >> comment added here: >> /* Note that the SIGINT, SIGTERM and SIGHUP signals are not handled >> * via signalfd, and so their handlers will still be invoked >> * asynchronously. This is done so that ^C can be used to interrupt >> * QEMU when it is being run under gdb. >> */ > > What about: > > /* SIGINT cannot be handled via signalfd, so that ^C can be used > * to interrupt QEMU when it is being run under gdb. SIGHUP and > * SIGTERM are also handled asynchronously, even though it is not > * strictly necessary, because they use the same handler as SIGINT. > */
Looks good to me, I'll use that wording. -- PMM