Commit 46113a54 authored by Josh Poimboeuf's avatar Josh Poimboeuf Committed by Arnaldo Carvalho de Melo

perf tools: Remove 'perf' from subcmd function and variable names

In preparation for moving exec_cmd.c and run-command.c out of perf and
into a library, remove 'perf' from all the symbol names.
Signed-off-by: default avatarJosh Poimboeuf <jpoimboe@redhat.com>
Cc: Jiri Olsa <jolsa@redhat.com>
Cc: Namhyung Kim <namhyung@kernel.org>
Cc: Peter Zijlstra <peterz@infradead.org>
Link: http://lkml.kernel.org/r/bc3ee82b40b8f396b644fa49e0f7260ce442635b.1450193761.git.jpoimboe@redhat.comSigned-off-by: default avatarArnaldo Carvalho de Melo <acme@redhat.com>
parent 901421a5
...@@ -407,7 +407,7 @@ static int get_html_page_path(struct strbuf *page_path, const char *page) ...@@ -407,7 +407,7 @@ static int get_html_page_path(struct strbuf *page_path, const char *page)
#ifndef open_html #ifndef open_html
static void open_html(const char *path) static void open_html(const char *path)
{ {
execl_perf_cmd("web--browse", "-c", "help.browser", path, NULL); execl_cmd("web--browse", "-c", "help.browser", path, NULL);
} }
#endif #endif
......
...@@ -1408,7 +1408,7 @@ static int list_available_scripts(const struct option *opt __maybe_unused, ...@@ -1408,7 +1408,7 @@ static int list_available_scripts(const struct option *opt __maybe_unused,
char first_half[BUFSIZ]; char first_half[BUFSIZ];
char *script_root; char *script_root;
snprintf(scripts_path, MAXPATHLEN, "%s/scripts", perf_exec_path()); snprintf(scripts_path, MAXPATHLEN, "%s/scripts", get_argv_exec_path());
scripts_dir = opendir(scripts_path); scripts_dir = opendir(scripts_path);
if (!scripts_dir) if (!scripts_dir)
...@@ -1529,7 +1529,7 @@ int find_scripts(char **scripts_array, char **scripts_path_array) ...@@ -1529,7 +1529,7 @@ int find_scripts(char **scripts_array, char **scripts_path_array)
if (!session) if (!session)
return -1; return -1;
snprintf(scripts_path, MAXPATHLEN, "%s/scripts", perf_exec_path()); snprintf(scripts_path, MAXPATHLEN, "%s/scripts", get_argv_exec_path());
scripts_dir = opendir(scripts_path); scripts_dir = opendir(scripts_path);
if (!scripts_dir) { if (!scripts_dir) {
...@@ -1587,7 +1587,7 @@ static char *get_script_path(const char *script_root, const char *suffix) ...@@ -1587,7 +1587,7 @@ static char *get_script_path(const char *script_root, const char *suffix)
char lang_path[MAXPATHLEN]; char lang_path[MAXPATHLEN];
char *__script_root; char *__script_root;
snprintf(scripts_path, MAXPATHLEN, "%s/scripts", perf_exec_path()); snprintf(scripts_path, MAXPATHLEN, "%s/scripts", get_argv_exec_path());
scripts_dir = opendir(scripts_path); scripts_dir = opendir(scripts_path);
if (!scripts_dir) if (!scripts_dir)
...@@ -1823,7 +1823,7 @@ int cmd_script(int argc, const char **argv, const char *prefix __maybe_unused) ...@@ -1823,7 +1823,7 @@ int cmd_script(int argc, const char **argv, const char *prefix __maybe_unused)
scripting_max_stack = itrace_synth_opts.callchain_sz; scripting_max_stack = itrace_synth_opts.callchain_sz;
/* make sure PERF_EXEC_PATH is set for scripts */ /* make sure PERF_EXEC_PATH is set for scripts */
perf_set_argv_exec_path(perf_exec_path()); set_argv_exec_path(get_argv_exec_path());
if (argc && !script_name && !rec_script_path && !rep_script_path) { if (argc && !script_name && !rec_script_path && !rep_script_path) {
int live_pipe[2]; int live_pipe[2];
......
...@@ -183,9 +183,9 @@ static int handle_options(const char ***argv, int *argc, int *envchanged) ...@@ -183,9 +183,9 @@ static int handle_options(const char ***argv, int *argc, int *envchanged)
if (!prefixcmp(cmd, CMD_EXEC_PATH)) { if (!prefixcmp(cmd, CMD_EXEC_PATH)) {
cmd += strlen(CMD_EXEC_PATH); cmd += strlen(CMD_EXEC_PATH);
if (*cmd == '=') if (*cmd == '=')
perf_set_argv_exec_path(cmd + 1); set_argv_exec_path(cmd + 1);
else { else {
puts(perf_exec_path()); puts(get_argv_exec_path());
exit(0); exit(0);
} }
} else if (!strcmp(cmd, "--html-path")) { } else if (!strcmp(cmd, "--html-path")) {
...@@ -538,7 +538,7 @@ int main(int argc, const char **argv) ...@@ -538,7 +538,7 @@ int main(int argc, const char **argv)
page_size = sysconf(_SC_PAGE_SIZE); page_size = sysconf(_SC_PAGE_SIZE);
cacheline_size = sysconf(_SC_LEVEL1_DCACHE_LINESIZE); cacheline_size = sysconf(_SC_LEVEL1_DCACHE_LINESIZE);
cmd = perf_extract_argv0_path(argv[0]); cmd = extract_argv0_path(argv[0]);
if (!cmd) if (!cmd)
cmd = "perf-help"; cmd = "perf-help";
......
...@@ -164,7 +164,7 @@ int test__attr(int subtest __maybe_unused) ...@@ -164,7 +164,7 @@ int test__attr(int subtest __maybe_unused)
return run_dir("./tests", "./perf"); return run_dir("./tests", "./perf");
/* Then installed path. */ /* Then installed path. */
snprintf(path_dir, PATH_MAX, "%s/tests", perf_exec_path()); snprintf(path_dir, PATH_MAX, "%s/tests", get_argv_exec_path());
snprintf(path_perf, PATH_MAX, "%s/perf", BINDIR); snprintf(path_perf, PATH_MAX, "%s/perf", BINDIR);
if (!lstat(path_dir, &st) && if (!lstat(path_dir, &st) &&
......
...@@ -32,7 +32,7 @@ char *system_path(const char *path) ...@@ -32,7 +32,7 @@ char *system_path(const char *path)
return buf; return buf;
} }
const char *perf_extract_argv0_path(const char *argv0) const char *extract_argv0_path(const char *argv0)
{ {
const char *slash; const char *slash;
...@@ -51,7 +51,7 @@ const char *perf_extract_argv0_path(const char *argv0) ...@@ -51,7 +51,7 @@ const char *perf_extract_argv0_path(const char *argv0)
return argv0; return argv0;
} }
void perf_set_argv_exec_path(const char *exec_path) void set_argv_exec_path(const char *exec_path)
{ {
argv_exec_path = exec_path; argv_exec_path = exec_path;
/* /*
...@@ -61,8 +61,8 @@ void perf_set_argv_exec_path(const char *exec_path) ...@@ -61,8 +61,8 @@ void perf_set_argv_exec_path(const char *exec_path)
} }
/* Returns the highest-priority, location to look for perf programs. */ /* Returns the highest-priority location to look for subprograms. */
char *perf_exec_path(void) char *get_argv_exec_path(void)
{ {
char *env; char *env;
...@@ -92,7 +92,7 @@ void setup_path(void) ...@@ -92,7 +92,7 @@ void setup_path(void)
{ {
const char *old_path = getenv("PATH"); const char *old_path = getenv("PATH");
char *new_path = NULL; char *new_path = NULL;
char *tmp = perf_exec_path(); char *tmp = get_argv_exec_path();
add_path(&new_path, tmp); add_path(&new_path, tmp);
add_path(&new_path, argv0_path); add_path(&new_path, argv0_path);
...@@ -108,7 +108,7 @@ void setup_path(void) ...@@ -108,7 +108,7 @@ void setup_path(void)
free(new_path); free(new_path);
} }
static const char **prepare_perf_cmd(const char **argv) static const char **prepare_exec_cmd(const char **argv)
{ {
int argc; int argc;
const char **nargv; const char **nargv;
...@@ -124,8 +124,8 @@ static const char **prepare_perf_cmd(const char **argv) ...@@ -124,8 +124,8 @@ static const char **prepare_perf_cmd(const char **argv)
return nargv; return nargv;
} }
int execv_perf_cmd(const char **argv) { int execv_cmd(const char **argv) {
const char **nargv = prepare_perf_cmd(argv); const char **nargv = prepare_exec_cmd(argv);
/* execvp() can only ever return if it fails */ /* execvp() can only ever return if it fails */
execvp(subcmd_config.exec_name, (char **)nargv); execvp(subcmd_config.exec_name, (char **)nargv);
...@@ -135,7 +135,7 @@ int execv_perf_cmd(const char **argv) { ...@@ -135,7 +135,7 @@ int execv_perf_cmd(const char **argv) {
} }
int execl_perf_cmd(const char *cmd,...) int execl_cmd(const char *cmd,...)
{ {
int argc; int argc;
const char *argv[MAX_ARGS + 1]; const char *argv[MAX_ARGS + 1];
...@@ -155,5 +155,5 @@ int execl_perf_cmd(const char *cmd,...) ...@@ -155,5 +155,5 @@ int execl_perf_cmd(const char *cmd,...)
return error("too many args to run %s", cmd); return error("too many args to run %s", cmd);
argv[argc] = NULL; argv[argc] = NULL;
return execv_perf_cmd(argv); return execv_cmd(argv);
} }
...@@ -4,13 +4,13 @@ ...@@ -4,13 +4,13 @@
extern void exec_cmd_init(const char *exec_name, const char *prefix, extern void exec_cmd_init(const char *exec_name, const char *prefix,
const char *exec_path, const char *exec_path_env); const char *exec_path, const char *exec_path_env);
extern void perf_set_argv_exec_path(const char *exec_path); extern void set_argv_exec_path(const char *exec_path);
extern const char *perf_extract_argv0_path(const char *path); extern const char *extract_argv0_path(const char *path);
extern void setup_path(void); extern void setup_path(void);
extern int execv_perf_cmd(const char **argv); /* NULL terminated */ extern int execv_cmd(const char **argv); /* NULL terminated */
extern int execl_perf_cmd(const char *cmd, ...); extern int execl_cmd(const char *cmd, ...);
/* perf_exec_path and system_path return malloc'd string, caller must free it */ /* get_argv_exec_path and system_path return malloc'd string, caller must free it */
extern char *perf_exec_path(void); extern char *get_argv_exec_path(void);
extern char *system_path(const char *path); extern char *system_path(const char *path);
#endif /* __PERF_EXEC_CMD_H */ #endif /* __PERF_EXEC_CMD_H */
...@@ -155,7 +155,7 @@ void load_command_list(const char *prefix, ...@@ -155,7 +155,7 @@ void load_command_list(const char *prefix,
struct cmdnames *other_cmds) struct cmdnames *other_cmds)
{ {
const char *env_path = getenv("PATH"); const char *env_path = getenv("PATH");
char *exec_path = perf_exec_path(); char *exec_path = get_argv_exec_path();
if (exec_path) { if (exec_path) {
list_commands_in_dir(main_cmds, exec_path, prefix); list_commands_in_dir(main_cmds, exec_path, prefix);
...@@ -200,7 +200,7 @@ void list_commands(const char *title, struct cmdnames *main_cmds, ...@@ -200,7 +200,7 @@ void list_commands(const char *title, struct cmdnames *main_cmds,
longest = other_cmds->names[i]->len; longest = other_cmds->names[i]->len;
if (main_cmds->cnt) { if (main_cmds->cnt) {
char *exec_path = perf_exec_path(); char *exec_path = get_argv_exec_path();
printf("available %s in '%s'\n", title, exec_path); printf("available %s in '%s'\n", title, exec_path);
printf("----------------"); printf("----------------");
mput_char('-', strlen(title) + strlen(exec_path)); mput_char('-', strlen(title) + strlen(exec_path));
......
...@@ -112,8 +112,8 @@ int start_command(struct child_process *cmd) ...@@ -112,8 +112,8 @@ int start_command(struct child_process *cmd)
} }
if (cmd->preexec_cb) if (cmd->preexec_cb)
cmd->preexec_cb(); cmd->preexec_cb();
if (cmd->perf_cmd) { if (cmd->exec_cmd) {
execv_perf_cmd(cmd->argv); execv_cmd(cmd->argv);
} else { } else {
execvp(cmd->argv[0], (char *const*) cmd->argv); execvp(cmd->argv[0], (char *const*) cmd->argv);
} }
...@@ -207,7 +207,7 @@ static void prepare_run_command_v_opt(struct child_process *cmd, ...@@ -207,7 +207,7 @@ static void prepare_run_command_v_opt(struct child_process *cmd,
memset(cmd, 0, sizeof(*cmd)); memset(cmd, 0, sizeof(*cmd));
cmd->argv = argv; cmd->argv = argv;
cmd->no_stdin = opt & RUN_COMMAND_NO_STDIN ? 1 : 0; cmd->no_stdin = opt & RUN_COMMAND_NO_STDIN ? 1 : 0;
cmd->perf_cmd = opt & RUN_PERF_CMD ? 1 : 0; cmd->exec_cmd = opt & RUN_EXEC_CMD ? 1 : 0;
cmd->stdout_to_stderr = opt & RUN_COMMAND_STDOUT_TO_STDERR ? 1 : 0; cmd->stdout_to_stderr = opt & RUN_COMMAND_STDOUT_TO_STDERR ? 1 : 0;
} }
......
...@@ -41,7 +41,7 @@ struct child_process { ...@@ -41,7 +41,7 @@ struct child_process {
unsigned no_stdin:1; unsigned no_stdin:1;
unsigned no_stdout:1; unsigned no_stdout:1;
unsigned no_stderr:1; unsigned no_stderr:1;
unsigned perf_cmd:1; /* if this is to be perf sub-command */ unsigned exec_cmd:1; /* if this is to be external sub-command */
unsigned stdout_to_stderr:1; unsigned stdout_to_stderr:1;
void (*preexec_cb)(void); void (*preexec_cb)(void);
}; };
...@@ -51,7 +51,7 @@ int finish_command(struct child_process *); ...@@ -51,7 +51,7 @@ int finish_command(struct child_process *);
int run_command(struct child_process *); int run_command(struct child_process *);
#define RUN_COMMAND_NO_STDIN 1 #define RUN_COMMAND_NO_STDIN 1
#define RUN_PERF_CMD 2 /*If this is to be perf sub-command */ #define RUN_EXEC_CMD 2 /*If this is to be external sub-command */
#define RUN_COMMAND_STDOUT_TO_STDERR 4 #define RUN_COMMAND_STDOUT_TO_STDERR 4
int run_command_v_opt(const char **argv, int opt); int run_command_v_opt(const char **argv, int opt);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment