Received: by 2002:a6b:500f:0:0:0:0:0 with SMTP id e15csp2629693iob; Fri, 6 May 2022 07:10:06 -0700 (PDT) X-Google-Smtp-Source: ABdhPJx1uI7SmLUPTk97W9KBOvi9Qu4LSUYwpgINPsN+jg3JJi9BtH1aWPh1OsWVOaDA5GGgp0tc X-Received: by 2002:a65:434b:0:b0:382:4fa9:3be6 with SMTP id k11-20020a65434b000000b003824fa93be6mr2916096pgq.459.1651846206186; Fri, 06 May 2022 07:10:06 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1651846206; cv=none; d=google.com; s=arc-20160816; b=vXPqYTfpE9zO1Rju1s3pC7FBVTb5L0jz88bjDOb+2lYtoZtmw3G5AkKfpW/aZOL8NP JofX3nHdezjS6QytwiVx8A9kE2VcrM1l8ls90QSYj5FFNugjaJWWzGMrcx5y6T9yJLFi vCtTLs22TGZtSyoiViNLLu1G0P/fZKa3dj1JlpIlNQDwWJeJVndBUbtAxd1piaNZIad7 LR7O48hm1uUSjm0Tp7FnEeYEA/tGZYfkW/Uq+eXGDLWVi/d+IQ3SDc1TEphGVvTVvEPA wrYqqNq7rHJUUuqWoGYb+YRuCRMY7DQbIey1NcWlu8cqkolD/LZ+XIu6E/B7Cd45sE9F H0Uw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :references:in-reply-to:message-id:date:subject:cc:to:from :dkim-signature; bh=2Q/yiQSP0UKyBh6YQJ5BaZo0I0IrAyfxQYHuply2jAk=; b=V+lFfY6TdjOmyvvwrj7ZTACQxzucq+rua1/n+pfDFPSemi+5H5zh5Av4RZMlekJETk bLwdiDCAy+/9uZpGKTMH4m4Z3D+d0Yf3qjlIS8jcCItJGOmzINR0GNzIYs17lKnsqLZU ytiA3VnJghVnsl2pL2v0XgaMC93ZNvc5yxzBCTpetgESGmGMaVPkdQLrqwIcgqT/5bMg DJldTPruXhjtg0e7q/ZZm9YGZItFheqLOujFd1TxzQa5QcZrL/DGLmItoIahvxhM4JWd Ca92MOSooOfdmMVF1w9BKr2y8yYtkMUv//GZvBAFs0aFlEgDJpOzI9yTGLYehvupC5SI HWBg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@amazon.com header.s=amazon201209 header.b=hf2IYhbT; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=amazon.com Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id f4-20020a631f04000000b0039daee7ecf1si5019033pgf.318.2022.05.06.07.09.48; Fri, 06 May 2022 07:10:06 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@amazon.com header.s=amazon201209 header.b=hf2IYhbT; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=amazon.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1353359AbiEDQ0q (ORCPT + 99 others); Wed, 4 May 2022 12:26:46 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:59702 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S234286AbiEDQ0p (ORCPT ); Wed, 4 May 2022 12:26:45 -0400 Received: from smtp-fw-80007.amazon.com (smtp-fw-80007.amazon.com [99.78.197.218]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 0B31A39813; Wed, 4 May 2022 09:23:09 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=amazon.com; i=@amazon.com; q=dns/txt; s=amazon201209; t=1651681389; x=1683217389; h=from:to:cc:subject:date:message-id:in-reply-to: references:mime-version:content-transfer-encoding; bh=2Q/yiQSP0UKyBh6YQJ5BaZo0I0IrAyfxQYHuply2jAk=; b=hf2IYhbTP7/C5uI5Kqo6jNwp5esTQ5HkllgxU6KLMNpVxw0TrNzazoT3 DTejx9bgJW+tXUB1EtEzMMF9Hs89opPjCk7dQ0EZoVEb2hUX7pjY0edfr W/ffJU0WtndriOjGqd5Mbw2F8tVxIdUMvfOEdFCJimA6ryHT07GqgpRoB 4=; X-IronPort-AV: E=Sophos;i="5.91,198,1647302400"; d="scan'208";a="85452775" Received: from pdx4-co-svc-p1-lb2-vlan2.amazon.com (HELO email-inbound-relay-iad-1d-1c3c2014.us-east-1.amazon.com) ([10.25.36.210]) by smtp-border-fw-80007.pdx80.corp.amazon.com with ESMTP; 04 May 2022 16:22:35 +0000 Received: from EX13MTAUWC001.ant.amazon.com (iad12-ws-svc-p26-lb9-vlan2.iad.amazon.com [10.40.163.34]) by email-inbound-relay-iad-1d-1c3c2014.us-east-1.amazon.com (Postfix) with ESMTPS id 60F5DE42C8; Wed, 4 May 2022 16:22:30 +0000 (UTC) Received: from EX13D02UWC001.ant.amazon.com (10.43.162.243) by EX13MTAUWC001.ant.amazon.com (10.43.162.135) with Microsoft SMTP Server (TLS) id 15.0.1497.32; Wed, 4 May 2022 16:22:23 +0000 Received: from EX13MTAUEA001.ant.amazon.com (10.43.61.82) by EX13D02UWC001.ant.amazon.com (10.43.162.243) with Microsoft SMTP Server (TLS) id 15.0.1497.32; Wed, 4 May 2022 16:22:23 +0000 Received: from dev-dsk-alisaidi-1d-b9a0e636.us-east-1.amazon.com (172.19.181.128) by mail-relay.amazon.com (10.43.61.243) with Microsoft SMTP Server id 15.0.1497.32 via Frontend Transport; Wed, 4 May 2022 16:22:22 +0000 Received: by dev-dsk-alisaidi-1d-b9a0e636.us-east-1.amazon.com (Postfix, from userid 5131138) id B54541D1A; Wed, 4 May 2022 16:22:22 +0000 (UTC) From: Ali Saidi To: CC: , , , , , , , , , , , , , , , , , , , , Subject: Re: [PATCH v7 5/5] perf arm-spe: Use SPE data source for neoverse cores Date: Wed, 4 May 2022 16:22:20 +0000 Message-ID: <20220504162220.28528-1-alisaidi@amazon.com> X-Mailer: git-send-email 2.32.0 In-Reply-To: <458a2de1-dc93-7e2d-5dc5-fbcd670572b6@arm.com> References: <458a2de1-dc93-7e2d-5dc5-fbcd670572b6@arm.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Content-Type: text/plain X-Spam-Status: No, score=-12.5 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_MED, SPF_HELO_NONE,SPF_PASS,T_SCC_BODY_TEXT_LINE,USER_IN_DEF_SPF_WL autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi German, Thanks for taking another look! On Tue, 3 May 2022 09:58:15 +0000, German Gomez wrote: > Hi Ali, Leo > > Some minor comments below. > > On 26/04/2022 14:59, Ali Saidi wrote: > > When synthesizing data from SPE, augment the type with source information > > for Arm Neoverse cores. The field is IMPLDEF but the Neoverse cores all use > > the same encoding. I can't find encoding information for any other SPE > > implementations to unify their choices with Arm's thus that is left for > > future work. > > > > This change populates the mem_lvl_num for Neoverse cores as well as the > > deprecated mem_lvl namespace. > > > > Signed-off-by: Ali Saidi > > --- > > .../util/arm-spe-decoder/arm-spe-decoder.c | 1 + > > .../util/arm-spe-decoder/arm-spe-decoder.h | 12 ++ > > tools/perf/util/arm-spe.c | 130 +++++++++++++++--- > > 3 files changed, 127 insertions(+), 16 deletions(-) > > > [snip] > > + /* > > + * We have no data on the hit level or data source for stores in the > > + * Neoverse SPE records. > > + */ > > + if (record->op & ARM_SPE_ST) { > > + data_src->mem_lvl = PERF_MEM_LVL_NA; > > + data_src->mem_lvl_num = PERF_MEM_LVLNUM_ANY_CACHE; > > Is it correct to use ANY_CACHE as the NA value? The LVLNUM_* enum begins at 1, so it looks like this should be set to 0 instead (like in HOPS_*). I think you're making a good point here. To be consistent we should set mem_lvl_num to PERF_MEM_LVLNUM_NA just like mem_lvl. Thanks, Ali