From patchwork Tue Feb 11 09:01:23 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Tankut Baris Aktemur X-Patchwork-Id: 37906 Received: (qmail 118534 invoked by alias); 11 Feb 2020 09:03: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 117821 invoked by uid 89); 11 Feb 2020 09:03:28 -0000 Authentication-Results: sourceware.org; auth=none X-Spam-SWARE-Status: No, score=-23.4 required=5.0 tests=AWL, BAYES_00, GIT_PATCH_0, GIT_PATCH_1, GIT_PATCH_2, GIT_PATCH_3 autolearn=ham version=3.3.1 spammy= X-HELO: mga04.intel.com Received: from mga04.intel.com (HELO mga04.intel.com) (192.55.52.120) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Tue, 11 Feb 2020 09:03:21 +0000 Received: from orsmga003.jf.intel.com ([10.7.209.27]) by fmsmga104.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 11 Feb 2020 01:03:18 -0800 Received: from irvmail001.ir.intel.com ([163.33.26.43]) by orsmga003.jf.intel.com with ESMTP; 11 Feb 2020 01:03:17 -0800 Received: from ulvlx001.iul.intel.com (ulvlx001.iul.intel.com [172.28.207.17]) by irvmail001.ir.intel.com (8.14.3/8.13.6/MailSET/Hub) with ESMTP id 01B93GAE002159; Tue, 11 Feb 2020 09:03:17 GMT Received: from ulvlx001.iul.intel.com (localhost [127.0.0.1]) by ulvlx001.iul.intel.com with ESMTP id 01B93GK4010419; Tue, 11 Feb 2020 10:03:16 +0100 Received: (from taktemur@localhost) by ulvlx001.iul.intel.com with LOCAL id 01B93G2R010415; Tue, 11 Feb 2020 10:03:16 +0100 From: Tankut Baris Aktemur To: gdb-patches@sourceware.org Subject: [PATCH 11/58] gdbserver: turn target op 'wait' into a method Date: Tue, 11 Feb 2020 10:01:23 +0100 Message-Id: <78de6d2be971d05f6f637b260555440ade8cee95.1581410933.git.tankut.baris.aktemur@intel.com> In-Reply-To: References: In-Reply-To: References: X-IsSubscribed: yes gdbserver/ChangeLog: 2020-02-10 Tankut Baris Aktemur Make process_stratum_target's wait op a method of process_target. * target.h (struct process_stratum_target): Remove the target op. (class process_target): Add the target op. Update the derived structs and callers below. * target.c (target_wait): Update. * linux-low.c (linux_target_ops): Update. (linux_wait): Turn into ... (linux_process_target::wait): ... this. * linux-low.h (class linux_process_target): Update. * lynx-low.c (lynx_target_ops): Update. (lynx_wait): Turn into ... (lynx_process_target::wait): ... this. * lynx-low.h (class lynx_process_target): Update. * nto-low.c (nto_target_ops): Update. (nto_wait): Turn into ... (nto_process_target::wait): ... this. * nto-low.h (class nto_process_target): Update. * win32-low.c (win32_target_ops): Update. (win32_wait): Turn into ... (win32_process_target::wait): ... this. (do_initial_child_stuff): Update. * win32-low.h (class win32_process_target): Update. --- gdbserver/linux-low.c | 8 ++++---- gdbserver/linux-low.h | 3 +++ gdbserver/lynx-low.c | 6 +++--- gdbserver/lynx-low.h | 3 +++ gdbserver/nto-low.c | 7 +++---- gdbserver/nto-low.h | 3 +++ gdbserver/target.c | 2 +- gdbserver/target.h | 26 +++++++++++++------------- gdbserver/win32-low.c | 12 +++++------- gdbserver/win32-low.h | 3 +++ 10 files changed, 41 insertions(+), 32 deletions(-) diff --git a/gdbserver/linux-low.c b/gdbserver/linux-low.c index 5722752fd12..9f6f1064f45 100644 --- a/gdbserver/linux-low.c +++ b/gdbserver/linux-low.c @@ -3833,9 +3833,10 @@ async_file_mark (void) be awakened anyway. */ } -static ptid_t -linux_wait (ptid_t ptid, - struct target_waitstatus *ourstatus, int target_options) +ptid_t +linux_process_target::wait (ptid_t ptid, + target_waitstatus *ourstatus, + int target_options) { ptid_t event_ptid; @@ -7358,7 +7359,6 @@ linux_get_hwcap2 (int wordsize) static linux_process_target the_linux_target; static process_stratum_target linux_target_ops = { - linux_wait, linux_fetch_registers, linux_store_registers, linux_prepare_to_access_memory, diff --git a/gdbserver/linux-low.h b/gdbserver/linux-low.h index 48d92f73ced..c402d8e1c94 100644 --- a/gdbserver/linux-low.h +++ b/gdbserver/linux-low.h @@ -287,6 +287,9 @@ public: bool thread_alive (ptid_t pid) override; void resume (thread_resume *resume_info, size_t n) override; + + ptid_t wait (ptid_t ptid, target_waitstatus *status, + int options) override; }; #define get_thread_lwp(thr) ((struct lwp_info *) (thread_target_data (thr))) diff --git a/gdbserver/lynx-low.c b/gdbserver/lynx-low.c index b3eacd8b6c2..24830d1c289 100644 --- a/gdbserver/lynx-low.c +++ b/gdbserver/lynx-low.c @@ -505,8 +505,9 @@ retry: /* A wrapper around lynx_wait_1 that also prints debug traces when such debug traces have been activated. */ -static ptid_t -lynx_wait (ptid_t ptid, struct target_waitstatus *status, int options) +ptid_t +lynx_process_target::wait (ptid_t ptid, target_waitstatus *status, + int options) { ptid_t new_ptid; @@ -726,7 +727,6 @@ static lynx_process_target the_lynx_target; /* The LynxOS target_ops vector. */ static process_stratum_target lynx_target_ops = { - lynx_wait, lynx_fetch_registers, lynx_store_registers, NULL, /* prepare_to_access_memory */ diff --git a/gdbserver/lynx-low.h b/gdbserver/lynx-low.h index 6ad41545679..1b21ffb4c0a 100644 --- a/gdbserver/lynx-low.h +++ b/gdbserver/lynx-low.h @@ -73,6 +73,9 @@ public: bool thread_alive (ptid_t pid) override; void resume (thread_resume *resume_info, size_t n) override; + + ptid_t wait (ptid_t ptid, target_waitstatus *status, + int options) override; }; /* The inferior's target description. This is a global because the diff --git a/gdbserver/nto-low.c b/gdbserver/nto-low.c index aba63786203..462936f5eb7 100644 --- a/gdbserver/nto-low.c +++ b/gdbserver/nto-low.c @@ -518,9 +518,9 @@ nto_process_target::resume (thread_resume *resume_info, size_t n) Return ptid of thread that caused the event. */ -static ptid_t -nto_wait (ptid_t ptid, - struct target_waitstatus *ourstatus, int target_options) +ptid_t +nto_process_target::wait (ptid_t ptid, target_waitstatus *ourstatus, + int target_options) { sigset_t set; siginfo_t info; @@ -941,7 +941,6 @@ nto_sw_breakpoint_from_kind (int kind, int *size) static nto_process_target the_nto_target; static process_stratum_target nto_target_ops = { - nto_wait, nto_fetch_registers, nto_store_registers, NULL, /* prepare_to_access_memory */ diff --git a/gdbserver/nto-low.h b/gdbserver/nto-low.h index b0ca4247e69..faa7cab2941 100644 --- a/gdbserver/nto-low.h +++ b/gdbserver/nto-low.h @@ -63,6 +63,9 @@ public: bool thread_alive (ptid_t pid) override; void resume (thread_resume *resume_info, size_t n) override; + + ptid_t wait (ptid_t ptid, target_waitstatus *status, + int options) override; }; /* The inferior's target description. This is a global because the diff --git a/gdbserver/target.c b/gdbserver/target.c index eca53de982e..b73c4465f1c 100644 --- a/gdbserver/target.c +++ b/gdbserver/target.c @@ -223,7 +223,7 @@ target_stop_and_wait (ptid_t ptid) ptid_t target_wait (ptid_t ptid, struct target_waitstatus *status, int options) { - return (*the_target->wait) (ptid, status, options); + return the_target->pt->wait (ptid, status, options); } /* See target/target.h. */ diff --git a/gdbserver/target.h b/gdbserver/target.h index 0a07d7f8c85..43532161f34 100644 --- a/gdbserver/target.h +++ b/gdbserver/target.h @@ -70,19 +70,6 @@ class process_target; shared code. */ struct process_stratum_target { - /* Wait for the inferior process or thread to change state. Store - status through argument pointer STATUS. - - PTID = -1 to wait for any pid to do something, PTID(pid,0,0) to - wait for any thread of process pid to do something. Return ptid - of child, or -1 in case of error; store status through argument - pointer STATUS. OPTIONS is a bit set of options defined as - TARGET_W* above. If options contains TARGET_WNOHANG and there's - no child stop to report, return is - null_ptid/TARGET_WAITKIND_IGNORE. */ - - ptid_t (*wait) (ptid_t ptid, struct target_waitstatus *status, int options); - /* Fetch registers from the inferior process. If REGNO is -1, fetch all registers; otherwise, fetch at least REGNO. */ @@ -481,6 +468,19 @@ public: /* Resume the inferior process. */ virtual void resume (thread_resume *resume_info, size_t n) = 0; + + /* Wait for the inferior process or thread to change state. Store + status through argument pointer STATUS. + + PTID = -1 to wait for any pid to do something, PTID(pid,0,0) to + wait for any thread of process pid to do something. Return ptid + of child, or -1 in case of error; store status through argument + pointer STATUS. OPTIONS is a bit set of options defined as + TARGET_W* above. If options contains TARGET_WNOHANG and there's + no child stop to report, return is + null_ptid/TARGET_WAITKIND_IGNORE. */ + virtual ptid_t wait (ptid_t ptid, target_waitstatus *status, + int options) = 0; }; extern process_stratum_target *the_target; diff --git a/gdbserver/win32-low.c b/gdbserver/win32-low.c index fcbf3075e80..b2723b6d292 100644 --- a/gdbserver/win32-low.c +++ b/gdbserver/win32-low.c @@ -103,8 +103,6 @@ typedef BOOL (WINAPI *winapi_DebugSetProcessKillOnExit) (BOOL KillOnExit); typedef BOOL (WINAPI *winapi_DebugBreakProcess) (HANDLE); typedef BOOL (WINAPI *winapi_GenerateConsoleCtrlEvent) (DWORD, DWORD); -static ptid_t win32_wait (ptid_t ptid, struct target_waitstatus *ourstatus, - int options); static void win32_resume (struct thread_resume *resume_info, size_t n); #ifndef _WIN32_WCE static void win32_add_all_dlls (void); @@ -380,7 +378,7 @@ do_initial_child_stuff (HANDLE proch, DWORD pid, int attached) { struct target_waitstatus status; - win32_wait (minus_one_ptid, &status, 0); + the_target->pt->wait (minus_one_ptid, &status, 0); /* Note win32_wait doesn't return thread events. */ if (status.kind != TARGET_WAITKIND_LOADED) @@ -707,7 +705,7 @@ win32_process_target::create_inferior ( /* Wait till we are at 1st instruction in program, return new pid (assuming success). */ - cs.last_ptid = win32_wait (ptid_t (current_process_id), &cs.last_status, 0); + cs.last_ptid = wait (ptid_t (current_process_id), &cs.last_status, 0); return current_process_id; } @@ -1600,8 +1598,9 @@ get_child_debug_event (struct target_waitstatus *ourstatus) /* Wait for the inferior process to change state. STATUS will be filled in with a response code to send to GDB. Returns the signal which caused the process to stop. */ -static ptid_t -win32_wait (ptid_t ptid, struct target_waitstatus *ourstatus, int options) +ptid_t +win32_process_target::wait (ptid_t ptid, target_waitstatus *ourstatus, + int options) { struct regcache *regcache; @@ -1828,7 +1827,6 @@ win32_sw_breakpoint_from_kind (int kind, int *size) static win32_process_target the_win32_target; static process_stratum_target win32_target_ops = { - win32_wait, win32_fetch_inferior_registers, win32_store_inferior_registers, NULL, /* prepare_to_access_memory */ diff --git a/gdbserver/win32-low.h b/gdbserver/win32-low.h index 472be13d518..aa509f64ca8 100644 --- a/gdbserver/win32-low.h +++ b/gdbserver/win32-low.h @@ -122,6 +122,9 @@ public: bool thread_alive (ptid_t pid) override; void resume (thread_resume *resume_info, size_t n) override; + + ptid_t wait (ptid_t ptid, target_waitstatus *status, + int options) override; }; /* Retrieve the context for this thread, if not already retrieved. */