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 X-Spam-Level: X-Spam-Status: No, score=-14.5 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER, INCLUDES_PATCH,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 78066C4338F for ; Sat, 31 Jul 2021 06:39:21 +0000 (UTC) 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 mail.kernel.org (Postfix) with ESMTPS id 479A760FE7 for ; Sat, 31 Jul 2021 06:39:21 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.1 mail.kernel.org 479A760FE7 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=linaro.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=lists.infradead.org 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=QfHktq+SncA4j8sYhA4iVcANCi+2po5Lt3jHPMRGU8A=; b=Q00/25XhR8OEq6 JSPaxm8wztd4ZTk1Z24u8VWK/kOvBZCVLiNM8RW7qyytvkO7ujXjprYAdMi7bi69QOUpL4dAlhPWF 3Eb6kEpBsCl36MrUoq6aoRmYQBxudpyn3gTQOJRjHO58beNiKgVGtAzo/lDAJojrsePGAeYieLsG8 0ULRNT6yq00QAp5d11CR1Uui997uysb7x9qgPf9M31HtL8P4+cSWvCMS2IKiCyrvdthHXVReqJxP8 N7CexRfgICSG8YofyxVAfPsLDudWkwJRSZWpvINpbaB51z8eQKtJujwveo3ywE0LLS+55eqwiBpe/ 38a/Cb9S1UBeNxSPy7bQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1m9id1-00B4JC-UW; Sat, 31 Jul 2021 06:37:48 +0000 Received: from mail-pl1-x62f.google.com ([2607:f8b0:4864:20::62f]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1m9icx-00B4Ik-GS for linux-arm-kernel@lists.infradead.org; Sat, 31 Jul 2021 06:37:45 +0000 Received: by mail-pl1-x62f.google.com with SMTP id t21so13627144plr.13 for ; Fri, 30 Jul 2021 23:37:43 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to; bh=MMWnZBP/iiPeKj/j1MYwfBpBFKoHSl81Cqk/jEOD+dk=; b=Ex6B0uTyBCucaWwm4DkcyqTKLU5vqD52HVobkEzFBK/LreWdw3g5cbIDphJJ7rHRrI Bl5z+Fa3yETB01dx151ljz+vzjaNtGg8s4M67q2TPpjI1ZIdl+ZJl/QQ/ZGcvgIqXaxw BhKFPF5SnUIEdnC0ZOHQzKSn93KPi+32J96+xYBUl9Kf6ExlsAzOoaBraTeR7sHgyGmK aA8eyLJWDMrdRkcnvZB90Ts6Z/wvf7uiDqCeo/WIpdecmPKfJZvpT5LAmE6JUtLxMHat Q0eVQJVukaCBH+xfrvrByXA3aF6gh5uWHh7LZi9HhSZFbvHme5wmwj7jmJmu03AmFnub xkBw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to; bh=MMWnZBP/iiPeKj/j1MYwfBpBFKoHSl81Cqk/jEOD+dk=; b=Gatm091PS2ZStEtftXuHGHGAIKIIYitKcuCbs3beMbuQsgtJUpNAxcTufL3fzVCGWd Rpe9tb+9l4Rp2J0KuG1VDl7CwDf5CB8DWim11wqmtNccIokdF9HReguW26mrDOzJC+ZJ w90Rx8D0OmByIf5isv9zscT0RlVMl/FbSXEI6NjiXRDf9ZnDifoyU6u9N3BEfMcQeiFp TUWYqx3QCwea/nADqczpnPduOI8qBl2FANonRJ4qQLYrorKT3sQ+qg24FNp451pRNskq s4RhhOpjz2mkp6e1vwlUnEe3nqk4TaInMRm5axxffys98E9DmNTMl3gTWXUetM903A1b Bmyw== X-Gm-Message-State: AOAM5302NM6mSocoapd/b/m8cTCSERnI00Dgw2hE8NPEzq+O/t9+s1o9 Je2exah5j4o9OU7I/IyVvucnEA== X-Google-Smtp-Source: ABdhPJyZDJkISj/cYTjZwpsaiftbjyQdqIO8oupnyxMgOgBsQkcrLbSAuww67M64iJplMutf07cKwA== X-Received: by 2002:a62:52cc:0:b029:3b1:1f36:8bbc with SMTP id g195-20020a6252cc0000b02903b11f368bbcmr4057867pfb.16.1627713462550; Fri, 30 Jul 2021 23:37:42 -0700 (PDT) Received: from leoy-ThinkPad-X240s ([204.124.181.43]) by smtp.gmail.com with ESMTPSA id f4sm5359079pgs.3.2021.07.30.23.37.34 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 30 Jul 2021 23:37:41 -0700 (PDT) Date: Sat, 31 Jul 2021 14:37:31 +0800 From: Leo Yan To: Mike Leach Cc: James Clark , Arnaldo Carvalho de Melo , Mathieu Poirier , Coresight ML , Al Grant , "Suzuki K. Poulose" , Anshuman Khandual , John Garry , Will Deacon , Mark Rutland , Alexander Shishkin , Jiri Olsa , Namhyung Kim , linux-arm-kernel , Linux Kernel Mailing List , linux-perf-users@vger.kernel.org Subject: Re: [PATCH 3/6] perf cs-etm: Save TRCDEVARCH register Message-ID: <20210731063731.GC7437@leoy-ThinkPad-X240s> References: <20210721090706.21523-1-james.clark@arm.com> <20210721090706.21523-4-james.clark@arm.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20210730_233743_614210_60A2288B X-CRM114-Status: GOOD ( 41.81 ) 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 On Wed, Jul 21, 2021 at 10:48:25AM +0100, Mike Leach wrote: > HI James, > > On Wed, 21 Jul 2021 at 10:07, James Clark wrote: > > > > Now that the metadata has a length field we can add extra registers > > without breaking any previous versions of perf. > > > > Save the TRCDEVARCH register so that it can be used to configure the ETE > > decoder in the next commit. If the sysfs file doesn't exist then 0 will > > be saved which is an impossible register value and can also be used to > > signify that the file couldn't be read. > > > > Signed-off-by: James Clark > > --- > > tools/perf/arch/arm/util/cs-etm.c | 13 ++++++++++++- > > tools/perf/util/cs-etm.c | 1 + > > tools/perf/util/cs-etm.h | 5 +++-- > > 3 files changed, 16 insertions(+), 3 deletions(-) > > > > diff --git a/tools/perf/arch/arm/util/cs-etm.c b/tools/perf/arch/arm/util/cs-etm.c > > index 85168d87b2d7..65a863bdf5cc 100644 > > --- a/tools/perf/arch/arm/util/cs-etm.c > > +++ b/tools/perf/arch/arm/util/cs-etm.c > > @@ -53,6 +53,7 @@ static const char *metadata_etmv4_ro[CS_ETMV4_PRIV_MAX] = { > > [CS_ETMV4_TRCIDR2] = "trcidr/trcidr2", > > [CS_ETMV4_TRCIDR8] = "trcidr/trcidr8", > > [CS_ETMV4_TRCAUTHSTATUS] = "mgmt/trcauthstatus", > > + [CS_ETE_TRCDEVARCH] = "mgmt/trcdevarch" ETMv4 supports TRCDEVARCH, so I think it's good to use the naming "CS_ETMV4_TRCDEVARCH"? > > }; > > > > static bool cs_etm_is_etmv4(struct auxtrace_record *itr, int cpu); > > @@ -73,7 +74,7 @@ static int cs_etm_set_context_id(struct auxtrace_record *itr, > > if (!cs_etm_is_etmv4(itr, cpu)) > > goto out; > > > > - /* Get a handle on TRCIRD2 */ > > + /* Get a handle on TRCIDR2 */ This is typo fixing; it's irrelevant to the topic in this patch, it's good to use a separate patch for the typo fixing. > > snprintf(path, PATH_MAX, "cpu%d/%s", > > cpu, metadata_etmv4_ro[CS_ETMV4_TRCIDR2]); > > err = perf_pmu__scan_file(cs_etm_pmu, path, "%x", &val); > > @@ -643,6 +644,16 @@ static void cs_etm_get_metadata(int cpu, u32 *offset, > > cs_etm_get_ro(cs_etm_pmu, cpu, > > metadata_etmv4_ro > > [CS_ETMV4_TRCAUTHSTATUS]); > > + /* > > + * ETE uses the same registers as ETMv4 plus TRCDEVARCH. It's also backwards > > + * compatible, so don't change the magic number otherwise that will reduce the > > + * number of versions of perf that can open it. Just append TRCDEVARCH to the end of > > + * the register block and allow newer versions of perf to make use. cs_etm_get_ro() > > + * returns 0 if it couldn't be read. > > + */ > > ETE is a superset of ETMv4, but an old perf that only knows ETMv4 > cannot be guaranteed to decode all ETE due to new packet types. > Therefore do we want to allow old perfs to decode only some ETE, > possibly with errors? > > I think it would be better to add in a new magic number for the new > decoder rather than have some grey overlap area were an "older" perf > might work intermittently dependent on the packets generated in a > particular trace run. I checked ETMv4.3 and ETMv4.4 spec (ARM IHI0064E for ETMv4.3 and ARM IHI0064F for ETMv4.4), both clarify ETMv4 has the register TRCDEVARCH; thus TRCDEVARCH is not a new register introduced by ETE. For this case, it's good to directly add a new field in the metadata array for recording register TRCDEVARCH. If there have any new registers are introduced by ETE, then Mike's suggestion for using new magic number is the right thing to do. > > + info->priv[*offset + CS_ETE_TRCDEVARCH] = > > + cs_etm_get_ro(cs_etm_pmu, cpu, > > + metadata_etmv4_ro[CS_ETE_TRCDEVARCH]); > > > > /* How much space was used */ > > increment = CS_ETMV4_PRIV_MAX; > > diff --git a/tools/perf/util/cs-etm.c b/tools/perf/util/cs-etm.c > > index 62769a84a53f..68978f6707a8 100644 > > --- a/tools/perf/util/cs-etm.c > > +++ b/tools/perf/util/cs-etm.c > > @@ -2508,6 +2508,7 @@ static const char * const cs_etmv4_priv_fmts[] = { > > [CS_ETMV4_TRCIDR2] = " TRCIDR2 %llx\n", > > [CS_ETMV4_TRCIDR8] = " TRCIDR8 %llx\n", > > [CS_ETMV4_TRCAUTHSTATUS] = " TRCAUTHSTATUS %llx\n", > > + [CS_ETE_TRCDEVARCH] = " TRCDEVARCH %llx\n" > > }; > > > > static const char * const param_unk_fmt = > > diff --git a/tools/perf/util/cs-etm.h b/tools/perf/util/cs-etm.h > > index d65c7b19407d..52d82dce9d59 100644 > > --- a/tools/perf/util/cs-etm.h > > +++ b/tools/perf/util/cs-etm.h > > @@ -59,7 +59,7 @@ enum { > > /* define fixed version 0 length - allow new format reader to read old files. */ > > #define CS_ETM_NR_TRC_PARAMS_V0 (CS_ETM_ETMIDR - CS_ETM_ETMCR + 1) > > > > -/* ETMv4 metadata */ > > +/* ETMv4 + ETE metadata */ > > enum { > > /* Dynamic, configurable parameters */ > > CS_ETMV4_TRCCONFIGR = CS_ETM_COMMON_BLK_MAX_V1, > > @@ -70,7 +70,8 @@ enum { > > CS_ETMV4_TRCIDR2, > > CS_ETMV4_TRCIDR8, > > CS_ETMV4_TRCAUTHSTATUS, > > - CS_ETMV4_PRIV_MAX, > > + CS_ETE_TRCDEVARCH, > > + CS_ETMV4_PRIV_MAX Spurious change for "CS_ETMV4_PRIV_MAX"? Thanks, Leo _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel