[RFC PATCH v5 3/9] Add scoped_restore_regcache_ptid
Simon Marchi
simark@simark.ca
Sat Mar 17 18:08:00 GMT 2018
On 2018-03-12 11:31 AM, Philipp Rudo wrote:
> When a target and its target beneath use different ptids to identify a
> thread the regcaches ptid has to be set/restored when calls are passed down
> to the target beneath to e.g. fetch_registers. Add a scoped_restore to
> simplify this.
Though I don't understand yet why this is needed (I'll find out when reading
the following patches), the implementation looks ok.
I don't think this is an issue (at least for now), but there is this observer
"thread_ptid_changed" that indicates when a thread changes ptid. If this
observer is triggered while we have temporarily changed a regcache's ptid,
I guess we could miss updating a regcache's ptid, as
scoped_restore_regcache_ptid will reset it to the old ptid. It's probably
not a problem but I think it's good to be aware of this.
Simon
More information about the Gdb-patches
mailing list