From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from simark.ca by simark.ca with LMTP id t1NBNmYnuWBoIwAAWB0awg (envelope-from ) for ; Thu, 03 Jun 2021 15:03:02 -0400 Received: by simark.ca (Postfix, from userid 112) id D0E781F163; Thu, 3 Jun 2021 15:03:02 -0400 (EDT) X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on simark.ca X-Spam-Level: X-Spam-Status: No, score=-1.0 required=5.0 tests=MAILING_LIST_MULTI, URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.2 Received: from sourceware.org (server2.sourceware.org [8.43.85.97]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by simark.ca (Postfix) with ESMTPS id 347F91E01F for ; Thu, 3 Jun 2021 15:03:00 -0400 (EDT) Received: from server2.sourceware.org (localhost [IPv6:::1]) by sourceware.org (Postfix) with ESMTP id E86C0398EC33 for ; Thu, 3 Jun 2021 19:02:59 +0000 (GMT) Received: from mail-wm1-f41.google.com (mail-wm1-f41.google.com [209.85.128.41]) by sourceware.org (Postfix) with ESMTPS id C6E193857C75 for ; Thu, 3 Jun 2021 19:02:47 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org C6E193857C75 Authentication-Results: sourceware.org; dmarc=none (p=none dis=none) header.from=palves.net Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=gmail.com Received: by mail-wm1-f41.google.com with SMTP id u5-20020a7bc0450000b02901480e40338bso3238237wmc.1 for ; Thu, 03 Jun 2021 12:02:47 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:subject:date:message-id:mime-version :content-transfer-encoding; bh=8x3oAvgLL+KWyVXuRdrLSJ1SutlmMCvJKapAQ05YzA0=; b=uTnvBavvb/6Kmm1DG2mZswBSh3mo722Wt5I578d0VXBEn5XBULhYIcmhqpFGTPOqIz lw120RFT1AnvDaDzScMIzSIXgGvkr8DU9rXp3YV3H+oCeTavF7ndjLtJNtAUflzh7gyo wZLgVl07JlADRP64NlZxnN1Owz0l3XSieZ+9V+gV20dVr/XE9mFV5mTNiWeO9mF/5+cu pPUU40ZrLGGIxWfmYP6sPGUg9VnI3P3wLng1WGox/E8t3/CBfgcbX+rNLyL7Xh627h2V tWta7z6zi+xa9d1fqYUwoQwVoBZ7UQ4mR6Rz6KvpiIrX6w+MgHNdr+rFqXNJZMz8LUtA fMgA== X-Gm-Message-State: AOAM531NzeVDB3F1KHQ8S4/5Lv17nb991wS0G9dUCLSsihlw2qihbeCP NQtajUirNJA6NnNeSf4S5sL0wu7GlUg2Tg== X-Google-Smtp-Source: ABdhPJyIghapPFWoxASkkjo16YA9VeAxzsej5uZttRaUThkcpscGcJ9e9J1J31jDEdLNlh17CNzsCQ== X-Received: by 2002:a1c:6a14:: with SMTP id f20mr494716wmc.37.1622746966103; Thu, 03 Jun 2021 12:02:46 -0700 (PDT) Received: from localhost ([2001:8a0:f932:6a00:6b6e:c7b6:c5a7:aac3]) by smtp.gmail.com with ESMTPSA id k11sm6545771wmj.1.2021.06.03.12.02.44 for (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Thu, 03 Jun 2021 12:02:45 -0700 (PDT) From: Pedro Alves To: gdb-patches@sourceware.org Subject: [PATCH 00/17] Interrupting programs that block/ignore SIGINT Date: Thu, 3 Jun 2021 20:02:26 +0100 Message-Id: <20210603190243.2609886-1-pedro@palves.net> X-Mailer: git-send-email 2.26.2 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-BeenThere: gdb-patches@sourceware.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Gdb-patches mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: gdb-patches-bounces+public-inbox=simark.ca@sourceware.org Sender: "Gdb-patches" Currently, on GNU/Linux, it is not possible to interrupt with Ctrl-C programs that block or ignore SIGINT, with e.g., sigprocmask or signal(SIGINT, SIG_IGN). You type Ctrl-C, but nothing happens. Similarly, if a program uses sigwait to wait for SIGINT, and the program receives a SIGINT, the SIGINT is _not_ intercepted by ptrace, it goes straight to the inferior. These problems have been known for years, and recorded in gdb/9425, gdb/14559. This is a consequence of how GDB implements interrupting programs with Ctrl-C -- when GDB spawns a new process, it makes the process use the same terminal as GDB, and then makes the process's process group be the foreground process group in GDB's terminal. This means that when the process is running in the foreground, after e.g. "continue", when the user types Ctrl-C, the kernel sends a SIGINT to the foreground process group, which is the inferior. GDB then intercepts the SIGINT via ptrace, the same way it intercepts any other signal, stops all the other threads of the inferior if in all-stop, and presents the "Program received SIGINT" stop to the user. This series address the problem by turning Ctrl-C handling around such that the SIGINT always reaches GDB first, not the inferior. That is done by making GDB put inferiors in their own terminal/session created by GDB. I.e., GDB creates a pseudo-terminal master/slave pair, makes the inferior run with the slave as its terminal, and pumps output/input on the master end. Because the inferior is run with its own session/terminal, GDB is free to remain as the foreground process in its own terminal, which means that the Ctrl-C SIGINT always reaches GDB first, instead of reaching the inferior first, and then GDB reacting to the ptrace-intercepted SIGINT. Because GDB gets the SIGINT first, GDB is then free to handling it by interrupting the program any way it sees fit. The series will then make GDB interrupt the program with SIGSTOP instead of SIGINT, which always works even if the inferior blocks/ignores SIGINT -- SIGSTOP can't be ignored. (In the future GDB may even switch to PTRACE_INTERRUPT, though that's a project of its own.) Having the inferior in its own terminal also means that GDB is in control of when inferior output is flushed to the screen. When debugging with the CLI, this means that inferior output is now never interpersed with GDB's output in an unreadable fashion. This will also allow fixing the problem of inferior output really messing up the screen in the TUI, forcing users to Ctrl-L to refresh the screen. This series does not address the TUI part, but it shouldn't be too hard -- I wrote a quick&dirty prototype patch doing that a few years back, so I know it works. Tested on GNU/Linux native, gdbserver and gdbserver + "maint target set-non-stop on". Also build-tested on mingw32-w64, Solaris 11, and OpenBSD. More details in each of the patches in the series. The first patch adds tests that fail currently, but will pass at the end of the series. The main GDB changes are in patches #13 and #16. Patch #1-#6, #10-#12, #14-#15 could go in without the main changes. All documentation (manual, NEWS) is in the last patch -- patch #17. Pedro Alves (17): Test interrupting programs that block SIGINT [gdb/9425, gdb/14559] prefork_hook: Remove 'args' parameter Fix silent gdb.base/annota1.exp test coverage regression Make gdb.base/long-inferior-output.exp fail fast Fix gdb.multi/multi-term-settings.exp race Don't check parent pid in gdb.threads/{ia64-sigill,siginfo-threads,watchthreads-reorder}.c Special-case "set inferior-tty /dev/tty" Make inferior/GDB share terminal in run+detach testcases Make inferior/GDB share terminal in tests that exercise GDB/inferior reading same input gdb.mi/mi-logging.exp, don't send input to GDB while the inferior is running target_terminal::ours_for_output before printing signal received Move scoped_ignore_sigttou to gdbsupport/ Always put inferiors in their own terminal/session [gdb/9425, gdb/14559] exists_non_stop_target: Avoid flushing frames convert previous_inferior_ptid to strong reference to thread_info GNU/Linux: Interrupt/Ctrl-C with SIGSTOP instead of SIGINT [PR gdb/9425, PR gdb/14559] Document pseudo-terminal and interrupting changes gdb/doc/gdb.texinfo | 110 ++- gdb/NEWS | 23 + gdb/Makefile.in | 1 - gdb/event-top.c | 10 +- gdb/fork-child.c | 10 +- gdb/inf-ptrace.c | 24 +- gdb/inf-ptrace.h | 8 + gdb/infcmd.c | 63 +- gdb/inferior.h | 4 + gdb/inflow.c | 731 +++++++++++++++--- gdb/infrun.c | 179 ++++- gdb/infrun.h | 16 + gdb/linux-nat.c | 115 ++- gdb/linux-nat.h | 2 + gdb/nat/fork-inferior.c | 93 ++- gdb/nat/fork-inferior.h | 30 +- gdb/procfs.c | 1 - gdb/ser-unix.c | 2 +- gdb/target.c | 40 +- gdb/terminal.h | 7 + .../gdb.base/annota-input-while-running.exp | 4 + gdb/testsuite/gdb.base/annota1.exp | 12 +- gdb/testsuite/gdb.base/attach-wait-input.exp | 4 + .../gdb.base/auto-connect-native-target.exp | 3 + .../gdb.base/bp-cmds-continue-ctrl-c.exp | 3 + .../gdb.base/continue-all-already-running.exp | 3 + gdb/testsuite/gdb.base/detach.exp | 19 +- gdb/testsuite/gdb.base/dprintf-detach.exp | 9 +- gdb/testsuite/gdb.base/infcall-input.exp | 12 +- .../gdb.base/interrupt-daemon-attach.exp | 2 +- gdb/testsuite/gdb.base/interrupt-daemon.exp | 4 +- gdb/testsuite/gdb.base/interrupt-noterm.exp | 2 +- gdb/testsuite/gdb.base/interrupt.exp | 4 +- gdb/testsuite/gdb.base/jit-elf.exp | 10 +- .../gdb.base/long-inferior-output.exp | 31 +- gdb/testsuite/gdb.base/multi-forks.exp | 3 + gdb/testsuite/gdb.base/random-signal.exp | 3 +- gdb/testsuite/gdb.base/range-stepping.exp | 2 +- gdb/testsuite/gdb.base/sigint-masked-out.c | 43 ++ gdb/testsuite/gdb.base/sigint-masked-out.exp | 109 +++ gdb/testsuite/gdb.base/sigint-sigwait.c | 80 ++ gdb/testsuite/gdb.base/sigint-sigwait.exp | 113 +++ .../gdb.base/watchpoint-hw-attach.exp | 3 + gdb/testsuite/gdb.gdb/selftest.exp | 4 +- gdb/testsuite/gdb.mi/mi-detach.exp | 6 + gdb/testsuite/gdb.mi/mi-logging.exp | 94 ++- gdb/testsuite/gdb.mi/new-ui-mi-sync.exp | 2 +- .../gdb.multi/multi-target-interrupt.exp | 2 +- .../gdb.multi/multi-target-no-resumed.exp | 2 +- .../gdb.multi/multi-term-settings.exp | 92 ++- .../gdb.multi/watchpoint-multi-exit.exp | 3 + gdb/testsuite/gdb.server/reconnect-ctrl-c.exp | 4 +- gdb/testsuite/gdb.threads/async.exp | 2 +- .../gdb.threads/continue-pending-status.exp | 2 +- gdb/testsuite/gdb.threads/ia64-sigill.c | 5 - gdb/testsuite/gdb.threads/leader-exit.exp | 2 +- gdb/testsuite/gdb.threads/manythreads.exp | 4 +- .../process-dies-while-detaching.exp | 3 + gdb/testsuite/gdb.threads/pthreads.exp | 2 +- gdb/testsuite/gdb.threads/schedlock.exp | 11 +- gdb/testsuite/gdb.threads/siginfo-threads.c | 5 - gdb/testsuite/gdb.threads/sigthread.exp | 2 +- gdb/testsuite/gdb.threads/threadapply.exp | 3 + .../gdb.threads/watchthreads-reorder.c | 5 - gdb/testsuite/lib/gdb.exp | 19 + gdb/tui/tui-win.c | 3 + gdbserver/fork-child.cc | 15 +- gdbsupport/Makefile.am | 1 + gdbsupport/Makefile.in | 15 +- gdbsupport/managed-tty.cc | 22 + gdbsupport/managed-tty.h | 42 + .../scoped_ignore_sigttou.h | 8 +- 72 files changed, 2059 insertions(+), 268 deletions(-) create mode 100644 gdb/testsuite/gdb.base/sigint-masked-out.c create mode 100644 gdb/testsuite/gdb.base/sigint-masked-out.exp create mode 100644 gdb/testsuite/gdb.base/sigint-sigwait.c create mode 100644 gdb/testsuite/gdb.base/sigint-sigwait.exp create mode 100644 gdbsupport/managed-tty.cc create mode 100644 gdbsupport/managed-tty.h rename gdb/inflow.h => gdbsupport/scoped_ignore_sigttou.h (90%) base-commit: 7cfa3e63f9fe7d3a17b8d5d28dff964ee32ecec4 -- 2.26.2