From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from barracuda.ebox.ca (barracuda.ebox.ca [96.127.255.19]) by sourceware.org (Postfix) with ESMTPS id 796943858416 for ; Tue, 18 Jan 2022 04:09:48 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org 796943858416 X-ASG-Debug-ID: 1642478978-0c856e06ad131c00001-fS2M51 Received: from smtp.ebox.ca (smtp.ebox.ca [96.127.255.82]) by barracuda.ebox.ca with ESMTP id lhq3gp5mx19PAVeJ (version=TLSv1 cipher=DHE-RSA-AES256-SHA bits=256 verify=NO); Mon, 17 Jan 2022 23:09:38 -0500 (EST) X-Barracuda-Envelope-From: simon.marchi@polymtl.ca X-Barracuda-RBL-Trusted-Forwarder: 96.127.255.82 Received: from simark.localdomain (192-222-157-6.qc.cable.ebox.net [192.222.157.6]) by smtp.ebox.ca (Postfix) with ESMTP id 920D7441D67; Mon, 17 Jan 2022 23:09:38 -0500 (EST) From: Simon Marchi X-Barracuda-RBL-IP: 192.222.157.6 X-Barracuda-Effective-Source-IP: 192-222-157-6.qc.cable.ebox.net[192.222.157.6] X-Barracuda-Apparent-Source-IP: 192.222.157.6 To: gdb-patches@sourceware.org Subject: [PATCH v2 4/9] gdb/infrun: add inferior parameters to stop_all_threads and restart_threads Date: Mon, 17 Jan 2022 23:09:32 -0500 X-ASG-Orig-Subj: [PATCH v2 4/9] gdb/infrun: add inferior parameters to stop_all_threads and restart_threads Message-Id: <20220118040937.730282-5-simon.marchi@polymtl.ca> X-Mailer: git-send-email 2.34.1 In-Reply-To: <20220118040937.730282-1-simon.marchi@polymtl.ca> References: <20220118040937.730282-1-simon.marchi@polymtl.ca> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Barracuda-Connect: smtp.ebox.ca[96.127.255.82] X-Barracuda-Start-Time: 1642478978 X-Barracuda-Encrypted: DHE-RSA-AES256-SHA X-Barracuda-URL: https://96.127.255.19:443/cgi-mod/mark.cgi X-Virus-Scanned: by bsmtpd at ebox.ca X-Barracuda-Scan-Msg-Size: 5036 X-Barracuda-BRTS-Status: 1 X-Barracuda-Spam-Score: 0.00 X-Barracuda-Spam-Status: No, SCORE=0.00 using global scores of TAG_LEVEL=1000.0 QUARANTINE_LEVEL=1000.0 KILL_LEVEL=8.0 tests= X-Barracuda-Spam-Report: Code version 3.2, rules version 3.2.3.95404 Rule breakdown below pts rule name description ---- ---------------------- -------------------------------------------------- X-Spam-Status: No, score=-15.8 required=5.0 tests=BAYES_00, GIT_PATCH_0, KAM_DMARC_QUARANTINE, KAM_DMARC_STATUS, RCVD_IN_DNSWL_LOW, SPF_HELO_NONE, 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: Tue, 18 Jan 2022 04:09:50 -0000 A following patch will want to stop all threads of a given inferior (as opposed to all threads of all inferiors) while handling a vfork, and restart them after. To help with this, add inferior parameters to stop_all_threads and restart_threads. This is done as a separate patch to make sure this doesn't cause regressions on its own, and to keep the following patches more concise. No visible changes are expected here, since all calls sites pass nullptr, which should keep the existing behavior. Change-Id: I4d9ba886ce842042075b4e346cfa64bbe2580dbf --- gdb/infrun.c | 39 +++++++++++++++++++++++++++++++-------- gdb/infrun.h | 7 +++++-- 2 files changed, 36 insertions(+), 10 deletions(-) diff --git a/gdb/infrun.c b/gdb/infrun.c index 40b4bdd73130..4d12c72f89fd 100644 --- a/gdb/infrun.c +++ b/gdb/infrun.c @@ -3754,7 +3754,8 @@ struct wait_one_event }; static bool handle_one (const wait_one_event &event); -static void restart_threads (struct thread_info *event_thread); +static void restart_threads (struct thread_info *event_thread, + inferior *inf = nullptr); /* Prepare and stabilize the inferior for detaching it. E.g., detaching while a thread is displaced stepping is a recipe for @@ -4919,7 +4920,7 @@ handle_one (const wait_one_event &event) /* See infrun.h. */ void -stop_all_threads (const char *reason) +stop_all_threads (const char *reason, inferior *inf) { /* We may need multiple passes to discover all threads. */ int pass; @@ -4927,22 +4928,29 @@ stop_all_threads (const char *reason) gdb_assert (exists_non_stop_target ()); - INFRUN_SCOPED_DEBUG_START_END ("reason=%s", reason); + INFRUN_SCOPED_DEBUG_START_END ("reason=%s, inf=%d", reason, + inf != nullptr ? inf->num : -1); scoped_restore_current_thread restore_thread; - /* Enable thread events of all targets. */ + /* Enable thread events on relevant targets. */ for (auto *target : all_non_exited_process_targets ()) { + if (inf != nullptr && inf->process_target () != target) + continue; + switch_to_target_no_thread (target); target_thread_events (true); } SCOPE_EXIT { - /* Disable thread events of all targets. */ + /* Disable thread events on relevant targets. */ for (auto *target : all_non_exited_process_targets ()) { + if (inf != nullptr && inf->process_target () != target) + continue; + switch_to_target_no_thread (target); target_thread_events (false); } @@ -4967,6 +4975,9 @@ stop_all_threads (const char *reason) for (auto *target : all_non_exited_process_targets ()) { + if (inf != nullptr && inf->process_target () != target) + continue; + switch_to_target_no_thread (target); update_thread_list (); } @@ -4975,6 +4986,9 @@ stop_all_threads (const char *reason) to tell the target to stop. */ for (thread_info *t : all_non_exited_threads ()) { + if (inf != nullptr && t->inf != inf) + continue; + /* For a single-target setting with an all-stop target, we would not even arrive here. For a multi-target setting, until GDB is able to handle a mixture of @@ -5717,17 +5731,26 @@ handle_inferior_event (struct execution_control_state *ecs) } /* Restart threads back to what they were trying to do back when we - paused them for an in-line step-over. The EVENT_THREAD thread is - ignored. */ + paused them (because of an in-line step-over or vfork, for example). + The EVENT_THREAD thread is ignored (not restarted). + + If INF is non-nullptr, only resume threads from INF. */ static void -restart_threads (struct thread_info *event_thread) +restart_threads (struct thread_info *event_thread, inferior *inf) { + INFRUN_SCOPED_DEBUG_START_END ("event_thread=%s, inf=%d", + event_thread->ptid.to_string ().c_str (), + inf != nullptr ? inf->num : -1); + /* In case the instruction just stepped spawned a new thread. */ update_thread_list (); for (thread_info *tp : all_non_exited_threads ()) { + if (inf != nullptr && tp->inf != inf) + continue; + if (tp->inf->detaching) { infrun_debug_printf ("restart threads: [%s] inferior detaching", diff --git a/gdb/infrun.h b/gdb/infrun.h index be600fd8f7da..7e135ed420b7 100644 --- a/gdb/infrun.h +++ b/gdb/infrun.h @@ -142,8 +142,11 @@ extern void nullify_last_target_wait_ptid (); /* Stop all threads. Only returns after everything is halted. REASON is a string indicating the reason why we stop all threads, used in - debug messages. */ -extern void stop_all_threads (const char *reason); + debug messages. + + If INF is non-nullptr, stop all threads of that inferior. Otherwise, stop + all threads of all inferiors. */ +extern void stop_all_threads (const char *reason, inferior *inf = nullptr); extern void prepare_for_detach (void); -- 2.34.1