From 5631d100f98390f82383eb271b9f503400514b44 Mon Sep 17 00:00:00 2001 From: Jiri Olsa Date: Mon, 8 Feb 2021 21:08:47 +0100 Subject: [PATCH] perf daemon: Add base option Add a base option allowing the user to specify a base directory. It will have precedence over config file base definition coming in the following patches. Signed-off-by: Jiri Olsa Cc: Alexander Shishkin Cc: Alexei Budankov Cc: Ian Rogers Cc: Ingo Molnar Cc: Mark Rutland Cc: Michael Petlan Cc: Namhyung Kim Cc: Peter Zijlstra Link: https://lore.kernel.org/r/20210208200908.1019149-4-jolsa@kernel.org Signed-off-by: Arnaldo Carvalho de Melo --- tools/perf/Documentation/perf-daemon.txt | 4 ++++ tools/perf/builtin-daemon.c | 11 +++++++++++ 2 files changed, 15 insertions(+) diff --git a/tools/perf/Documentation/perf-daemon.txt b/tools/perf/Documentation/perf-daemon.txt index ba3f88510aee1..1a4158cd973ef 100644 --- a/tools/perf/Documentation/perf-daemon.txt +++ b/tools/perf/Documentation/perf-daemon.txt @@ -31,6 +31,10 @@ OPTIONS Config file path. If not provided, perf will check system and default locations (/etc/perfconfig, $HOME/.perfconfig). +--base=:: + Base directory path. Each daemon instance is running on top + of base directory. + All generic options are available also under commands. diff --git a/tools/perf/builtin-daemon.c b/tools/perf/builtin-daemon.c index 90b5a8ea9ddae..ce0373f453d67 100644 --- a/tools/perf/builtin-daemon.c +++ b/tools/perf/builtin-daemon.c @@ -6,6 +6,7 @@ #include #include #include +#include #include "builtin.h" #include "perf.h" #include "debug.h" @@ -15,6 +16,7 @@ struct daemon { const char *config; char *config_real; + const char *base_user; char *base; FILE *out; char perf[PATH_MAX]; @@ -38,10 +40,17 @@ static void sig_handler(int sig __maybe_unused) static void daemon__exit(struct daemon *daemon) { free(daemon->config_real); + free(daemon->base); } static int setup_config(struct daemon *daemon) { + if (daemon->base_user) { + daemon->base = strdup(daemon->base_user); + if (!daemon->base) + return -ENOMEM; + } + if (daemon->config) { char *real = realpath(daemon->config, NULL); @@ -104,6 +113,8 @@ int cmd_daemon(int argc, const char **argv) OPT_INCR('v', "verbose", &verbose, "be more verbose"), OPT_STRING(0, "config", &__daemon.config, "config file", "config file path"), + OPT_STRING(0, "base", &__daemon.base_user, + "directory", "base directory"), OPT_END() };