espressif/binutils-gdb
1
0
Fork 0
mirror of https://github.com/espressif/binutils-gdb.git synced 2025-08-06 14:49:38 +08:00
Code Issues Packages Projects Releases Wiki Activity

common/ptid.h: Mention that process_stratum targets should prefer ptid.lwp.

Browse Source 
It's best that we standardize on process_stratum targets using the
ptid.lwp field to store thread ids.  The idea being leave the ptid.tid
field free for any thread_stratum target that might want to sit on
top.  This patch adds a comment in that direction to struct ptid's
definition.

gdb/
2014-02-19  Pedro Alves  <palves@redhat.com>

	* common/ptid.h (struct ptid): Mention that process_stratum
	targets should prefer ptid.lwp.
This commit is contained in:
Pedro Alves
2014-02-19 18:25:41 +00:00
parent ba3481708d
commit c658158d93
2 changed files with 11 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
gdb/ChangeLog
View File

@ -1,3 +1,8 @@
2014-02-19 Pedro Alves <palves@redhat.com>
* common/ptid.h (struct ptid): Mention that process_stratum
targets should prefer ptid.lwp.
2014-02-19 Pedro Alves <palves@redhat.com> 2014-02-19 Pedro Alves <palves@redhat.com>
* remote.c (remote_thread_alive, write_ptid, read_ptid) * remote.c (remote_thread_alive, write_ptid, read_ptid)

7
gdb/common/ptid.h
View File

@ -25,7 +25,12 @@
consists of the process id (pid), lightweight process id (lwp) and consists of the process id (pid), lightweight process id (lwp) and
thread id (tid). When manipulating ptids, the constructors, thread id (tid). When manipulating ptids, the constructors,
accessors, and predicates declared in this file should be used. Do accessors, and predicates declared in this file should be used. Do
NOT access the struct ptid members directly. */ NOT access the struct ptid members directly.
process_stratum targets that handle threading themselves should
prefer using the ptid.lwp field, leaving the ptid.tid field for any
thread_stratum target that might want to sit on top.
*/
struct ptid struct ptid
{ {