From: Sergio Durigan Junior <sergiodj@redhat.com>
To: "Jose E. Marchesi" <jose.marchesi@oracle.com>
Cc: gdb-patches@sourceware.org
Subject: Re: [PATCH V4 6/9] Support for DTrace USDT probes in x86_64 targets.
Date: Tue, 17 Feb 2015 01:37:00 -0000 [thread overview]
Message-ID: <87mw4d71yq.fsf@redhat.com> (raw)
In-Reply-To: <1422874968-382-7-git-send-email-jose.marchesi@oracle.com> (Jose E. Marchesi's message of "Mon, 2 Feb 2015 12:02:45 +0100")
On Monday, February 02 2015, Jose E. Marchesi wrote:
> This patch adds the target-specific code in order to support the
> calculation of DTrace probes arguments in x86_64 targets, and also the
> enabling and disabling of probes. This is done by implementing the
> `dtrace_*' gdbarch handlers.
Thanks for the patch. This was approved already; no further comments. OK.
> gdb/ChangeLog:
>
> 2015-02-02 Jose E. Marchesi <jose.marchesi@oracle.com>
>
> * amd64-linux-tdep.c: Include "parser-defs.h" and "user-regs.h".
> (amd64_dtrace_parse_probe_argument): New function.
> (amd64_dtrace_probe_is_enabled): Likewise.
> (amd64_dtrace_enable_probe): Likewise.
> (amd64_dtrace_disable_probe): Likewise.
> (amd64_linux_init_abi): Register the
> `gdbarch_dtrace_probe_argument', `gdbarch_dtrace_enable_probe',
> `gdbarch_dtrace_disable_probe' and
> `gdbarch_dtrace_probe_is_enabled' hooks.
> (amd64_dtrace_disabled_probe_sequence_1): New constant.
> (amd64_dtrace_disabled_probe_sequence_2): Likewise.
> (amd64_dtrace_enable_probe_sequence): Likewise.
> (amd64_dtrace_disable_probe_sequence): Likewise.
> ---
> gdb/ChangeLog | 16 ++++++
> gdb/amd64-linux-tdep.c | 148 ++++++++++++++++++++++++++++++++++++++++++++++++
> 2 files changed, 164 insertions(+)
>
> diff --git a/gdb/amd64-linux-tdep.c b/gdb/amd64-linux-tdep.c
> index 42d884e..ffa85f4 100644
> --- a/gdb/amd64-linux-tdep.c
> +++ b/gdb/amd64-linux-tdep.c
> @@ -28,6 +28,8 @@
> #include "gdbtypes.h"
> #include "reggroups.h"
> #include "regset.h"
> +#include "parser-defs.h"
> +#include "user-regs.h"
> #include "amd64-linux-tdep.h"
> #include "i386-linux-tdep.h"
> #include "linux-tdep.h"
> @@ -1643,6 +1645,146 @@ amd64_linux_iterate_over_regset_sections (struct gdbarch *gdbarch,
> &amd64_linux_xstateregset, "XSAVE extended state", cb_data);
> }
>
> +/* The instruction sequences used in x86_64 machines for a
> + disabled is-enabled probe. */
> +
> +const gdb_byte amd64_dtrace_disabled_probe_sequence_1[] = {
> + /* xor %rax, %rax */ 0x48, 0x33, 0xc0,
> + /* nop */ 0x90,
> + /* nop */ 0x90
> +};
> +
> +const gdb_byte amd64_dtrace_disabled_probe_sequence_2[] = {
> + /* xor %rax, %rax */ 0x48, 0x33, 0xc0,
> + /* ret */ 0xc3,
> + /* nop */ 0x90
> +};
> +
> +/* The instruction sequence used in x86_64 machines for enabling a
> + DTrace is-enabled probe. */
> +
> +const gdb_byte amd64_dtrace_enable_probe_sequence[] = {
> + /* mov $0x1, %eax */ 0xb8, 0x01, 0x00, 0x00, 0x00
> +};
> +
> +/* The instruction sequence used in x86_64 machines for disabling a
> + DTrace is-enabled probe. */
> +
> +const gdb_byte amd64_dtrace_disable_probe_sequence[] = {
> + /* xor %rax, %rax; nop; nop */ 0x48, 0x33, 0xC0, 0x90, 0x90
> +};
> +
> +/* Implementation of `gdbarch_dtrace_probe_is_enabled', as defined in
> + gdbarch.h. */
> +
> +static int
> +amd64_dtrace_probe_is_enabled (struct gdbarch *gdbarch, CORE_ADDR addr)
> +{
> + gdb_byte buf[5];
> +
> + /* This function returns 1 if the instructions at ADDR do _not_
> + follow any of the amd64_dtrace_disabled_probe_sequence_*
> + patterns.
> +
> + Note that ADDR is offset 3 bytes from the beginning of these
> + sequences. */
> +
> + read_code (addr - 3, buf, 5);
> + return (memcmp (buf, amd64_dtrace_disabled_probe_sequence_1, 5) != 0
> + && memcmp (buf, amd64_dtrace_disabled_probe_sequence_2, 5) != 0);
> +}
> +
> +/* Implementation of `gdbarch_dtrace_enable_probe', as defined in
> + gdbarch.h. */
> +
> +static void
> +amd64_dtrace_enable_probe (struct gdbarch *gdbarch, CORE_ADDR addr)
> +{
> + /* Note also that ADDR is offset 3 bytes from the beginning of
> + amd64_dtrace_enable_probe_sequence. */
> +
> + write_memory (addr - 3, amd64_dtrace_enable_probe_sequence, 5);
> +}
> +
> +/* Implementation of `gdbarch_dtrace_disable_probe', as defined in
> + gdbarch.h. */
> +
> +static void
> +amd64_dtrace_disable_probe (struct gdbarch *gdbarch, CORE_ADDR addr)
> +{
> + /* Note also that ADDR is offset 3 bytes from the beginning of
> + amd64_dtrace_disable_probe_sequence. */
> +
> + write_memory (addr - 3, amd64_dtrace_disable_probe_sequence, 5);
> +}
> +
> +/* Implementation of `gdbarch_dtrace_parse_probe_argument', as defined
> + in gdbarch.h. */
> +
> +static void
> +amd64_dtrace_parse_probe_argument (struct gdbarch *gdbarch,
> + struct parser_state *pstate,
> + int narg)
> +{
> + enum bfd_endian byte_order = gdbarch_byte_order (gdbarch);
> + struct frame_info *this_frame = get_selected_frame (NULL);
> + struct stoken str;
> +
> + /* DTrace probe arguments can be found on the ABI-defined places for
> + regular arguments at the current PC. The probe abstraction
> + currently supports up to 12 arguments for probes. */
> +
> + if (narg < 6)
> + {
> + static const int arg_reg_map[6] =
> + {
> + AMD64_RDI_REGNUM, /* Arg 1. */
> + AMD64_RSI_REGNUM, /* Arg 2. */
> + AMD64_RDX_REGNUM, /* Arg 3. */
> + AMD64_RCX_REGNUM, /* Arg 4. */
> + AMD64_R8_REGNUM, /* Arg 5. */
> + AMD64_R9_REGNUM /* Arg 6. */
> + };
> + int regno = arg_reg_map[narg];
> + const char *regname = user_reg_map_regnum_to_name (gdbarch, regno);
> +
> + write_exp_elt_opcode (pstate, OP_REGISTER);
> + str.ptr = regname;
> + str.length = strlen (regname);
> + write_exp_string (pstate, str);
> + write_exp_elt_opcode (pstate, OP_REGISTER);
> + }
> + else
> + {
> + /* Additional arguments are passed on the stack. */
> + CORE_ADDR sp;
> + const char *regname = user_reg_map_regnum_to_name (gdbarch, AMD64_RSP_REGNUM);
> +
> + /* Displacement. */
> + write_exp_elt_opcode (pstate, OP_LONG);
> + write_exp_elt_type (pstate, builtin_type (gdbarch)->builtin_long);
> + write_exp_elt_longcst (pstate, narg - 6);
> + write_exp_elt_opcode (pstate, OP_LONG);
> +
> + /* Register: SP. */
> + write_exp_elt_opcode (pstate, OP_REGISTER);
> + str.ptr = regname;
> + str.length = strlen (regname);
> + write_exp_string (pstate, str);
> + write_exp_elt_opcode (pstate, OP_REGISTER);
> +
> + write_exp_elt_opcode (pstate, BINOP_ADD);
> +
> + /* Cast to long. */
> + write_exp_elt_opcode (pstate, UNOP_CAST);
> + write_exp_elt_type (pstate,
> + lookup_pointer_type (builtin_type (gdbarch)->builtin_long));
> + write_exp_elt_opcode (pstate, UNOP_CAST);
> +
> + write_exp_elt_opcode (pstate, UNOP_IND);
> + }
> +}
> +
> static void
> amd64_linux_init_abi_common(struct gdbarch_info info, struct gdbarch *gdbarch)
> {
> @@ -1907,6 +2049,12 @@ amd64_linux_init_abi (struct gdbarch_info info, struct gdbarch *gdbarch)
> /* GNU/Linux uses SVR4-style shared libraries. */
> set_solib_svr4_fetch_link_map_offsets
> (gdbarch, svr4_lp64_fetch_link_map_offsets);
> +
> + /* Register DTrace handlers. */
> + set_gdbarch_dtrace_parse_probe_argument (gdbarch, amd64_dtrace_parse_probe_argument);
> + set_gdbarch_dtrace_probe_is_enabled (gdbarch, amd64_dtrace_probe_is_enabled);
> + set_gdbarch_dtrace_enable_probe (gdbarch, amd64_dtrace_enable_probe);
> + set_gdbarch_dtrace_disable_probe (gdbarch, amd64_dtrace_disable_probe);
> }
>
> static void
> --
> 1.7.10.4
--
Sergio
GPG key ID: 0x65FC5E36
Please send encrypted e-mail if possible
http://sergiodj.net/
next prev parent reply other threads:[~2015-02-17 1:37 UTC|newest]
Thread overview: 51+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-02-02 10:57 [PATCH V4 0/9] Add support for DTrace USDT probes to gdb Jose E. Marchesi
2015-02-02 10:57 ` [PATCH V4 1/9] Adapt `info probes' to support printing probes of different types Jose E. Marchesi
2015-02-17 1:12 ` Sergio Durigan Junior
2015-02-02 10:57 ` [PATCH V4 6/9] Support for DTrace USDT probes in x86_64 targets Jose E. Marchesi
2015-02-17 1:37 ` Sergio Durigan Junior [this message]
2015-02-02 10:57 ` [PATCH V4 3/9] New commands `enable probe' and `disable probe' Jose E. Marchesi
2015-02-02 16:01 ` Eli Zaretskii
2015-02-17 1:54 ` Sergio Durigan Junior
2015-02-02 10:57 ` [PATCH V4 9/9] Announce the DTrace USDT probes support in NEWS Jose E. Marchesi
2015-02-02 16:03 ` Eli Zaretskii
2015-02-02 10:57 ` [PATCH V4 7/9] Simple testsuite for DTrace USDT probes Jose E. Marchesi
2015-02-02 11:18 ` Jose E. Marchesi
2015-02-17 1:53 ` Sergio Durigan Junior
2015-02-17 1:58 ` Sergio Durigan Junior
2015-02-17 11:32 ` Pedro Alves
2015-02-02 10:57 ` [PATCH V4 4/9] New gdbarch functions: dtrace_parse_probe_argument, dtrace_probe_is_enabled, dtrace_enable_probe, dtrace_disable_probe Jose E. Marchesi
2015-02-17 1:14 ` Sergio Durigan Junior
2015-02-02 10:57 ` [PATCH V4 8/9] Documentation for DTrace USDT probes Jose E. Marchesi
2015-02-02 16:03 ` Eli Zaretskii
2015-02-02 19:47 ` Jose E. Marchesi
2015-02-02 10:57 ` [PATCH V4 2/9] Move `compute_probe_arg' and `compile_probe_arg' to probe.c Jose E. Marchesi
2015-02-17 1:13 ` Sergio Durigan Junior
2015-02-02 10:57 ` [PATCH V4 5/9] New probe type: DTrace USDT probes Jose E. Marchesi
2015-02-17 1:35 ` Sergio Durigan Junior
2015-03-25 19:14 ` Joel Brobecker
2015-03-26 16:15 ` Jose E. Marchesi
2015-03-26 17:50 ` Joel Brobecker
2015-03-26 18:43 ` Joel Brobecker
2015-03-26 18:53 ` Sergio Durigan Junior
2015-03-26 21:00 ` Joel Brobecker
2015-03-27 9:47 ` gdb fails to compile with GCC 4.4.7 (was: [PATCH V4 5/9] New probe type: DTrace USDT probes.) Tobias Burnus
2015-03-27 13:42 ` Joel Brobecker
2015-03-27 15:18 ` Tobias Burnus
2015-03-27 15:27 ` [pushed] " Joel Brobecker
2015-03-27 16:58 ` H.J. Lu
2015-03-26 23:39 ` [PATCH V4 5/9] New probe type: DTrace USDT probes Jose E. Marchesi
2015-03-31 17:29 ` Jose E. Marchesi
2015-03-31 18:47 ` Joel Brobecker
2015-03-31 19:54 ` Jose E. Marchesi
2015-08-06 21:31 ` Joel Brobecker
2015-08-07 2:03 ` Sergio Durigan Junior
2015-08-07 15:20 ` Joel Brobecker
2015-08-07 13:05 ` Jose E. Marchesi
2015-08-07 13:14 ` Jose E. Marchesi
2015-08-07 14:11 ` Jose E. Marchesi
2015-08-07 15:12 ` Joel Brobecker
2015-08-10 3:21 ` Sergio Durigan Junior
2015-08-10 14:31 ` Jose E. Marchesi
2015-02-16 13:20 ` [PATCH V4 0/9] Add support for DTrace USDT probes to gdb Jose E. Marchesi
2015-02-17 1:57 ` Sergio Durigan Junior
2015-02-17 11:56 ` Jose E. Marchesi
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87mw4d71yq.fsf@redhat.com \
--to=sergiodj@redhat.com \
--cc=gdb-patches@sourceware.org \
--cc=jose.marchesi@oracle.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).