linux-kselftest.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: rostedt at goodmis.org (Steven Rostedt)
Subject: [PATCH v2 19/32] selftests/ftrace: Add a testcase for nop tracer
Date: Fri, 24 Aug 2018 18:53:42 -0400	[thread overview]
Message-ID: <20180824185342.542d8a6a@gandalf.local.home> (raw)
In-Reply-To: <153443748943.23257.13855745627496095446.stgit@devbox>

On Fri, 17 Aug 2018 01:38:09 +0900
Masami Hiramatsu <mhiramat at kernel.org> wrote:

> Add a testcase for nop tracer. This tracer has 2 test
> options, so check it too.
> 

I'd rather not include this. In fact, I'd love to get rid of the nop
options. They were added to make sure options worked when we first
added them. They are actually pointless.

The only point of this test is to know if options work or not. Hmm, I
guess that is something that could possibly be valuable :-/

-- Steve


> Signed-off-by: Masami Hiramatsu <mhiramat at kernel.org>
> ---
>  .../testing/selftests/ftrace/test.d/tracer/nop.tc  |   22 ++++++++++++++++++++
>  1 file changed, 22 insertions(+)
>  create mode 100644 tools/testing/selftests/ftrace/test.d/tracer/nop.tc
> 
> diff --git a/tools/testing/selftests/ftrace/test.d/tracer/nop.tc b/tools/testing/selftests/ftrace/test.d/tracer/nop.tc
> new file mode 100644
> index 000000000000..ca63e316e0f5
> --- /dev/null
> +++ b/tools/testing/selftests/ftrace/test.d/tracer/nop.tc
> @@ -0,0 +1,22 @@
> +#!/bin/sh
> +# SPDX-License-Identifier: GPL-2.0
> +# description: Test nop tracer
> +
> +# nop must be supported always
> +grep -q "nop" available_tracers
> +
> +echo "nop" > current_tracer
> +
> +# nop provides 2 test options
> +grep -q "test_nop_accept" trace_options
> +grep -q "test_nop_refuse" trace_options
> +
> +# nop_accept always success
> +echo "test_nop_accept" > trace_options
> +echo "notest_nop_accept" > trace_options
> +
> +# nop_refuse always failed
> +! echo "test_nop_refuse" > trace_options
> +! echo "notest_nop_refuse" > trace_options
> +
> +exit 0

WARNING: multiple messages have this Message-ID (diff)
From: rostedt@goodmis.org (Steven Rostedt)
Subject: [PATCH v2 19/32] selftests/ftrace: Add a testcase for nop tracer
Date: Fri, 24 Aug 2018 18:53:42 -0400	[thread overview]
Message-ID: <20180824185342.542d8a6a@gandalf.local.home> (raw)
Message-ID: <20180824225342.EdG-jqaXrq856_0qDK12-8i5uAs0zIYMZX6h6F_j6Sc@z> (raw)
In-Reply-To: <153443748943.23257.13855745627496095446.stgit@devbox>

On Fri, 17 Aug 2018 01:38:09 +0900
Masami Hiramatsu <mhiramat@kernel.org> wrote:

> Add a testcase for nop tracer. This tracer has 2 test
> options, so check it too.
> 

I'd rather not include this. In fact, I'd love to get rid of the nop
options. They were added to make sure options worked when we first
added them. They are actually pointless.

The only point of this test is to know if options work or not. Hmm, I
guess that is something that could possibly be valuable :-/

-- Steve


