From patchwork Tue Oct 29 17:57:56 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "Simon Marchi (Code Review)" X-Patchwork-Id: 35426 Received: (qmail 52925 invoked by alias); 29 Oct 2019 17:58:32 -0000 Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Unsubscribe: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org Delivered-To: mailing list gdb-patches@sourceware.org Received: (qmail 49853 invoked by uid 89); 29 Oct 2019 17:58:17 -0000 Authentication-Results: sourceware.org; auth=none X-Spam-SWARE-Status: No, score=-26.9 required=5.0 tests=BAYES_00, GIT_PATCH_0, GIT_PATCH_1, GIT_PATCH_2, GIT_PATCH_3 autolearn=ham version=3.3.1 spammy= X-HELO: mx1.osci.io Received: from polly.osci.io (HELO mx1.osci.io) (8.43.85.229) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Tue, 29 Oct 2019 17:58:15 +0000 Received: by mx1.osci.io (Postfix, from userid 994) id 840CE210A7; Tue, 29 Oct 2019 13:58:12 -0400 (EDT) Received: from gnutoolchain-gerrit.osci.io (gnutoolchain-gerrit.osci.io [IPv6:2620:52:3:1:5054:ff:fe06:16ca]) by mx1.osci.io (Postfix) with ESMTP id 97A4E215AA for ; Tue, 29 Oct 2019 13:57:56 -0400 (EDT) Received: from localhost (localhost [127.0.0.1]) by gnutoolchain-gerrit.osci.io (Postfix) with ESMTP id 892172A7D9 for ; Tue, 29 Oct 2019 13:57:56 -0400 (EDT) X-Gerrit-PatchSet: 1 Date: Tue, 29 Oct 2019 13:57:56 -0400 From: "Tom Tromey (Code Review)" To: gdb-patches@sourceware.org Message-ID: Auto-Submitted: auto-generated X-Gerrit-MessageType: newchange Subject: [review] Introduce fetch_pending_stop X-Gerrit-Change-Id: I330cb8bda61ff9460f22d9e0eaaa96740d256202 X-Gerrit-Change-Number: 427 X-Gerrit-ChangeURL: X-Gerrit-Commit: 1446759c2733cceb960601dc2f3cf9b983287526 References: Reply-To: tromey@sourceware.org, gdb-patches@sourceware.org MIME-Version: 1.0 Content-Disposition: inline User-Agent: Gerrit/3.0.3-74-g460fb0f7e9 Change URL: https://gnutoolchain-gerrit.osci.io/r/c/binutils-gdb/+/427 ...................................................................... Introduce fetch_pending_stop This introduces a new "fetch_pending_stop" function and changes gdb to use it. This function removes the first matching pending stop from the list of such stops. Change-Id: I2d289dbf3ab17ae5c13c59afdf89b724301e962f gdb/ChangeLog 2019-10-29 Tom Tromey * windows-nat.c (get_windows_debug_event): Use fetch_pending_stop. * nat/windows-nat.h (fetch_pending_stop): Declare. * nat/windows-nat.c (fetch_pending_stop): New function. Change-Id: I330cb8bda61ff9460f22d9e0eaaa96740d256202 --- M gdb/ChangeLog M gdb/nat/windows-nat.c M gdb/nat/windows-nat.h M gdb/windows-nat.c 4 files changed, 49 insertions(+), 20 deletions(-) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 0d38851..e4bdd16 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,5 +1,12 @@ 2019-10-29 Tom Tromey + * windows-nat.c (get_windows_debug_event): Use + fetch_pending_stop. + * nat/windows-nat.h (fetch_pending_stop): Declare. + * nat/windows-nat.c (fetch_pending_stop): New function. + +2019-10-29 Tom Tromey + * windows-nat.c (windows_continue): Use matching_pending_stop and continue_last_debug_event. * nat/windows-nat.h (matching_pending_stop) diff --git a/gdb/nat/windows-nat.c b/gdb/nat/windows-nat.c index 163e3b1..7ac06f4 100644 --- a/gdb/nat/windows-nat.c +++ b/gdb/nat/windows-nat.c @@ -322,6 +322,33 @@ /* See nat/windows-nat.h. */ +bool +fetch_pending_stop (bool debug_events, pending_stop *result) +{ + for (auto iter = pending_stops.begin (); + iter != pending_stops.end (); + ++iter) + { + if (desired_stop_thread_id == -1 + || desired_stop_thread_id == iter->thread_id) + { + *result = *iter; + current_event = iter->event; + + DEBUG_EVENTS (("get_windows_debug_event - " + "pending stop found in 0x%x (desired=0x%x)\n", + iter->thread_id, desired_stop_thread_id)); + + pending_stops.erase (iter); + return true; + } + } + + return false; +} + +/* See nat/windows-nat.h. */ + BOOL continue_last_debug_event (DWORD continue_status, bool debug_events) { diff --git a/gdb/nat/windows-nat.h b/gdb/nat/windows-nat.h index 42e2e75..2a5d4a1 100644 --- a/gdb/nat/windows-nat.h +++ b/gdb/nat/windows-nat.h @@ -216,6 +216,12 @@ extern bool matching_pending_stop (bool debug_events); +/* See if a pending stop matches DESIRED_STOP_THREAD_ID. If so, + remove it from the list of pending stops, store it in *RESULT, set + 'current_event', and return true. Otherwise, return false. */ + +extern bool fetch_pending_stop (bool debug_events, pending_stop *result); + /* A simple wrapper for ContinueDebugEvent that continues the last waited-for event. */ diff --git a/gdb/windows-nat.c b/gdb/windows-nat.c index 6884411..e91e052 100644 --- a/gdb/windows-nat.c +++ b/gdb/windows-nat.c @@ -1301,29 +1301,18 @@ /* If there is a relevant pending stop, report it now. See the comment by the definition of "pending_stops" for details on why this is needed. */ - for (auto iter = pending_stops.begin (); - iter != pending_stops.end (); - ++iter) + pending_stop stop; + if (fetch_pending_stop (debug_events, &stop)) { - if (desired_stop_thread_id == -1 - || desired_stop_thread_id == iter->thread_id) - { - thread_id = iter->thread_id; - *ourstatus = iter->status; - current_event = iter->event; + thread_id = stop.thread_id; + *ourstatus = stop.status; - inferior_ptid = ptid_t (current_event.dwProcessId, thread_id, 0); - current_windows_thread = thread_rec (inferior_ptid, - INVALIDATE_CONTEXT); - current_windows_thread->reload_context = 1; + inferior_ptid = ptid_t (current_event.dwProcessId, thread_id, 0); + current_windows_thread = thread_rec (inferior_ptid, + INVALIDATE_CONTEXT); + current_windows_thread->reload_context = 1; - DEBUG_EVENTS (("get_windows_debug_event - " - "pending stop found in 0x%x (desired=0x%x)\n", - thread_id, desired_stop_thread_id)); - - pending_stops.erase (iter); - return thread_id; - } + return thread_id; } last_sig = GDB_SIGNAL_0;