diff mbox series

[v6,42/47] perf dsos: Remove dsosaddnew

Message ID	20231207011722.1220634-43-irogers@google.com
State	New
Headers	Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.32 as permitted sender) client-ip=23.128.96.32; Date: Wed, 6 Dec 2023 17:17:16 -0800 In-Reply-To: <20231207011722.1220634-1-irogers@google.com> Message-Id: <20231207011722.1220634-43-irogers@google.com> Mime-Version: 1.0 References: <20231207011722.1220634-1-irogers@google.com> Subject: [PATCH v6 42/47] perf dsos: Remove __dsos__addnew From: Ian Rogers <irogers@google.com> To: Peter Zijlstra <peterz@infradead.org>, Ingo Molnar <mingo@redhat.com>, Arnaldo Carvalho de Melo <acme@kernel.org>, Mark Rutland <mark.rutland@arm.com>, Alexander Shishkin <alexander.shishkin@linux.intel.com>, Jiri Olsa <jolsa@kernel.org>, Namhyung Kim <namhyung@kernel.org>, Ian Rogers <irogers@google.com>, Adrian Hunter <adrian.hunter@intel.com>, Nick Terrell <terrelln@fb.com>, Kan Liang <kan.liang@linux.intel.com>, Andi Kleen <ak@linux.intel.com>, Kajol Jain <kjain@linux.ibm.com>, Athira Rajeev <atrajeev@linux.vnet.ibm.com>, Huacai Chen <chenhuacai@kernel.org>, Masami Hiramatsu <mhiramat@kernel.org>, Vincent Whitchurch <vincent.whitchurch@axis.com>, "Steinar H. Gunderson" <sesse@google.com>, Liam Howlett <liam.howlett@oracle.com>, Miguel Ojeda <ojeda@kernel.org>, Colin Ian King <colin.i.king@gmail.com>, Dmitrii Dolgov <9erthalion6@gmail.com>, Yang Jihong <yangjihong1@huawei.com>, Ming Wang <wangming01@loongson.cn>, James Clark <james.clark@arm.com>, K Prateek Nayak <kprateek.nayak@amd.com>, Sean Christopherson <seanjc@google.com>, Leo Yan <leo.yan@linaro.org>, Ravi Bangoria <ravi.bangoria@amd.com>, German Gomez <german.gomez@arm.com>, Changbin Du <changbin.du@huawei.com>, Paolo Bonzini <pbonzini@redhat.com>, Li Dong <lidong@vivo.com>, Sandipan Das <sandipan.das@amd.com>, liuwenyu <liuwenyu7@huawei.com>, linux-kernel@vger.kernel.org, linux-perf-users@vger.kernel.org, Guilherme Amadio <amadio@gentoo.org> Content-Type: text/plain; charset="UTF-8" Precedence: bulk
Series	maps/threads/dsos memory improvements and fixes \| [v6,00/47] maps/threads/dsos memory improvements and fixes [v6,01/47] perf map: Improve map/unmap parameter names [v6,02/47] perf maps: Add maps__for_each_map to iterate maps holding the lock [v6,03/47] perf events x86: Use function to add missing lock [v6,04/47] perf report: Use function to add missing maps lock [v6,05/47] perf tests: Use function to add missing maps lock [v6,06/47] perf machine: Use function to add missing maps lock [v6,07/47] perf probe-event: Use function to add missing maps lock [v6,08/47] perf symbol: Use function to add missing maps lock [v6,09/47] perf synthetic-events: Use function to add missing maps lock [v6,10/47] perf thread: Use function to add missing maps lock [v6,11/47] perf unwind: Use function to add missing maps lock [v6,12/47] perf vdso: Use function to add missing maps lock [v6,13/47] perf maps: Reduce scope of maps__for_each_entry [v6,14/47] perf maps: Add remove maps function to remove a map based on callback [v6,15/47] perf debug: Expose debug file [v6,16/47] perf maps: Refactor maps__fixup_overlappings [v6,17/47] perf maps: Do simple merge if given map doesn't overlap [v6,18/47] perf maps: Rename clone to copy from [v6,19/47] perf maps: Add maps__load_first [v6,20/47] perf maps: Add find next entry to give entry after the given map [v6,21/47] perf maps: Reduce scope of map_rb_node and maps internals [v6,22/47] perf maps: Fix up overlaps during fixup_end [v6,23/47] perf maps: Switch from rbtree to lazily sorted array for addresses [v6,24/47] perf maps: Get map before returning in maps__find [v6,25/47] perf maps: Get map before returning in maps__find_by_name [v6,26/47] perf maps: Get map before returning in maps__find_next_entry [v6,27/47] perf maps: Hide maps internals [v6,28/47] perf maps: Locking tidy up of nr_maps [v6,29/47] perf dso: Reorder variables to save space in struct dso [v6,30/47] perf report: Sort child tasks by tid [v6,31/47] perf trace: Ignore thread hashing in summary [v6,32/47] perf machine: Move fprintf to for_each loop and a callback [v6,33/47] perf threads: Move threads to its own files [v6,34/47] perf threads: Switch from rbtree to hashmap [v6,35/47] perf threads: Reduce table size from 256 to 8 [v6,36/47] perf dsos: Attempt to better abstract dsos internals [v6,37/47] perf dsos: Tidy reference counting and locking [v6,38/47] perf dsos: Add dsos__for_each_dso [v6,39/47] perf dso: Move dso functions out of dsos [v6,40/47] perf dsos: Switch more loops to dsos__for_each_dso [v6,41/47] perf dsos: Switch backing storage to array from rbtree/list [v6,42/47] perf dsos: Remove __dsos__addnew [v6,43/47] perf dsos: Remove __dsos__findnew_link_by_longname_id [v6,44/47] perf dsos: Switch hand code to bsearch [v6,45/47] perf dso: Add reference count checking and accessor functions [v6,46/47] perf dso: Reference counting related fixes [v6,47/47] perf dso: Use container_of to avoid a pointer in dso_data

