From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-alma10-1.taild15c8.ts.net [100.103.45.18]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 353373CB2DC; Tue, 29 Sep 2026 01:11:51 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=100.103.45.18 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1790644312; cv=none; b=OJa45YFbEltkvHA3nI4lwb6caqxInWJhMbd+fJgZNRsEozEodG+BfWFd6+dENSzM+bFgpMCUrXZPunbX54DhydARiFVZvD0E/BiKIr+o1OTYQC+yrBXXNUxGuOXMTd8h7YO45ZCw/vyoCjPbTPKR6XY1nVzgt7Pz7792c2NPY+U= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1790644312; c=relaxed/simple; bh=CRtU+1k243ibQLHq8l0S+PRq0d+fGOayydRrAGqLNyc=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=Q1JO3VxxaQVy5GFo+vaz+Xafq9tcdm9U10uFmUCbxeaRnA67zanUcL6taJApLrgbaH1bGl7RvVflPPFUzRCBCEenN6xtFRMbMFxsEN9jifIqc5w6hTzEgj+LAf/tuK4lTB47HiXemS1A/tIyjjr0TrU0LGyta0qpsi0mVo8S4Jo= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=gb3YYXqw; arc=none smtp.client-ip=100.103.45.18 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="gb3YYXqw" Received: by smtp.kernel.org (Postfix) with ESMTPSA id AD1AF1F00898; Tue, 29 Sep 2026 01:11:50 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=kernel.org; s=k20260515; t=1790644311; bh=mspPPPOReD501chwfe0L6MMJZzvJs0Tlpc4u7exl5ts=; h=From:To:Cc:Subject:Date:In-Reply-To:References; b=gb3YYXqwTUeWPE9gfif4Rf32oJPl741dm3ijDBXTFPtcJdv08nRuW5vMgpDutGgH6 zknaLDZwAvuuBSvUdGLlcB1369xffRzzLQkta4FTHNXVXmbcL8tM2M138cBuhiKFrO Bnb2gkyvnmwwJUy97BwPoi02pes5mnQ/V0fUnPLYi84lO6bkYXUA9tFPCCevUmd6r5 4dX2jN5qcvfKTHtI2UPD6sq/JQ1Xlhsy0Lc24rW2ZY+f9zHIQKj0f6q5ihkTu8dynH PanhxWBfI5tYmXEiRup2zNYDWlKmcHrCKWGw8cjRdKxTjrNEVJw/wSES/23odehtpR NcF5tmrs1t0VQ== From: Namhyung Kim To: Arnaldo Carvalho de Melo , Ian Rogers , James Clark Cc: Jiri Olsa , Adrian Hunter , Peter Zijlstra , Ingo Molnar , LKML , linux-perf-users@vger.kernel.org, Masami Hiramatsu , Arnaldo Carvalho de Melo Subject: [PATCH v2 4/8] perf tools: Export dso__get_filename() with type argument Date: Mon, 28 Sep 2026 18:11:35 -0700 Message-ID: <20260929011139.53242-5-namhyung@kernel.org> X-Mailer: git-send-email 2.55.0 In-Reply-To: <20260929011139.53242-1-namhyung@kernel.org> References: <20260929011139.53242-1-namhyung@kernel.org> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit The dso__get_filename() returns a new string of filepath for the given binary type. It handles chroot and decompressed kernel modules so that callers can read the file contents. Thus it'd be better to use it to check available binary types. Add a type argument to handle a different path for symbol or debug info. Reviewed-by: Arnaldo Carvalho de Melo Signed-off-by: Namhyung Kim --- tools/perf/util/debuginfo.c | 17 +++++++++++------ tools/perf/util/dso.c | 29 ++++++++++++++++++----------- tools/perf/util/dso.h | 2 ++ 3 files changed, 31 insertions(+), 17 deletions(-) diff --git a/tools/perf/util/debuginfo.c b/tools/perf/util/debuginfo.c index 2669d5f5ee6121d4..38e9299765182bd7 100644 --- a/tools/perf/util/debuginfo.c +++ b/tools/perf/util/debuginfo.c @@ -93,10 +93,11 @@ static struct debuginfo *__debuginfo__new(const char *path) struct debuginfo *debuginfo__new(const char *path) { - char buf[PATH_MAX], nil = '\0'; struct dso *dso; - struct debuginfo *dinfo = NULL; - struct build_id bid = { .size = 0}; + struct debuginfo *dinfo; + struct build_id bid = { .size = 0 }; + char *filename; + bool decomp = false; /* Try to open distro debuginfo files */ dso = dso__new(path); @@ -112,11 +113,15 @@ struct debuginfo *debuginfo__new(const char *path) dso__set_build_id(dso, &bid); dso__find_dbginfo_type(dso); - if (dso__read_binary_type_filename(dso, dso__dbginfo_type(dso), &nil, - buf, PATH_MAX) < 0) + filename = dso__get_filename(dso, "", &decomp, dso__dbginfo_type(dso)); + if (filename == NULL) return NULL; - dinfo = __debuginfo__new(buf); + dinfo = __debuginfo__new(filename); + + if (decomp) + unlink(filename); + free(filename); dso__put(dso); return dinfo; diff --git a/tools/perf/util/dso.c b/tools/perf/util/dso.c index ceebb32d0a48a96f..7fd6b64583d8b61a 100644 --- a/tools/perf/util/dso.c +++ b/tools/perf/util/dso.c @@ -590,8 +590,8 @@ char *dso__filename_with_chroot(const struct dso *dso, const char *filename) return filename_with_chroot(nsinfo__pid(dso__nsinfo_const(dso)), filename); } -static char *dso__get_filename(struct dso *dso, const char *root_dir, - bool *decomp) +char *dso__get_filename(struct dso *dso, const char *root_dir, bool *decomp, + enum dso_binary_type type) { char *name = malloc(PATH_MAX); @@ -600,8 +600,7 @@ static char *dso__get_filename(struct dso *dso, const char *root_dir, if (name == NULL) return NULL; - if (dso__read_binary_type_filename(dso, dso__binary_type(dso), - root_dir, name, PATH_MAX)) + if (dso__read_binary_type_filename(dso, type, root_dir, name, PATH_MAX)) goto out; if (!is_regular_file(name)) { @@ -671,7 +670,8 @@ static int __open_dso(struct dso *dso, struct machine *machine) mutex_lock(dso__lock(dso)); - name = dso__get_filename(dso, machine ? machine->root_dir : "", &decomp); + name = dso__get_filename(dso, machine ? machine->root_dir : "", &decomp, + dso__binary_type(dso)); if (name) { fd = do_open(name); } else { @@ -2087,7 +2087,7 @@ struct debuginfo *dso__debuginfo(struct dso *dso) mutex_lock(dso__lock(dso)); - name = dso__get_filename(dso, "", &decomp); + name = dso__get_filename(dso, "", &decomp, dso__dbginfo_type(dso)); if (name) dinfo = debuginfo__new(name); @@ -2121,18 +2121,25 @@ void dso__find_dbginfo_type(struct dso *dso) DSO_BINARY_TYPE__NOT_FOUND, }; const enum dso_binary_type *type; - char buf[PATH_MAX]; + char *path; + bool found, decomp = false; if (dso__dbginfo_type(dso) != DSO_BINARY_TYPE__NOT_FOUND) return; for (type = dbginfo_types; *type != DSO_BINARY_TYPE__NOT_FOUND; type++) { - if (dso__read_binary_type_filename(dso, *type, "", - buf, PATH_MAX) < 0) + path = dso__get_filename(dso, "", &decomp, *type); + if (path == NULL) continue; - if (filename__has_section(buf, ".debug_info") || - filename__has_section(buf, ".zdebug_info")) + found = filename__has_section(path, ".debug_info") || + filename__has_section(path, ".zdebug_info"); + + if (decomp) + unlink(path); + free(path); + + if (found) break; } diff --git a/tools/perf/util/dso.h b/tools/perf/util/dso.h index a152957322f6b31c..94cc7ea89ceb5bd9 100644 --- a/tools/perf/util/dso.h +++ b/tools/perf/util/dso.h @@ -841,6 +841,8 @@ int dso__kernel_module_get_build_id(struct dso *dso, const char *root_dir); char dso__symtab_origin(const struct dso *dso); int dso__read_binary_type_filename(const struct dso *dso, enum dso_binary_type type, const char *root_dir, char *filename, size_t size); +char *dso__get_filename(struct dso *dso, const char *root_dir, bool *decomp, + enum dso_binary_type type); bool is_kernel_module(const char *pathname, int cpumode); bool dso__needs_decompress(struct dso *dso); int dso__decompress_kmodule_fd(struct dso *dso, const char *name); -- 2.55.0