Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1480159

Re: [BUGFIX] Re: [PATCH v2 3/3] perf-probe: Support probing on offline cross-arch binary

From Arnaldo Carvalho de Melo <acme@kernel.org>
Newsgroups linux.kernel
Subject Re: [BUGFIX] Re: [PATCH v2 3/3] perf-probe: Support probing on offline cross-arch binary
Date 2016-09-09 17:50 +0200
Message-ID <sfwdX-4Pt-1@gated-at.bofh.it> (permalink)
References (2 earlier) <sckM2-7eb-39@gated-at.bofh.it> <scn7c-pj-5@gated-at.bofh.it> <sfv8e-45z-21@gated-at.bofh.it> <sfvBf-4BZ-3@gated-at.bofh.it> <sfvUC-4IV-57@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


Em Sat, Sep 10, 2016 at 12:28:07AM +0900, Masami Hiramatsu escreveu:
> On Fri, 9 Sep 2016 11:59:55 -0300
> Arnaldo Carvalho de Melo <acme@kernel.org> wrote:
> > # diff /tmp/fail /tmp/ok
> > <SNIP>
> > -Writing event: p:perf_bpf_probe/func _text+5197232 f_mode=+68(%di):x32 offset=%si:s64 orig=dx:s32
> > -Failed to write event: Invalid argument
> > -bpf_probe: failed to apply perf probe eventsFailed to add events selected by BPF
> > +Writing event: p:perf_bpf_probe/func _text+5197232 f_mode=+68(%di):x32 offset=%si:s64 orig=%dx:s32
> > <SNIP>

> > Fixed by this patch:

> > +++ b/tools/perf/arch/x86/include/dwarf-regs-table.h
> > @@ -7,7 +7,7 @@ static const char * const x86_32_regstr_tbl[] = {
> >  };

> >  static const char * const x86_64_regstr_tbl[] = {
> > -	"%ax", "dx", "%cx", "%bx", "%si", "%di",
> > +	"%ax", "%dx", "%cx", "%bx", "%si", "%di",
> >  	"%bp", "%sp", "%r8", "%r9", "%r10", "%r11",
> >  	"%r12", "%r13", "%r14", "%r15",
> >  };
 
> Oops, it's my typo... thank you for reporting and fixing!

Right, but it was my fault as well, I should've payed attention to the
output of 'perf test' and replied to you so that you could figure this
out and resend.

Anyway, its good that this gets exercised by that test, its just a
matter of us making sure that no regressions appear on 'perf test'
before pushing things upstream.
 
> Acked-by: Masami Hiramatsu <mhiramat@kernel.org>

Thanks, added it to the patch.

- Arnaldo

- Arnaldo

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread


Thread

[PATCH v2 0/3] perf probe: Introduce remote cross-arch probes Masami Hiramatsu <mhiramat@kernel.org> - 2016-08-25 18:30 +0200
  [PATCH v2 3/3] perf-probe: Support probing on offline cross-arch binary Masami Hiramatsu <mhiramat@kernel.org> - 2016-08-25 18:30 +0200
    Re: [PATCH v2 3/3] perf-probe: Support probing on offline cross-arch  binary Arnaldo Carvalho de Melo <acme@kernel.org> - 2016-08-31 23:00 +0200
      Re: [PATCH v2 3/3] perf-probe: Support probing on offline  cross-arch binary Masami Hiramatsu <mhiramat@kernel.org> - 2016-09-01 01:30 +0200
        [BUG] Re: [PATCH v2 3/3] perf-probe: Support probing on offline  cross-arch binary Arnaldo Carvalho de Melo <acme@kernel.org> - 2016-09-09 16:40 +0200
          Re: [BUGFIX] Re: [PATCH v2 3/3] perf-probe: Support probing on  offline cross-arch binary Arnaldo Carvalho de Melo <acme@kernel.org> - 2016-09-09 17:10 +0200
            Re: [BUGFIX] Re: [PATCH v2 3/3] perf-probe: Support probing on  offline cross-arch binary Masami Hiramatsu <mhiramat@kernel.org> - 2016-09-09 17:30 +0200
              Re: [BUGFIX] Re: [PATCH v2 3/3] perf-probe: Support probing on  offline cross-arch binary Arnaldo Carvalho de Melo <acme@kernel.org> - 2016-09-09 17:50 +0200
    [tip:perf/core] perf probe: Support probing on offline cross-arch  binary tip-bot for Masami Hiramatsu <tipbot@zytor.com> - 2016-09-05 15:30 +0200
  [PATCH v2 1/3] perf-probe: Show trace event definition Masami Hiramatsu <mhiramat@kernel.org> - 2016-08-25 18:30 +0200
    [tip:perf/core] perf probe: Show trace event definition tip-bot for Masami Hiramatsu <tipbot@zytor.com> - 2016-09-05 15:30 +0200
  [PATCH v2 4/4] perf-probe: Ignore vmlinux Build-id when offline vmlinux given Masami Hiramatsu <mhiramat@kernel.org> - 2016-08-26 17:00 +0200
    [tip:perf/core] perf probe: Ignore vmlinux Build-id when offline  vmlinux given tip-bot for Masami Hiramatsu <tipbot@zytor.com> - 2016-09-05 15:30 +0200
  Re: [PATCH v2 0/3] perf probe: Introduce remote cross-arch probes Masami Hiramatsu <mhiramat@kernel.org> - 2016-08-31 03:10 +0200

csiph-web