> Signed-off-by: Masami Hiramatsu <mhiramat at kernel.org>
> ---
>  .../testing/selftests/ftrace/test.d/tracer/nop.tc  |   22 ++++++++++++++++++++
>  1 file changed, 22 insertions(+)
>  create mode 100644 tools/testing/selftests/ftrace/test.d/tracer/nop.tc
> 
> diff --git a/tools/testing/selftests/ftrace/test.d/tracer/nop.tc b/tools/testing/selftests/ftrace/test.d/tracer/nop.tc
> new file mode 100644
> index 000000000000..ca63e316e0f5
> --- /dev/null
> +++ b/tools/testing/selftests/ftrace/test.d/tracer/nop.tc
> @@ -0,0 +1,22 @@
> +#!/bin/sh
> +# SPDX-License-Identifier: GPL-2.0
> +# description: Test nop tracer
> +
> +# nop must be supported always
> +grep -q "nop" available_tracers
> +
> +echo "nop" > current_tracer
> +
> +# nop provides 2 test options
> +grep -q "test_nop_accept" trace_options
> +grep -q "test_nop_refuse" trace_options
> +
> +# nop_accept always success
> +echo "test_nop_accept" > trace_options
> +echo "notest_nop_accept" > trace_options
> +
> +# nop_refuse always failed
> +! echo "test_nop_refuse" > trace_options
> +! echo "notest_nop_refuse" > trace_options
> +
> +exit 0

  parent reply	other threads:[~2018-08-24 22:53 UTC|newest]

