This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [PATCH] remove registers_changed from switch_to_thread
- From: Pedro Alves <palves at redhat dot com>
- To: Yao Qi <yao at codesourcery dot com>
- Cc: gdb-patches at sourceware dot org
- Date: Thu, 26 Jul 2012 15:57:12 +0100
- Subject: Re: [PATCH] remove registers_changed from switch_to_thread
- References: <1343313892-30424-1-git-send-email-yao@codesourcery.com>
On 07/26/2012 03:44 PM, Yao Qi wrote:
> When reading the code, I find registers_changed is called from
> switch_to_thread, which confuses me a little bit. We have
> regcache, and each thread has its own regcache. Why do we have
> to invalidate all regcaches during thread switch?
>
> The line of calling registers_changed in switch_to_thread is quite
> old, it was written in gdb 4.10 (1993). At that time, we didn't
> have regcache, and register state is a global state. There is also
> a global variable 'registers_pid' associated with the global
> register state. When thread switches, 'registers_pid' should be
> updated. That is the reason why registers_changed should
> be called in switch_to_thread. So we don't have to do it now.
> This patch is to remove this line. Regression tested on x86_64
> native and gdbserver. OK to apply?
Nice! OK.
--
Pedro Alves