subject:"\[tip\:perf\/core\] perf tools\: Remove unused 'prefix' from builtin functions"

[tip:perf/core] perf tools: Remove unused 'prefix' from builtin functions

2017-03-27 Thread tip-bot for Arnaldo Carvalho de Melo

Commit-ID:  b0ad8ea66445d64a469df0c710947f4cdb8ef16b
Gitweb: http://git.kernel.org/tip/b0ad8ea66445d64a469df0c710947f4cdb8ef16b
Author: Arnaldo Carvalho de Melo 
AuthorDate: Mon, 27 Mar 2017 11:47:20 -0300
Committer:  Arnaldo Carvalho de Melo 
CommitDate: Mon, 27 Mar 2017 11:58:09 -0300

perf tools: Remove unused 'prefix' from builtin functions

We got it from the git sources but never used it for anything, with the
place where this would be somehow used remaining:

  static int run_builtin(struct cmd_struct *p, int argc, const char **argv)
  {
prefix = NULL;
if (p->option & RUN_SETUP)
prefix = NULL; /* setup_perf_directory(); */

Ditch it.

Cc: Adrian Hunter 
Cc: David Ahern 
Cc: Jiri Olsa 
Cc: Namhyung Kim 
Cc: Wang Nan 
Link: http://lkml.kernel.org/n/tip-uw5swz05vol0qpr32c5lp...@git.kernel.org
Signed-off-by: Arnaldo Carvalho de Melo 
---
 tools/perf/bench/bench.h   | 20 ++--
 tools/perf/bench/futex-hash.c  |  3 +-
 tools/perf/bench/futex-lock-pi.c   |  3 +-
 tools/perf/bench/futex-requeue.c   |  3 +-
 tools/perf/bench/futex-wake-parallel.c |  3 +-
 tools/perf/bench/futex-wake.c  |  3 +-
 tools/perf/bench/mem-functions.c   |  4 +--
 tools/perf/bench/numa.c|  2 +-
 tools/perf/bench/sched-messaging.c |  3 +-
 tools/perf/bench/sched-pipe.c  |  2 +-
 tools/perf/builtin-annotate.c  |  2 +-
 tools/perf/builtin-bench.c | 12 +++
 tools/perf/builtin-buildid-cache.c |  3 +-
 tools/perf/builtin-buildid-list.c  |  3 +-
 tools/perf/builtin-c2c.c   |  4 +--
 tools/perf/builtin-config.c|  2 +-
 tools/perf/builtin-data.c  |  9 +++---
 tools/perf/builtin-diff.c  |  2 +-
 tools/perf/builtin-evlist.c|  2 +-
 tools/perf/builtin-ftrace.c|  2 +-
 tools/perf/builtin-help.c  |  2 +-
 tools/perf/builtin-inject.c|  2 +-
 tools/perf/builtin-kallsyms.c  |  2 +-
 tools/perf/builtin-kmem.c  |  4 +--
 tools/perf/builtin-kvm.c   | 16 +-
 tools/perf/builtin-list.c  |  2 +-
 tools/perf/builtin-lock.c  |  6 ++--
 tools/perf/builtin-mem.c   |  6 ++--
 tools/perf/builtin-probe.c |  6 ++--
 tools/perf/builtin-record.c|  2 +-
 tools/perf/builtin-report.c|  2 +-
 tools/perf/builtin-sched.c |  6 ++--
 tools/perf/builtin-script.c|  4 +--
 tools/perf/builtin-stat.c  |  2 +-
 tools/perf/builtin-timechart.c |  7 ++--
 tools/perf/builtin-top.c   |  2 +-
 tools/perf/builtin-trace.c |  4 +--
 tools/perf/builtin-version.c   |  3 +-
 tools/perf/builtin.h   | 58 +-
 tools/perf/perf.c  | 11 ++-
 tools/perf/tests/builtin-test.c|  2 +-
 41 files changed, 110 insertions(+), 126 deletions(-)

diff --git a/tools/perf/bench/bench.h b/tools/perf/bench/bench.h
index 579a592..842ab27 100644
--- a/tools/perf/bench/bench.h
+++ b/tools/perf/bench/bench.h
@@ -25,17 +25,17 @@
 # endif
 #endif
 
-int bench_numa(int argc, const char **argv, const char *prefix);
-int bench_sched_messaging(int argc, const char **argv, const char *prefix);
-int bench_sched_pipe(int argc, const char **argv, const char *prefix);
-int bench_mem_memcpy(int argc, const char **argv, const char *prefix);
-int bench_mem_memset(int argc, const char **argv, const char *prefix);
-int bench_futex_hash(int argc, const char **argv, const char *prefix);
-int bench_futex_wake(int argc, const char **argv, const char *prefix);
-int bench_futex_wake_parallel(int argc, const char **argv, const char *prefix);
-int bench_futex_requeue(int argc, const char **argv, const char *prefix);
+int bench_numa(int argc, const char **argv);
+int bench_sched_messaging(int argc, const char **argv);
+int bench_sched_pipe(int argc, const char **argv);
+int bench_mem_memcpy(int argc, const char **argv);
+int bench_mem_memset(int argc, const char **argv);
+int bench_futex_hash(int argc, const char **argv);
+int bench_futex_wake(int argc, const char **argv);
+int bench_futex_wake_parallel(int argc, const char **argv);
+int bench_futex_requeue(int argc, const char **argv);
 /* pi futexes */
-int bench_futex_lock_pi(int argc, const char **argv, const char *prefix);
+int bench_futex_lock_pi(int argc, const char **argv);
 
 #define BENCH_FORMAT_DEFAULT_STR   "default"
 #define BENCH_FORMAT_DEFAULT   0
diff --git a/tools/perf/bench/futex-hash.c b/tools/perf/bench/futex-hash.c
index 2499e1b..fe16b31 100644
--- a/tools/perf/bench/futex-hash.c
+++ b/tools/perf/bench/futex-hash.c
@@ -114,8 +114,7 @@ static void print_summary(void)
   (int) runtime.tv_sec);
 }