Thread overview: 152+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-08-16 16:29 [PATCH v2 00/32] selftests/ftrace: Improve ftracetest with coverage check mhiramat
2018-08-16 16:29 ` Masami Hiramatsu
2018-08-16 16:29 ` [PATCH v2 01/32] tracing: Allow gcov profiling on only ftrace subsystem mhiramat
2018-08-16 16:29   ` Masami Hiramatsu
2018-08-20 15:32   ` rostedt
2018-08-20 15:32     ` Steven Rostedt
2018-08-20 15:33     ` rostedt
2018-08-20 15:33       ` Steven Rostedt
2018-08-21  6:53       ` mhiramat
2018-08-21  6:53         ` Masami Hiramatsu
2018-08-16 16:30 ` [PATCH v2 02/32] selftests/ftrace: Add --stop-fail hidden option for debug mhiramat
2018-08-16 16:30   ` Masami Hiramatsu
2018-08-24 20:38   ` rostedt
2018-08-24 20:38     ` Steven Rostedt
2018-08-16 16:30 ` [PATCH v2 03/32] selftests/ftrace: Add --console hidden option mhiramat
2018-08-16 16:30   ` Masami Hiramatsu
2018-08-24 20:37   ` rostedt
2018-08-24 20:37     ` Steven Rostedt
2018-08-25  1:35     ` mhiramat
2018-08-25  1:35       ` Masami Hiramatsu
2018-08-25  1:42       ` rostedt
2018-08-25  1:42         ` Steven Rostedt
2018-08-26  7:28         ` mhiramat
2018-08-26  7:28           ` Masami Hiramatsu
2018-08-16 16:31 ` [PATCH v2 04/32] selftests/ftrace: Add case number prefix to logfile mhiramat
2018-08-16 16:31   ` Masami Hiramatsu
2018-08-24 21:47   ` rostedt
2018-08-24 21:47     ` Steven Rostedt
2018-08-16 16:31 ` [PATCH v2 05/32] selftests/ftrace: More initialize features in initialize_ftrace mhiramat
2018-08-16 16:31   ` Masami Hiramatsu
2018-08-24 21:24   ` rostedt
2018-08-24 21:24     ` Steven Rostedt
2018-08-24 21:46   ` rostedt
2018-08-24 21:46     ` Steven Rostedt
2018-08-24 21:46     ` rostedt
2018-08-24 21:46       ` Steven Rostedt
2018-08-16 16:32 ` [PATCH v2 06/32] selftests/ftrace: Add SPDX License Identifier to template mhiramat
2018-08-16 16:32   ` Masami Hiramatsu
2018-08-16 16:32 ` [PATCH v2 07/32] selftests/ftrace: Cleanup ftrace after running test mhiramat
2018-08-16 16:32   ` Masami Hiramatsu
2018-08-16 16:32 ` [PATCH v2 08/32] selftests/ftrace: Remove unneeded per-test init/cleanup ftrace mhiramat
2018-08-16 16:32   ` Masami Hiramatsu
2018-08-16 16:33 ` [PATCH v2 09/32] selftests/ftrace: Fix to test kprobe $comm arg only if available mhiramat
2018-08-16 16:33   ` Masami Hiramatsu
2018-08-16 16:33 ` [PATCH v2 10/32] selftests/ftrace: Fix checkbashisms errors mhiramat
2018-08-16 16:33   ` Masami Hiramatsu
2018-08-24 21:50   ` rostedt
2018-08-24 21:50     ` Steven Rostedt
2018-08-16 16:34 ` [PATCH v2 11/32] selftests/ftrace: Use loopback address instead of localhost mhiramat
2018-08-16 16:34   ` Masami Hiramatsu
2018-08-24 21:52   ` rostedt
2018-08-24 21:52     ` Steven Rostedt
2018-08-25  1:47     ` mhiramat
2018-08-25  1:47       ` Masami Hiramatsu
2018-08-16 16:34 ` [PATCH v2 12/32] selftests/ftrace: Improve kprobe on module testcase to load/unload module mhiramat
2018-08-16 16:34   ` Masami Hiramatsu
2018-08-24 23:07   ` rostedt
2018-08-24 23:07     ` Steven Rostedt
2018-08-16 16:35 ` [PATCH v2 13/32] selftests/ftrace: Improve kprobe testcase to check log data mhiramat
2018-08-16 16:35   ` Masami Hiramatsu
2018-08-16 16:35 ` [PATCH v2 14/32] selftests/ftrace: Improve kretprobe " mhiramat
2018-08-16 16:35   ` Masami Hiramatsu
2018-08-16 16:36 ` [PATCH v2 15/32] selftests/ftrace: Test kprobe-event argument with various bitsize mhiramat
2018-08-16 16:36   ` Masami Hiramatsu
2018-08-16 16:36 ` [PATCH v2 16/32] selftests/ftrace: Check set_event_pid result mhiramat
2018-08-16 16:36   ` Masami Hiramatsu
2018-08-16 16:37 ` [PATCH v2 17/32] selftests/ftrace: Add kprobe event with $comm argument testcase mhiramat
2018-08-16 16:37   ` Masami Hiramatsu
2018-08-16 16:37 ` [PATCH v2 18/32] selftests/ftrace: Add kprobe profile testcase mhiramat
2018-08-16 16:37   ` Masami Hiramatsu
2018-08-16 16:38 ` [PATCH v2 19/32] selftests/ftrace: Add a testcase for nop tracer mhiramat
2018-08-16 16:38   ` Masami Hiramatsu
2018-08-24 22:53   ` rostedt [this message]
2018-08-24 22:53     ` Steven Rostedt
2018-08-26  7:30     ` mhiramat
2018-08-26  7:30       ` Masami Hiramatsu
2018-08-16 16:38 ` [PATCH v2 20/32] selftests/ftrace: Add kprobe-event with symbol argument test mhiramat
2018-08-16 16:38   ` Masami Hiramatsu
2018-08-16 16:39 ` [PATCH v2 21/32] selftests/ftrace: Add trace_printk sample module test mhiramat
2018-08-16 16:39   ` Masami Hiramatsu
2018-08-24 23:06   ` rostedt
2018-08-24 23:06     ` Steven Rostedt
2018-08-26  8:43     ` mhiramat
2018-08-26  8:43       ` Masami Hiramatsu
2018-08-16 16:39 ` [PATCH v2 22/32] selftests/ftrace: Add ringbuffer size changing testcase mhiramat
2018-08-16 16:39   ` Masami Hiramatsu
2018-08-24 23:18   ` rostedt
2018-08-24 23:18     ` Steven Rostedt
2018-08-30  7:12     ` mhiramat
2018-08-30  7:12       ` Masami Hiramatsu
2018-08-30 15:50       ` rostedt
2018-08-30 15:50         ` Steven Rostedt
2018-08-30 23:35         ` mhiramat
2018-08-30 23:35           ` Masami Hiramatsu
2018-08-30 23:40           ` rostedt
2018-08-30 23:40             ` Steven Rostedt
2018-08-16 16:40 ` [PATCH v2 23/32] selftests/ftrace: Add function profiling stat testcase mhiramat
2018-08-16 16:40   ` Masami Hiramatsu
2018-08-24 23:20   ` rostedt
2018-08-24 23:20     ` Steven Rostedt
2018-08-26 11:20     ` mhiramat
2018-08-26 11:20       ` Masami Hiramatsu
2018-08-16 16:40 ` [PATCH v2 24/32] selftests/ftrace: Add max stack tracer testcase mhiramat
2018-08-16 16:40   ` Masami Hiramatsu
2018-08-24 23:23   ` rostedt
2018-08-24 23:23     ` Steven Rostedt
2018-08-25  1:45     ` mhiramat
2018-08-25  1:45       ` Masami Hiramatsu
2018-08-25  2:41       ` rostedt
2018-08-25  2:41         ` Steven Rostedt
2018-08-26  8:42         ` mhiramat
2018-08-26  8:42           ` Masami Hiramatsu
2018-08-16 16:40 ` [PATCH v2 25/32] selftests/ftrace: Add function filter on module testcase mhiramat
2018-08-16 16:40   ` Masami Hiramatsu
2018-08-24 23:25   ` rostedt
2018-08-24 23:25     ` Steven Rostedt
2018-08-16 16:41 ` [PATCH v2 26/32] selftests/ftrace: Add trace_pipe testcase mhiramat
2018-08-16 16:41   ` Masami Hiramatsu
2018-08-24 23:26   ` rostedt
2018-08-24 23:26     ` Steven Rostedt
2018-08-26 11:20     ` mhiramat
2018-08-26 11:20       ` Masami Hiramatsu
2018-08-16 16:41 ` [PATCH v2 27/32] selftests/ftrace: Add stacktrace ftrace filter command testcase mhiramat
2018-08-16 16:41   ` Masami Hiramatsu
2018-08-24 23:28   ` rostedt
2018-08-24 23:28     ` Steven Rostedt
2018-08-16 16:42 ` [PATCH v2 28/32] selftests/ftrace: Add wakeup tracer testcase mhiramat
2018-08-16 16:42   ` Masami Hiramatsu
2018-08-16 16:42 ` [PATCH v2 29/32] selftests/ftrace: Add wakeup_rt " mhiramat
2018-08-16 16:42   ` Masami Hiramatsu
2018-08-25  2:09   ` rostedt
2018-08-25  2:09     ` Steven Rostedt
2018-08-26 11:22     ` mhiramat
2018-08-26 11:22       ` Masami Hiramatsu
2018-08-16 16:43 ` [PATCH v2 30/32] selftests/ftrace: Add ftrace cpumask testcase mhiramat
2018-08-16 16:43   ` Masami Hiramatsu
2018-08-25  2:18   ` rostedt
2018-08-25  2:18     ` Steven Rostedt
2018-08-27 12:16     ` mhiramat
2018-08-27 12:16       ` Masami Hiramatsu
2018-08-16 16:43 ` [PATCH v2 31/32] selftests/ftrace: Add output format testcase mhiramat
2018-08-16 16:43   ` Masami Hiramatsu
2018-08-25  2:19   ` rostedt
2018-08-25  2:19     ` Steven Rostedt
2018-08-30  7:48     ` mhiramat
2018-08-30  7:48       ` Masami Hiramatsu
2018-08-16 16:44 ` [PATCH v2 32/32] selftests/ftrace: Add blktrace testcase mhiramat
2018-08-16 16:44   ` Masami Hiramatsu
2018-08-25  2:22   ` rostedt
2018-08-25  2:22     ` Steven Rostedt
2018-08-30  7:53     ` mhiramat
2018-08-30  7:53       ` Masami Hiramatsu

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=20180824185342.542d8a6a@gandalf.local.home \
    --to=linux-kselftest@vger.kernel.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).