From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail.baldwin.cx (bigwig.baldwin.cx [IPv6:2607:f138:0:13::2]) by sourceware.org (Postfix) with ESMTPS id 4BCB5385840D for ; Fri, 21 Jan 2022 20:16:40 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org 4BCB5385840D Authentication-Results: sourceware.org; dmarc=none (p=none dis=none) header.from=FreeBSD.org Authentication-Results: sourceware.org; spf=fail smtp.mailfrom=FreeBSD.org Received: from ralph.baldwin.cx (ralph.baldwin.cx [66.234.199.215]) by mail.baldwin.cx (Postfix) with ESMTPSA id 752FA1A84C70 for ; Fri, 21 Jan 2022 15:16:38 -0500 (EST) From: John Baldwin To: gdb-patches@sourceware.org Subject: [PATCH v5 02/15] gdb linux-nat: Convert linux_nat_event_pipe to the event_pipe class. Date: Fri, 21 Jan 2022 12:16:18 -0800 Message-Id: <20220121201631.63530-3-jhb@FreeBSD.org> X-Mailer: git-send-email 2.34.1 In-Reply-To: <20220121201631.63530-1-jhb@FreeBSD.org> References: <20220121201631.63530-1-jhb@FreeBSD.org> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Greylist: Sender succeeded SMTP AUTH, not delayed by milter-greylist-4.6.4 (mail.baldwin.cx [0.0.0.0]); Fri, 21 Jan 2022 15:16:38 -0500 (EST) X-Virus-Scanned: clamav-milter 0.103.1 at mail.baldwin.cx X-Virus-Status: Clean X-Spam-Status: No, score=-12.3 required=5.0 tests=BAYES_00, FORGED_SPF_HELO, GIT_PATCH_0, KAM_DMARC_STATUS, KHOP_HELO_FCRDNS, SPF_HELO_PASS, SPF_SOFTFAIL, TXREP autolearn=ham autolearn_force=no version=3.4.4 X-Spam-Checker-Version: SpamAssassin 3.4.4 (2020-01-24) on server2.sourceware.org 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: , X-List-Received-Date: Fri, 21 Jan 2022 20:16:43 -0000 Use event_pipe from gdbsupport in place of the existing file descriptor array. --- gdb/linux-nat.c | 59 ++++++++++++++----------------------------------- 1 file changed, 16 insertions(+), 43 deletions(-) diff --git a/gdb/linux-nat.c b/gdb/linux-nat.c index 0cc6923ad02..db3585dfa23 100644 --- a/gdb/linux-nat.c +++ b/gdb/linux-nat.c @@ -48,6 +48,7 @@ #include /* for O_RDONLY */ #include "inf-loop.h" #include "gdbsupport/event-loop.h" +#include "gdbsupport/event-pipe.h" #include "event-top.h" #include #include @@ -111,11 +112,11 @@ and target events, so neither blocking waitpid nor sigsuspend are viable options. Instead, we should asynchronously notify the GDB main event loop whenever there's an unprocessed event from the target. We detect asynchronous target events by handling SIGCHLD signals. To -notify the event loop about target events, the self-pipe trick is used ---- a pipe is registered as waitable event source in the event loop, +notify the event loop about target events, an event pipe is used +--- the pipe is registered as waitable event source in the event loop, the event loop select/poll's on the read end of this pipe (as well on -other event sources, e.g., stdin), and the SIGCHLD handler writes a -byte to this pipe. This is more portable than relying on +other event sources, e.g., stdin), and the SIGCHLD handler marks the +event pipe to raise an event. This is more portable than relying on pselect/ppoll, since on kernels that lack those syscalls, libc emulates them with select/poll+sigprocmask, and that is racy (a.k.a. plain broken). @@ -228,26 +229,18 @@ static int report_thread_events; /* Async mode support. */ -/* The read/write ends of the pipe registered as waitable file in the - event loop. */ -static int linux_nat_event_pipe[2] = { -1, -1 }; +/* The event pipe registered as a waitable file in the event loop. */ +static event_pipe linux_nat_event_pipe; /* True if we're currently in async mode. */ -#define linux_is_async_p() (linux_nat_event_pipe[0] != -1) +#define linux_is_async_p() (linux_nat_event_pipe.is_open ()) /* Flush the event pipe. */ static void async_file_flush (void) { - int ret; - char buf; - - do - { - ret = read (linux_nat_event_pipe[0], &buf, 1); - } - while (ret >= 0 || (ret == -1 && errno == EINTR)); + linux_nat_event_pipe.flush (); } /* Put something (anything, doesn't matter what, or how much) in event @@ -257,21 +250,7 @@ async_file_flush (void) static void async_file_mark (void) { - int ret; - - /* It doesn't really matter what the pipe contains, as long we end - up with something in it. Might as well flush the previous - left-overs. */ - async_file_flush (); - - do - { - ret = write (linux_nat_event_pipe[1], "+", 1); - } - while (ret == -1 && errno == EINTR); - - /* Ignore EAGAIN. If the pipe is full, the event loop will already - be awakened anyway. */ + linux_nat_event_pipe.mark (); } static int kill_lwp (int lwpid, int signo); @@ -4202,7 +4181,7 @@ sigchld_handler (int signo) gdb_stdlog->write_async_safe ("sigchld\n", sizeof ("sigchld\n") - 1); if (signo == SIGCHLD - && linux_nat_event_pipe[0] != -1) + && linux_nat_event_pipe.is_open ()) async_file_mark (); /* Let the event loop know that there are events to handle. */ @@ -4234,19 +4213,13 @@ linux_async_pipe (int enable) if (enable) { - if (gdb_pipe_cloexec (linux_nat_event_pipe) == -1) + if (!linux_nat_event_pipe.open ()) internal_error (__FILE__, __LINE__, "creating event pipe failed."); - - fcntl (linux_nat_event_pipe[0], F_SETFL, O_NONBLOCK); - fcntl (linux_nat_event_pipe[1], F_SETFL, O_NONBLOCK); } else { - close (linux_nat_event_pipe[0]); - close (linux_nat_event_pipe[1]); - linux_nat_event_pipe[0] = -1; - linux_nat_event_pipe[1] = -1; + linux_nat_event_pipe.close (); } restore_child_signals_mask (&prev_mask); @@ -4258,7 +4231,7 @@ linux_async_pipe (int enable) int linux_nat_target::async_wait_fd () { - return linux_nat_event_pipe[0]; + return linux_nat_event_pipe.event_fd (); } /* target_async implementation. */ @@ -4270,7 +4243,7 @@ linux_nat_target::async (int enable) { if (!linux_async_pipe (1)) { - add_file_handler (linux_nat_event_pipe[0], + add_file_handler (linux_nat_event_pipe.event_fd (), handle_target_event, NULL, "linux-nat"); /* There may be pending events to handle. Tell the event loop @@ -4280,7 +4253,7 @@ linux_nat_target::async (int enable) } else { - delete_file_handler (linux_nat_event_pipe[0]); + delete_file_handler (linux_nat_event_pipe.event_fd ()); linux_async_pipe (0); } return; -- 2.34.1