gdbserver: turn fast tracepoint target ops into methods
gdbserver/ChangeLog: 2020-02-20 Tankut Baris Aktemur <tankut.baris.aktemur@intel.com> Turn process_stratum_target's install_fast_tracepoint_jump_pad and get_min_fast_tracepoint_insn_len ops into methods of process_target. * target.h (struct process_stratum_target): Remove the target ops. (class process_target): Add the target ops. Also add 'supports_fast_tracepoints'. (target_supports_fast_tracepoints): Update the macro. (target_get_min_fast_tracepoint_insn_len): Update the macro. (install_fast_tracepoint_jump_pad): Update and rename the macro to ... (target_install_fast_tracepoint_jump_pad): ... this. * target.cc (process_target::supports_fast_tracepoints): Define. (process_target::install_fast_tracepoint_jump_pad): Define. (process_target::get_min_fast_tracepoint_insn_len): Define. Update the derived classes and callers below. * tracepoint.cc (install_fast_tracepoint): Update. * linux-low.cc (linux_target_ops): Update. (linux_process_target::supports_fast_tracepoints): Define. (linux_install_fast_tracepoint_jump_pad): Turn into ... (linux_process_target::install_fast_tracepoint_jump_pad): ... this. (linux_get_min_fast_tracepoint_insn_len): Turn into ... (linux_process_target::get_min_fast_tracepoint_insn_len): ... this. * linux-low.h (class linux_process_target): Update. * lynx-low.cc (lynx_target_ops): Update. * nto-low.cc (nto_target_ops): Update. * win32-low.cc (win32_target_ops): Update.
This commit is contained in:
parent
5c9eb2f2b5
commit
c23c939174
|
@ -1,3 +1,35 @@
|
||||||
|
2020-02-20 Tankut Baris Aktemur <tankut.baris.aktemur@intel.com>
|
||||||
|
|
||||||
|
Turn process_stratum_target's install_fast_tracepoint_jump_pad
|
||||||
|
and get_min_fast_tracepoint_insn_len ops into methods of
|
||||||
|
process_target.
|
||||||
|
|
||||||
|
* target.h (struct process_stratum_target): Remove the target ops.
|
||||||
|
(class process_target): Add the target ops. Also add
|
||||||
|
'supports_fast_tracepoints'.
|
||||||
|
(target_supports_fast_tracepoints): Update the macro.
|
||||||
|
(target_get_min_fast_tracepoint_insn_len): Update the macro.
|
||||||
|
(install_fast_tracepoint_jump_pad): Update and rename the macro
|
||||||
|
to ...
|
||||||
|
(target_install_fast_tracepoint_jump_pad): ... this.
|
||||||
|
* target.cc (process_target::supports_fast_tracepoints): Define.
|
||||||
|
(process_target::install_fast_tracepoint_jump_pad): Define.
|
||||||
|
(process_target::get_min_fast_tracepoint_insn_len): Define.
|
||||||
|
|
||||||
|
Update the derived classes and callers below.
|
||||||
|
|
||||||
|
* tracepoint.cc (install_fast_tracepoint): Update.
|
||||||
|
* linux-low.cc (linux_target_ops): Update.
|
||||||
|
(linux_process_target::supports_fast_tracepoints): Define.
|
||||||
|
(linux_install_fast_tracepoint_jump_pad): Turn into ...
|
||||||
|
(linux_process_target::install_fast_tracepoint_jump_pad): ... this.
|
||||||
|
(linux_get_min_fast_tracepoint_insn_len): Turn into ...
|
||||||
|
(linux_process_target::get_min_fast_tracepoint_insn_len): ... this.
|
||||||
|
* linux-low.h (class linux_process_target): Update.
|
||||||
|
* lynx-low.cc (lynx_target_ops): Update.
|
||||||
|
* nto-low.cc (nto_target_ops): Update.
|
||||||
|
* win32-low.cc (win32_target_ops): Update.
|
||||||
|
|
||||||
2020-02-20 Tankut Baris Aktemur <tankut.baris.aktemur@intel.com>
|
2020-02-20 Tankut Baris Aktemur <tankut.baris.aktemur@intel.com>
|
||||||
|
|
||||||
Turn process_stratum_target's stabilize_threads op into a
|
Turn process_stratum_target's stabilize_threads op into a
|
||||||
|
|
|
@ -6620,19 +6620,20 @@ linux_process_target::done_accessing_memory ()
|
||||||
target_unpause_all (true);
|
target_unpause_all (true);
|
||||||
}
|
}
|
||||||
|
|
||||||
static int
|
bool
|
||||||
linux_install_fast_tracepoint_jump_pad (CORE_ADDR tpoint, CORE_ADDR tpaddr,
|
linux_process_target::supports_fast_tracepoints ()
|
||||||
CORE_ADDR collector,
|
{
|
||||||
CORE_ADDR lockaddr,
|
return the_low_target.install_fast_tracepoint_jump_pad != nullptr;
|
||||||
ULONGEST orig_size,
|
}
|
||||||
CORE_ADDR *jump_entry,
|
|
||||||
CORE_ADDR *trampoline,
|
int
|
||||||
ULONGEST *trampoline_size,
|
linux_process_target::install_fast_tracepoint_jump_pad
|
||||||
unsigned char *jjump_pad_insn,
|
(CORE_ADDR tpoint, CORE_ADDR tpaddr, CORE_ADDR collector,
|
||||||
ULONGEST *jjump_pad_insn_size,
|
CORE_ADDR lockaddr, ULONGEST orig_size, CORE_ADDR *jump_entry,
|
||||||
CORE_ADDR *adjusted_insn_addr,
|
CORE_ADDR *trampoline, ULONGEST *trampoline_size,
|
||||||
CORE_ADDR *adjusted_insn_addr_end,
|
unsigned char *jjump_pad_insn, ULONGEST *jjump_pad_insn_size,
|
||||||
char *err)
|
CORE_ADDR *adjusted_insn_addr, CORE_ADDR *adjusted_insn_addr_end,
|
||||||
|
char *err)
|
||||||
{
|
{
|
||||||
return (*the_low_target.install_fast_tracepoint_jump_pad)
|
return (*the_low_target.install_fast_tracepoint_jump_pad)
|
||||||
(tpoint, tpaddr, collector, lockaddr, orig_size,
|
(tpoint, tpaddr, collector, lockaddr, orig_size,
|
||||||
|
@ -6651,8 +6652,8 @@ linux_emit_ops (void)
|
||||||
return NULL;
|
return NULL;
|
||||||
}
|
}
|
||||||
|
|
||||||
static int
|
int
|
||||||
linux_get_min_fast_tracepoint_insn_len (void)
|
linux_process_target::get_min_fast_tracepoint_insn_len ()
|
||||||
{
|
{
|
||||||
return (*the_low_target.get_min_fast_tracepoint_insn_len) ();
|
return (*the_low_target.get_min_fast_tracepoint_insn_len) ();
|
||||||
}
|
}
|
||||||
|
@ -7455,10 +7456,8 @@ linux_get_hwcap2 (int wordsize)
|
||||||
static linux_process_target the_linux_target;
|
static linux_process_target the_linux_target;
|
||||||
|
|
||||||
static process_stratum_target linux_target_ops = {
|
static process_stratum_target linux_target_ops = {
|
||||||
linux_install_fast_tracepoint_jump_pad,
|
|
||||||
linux_emit_ops,
|
linux_emit_ops,
|
||||||
linux_supports_disable_randomization,
|
linux_supports_disable_randomization,
|
||||||
linux_get_min_fast_tracepoint_insn_len,
|
|
||||||
linux_qxfer_libraries_svr4,
|
linux_qxfer_libraries_svr4,
|
||||||
linux_supports_agent,
|
linux_supports_agent,
|
||||||
#ifdef HAVE_LINUX_BTRACE
|
#ifdef HAVE_LINUX_BTRACE
|
||||||
|
|
|
@ -402,6 +402,24 @@ public:
|
||||||
void unpause_all (bool unfreeze) override;
|
void unpause_all (bool unfreeze) override;
|
||||||
|
|
||||||
void stabilize_threads () override;
|
void stabilize_threads () override;
|
||||||
|
|
||||||
|
bool supports_fast_tracepoints () override;
|
||||||
|
|
||||||
|
int install_fast_tracepoint_jump_pad (CORE_ADDR tpoint,
|
||||||
|
CORE_ADDR tpaddr,
|
||||||
|
CORE_ADDR collector,
|
||||||
|
CORE_ADDR lockaddr,
|
||||||
|
ULONGEST orig_size,
|
||||||
|
CORE_ADDR *jump_entry,
|
||||||
|
CORE_ADDR *trampoline,
|
||||||
|
ULONGEST *trampoline_size,
|
||||||
|
unsigned char *jjump_pad_insn,
|
||||||
|
ULONGEST *jjump_pad_insn_size,
|
||||||
|
CORE_ADDR *adjusted_insn_addr,
|
||||||
|
CORE_ADDR *adjusted_insn_addr_end,
|
||||||
|
char *err) override;
|
||||||
|
|
||||||
|
int get_min_fast_tracepoint_insn_len () override;
|
||||||
};
|
};
|
||||||
|
|
||||||
#define get_thread_lwp(thr) ((struct lwp_info *) (thread_target_data (thr)))
|
#define get_thread_lwp(thr) ((struct lwp_info *) (thread_target_data (thr)))
|
||||||
|
|
|
@ -735,10 +735,8 @@ static lynx_process_target the_lynx_target;
|
||||||
/* The LynxOS target_ops vector. */
|
/* The LynxOS target_ops vector. */
|
||||||
|
|
||||||
static process_stratum_target lynx_target_ops = {
|
static process_stratum_target lynx_target_ops = {
|
||||||
NULL, /* install_fast_tracepoint_jump_pad */
|
|
||||||
NULL, /* emit_ops */
|
NULL, /* emit_ops */
|
||||||
NULL, /* supports_disable_randomization */
|
NULL, /* supports_disable_randomization */
|
||||||
NULL, /* get_min_fast_tracepoint_insn_len */
|
|
||||||
NULL, /* qxfer_libraries_svr4 */
|
NULL, /* qxfer_libraries_svr4 */
|
||||||
NULL, /* support_agent */
|
NULL, /* support_agent */
|
||||||
NULL, /* enable_btrace */
|
NULL, /* enable_btrace */
|
||||||
|
|
|
@ -947,10 +947,8 @@ nto_sw_breakpoint_from_kind (int kind, int *size)
|
||||||
static nto_process_target the_nto_target;
|
static nto_process_target the_nto_target;
|
||||||
|
|
||||||
static process_stratum_target nto_target_ops = {
|
static process_stratum_target nto_target_ops = {
|
||||||
NULL, /* install_fast_tracepoint_jump_pad */
|
|
||||||
NULL, /* emit_ops */
|
NULL, /* emit_ops */
|
||||||
NULL, /* supports_disable_randomization */
|
NULL, /* supports_disable_randomization */
|
||||||
NULL, /* get_min_fast_tracepoint_insn_len */
|
|
||||||
NULL, /* qxfer_libraries_svr4 */
|
NULL, /* qxfer_libraries_svr4 */
|
||||||
NULL, /* support_agent */
|
NULL, /* support_agent */
|
||||||
NULL, /* enable_btrace */
|
NULL, /* enable_btrace */
|
||||||
|
|
|
@ -668,3 +668,28 @@ process_target::stabilize_threads ()
|
||||||
{
|
{
|
||||||
/* Nop. */
|
/* Nop. */
|
||||||
}
|
}
|
||||||
|
|
||||||
|
bool
|
||||||
|
process_target::supports_fast_tracepoints ()
|
||||||
|
{
|
||||||
|
return false;
|
||||||
|
}
|
||||||
|
|
||||||
|
int
|
||||||
|
process_target::install_fast_tracepoint_jump_pad
|
||||||
|
(CORE_ADDR tpoint, CORE_ADDR tpaddr, CORE_ADDR collector,
|
||||||
|
CORE_ADDR lockaddr, ULONGEST orig_size, CORE_ADDR *jump_entry,
|
||||||
|
CORE_ADDR *trampoline, ULONGEST *trampoline_size,
|
||||||
|
unsigned char *jjump_pad_insn, ULONGEST *jjump_pad_insn_size,
|
||||||
|
CORE_ADDR *adjusted_insn_addr, CORE_ADDR *adjusted_insn_addr_end,
|
||||||
|
char *err)
|
||||||
|
{
|
||||||
|
gdb_assert_not_reached ("target op install_fast_tracepoint_jump_pad "
|
||||||
|
"not supported");
|
||||||
|
}
|
||||||
|
|
||||||
|
int
|
||||||
|
process_target::get_min_fast_tracepoint_insn_len ()
|
||||||
|
{
|
||||||
|
return 0;
|
||||||
|
}
|
||||||
|
|
|
@ -70,34 +70,6 @@ class process_target;
|
||||||
shared code. */
|
shared code. */
|
||||||
struct process_stratum_target
|
struct process_stratum_target
|
||||||
{
|
{
|
||||||
/* Install a fast tracepoint jump pad. TPOINT is the address of the
|
|
||||||
tracepoint internal object as used by the IPA agent. TPADDR is
|
|
||||||
the address of tracepoint. COLLECTOR is address of the function
|
|
||||||
the jump pad redirects to. LOCKADDR is the address of the jump
|
|
||||||
pad lock object. ORIG_SIZE is the size in bytes of the
|
|
||||||
instruction at TPADDR. JUMP_ENTRY points to the address of the
|
|
||||||
jump pad entry, and on return holds the address past the end of
|
|
||||||
the created jump pad. If a trampoline is created by the function,
|
|
||||||
then TRAMPOLINE and TRAMPOLINE_SIZE return the address and size of
|
|
||||||
the trampoline, else they remain unchanged. JJUMP_PAD_INSN is a
|
|
||||||
buffer containing a copy of the instruction at TPADDR.
|
|
||||||
ADJUST_INSN_ADDR and ADJUST_INSN_ADDR_END are output parameters that
|
|
||||||
return the address range where the instruction at TPADDR was relocated
|
|
||||||
to. If an error occurs, the ERR may be used to pass on an error
|
|
||||||
message. */
|
|
||||||
int (*install_fast_tracepoint_jump_pad) (CORE_ADDR tpoint, CORE_ADDR tpaddr,
|
|
||||||
CORE_ADDR collector,
|
|
||||||
CORE_ADDR lockaddr,
|
|
||||||
ULONGEST orig_size,
|
|
||||||
CORE_ADDR *jump_entry,
|
|
||||||
CORE_ADDR *trampoline,
|
|
||||||
ULONGEST *trampoline_size,
|
|
||||||
unsigned char *jjump_pad_insn,
|
|
||||||
ULONGEST *jjump_pad_insn_size,
|
|
||||||
CORE_ADDR *adjusted_insn_addr,
|
|
||||||
CORE_ADDR *adjusted_insn_addr_end,
|
|
||||||
char *err);
|
|
||||||
|
|
||||||
/* Return the bytecode operations vector for the current inferior.
|
/* Return the bytecode operations vector for the current inferior.
|
||||||
Returns NULL if bytecode compilation is not supported. */
|
Returns NULL if bytecode compilation is not supported. */
|
||||||
struct emit_ops *(*emit_ops) (void);
|
struct emit_ops *(*emit_ops) (void);
|
||||||
|
@ -105,10 +77,6 @@ struct process_stratum_target
|
||||||
/* Returns true if the target supports disabling randomization. */
|
/* Returns true if the target supports disabling randomization. */
|
||||||
int (*supports_disable_randomization) (void);
|
int (*supports_disable_randomization) (void);
|
||||||
|
|
||||||
/* Return the minimum length of an instruction that can be safely overwritten
|
|
||||||
for use as a fast tracepoint. */
|
|
||||||
int (*get_min_fast_tracepoint_insn_len) (void);
|
|
||||||
|
|
||||||
/* Read solib info on SVR4 platforms. */
|
/* Read solib info on SVR4 platforms. */
|
||||||
int (*qxfer_libraries_svr4) (const char *annex, unsigned char *readbuf,
|
int (*qxfer_libraries_svr4) (const char *annex, unsigned char *readbuf,
|
||||||
unsigned const char *writebuf,
|
unsigned const char *writebuf,
|
||||||
|
@ -494,6 +462,37 @@ public:
|
||||||
|
|
||||||
/* Stabilize all threads. That is, force them out of jump pads. */
|
/* Stabilize all threads. That is, force them out of jump pads. */
|
||||||
virtual void stabilize_threads ();
|
virtual void stabilize_threads ();
|
||||||
|
|
||||||
|
/* Return true if the install_fast_tracepoint_jump_pad op is
|
||||||
|
supported. */
|
||||||
|
virtual bool supports_fast_tracepoints ();
|
||||||
|
|
||||||
|
/* Install a fast tracepoint jump pad. TPOINT is the address of the
|
||||||
|
tracepoint internal object as used by the IPA agent. TPADDR is
|
||||||
|
the address of tracepoint. COLLECTOR is address of the function
|
||||||
|
the jump pad redirects to. LOCKADDR is the address of the jump
|
||||||
|
pad lock object. ORIG_SIZE is the size in bytes of the
|
||||||
|
instruction at TPADDR. JUMP_ENTRY points to the address of the
|
||||||
|
jump pad entry, and on return holds the address past the end of
|
||||||
|
the created jump pad. If a trampoline is created by the function,
|
||||||
|
then TRAMPOLINE and TRAMPOLINE_SIZE return the address and size of
|
||||||
|
the trampoline, else they remain unchanged. JJUMP_PAD_INSN is a
|
||||||
|
buffer containing a copy of the instruction at TPADDR.
|
||||||
|
ADJUST_INSN_ADDR and ADJUST_INSN_ADDR_END are output parameters that
|
||||||
|
return the address range where the instruction at TPADDR was relocated
|
||||||
|
to. If an error occurs, the ERR may be used to pass on an error
|
||||||
|
message. */
|
||||||
|
virtual int install_fast_tracepoint_jump_pad
|
||||||
|
(CORE_ADDR tpoint, CORE_ADDR tpaddr, CORE_ADDR collector,
|
||||||
|
CORE_ADDR lockaddr, ULONGEST orig_size, CORE_ADDR *jump_entry,
|
||||||
|
CORE_ADDR *trampoline, ULONGEST *trampoline_size,
|
||||||
|
unsigned char *jjump_pad_insn, ULONGEST *jjump_pad_insn_size,
|
||||||
|
CORE_ADDR *adjusted_insn_addr, CORE_ADDR *adjusted_insn_addr_end,
|
||||||
|
char *err);
|
||||||
|
|
||||||
|
/* Return the minimum length of an instruction that can be safely
|
||||||
|
overwritten for use as a fast tracepoint. */
|
||||||
|
virtual int get_min_fast_tracepoint_insn_len ();
|
||||||
};
|
};
|
||||||
|
|
||||||
extern process_stratum_target *the_target;
|
extern process_stratum_target *the_target;
|
||||||
|
@ -559,11 +558,10 @@ int kill_inferior (process_info *proc);
|
||||||
the_target->pt->supports_tracepoints ()
|
the_target->pt->supports_tracepoints ()
|
||||||
|
|
||||||
#define target_supports_fast_tracepoints() \
|
#define target_supports_fast_tracepoints() \
|
||||||
(the_target->install_fast_tracepoint_jump_pad != NULL)
|
the_target->pt->supports_fast_tracepoints ()
|
||||||
|
|
||||||
#define target_get_min_fast_tracepoint_insn_len() \
|
#define target_get_min_fast_tracepoint_insn_len() \
|
||||||
(the_target->get_min_fast_tracepoint_insn_len \
|
the_target->pt->get_min_fast_tracepoint_insn_len ()
|
||||||
? (*the_target->get_min_fast_tracepoint_insn_len) () : 0)
|
|
||||||
|
|
||||||
#define target_thread_stopped(thread) \
|
#define target_thread_stopped(thread) \
|
||||||
the_target->pt->thread_stopped (thread)
|
the_target->pt->thread_stopped (thread)
|
||||||
|
@ -577,26 +575,26 @@ int kill_inferior (process_info *proc);
|
||||||
#define target_stabilize_threads() \
|
#define target_stabilize_threads() \
|
||||||
the_target->pt->stabilize_threads ()
|
the_target->pt->stabilize_threads ()
|
||||||
|
|
||||||
#define install_fast_tracepoint_jump_pad(tpoint, tpaddr, \
|
#define target_install_fast_tracepoint_jump_pad(tpoint, tpaddr, \
|
||||||
collector, lockaddr, \
|
collector, lockaddr, \
|
||||||
orig_size, \
|
orig_size, \
|
||||||
jump_entry, \
|
jump_entry, \
|
||||||
trampoline, trampoline_size, \
|
trampoline, trampoline_size, \
|
||||||
jjump_pad_insn, \
|
jjump_pad_insn, \
|
||||||
jjump_pad_insn_size, \
|
jjump_pad_insn_size, \
|
||||||
adjusted_insn_addr, \
|
adjusted_insn_addr, \
|
||||||
adjusted_insn_addr_end, \
|
adjusted_insn_addr_end, \
|
||||||
err) \
|
err) \
|
||||||
(*the_target->install_fast_tracepoint_jump_pad) (tpoint, tpaddr, \
|
the_target->pt->install_fast_tracepoint_jump_pad (tpoint, tpaddr, \
|
||||||
collector,lockaddr, \
|
collector,lockaddr, \
|
||||||
orig_size, jump_entry, \
|
orig_size, jump_entry, \
|
||||||
trampoline, \
|
trampoline, \
|
||||||
trampoline_size, \
|
trampoline_size, \
|
||||||
jjump_pad_insn, \
|
jjump_pad_insn, \
|
||||||
jjump_pad_insn_size, \
|
jjump_pad_insn_size, \
|
||||||
adjusted_insn_addr, \
|
adjusted_insn_addr, \
|
||||||
adjusted_insn_addr_end, \
|
adjusted_insn_addr_end, \
|
||||||
err)
|
err)
|
||||||
|
|
||||||
#define target_emit_ops() \
|
#define target_emit_ops() \
|
||||||
(the_target->emit_ops ? (*the_target->emit_ops) () : NULL)
|
(the_target->emit_ops ? (*the_target->emit_ops) () : NULL)
|
||||||
|
|
|
@ -3117,17 +3117,11 @@ install_fast_tracepoint (struct tracepoint *tpoint, char *errbuf)
|
||||||
trampoline_size = 0;
|
trampoline_size = 0;
|
||||||
|
|
||||||
/* Install the jump pad. */
|
/* Install the jump pad. */
|
||||||
err = install_fast_tracepoint_jump_pad (tpoint->obj_addr_on_target,
|
err = target_install_fast_tracepoint_jump_pad
|
||||||
tpoint->address,
|
(tpoint->obj_addr_on_target, tpoint->address, collect,
|
||||||
collect,
|
ipa_sym_addrs.addr_collecting, tpoint->orig_size, &jentry,
|
||||||
ipa_sym_addrs.addr_collecting,
|
&trampoline, &trampoline_size, fjump, &fjump_size,
|
||||||
tpoint->orig_size,
|
&tpoint->adjusted_insn_addr, &tpoint->adjusted_insn_addr_end, errbuf);
|
||||||
&jentry,
|
|
||||||
&trampoline, &trampoline_size,
|
|
||||||
fjump, &fjump_size,
|
|
||||||
&tpoint->adjusted_insn_addr,
|
|
||||||
&tpoint->adjusted_insn_addr_end,
|
|
||||||
errbuf);
|
|
||||||
|
|
||||||
if (err)
|
if (err)
|
||||||
return 1;
|
return 1;
|
||||||
|
|
|
@ -1858,10 +1858,8 @@ win32_sw_breakpoint_from_kind (int kind, int *size)
|
||||||
static win32_process_target the_win32_target;
|
static win32_process_target the_win32_target;
|
||||||
|
|
||||||
static process_stratum_target win32_target_ops = {
|
static process_stratum_target win32_target_ops = {
|
||||||
NULL, /* install_fast_tracepoint_jump_pad */
|
|
||||||
NULL, /* emit_ops */
|
NULL, /* emit_ops */
|
||||||
NULL, /* supports_disable_randomization */
|
NULL, /* supports_disable_randomization */
|
||||||
NULL, /* get_min_fast_tracepoint_insn_len */
|
|
||||||
NULL, /* qxfer_libraries_svr4 */
|
NULL, /* qxfer_libraries_svr4 */
|
||||||
NULL, /* support_agent */
|
NULL, /* support_agent */
|
||||||
NULL, /* enable_btrace */
|
NULL, /* enable_btrace */
|
||||||
|
|
Loading…
Reference in New Issue