[tip:perf/core] perf tools: Remove unused 'prefix' from builtin functions

2017-03-27 Thread tip-bot for Arnaldo Carvalho de Melo

Commit-ID:  b0ad8ea66445d64a469df0c710947f4cdb8ef16b
Gitweb: http://git.kernel.org/tip/b0ad8ea66445d64a469df0c710947f4cdb8ef16b
Author: Arnaldo Carvalho de Melo 
AuthorDate: Mon, 27 Mar 2017 11:47:20 -0300
Committer:  Arnaldo Carvalho de Melo 
CommitDate: Mon, 27 Mar 2017 11:58:09 -0300

perf tools: Remove unused 'prefix' from builtin functions

We got it from the git sources but never used it for anything, with the
place where this would be somehow used remaining:

  static int run_builtin(struct cmd_struct *p, int argc, const char **argv)
  {
prefix = NULL;
if (p->option & RUN_SETUP)
prefix = NULL; /* setup_perf_directory(); */

Ditch it.

Cc: Adrian Hunter 
Cc: David Ahern 
Cc: Jiri Olsa 
Cc: Namhyung Kim 
Cc: Wang Nan 
Link: http://lkml.kernel.org/n/tip-uw5swz05vol0qpr32c5lp...@git.kernel.org
Signed-off-by: Arnaldo Carvalho de Melo 
---
 tools/perf/bench/bench.h   | 20 ++--
 tools/perf/bench/futex-hash.c  |  3 +-
 tools/perf/bench/futex-lock-pi.c   |  3 +-
 tools/perf/bench/futex-requeue.c   |  3 +-
 tools/perf/bench/futex-wake-parallel.c |  3 +-
 tools/perf/bench/futex-wake.c  |  3 +-
 tools/perf/bench/mem-functions.c   |  4 +--
 tools/perf/bench/numa.c|  2 +-
 tools/perf/bench/sched-messaging.c |  3 +-
 tools/perf/bench/sched-pipe.c  |  2 +-
 tools/perf/builtin-annotate.c  |  2 +-
 tools/perf/builtin-bench.c | 12 +++
 tools/perf/builtin-buildid-cache.c |  3 +-
 tools/perf/builtin-buildid-list.c  |  3 +-
 tools/perf/builtin-c2c.c   |  4 +--
 tools/perf/builtin-config.c|  2 +-
 tools/perf/builtin-data.c  |  9 +++---
 tools/perf/builtin-diff.c  |  2 +-
 tools/perf/builtin-evlist.c|  2 +-
 tools/perf/builtin-ftrace.c|  2 +-
 tools/perf/builtin-help.c  |  2 +-
 tools/perf/builtin-inject.c|  2 +-
 tools/perf/builtin-kallsyms.c  |  2 +-
 tools/perf/builtin-kmem.c  |  4 +--
 tools/perf/builtin-kvm.c   | 16 +-
 tools/perf/builtin-list.c  |  2 +-
 tools/perf/builtin-lock.c  |  6 ++--
 tools/perf/builtin-mem.c   |  6 ++--
 tools/perf/builtin-probe.c |  6 ++--
 tools/perf/builtin-record.c|  2 +-
 tools/perf/builtin-report.c|  2 +-
 tools/perf/builtin-sched.c |  6 ++--
 tools/perf/builtin-script.c|  4 +--
 tools/perf/builtin-stat.c  |  2 +-
 tools/perf/builtin-timechart.c |  7 ++--
 tools/perf/builtin-top.c   |  2 +-
 tools/perf/builtin-trace.c |  4 +--
 tools/perf/builtin-version.c   |  3 +-
 tools/perf/builtin.h   | 58 +-
 tools/perf/perf.c  | 11 ++-
 tools/perf/tests/builtin-test.c|  2 +-
 41 files changed, 110 insertions(+), 126 deletions(-)

diff --git a/tools/perf/bench/bench.h b/tools/perf/bench/bench.h
index 579a592..842ab27 100644
--- a/tools/perf/bench/bench.h
+++ b/tools/perf/bench/bench.h
@@ -25,17 +25,17 @@
 # endif
 #endif
 
-int bench_numa(int argc, const char **argv, const char *prefix);
-int bench_sched_messaging(int argc, const char **argv, const char *prefix);
-int bench_sched_pipe(int argc, const char **argv, const char *prefix);
-int bench_mem_memcpy(int argc, const char **argv, const char *prefix);
-int bench_mem_memset(int argc, const char **argv, const char *prefix);
-int bench_futex_hash(int argc, const char **argv, const char *prefix);
-int bench_futex_wake(int argc, const char **argv, const char *prefix);
-int bench_futex_wake_parallel(int argc, const char **argv, const char *prefix);
-int bench_futex_requeue(int argc, const char **argv, const char *prefix);
+int bench_numa(int argc, const char **argv);
+int bench_sched_messaging(int argc, const char **argv);
+int bench_sched_pipe(int argc, const char **argv);
+int bench_mem_memcpy(int argc, const char **argv);
+int bench_mem_memset(int argc, const char **argv);
+int bench_futex_hash(int argc, const char **argv);
+int bench_futex_wake(int argc, const char **argv);
+int bench_futex_wake_parallel(int argc, const char **argv);
+int bench_futex_requeue(int argc, const char **argv);
 /* pi futexes */
-int bench_futex_lock_pi(int argc, const char **argv, const char *prefix);
+int bench_futex_lock_pi(int argc, const char **argv);
 
 #define BENCH_FORMAT_DEFAULT_STR   "default"
 #define BENCH_FORMAT_DEFAULT   0
diff --git a/tools/perf/bench/futex-hash.c b/tools/perf/bench/futex-hash.c
index 2499e1b..fe16b31 100644
--- a/tools/perf/bench/futex-hash.c
+++ b/tools/perf/bench/futex-hash.c
@@ -114,8 +114,7 @@ static void print_summary(void)
   (int) runtime.tv_sec);
 }
 
-int bench_futex_hash(int argc, const char **argv,
-const char *prefix __maybe_unused)
+int bench_futex_hash(int argc, const char

[tip:perf/core] perf tools: Remove unused 'prefix' from builtin functions

[tip:perf/core] perf tools: Remove unused 'prefix' from builtin functions

2 matches

Site Navigation

Mail list logo

Footer information