diff options
| author | Tankut Baris Aktemur <tankut.baris.aktemur@intel.com> | 2025-01-29 10:50:31 +0100 |
|---|---|---|
| committer | Tankut Baris Aktemur <tankut.baris.aktemur@intel.com> | 2025-01-29 11:17:35 +0100 |
| commit | 84da4a1ea0ae58ef13ce81586f031ca53eb40112 (patch) | |
| tree | 14c2022d4de744d8e5af2e1d5d04b5a66b2d7785 | |
| parent | fe1b4d6dd9e68c80e04776d78b54d3316fdf8883 (diff) | |
| download | binutils-84da4a1ea0ae58ef13ce81586f031ca53eb40112.zip binutils-84da4a1ea0ae58ef13ce81586f031ca53eb40112.tar.gz binutils-84da4a1ea0ae58ef13ce81586f031ca53eb40112.tar.bz2 | |
gdbserver: refactor the definition and uses of supply_regblock
The supply_regblock function takes a pointer to a buffer as an
argument and implements two different behavior based on the pointer
being null. There are two cases where we pass nullptr, all in
tracepoint.cc, where we are essentially doing a reset on the regcache.
In fast_tracepoint_ctx::regcache, register_status array does not
even exist. Hence, that use simply boils down to zeroing of register
data. Do this at the time of creating the buffer and remove the call
to supply_regblock.
In fetch_traceframe_registers, inline the use with a call to `reset`.
Hence, there are no more cases left, where a nullptr would be passed
to supply_regblock. Assert that the buffer argument is non-null and
simplify the implementation.
Approved-By: Simon Marchi <simon.marchi@efficios.com>
| -rw-r--r-- | gdbserver/regcache.cc | 14 | ||||
| -rw-r--r-- | gdbserver/tracepoint.cc | 5 |
2 files changed, 8 insertions, 11 deletions
diff --git a/gdbserver/regcache.cc b/gdbserver/regcache.cc index bd053c0..06ac372 100644 --- a/gdbserver/regcache.cc +++ b/gdbserver/regcache.cc @@ -360,18 +360,14 @@ supply_register_by_name_zeroed (struct regcache *regcache, void supply_regblock (struct regcache *regcache, const void *buf) { - if (buf) - { - const struct target_desc *tdesc = regcache->tdesc; + gdb_assert (buf != nullptr); + const struct target_desc *tdesc = regcache->tdesc; - memcpy (regcache->registers, buf, tdesc->registers_size); + memcpy (regcache->registers, buf, tdesc->registers_size); #ifndef IN_PROCESS_AGENT - for (int i = 0; i < tdesc->reg_defs.size (); i++) - regcache->set_register_status (i, REG_VALID); + for (int i = 0; i < tdesc->reg_defs.size (); i++) + regcache->set_register_status (i, REG_VALID); #endif - } - else - regcache->reset (REG_UNAVAILABLE); } #ifndef IN_PROCESS_AGENT diff --git a/gdbserver/tracepoint.cc b/gdbserver/tracepoint.cc index 0ed8e8f..5e3792e 100644 --- a/gdbserver/tracepoint.cc +++ b/gdbserver/tracepoint.cc @@ -1198,7 +1198,6 @@ struct fast_tracepoint_ctx : public tracepoint_hit_ctx if (!this->m_regcache.has_value ()) { this->m_regcache.emplace (ipa_tdesc, this->regspace); - supply_regblock (&this->m_regcache.value (), nullptr); supply_fast_tracepoint_registers (&this->m_regcache.value (), this->regs); } @@ -4815,7 +4814,7 @@ fetch_traceframe_registers (int tfnum, struct regcache *regcache, int regnum) if (dataptr == NULL) { /* Mark registers unavailable. */ - supply_regblock (regcache, NULL); + regcache->reset (REG_UNAVAILABLE); /* We can generally guess at a PC, although this will be misleading for while-stepping frames and multi-location @@ -5370,6 +5369,8 @@ gdb_collect (struct tracepoint *tpoint, unsigned char *regs) return; } + memset (ctx.regspace, 0, ipa_tdesc->registers_size); + for (ctx.tpoint = tpoint; ctx.tpoint != NULL && ctx.tpoint->address == tpoint->address; ctx.tpoint = ctx.tpoint->next) |