linux-perf-users.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Michael Petlan <mpetlan@redhat.com>
To: Masami Hiramatsu <masami.hiramatsu.pt@hitachi.com>
Cc: linux-perf-users@vger.kernel.org,
	Arnaldo Carvalho de Melo <acme@kernel.org>
Subject: Re: [PATCH perf/core 0/2] perf/runtests: Add runtests.sh shellscript testcases for perf probe
Date: Mon, 2 Nov 2015 19:43:49 -0500 (EST)	[thread overview]
Message-ID: <1499210318.1567245.1446511429277.JavaMail.zimbra@redhat.com> (raw)
In-Reply-To: <20151022121615.13192.71967.stgit@localhost.localdomain>

Hi Masami Hiramatsu,

I am sorry for responding late, I had a lot of other work.
Thanks for sharing the testsuite, I like it very much, so
I added some of your testcases into my skeleton. There is
still some work to do regarding that.

I develop the testsuite using my own github repository[1],
if you are interested, you may have look at it, comment...

The commit cc094da02683b0908512b6b8676c0648e73fa2c8 added
the first part of your suite there.

Thank you again for your interest.

Michael


[1] https://github.com/rfmvh/perftool-testsuite


----- Mensaje original -----
> De: "Masami Hiramatsu" <masami.hiramatsu.pt@hitachi.com>
> Para: "Michael Petlan" <mpetlan@redhat.com>, linux-perf-users@vger.kernel.org
> CC: "Arnaldo Carvalho de Melo" <acme@kernel.org>
> Enviados: Jueves, 22 de Octubre 2015 14:16:15
> Asunto: [PATCH perf/core 0/2] perf/runtests: Add runtests.sh shellscript testcases for perf probe
> 
> Hi Michael,
> 
> Here is a couple of patches to add my testsuite(runtest.sh) to perf.
> 
> This runs as ftracetest does. I've added some testcases which cover
> perf-probe features :)
> 
>   ----
>   # ./runtests.sh
>   === Run tests ===
>   [1] Basinc probe adding [PASS]
>   [2] Basic probe adding on user binary   [PASS]
>   [3] Skip blacklist function     [PASS]
>   [4] Basic probe deleting        [PASS]
>   [5] probe adding with --force   [PASS]
>   [6] Cut off the function suffix from name       [PASS]
>   [7] Basic function listup command       [PASS]
>   [8] help messages       [PASS]
>   [9] invalid commands    [PASS]
>   [10] perf probe --list outputs in stdout        [PASS]
>   [11] list command parsing       [PASS]
>   [12] Skip out-of-text functions [PASS]
>   [13] Quiet option check [PASS]
>   [14] --line semantic errors     [PASS]
>   [15] readable vars command      [PASS]
>   [16] perf-probe wildcard support        [PASS]
>   
>   # of passed:  16
>   # of failed:  0
>   # of unresolved:  0
>   # of untested:  0
>   # of unsupported:  0
>   # of xfailed:  0
>   # of undefined(test bug):  0
>   ----
> 
> Thank you,
> 
> ---
> 
> Masami Hiramatsu (2):
>       perf/runtests: Add runtests.sh shellscript testsuite
>       perf/runtests: Add testcases for perf probe
> 
> 
>  tools/perf/runtests/runtests.conf                  |    5
>  tools/perf/runtests/runtests.sh                    |  264
>  ++++++++++++++++++++
>  tools/perf/runtests/test.d/functions               |   13 +
>  tools/perf/runtests/test.d/probe/addprobes.tc      |   15 +
>  tools/perf/runtests/test.d/probe/adduprobes.tc     |   13 +
>  tools/perf/runtests/test.d/probe/blacklist.tc      |   14 +
>  tools/perf/runtests/test.d/probe/delete.tc         |    8 +
>  tools/perf/runtests/test.d/probe/forceadd.tc       |    8 +
>  .../perf/runtests/test.d/probe/funcname-suffix.tc  |   17 +
>  tools/perf/runtests/test.d/probe/funcs.tc          |    8 +
>  tools/perf/runtests/test.d/probe/help.tc           |    5
>  .../perf/runtests/test.d/probe/invalid_command.tc  |   27 ++
>  tools/perf/runtests/test.d/probe/list-stdout.tc    |    7 +
>  tools/perf/runtests/test.d/probe/list.tc           |   10 +
>  tools/perf/runtests/test.d/probe/out-of-text.tc    |   10 +
>  tools/perf/runtests/test.d/probe/quiet.tc          |    6
>  tools/perf/runtests/test.d/probe/semantic-line.tc  |   20 ++
>  tools/perf/runtests/test.d/probe/vars.tc           |    9 +
>  tools/perf/runtests/test.d/probe/wildcard.tc       |    6
>  19 files changed, 465 insertions(+)
>  create mode 100644 tools/perf/runtests/runtests.conf
>  create mode 100755 tools/perf/runtests/runtests.sh
>  create mode 100644 tools/perf/runtests/test.d/functions
>  create mode 100644 tools/perf/runtests/test.d/probe/addprobes.tc
>  create mode 100644 tools/perf/runtests/test.d/probe/adduprobes.tc
>  create mode 100644 tools/perf/runtests/test.d/probe/blacklist.tc
>  create mode 100644 tools/perf/runtests/test.d/probe/delete.tc
>  create mode 100644 tools/perf/runtests/test.d/probe/forceadd.tc
>  create mode 100644 tools/perf/runtests/test.d/probe/funcname-suffix.tc
>  create mode 100644 tools/perf/runtests/test.d/probe/funcs.tc
>  create mode 100644 tools/perf/runtests/test.d/probe/help.tc
>  create mode 100644 tools/perf/runtests/test.d/probe/invalid_command.tc
>  create mode 100644 tools/perf/runtests/test.d/probe/list-stdout.tc
>  create mode 100644 tools/perf/runtests/test.d/probe/list.tc
>  create mode 100644 tools/perf/runtests/test.d/probe/out-of-text.tc
>  create mode 100644 tools/perf/runtests/test.d/probe/quiet.tc
>  create mode 100644 tools/perf/runtests/test.d/probe/semantic-line.tc
>  create mode 100644 tools/perf/runtests/test.d/probe/vars.tc
>  create mode 100644 tools/perf/runtests/test.d/probe/wildcard.tc
> 
> --
> --
> To unsubscribe from this list: send the line "unsubscribe linux-perf-users"
> in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 

  parent reply	other threads:[~2015-11-03  0:43 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-10-21 23:08 [PATCH 0/3] introducing tools/perf/testsuite Michael Petlan
2015-10-22 10:11 ` 平松雅巳 / HIRAMATU,MASAMI
2015-10-22 12:16 ` [PATCH perf/core 0/2] perf/runtests: Add runtests.sh shellscript testcases for perf probe Masami Hiramatsu
2015-10-22 12:16   ` [PATCH perf/core 1/2] perf/runtests: Add runtests.sh shellscript testsuite Masami Hiramatsu
2015-10-22 12:16   ` [PATCH perf/core 2/2] perf/runtests: Add testcases for perf probe Masami Hiramatsu
2015-11-03  0:43   ` Michael Petlan [this message]
2015-11-03  3:24     ` perf/runtests: Add runtests.sh shellscript " 平松雅巳 / HIRAMATU,MASAMI
2015-11-03 15:45       ` Arnaldo Carvalho de Melo
2015-11-03 19:55         ` Michael Petlan

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=1499210318.1567245.1446511429277.JavaMail.zimbra@redhat.com \
    --to=mpetlan@redhat.com \
    --cc=acme@kernel.org \
    --cc=linux-perf-users@vger.kernel.org \
    --cc=masami.hiramatsu.pt@hitachi.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 NNTP newsgroup(s).