Commit Message

Ian Rogers Dec. 7, 2023, 1:17 a.m. UTC

  Function no longer used so remove.

Signed-off-by: Ian Rogers <irogers@google.com>
---
 tools/perf/util/dsos.c | 5 -----
 tools/perf/util/dsos.h | 1 -
 2 files changed, 6 deletions(-)

diff mbox series

Patch

diff --git a/tools/perf/util/dsos.c b/tools/perf/util/dsos.c
index cfc10e1a6802..1495ab1cd7a0 100644
--- a/tools/perf/util/dsos.c
+++ b/tools/perf/util/dsos.c
@@ -341,11 +341,6 @@  static struct dso *__dsos__addnew_id(struct dsos *dsos, const char *name, struct
 	return dso;
 }
 
-struct dso *__dsos__addnew(struct dsos *dsos, const char *name)
-{
-	return __dsos__addnew_id(dsos, name, NULL);
-}
-
 static struct dso *__dsos__findnew_id(struct dsos *dsos, const char *name, struct dso_id *id)
 {
 	struct dso *dso = __dsos__find_id(dsos, name, id, false, /*write_locked=*/true);
diff --git a/tools/perf/util/dsos.h b/tools/perf/util/dsos.h
index c1b3979ad4bd..d1497b11d64c 100644
--- a/tools/perf/util/dsos.h
+++ b/tools/perf/util/dsos.h
@@ -30,7 +30,6 @@  void dsos__exit(struct dsos *dsos);
 
 int __dsos__add(struct dsos *dsos, struct dso *dso);
 int dsos__add(struct dsos *dsos, struct dso *dso);
-struct dso *__dsos__addnew(struct dsos *dsos, const char *name);
 struct dso *dsos__find(struct dsos *dsos, const char *name, bool cmp_short);
 
 struct dso *dsos__findnew_id(struct dsos *dsos, const char *name, struct dso_id *id);

[v6,42/47] perf dsos: Remove __dsos__addnew

Commit Message

Patch

[v6,42/47] perf dsos: Remove dsosaddnew