diff options
| author | Tankut Baris Aktemur <tankut.baris.aktemur@intel.com> | 2021-12-13 12:22:48 +0100 |
|---|---|---|
| committer | Tankut Baris Aktemur <tankut.baris.aktemur@intel.com> | 2021-12-13 12:22:48 +0100 |
| commit | 24583e45efc823c1a9ea009dfddc36f9b74dcbcd (patch) | |
| tree | ac00cde441192c880eee0f36a7e4346e0ba53c58 /gdbserver/remote-utils.cc | |
| parent | f24791b72e764ada576901c0e866bf7768773a16 (diff) | |
| download | binutils-24583e45efc823c1a9ea009dfddc36f9b74dcbcd.zip binutils-24583e45efc823c1a9ea009dfddc36f9b74dcbcd.tar.gz binutils-24583e45efc823c1a9ea009dfddc36f9b74dcbcd.tar.bz2 | |
gdbserver: replace direct assignments to current_thread
Replace the direct assignments to current_thread with
switch_to_thread. Use scoped_restore_current_thread when appropriate.
There is one instance remaining in linux-low.cc's wait_for_sigstop.
This will be handled in a separate patch.
Regression-tested on X86-64 Linux using the native-gdbserver and
native-extended-gdbserver board files.
Diffstat (limited to 'gdbserver/remote-utils.cc')
| -rw-r--r-- | gdbserver/remote-utils.cc | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/gdbserver/remote-utils.cc b/gdbserver/remote-utils.cc index 8202365..ae1a85f 100644 --- a/gdbserver/remote-utils.cc +++ b/gdbserver/remote-utils.cc @@ -1099,7 +1099,6 @@ prepare_resume_reply (char *buf, ptid_t ptid, const target_waitstatus &status) case TARGET_WAITKIND_SYSCALL_ENTRY: case TARGET_WAITKIND_SYSCALL_RETURN: { - struct thread_info *saved_thread; const char **regp; struct regcache *regcache; @@ -1182,7 +1181,7 @@ prepare_resume_reply (char *buf, ptid_t ptid, const target_waitstatus &status) buf += strlen (buf); - saved_thread = current_thread; + scoped_restore_current_thread restore_thread; switch_to_thread (the_target, ptid); @@ -1273,8 +1272,6 @@ prepare_resume_reply (char *buf, ptid_t ptid, const target_waitstatus &status) buf += strlen (buf); current_process ()->dlls_changed = false; } - - current_thread = saved_thread; } break; case TARGET_WAITKIND_EXITED: |