From: Arnaldo Carvalho de Melo <arnaldo.melo@gmail.com>
To: Guo Ren <guoren@kernel.org>
Cc: Mao Han <han_mao@c-sky.com>,
linux-kernel@vger.kernel.org, linux-csky@vger.kernel.org,
Peter Zijlstra <peterz@infradead.org>,
Ingo Molnar <mingo@redhat.com>,
Alexander Shishkin <alexander.shishkin@linux.intel.com>,
Jiri Olsa <jolsa@redhat.com>, Namhyung Kim <namhyung@kernel.org>
Subject: Re: [PATCH 1/1] perf annotate csky: Add perf annotate support
Date: Wed, 26 Jun 2019 15:51:56 -0300 [thread overview]
Message-ID: <20190626185156.GC3902@kernel.org> (raw)
In-Reply-To: <CAJF2gTRyma8sDMJaWCde1eOe6KSwn4_e=tJOT4d3kgmvzOxz8g@mail.gmail.com>
Em Wed, Jun 26, 2019 at 02:56:55PM +0800, Guo Ren escreveu:
> Thx Mao,
>
> Approved!
I guess I can take this as a:
Acked-by: Guo Ren <guoren@kernel.org>
Or would this better be:
Reviewed-by: Guo Ren <guoren@kernel.org>
?
- Arnaldo
> Best Regards
> Guo Ren
>
> On Wed, Jun 26, 2019 at 2:53 PM Mao Han <han_mao@c-sky.com> wrote:
> >
> > This patch add basic arch initialization and instruction associate support
> > for csky.
> >
> > perf annotate --stdio2
> > Samples: 161 of event 'cpu-clock:pppH', 4000 Hz, Event count (approx.):
> > 40250000, [percent: local period]
> > test_4() /usr/lib/perf-test/callchain_test
> > Percent
> >
> > Disassembly of section .text:
> >
> > 00008420 <test_4>:
> > test_4():
> > subi sp, sp, 4
> > st.w r8, (sp, 0x0)
> > mov r8, sp
> > subi sp, sp, 8
> > subi r3, r8, 4
> > movi r2, 0
> > st.w r2, (r3, 0x0)
> > ↓ br 2e
> > 100.00 14: subi r3, r8, 4
> > ld.w r2, (r3, 0x0)
> > subi r3, r8, 8
> > st.w r2, (r3, 0x0)
> > subi r3, r8, 4
> > ld.w r3, (r3, 0x0)
> > addi r2, r3, 1
> > subi r3, r8, 4
> > st.w r2, (r3, 0x0)
> > 2e: subi r3, r8, 4
> > ld.w r2, (r3, 0x0)
> > lrw r3, 0x98967f // 8598 <main+0x28>
> > cmplt r3, r2
> > ↑ bf 14
> > mov r0, r0
> > mov r0, r0
> > mov sp, r8
> > ld.w r8, (sp, 0x0)
> > addi sp, sp, 4
> > ← rts
> >
> > Signed-off-by: Mao Han <han_mao@c-sky.com>
> > Cc: Guo Ren <guoren@kernel.org>
> > Cc: Peter Zijlstra <peterz@infradead.org>
> > Cc: Ingo Molnar <mingo@redhat.com>
> > Cc: Arnaldo Carvalho de Melo <acme@kernel.org>
> > Cc: Alexander Shishkin <alexander.shishkin@linux.intel.com>
> > Cc: Jiri Olsa <jolsa@redhat.com>
> > Cc: Namhyung Kim <namhyung@kernel.org>
> > ---
> > tools/perf/arch/csky/annotate/instructions.c | 48 ++++++++++++++++++++++++++++
> > tools/perf/util/annotate.c | 5 +++
> > 2 files changed, 53 insertions(+)
> > create mode 100644 tools/perf/arch/csky/annotate/instructions.c
> >
> > diff --git a/tools/perf/arch/csky/annotate/instructions.c b/tools/perf/arch/csky/annotate/instructions.c
> > new file mode 100644
> > index 0000000..5337bfb
> > --- /dev/null
> > +++ b/tools/perf/arch/csky/annotate/instructions.c
> > @@ -0,0 +1,48 @@
> > +// SPDX-License-Identifier: GPL-2.0
> > +// Copyright (C) 2019 Hangzhou C-SKY Microsystems co.,ltd.
> > +
> > +#include <linux/compiler.h>
> > +
> > +static struct ins_ops *csky__associate_ins_ops(struct arch *arch,
> > + const char *name)
> > +{
> > + struct ins_ops *ops = NULL;
> > +
> > + /* catch all kind of jumps */
> > + if (!strcmp(name, "bt") ||
> > + !strcmp(name, "bf") ||
> > + !strcmp(name, "bez") ||
> > + !strcmp(name, "bnez") ||
> > + !strcmp(name, "bnezad") ||
> > + !strcmp(name, "bhsz") ||
> > + !strcmp(name, "bhz") ||
> > + !strcmp(name, "blsz") ||
> > + !strcmp(name, "blz") ||
> > + !strcmp(name, "br") ||
> > + !strcmp(name, "jmpi") ||
> > + !strcmp(name, "jmp"))
> > + ops = &jump_ops;
> > +
> > + /* catch function call */
> > + if (!strcmp(name, "bsr") ||
> > + !strcmp(name, "jsri") ||
> > + !strcmp(name, "jsr"))
> > + ops = &call_ops;
> > +
> > + /* catch function return */
> > + if (!strcmp(name, "rts"))
> > + ops = &ret_ops;
> > +
> > + if (ops)
> > + arch__associate_ins_ops(arch, name, ops);
> > + return ops;
> > +}
> > +
> > +static int csky__annotate_init(struct arch *arch, char *cpuid __maybe_unused)
> > +{
> > + arch->initialized = true;
> > + arch->objdump.comment_char = '/';
> > + arch->associate_instruction_ops = csky__associate_ins_ops;
> > +
> > + return 0;
> > +}
> > diff --git a/tools/perf/util/annotate.c b/tools/perf/util/annotate.c
> > index 79db038..eb2456e 100644
> > --- a/tools/perf/util/annotate.c
> > +++ b/tools/perf/util/annotate.c
> > @@ -144,6 +144,7 @@ static int arch__associate_ins_ops(struct arch* arch, const char *name, struct i
> > #include "arch/arc/annotate/instructions.c"
> > #include "arch/arm/annotate/instructions.c"
> > #include "arch/arm64/annotate/instructions.c"
> > +#include "arch/csky/annotate/instructions.c"
> > #include "arch/x86/annotate/instructions.c"
> > #include "arch/powerpc/annotate/instructions.c"
> > #include "arch/s390/annotate/instructions.c"
> > @@ -163,6 +164,10 @@ static struct arch architectures[] = {
> > .init = arm64__annotate_init,
> > },
> > {
> > + .name = "csky",
> > + .init = csky__annotate_init,
> > + },
> > + {
> > .name = "x86",
> > .init = x86__annotate_init,
> > .instructions = x86__instructions,
> > --
> > 2.7.4
> >
>
>
> --
> Best Regards
> Guo Ren
>
> ML: https://lore.kernel.org/linux-csky/
--
- Arnaldo
next prev parent reply other threads:[~2019-06-26 18:52 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <cover.1561531557.git.han_mao@c-sky.com>
2019-06-26 6:52 ` [PATCH 1/1] perf annotate csky: Add perf annotate support Mao Han
2019-06-26 6:56 ` Guo Ren
2019-06-26 18:51 ` Arnaldo Carvalho de Melo [this message]
2019-06-27 1:50 ` Guo Ren
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=20190626185156.GC3902@kernel.org \
--to=arnaldo.melo@gmail.com \
--cc=alexander.shishkin@linux.intel.com \
--cc=guoren@kernel.org \
--cc=han_mao@c-sky.com \
--cc=jolsa@redhat.com \
--cc=linux-csky@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mingo@redhat.com \
--cc=namhyung@kernel.org \
--cc=peterz@infradead.org \
/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 NNTP newsgroup(s).