perf tools: Make has_kcore_dir() work also for guest kcore_dir
authorAdrian Hunter <adrian.hunter@intel.com>
Mon, 11 Jul 2022 09:32:04 +0000 (12:32 +0300)
committerArnaldo Carvalho de Melo <acme@redhat.com>
Wed, 20 Jul 2022 14:08:34 +0000 (11:08 -0300)
Copies of /proc/kallsyms, /proc/modules and an extract of /proc/kcore can
be stored in the perf.data output directory under the subdirectory named
kcore_dir. Guest machines will have their files also under subdirectories
beginning kcore_dir__ followed by the machine pid. Make has_kcore_dir()
return true also if there is a guest machine kcore_dir.

Signed-off-by: Adrian Hunter <adrian.hunter@intel.com>
Acked-by: Ian Rogers <irogers@google.com>
Cc: Andi Kleen <ak@linux.intel.com>
Cc: Jiri Olsa <jolsa@kernel.org>
Cc: Namhyung Kim <namhyung@kernel.org>
Cc: kvm@vger.kernel.org
Link: https://lore.kernel.org/r/20220711093218.10967-22-adrian.hunter@intel.com
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
tools/perf/util/data.c

index caabeac..9782ccb 100644 (file)
@@ -3,6 +3,7 @@
 #include <linux/kernel.h>
 #include <linux/string.h>
 #include <linux/zalloc.h>
+#include <linux/err.h>
 #include <sys/types.h>
 #include <sys/stat.h>
 #include <errno.h>
@@ -481,16 +482,21 @@ int perf_data__make_kcore_dir(struct perf_data *data, char *buf, size_t buf_sz)
 
 bool has_kcore_dir(const char *path)
 {
-       char *kcore_dir;
-       int ret;
-
-       if (asprintf(&kcore_dir, "%s/kcore_dir", path) < 0)
-               return false;
-
-       ret = access(kcore_dir, F_OK);
+       struct dirent *d = ERR_PTR(-EINVAL);
+       const char *name = "kcore_dir";
+       DIR *dir = opendir(path);
+       size_t n = strlen(name);
+       bool result = false;
+
+       if (dir) {
+               while (d && !result) {
+                       d = readdir(dir);
+                       result = d ? strncmp(d->d_name, name, n) : false;
+               }
+               closedir(dir);
+       }
 
-       free(kcore_dir);
-       return !ret;
+       return result;
 }
 
 char *perf_data__kallsyms_name(struct perf_data *data)