From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id A515FC7EE29 for ; Fri, 9 Jun 2023 15:50:15 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=jMhfYJskuot/gacQ6lT3AUo3SA/voqWQteEjF1pRVEE=; b=Kb1NkyyTA03vYx BA9g2/gDgR+wGmZUnMWFcQA0OySq+xnz/nN1My+9hpIcNMbRt7SurtD2UZ58rnYozhlrbI3OtiVxL kFH2fjpo+fG7GppbiiSnbeR5rJq1o4ic/ECBB8Jz3kqYxLv1XmGI6dN3/O5ZIB/h6TXUfSLSE7bWw bqtCcxM4EdeJbgPNiZUUu3eM6xsYbrJkbeDHr/uKzaoZ3/ezooyDIXEZgTO0cQWzKoww7BHIzMpBY vw8ockAnZ76ZQCdgdfi9CUmHm0p/vIq4jnxYJoU0r8aC5ulcrC13MUBqHPJs65H6aUdqTOrZhWocF LFZJaJU4DLlr8PxuAcwg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1q7eN3-00DVeH-0x; Fri, 09 Jun 2023 15:49:49 +0000 Received: from desiato.infradead.org ([2001:8b0:10b:1:d65d:64ff:fe57:4e05]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1q7eN2-00DVdy-26 for linux-arm-kernel@bombadil.infradead.org; Fri, 09 Jun 2023 15:49:48 +0000 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=infradead.org; s=desiato.20200630; h=In-Reply-To:Content-Type:MIME-Version: References:Message-ID:Subject:Cc:To:From:Date:Sender:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description; bh=34/nRYXsKbW34VNfMmlR89irO8lg7u6XmwTJ4+VkuwA=; b=FIXgcu0kVJHLKpZCwh7rRH5KFc rmTe4LwUhwddEufoQrr7KUiDR6w4GA0mSpPLyGAREMOOpVRN66u2k4pPENkWl6R4cG/zp8n+N4VSQ ir9oD2LCf3IWyj7GB8ghRs7XlA0OpKTy5X198RX9OeSAu9GHizi3feeCWdio1A5C1mCTrkF35bGmQ fRnPghEe5uiPwNoKok6ctWOtz5xd20mWbZBmBNnBti4zYg8HgffgjzBd8Hs18bWxOSWqc5gxKkT4T +6tyv/kl/IoA57aq2AxEPvSlYljcyFN0b+iHWWPPWcrvabZp+Cc9AS1vhomh/MTqil5vayY2noNrO pFq/ZWSQ==; Received: from [187.19.173.165] (helo=quaco.ghostprotocols.net) by desiato.infradead.org with esmtpsa (Exim 4.96 #2 (Red Hat Linux)) id 1q7cwy-007ClK-1i; Fri, 09 Jun 2023 14:18:49 +0000 Received: by quaco.ghostprotocols.net (Postfix, from userid 1000) id 61ADA40692; Fri, 9 Jun 2023 11:18:46 -0300 (-03) Date: Fri, 9 Jun 2023 11:18:46 -0300 From: Arnaldo Carvalho de Melo To: Ian Rogers Cc: John Garry , Will Deacon , James Clark , Mike Leach , Leo Yan , Peter Zijlstra , Ingo Molnar , Mark Rutland , Alexander Shishkin , Jiri Olsa , Namhyung Kim , Adrian Hunter , Suzuki K Poulose , "Naveen N. Rao" , Kan Liang , German Gomez , Ali Saidi , Jing Zhang , Athira Rajeev , Miguel Ojeda , ye xingchen , Liam Howlett , Dmitrii Dolgov <9erthalion6@gmail.com>, Yang Jihong , K Prateek Nayak , Changbin Du , Ravi Bangoria , Sean Christopherson , Andi Kleen , "Steinar H. Gunderson" , Yuan Can , Brian Robbins , liuwenyu , Ivan Babrou , Fangrui Song , linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-perf-users@vger.kernel.org, coresight@lists.linaro.org Subject: Re: [PATCH v2 05/26] perf addr_location: Move to its own header Message-ID: References: <20230608232823.4027869-1-irogers@google.com> <20230608232823.4027869-6-irogers@google.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20230608232823.4027869-6-irogers@google.com> X-Url: http://acmel.wordpress.com X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Em Thu, Jun 08, 2023 at 04:28:02PM -0700, Ian Rogers escreveu: > addr_location is a common abstraction, move it into its own header and > source file in preparation for wider clean up. Thanks, applied. - Arnaldo > Signed-off-by: Ian Rogers > --- > tools/perf/util/Build | 1 + > tools/perf/util/addr_location.c | 16 ++++++++++++++++ > tools/perf/util/addr_location.h | 28 ++++++++++++++++++++++++++++ > tools/perf/util/event.c | 12 ------------ > tools/perf/util/symbol.h | 17 +---------------- > 5 files changed, 46 insertions(+), 28 deletions(-) > create mode 100644 tools/perf/util/addr_location.c > create mode 100644 tools/perf/util/addr_location.h > > diff --git a/tools/perf/util/Build b/tools/perf/util/Build > index c449741adf30..ff2fd1a36bb8 100644 > --- a/tools/perf/util/Build > +++ b/tools/perf/util/Build > @@ -1,4 +1,5 @@ > perf-y += arm64-frame-pointer-unwind-support.o > +perf-y += addr_location.o > perf-y += annotate.o > perf-y += block-info.o > perf-y += block-range.o > diff --git a/tools/perf/util/addr_location.c b/tools/perf/util/addr_location.c > new file mode 100644 > index 000000000000..c73fc2aa236c > --- /dev/null > +++ b/tools/perf/util/addr_location.c > @@ -0,0 +1,16 @@ > +// SPDX-License-Identifier: GPL-2.0 > +#include "addr_location.h" > +#include "map.h" > +#include "thread.h" > + > +/* > + * The preprocess_sample method will return with reference counts for the > + * in it, when done using (and perhaps getting ref counts if needing to > + * keep a pointer to one of those entries) it must be paired with > + * addr_location__put(), so that the refcounts can be decremented. > + */ > +void addr_location__put(struct addr_location *al) > +{ > + map__zput(al->map); > + thread__zput(al->thread); > +} > diff --git a/tools/perf/util/addr_location.h b/tools/perf/util/addr_location.h > new file mode 100644 > index 000000000000..7dfa7417c0fe > --- /dev/null > +++ b/tools/perf/util/addr_location.h > @@ -0,0 +1,28 @@ > +/* SPDX-License-Identifier: GPL-2.0 */ > +#ifndef __PERF_ADDR_LOCATION > +#define __PERF_ADDR_LOCATION 1 > + > +#include > + > +struct thread; > +struct maps; > +struct map; > +struct symbol; > + > +struct addr_location { > + struct thread *thread; > + struct maps *maps; > + struct map *map; > + struct symbol *sym; > + const char *srcline; > + u64 addr; > + char level; > + u8 filtered; > + u8 cpumode; > + s32 cpu; > + s32 socket; > +}; > + > +void addr_location__put(struct addr_location *al); > + > +#endif /* __PERF_ADDR_LOCATION */ > diff --git a/tools/perf/util/event.c b/tools/perf/util/event.c > index e1ce7cb5e421..6ee23145ee7e 100644 > --- a/tools/perf/util/event.c > +++ b/tools/perf/util/event.c > @@ -767,18 +767,6 @@ int machine__resolve(struct machine *machine, struct addr_location *al, > return 0; > } > > -/* > - * The preprocess_sample method will return with reference counts for the > - * in it, when done using (and perhaps getting ref counts if needing to > - * keep a pointer to one of those entries) it must be paired with > - * addr_location__put(), so that the refcounts can be decremented. > - */ > -void addr_location__put(struct addr_location *al) > -{ > - map__zput(al->map); > - thread__zput(al->thread); > -} > - > bool is_bts_event(struct perf_event_attr *attr) > { > return attr->type == PERF_TYPE_HARDWARE && > diff --git a/tools/perf/util/symbol.h b/tools/perf/util/symbol.h > index 7558735543c2..5ca8665dd2c1 100644 > --- a/tools/perf/util/symbol.h > +++ b/tools/perf/util/symbol.h > @@ -9,6 +9,7 @@ > #include > #include > #include > +#include "addr_location.h" > #include "path.h" > #include "symbol_conf.h" > #include "spark.h" > @@ -120,22 +121,6 @@ struct ref_reloc_sym { > u64 unrelocated_addr; > }; > > -struct addr_location { > - struct thread *thread; > - struct maps *maps; > - struct map *map; > - struct symbol *sym; > - const char *srcline; > - u64 addr; > - char level; > - u8 filtered; > - u8 cpumode; > - s32 cpu; > - s32 socket; > -}; > - > -void addr_location__put(struct addr_location *al); > - > int dso__load(struct dso *dso, struct map *map); > int dso__load_vmlinux(struct dso *dso, struct map *map, > const char *vmlinux, bool vmlinux_allocated); > -- > 2.41.0.162.gfafddb0af9-goog > -- - Arnaldo _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel