Received: by 2002:a05:6a10:5bc5:0:0:0:0 with SMTP id os5csp555261pxb; Tue, 19 Oct 2021 08:18:21 -0700 (PDT) X-Google-Smtp-Source: ABdhPJzAlEZ/NclHQ4nfflmeIJrXZDFDqtVapI3GAVlRKIZwpxHWw71vGR3t484k5AapoRDIX3+P X-Received: by 2002:a5d:4b85:: with SMTP id b5mr45410661wrt.362.1634656700947; Tue, 19 Oct 2021 08:18:20 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1634656700; cv=none; d=google.com; s=arc-20160816; b=ZPD2Zj8OVQqam2IhDX3yGvvmF2P2l99jUu0HERuE+V1fPgsMSJbVh9YSHZuYNsT4mw JvUuwUENsJFGB7JergPZNMFBoZEH5812FTIlVqOEHSRMR94P6NPfO7N/UoAApL/mLTUy ogNVLqQMDV+KiPuSVaCCpevbs5fvqzWNxW41QhIxzbTgZaa5On3Fmw+RfG9wcHkAnDUg SVLtx2MdfSxPL1LDDTYLijgaC7FeVpcUuIGG0HW4GlKU2/ZFY+bdtoFULGHUy2ilsB3O e9FpNM9OJdCLcKD9X/V5wF08jjaJeJPOE/pT0tBfsQzQrf4KaQUf0SeBZNALzfRX4hbP Stdg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:in-reply-to:content-transfer-encoding :content-disposition:mime-version:references:message-id:subject:cc :to:from:date:dkim-signature; bh=BmJ4UICMQV+oSRBHK4q7IZ1G3CgY24IIWlpgmWKeEvM=; b=j2hFNcaae17ZdT4+xIajqJv5Bcnjh+QQE+lzthZYN98b1IvZjKEPLEVVoiFccTGJkS LJxOAWJt7SWPohChmTNh6jDuZgGG1qesSLfh+5y5Pk5Nidh8ZhnpBvt4ocKXGlqu1ZCf ZxsGxJ4LDM2y+4QiXpYeztdMEOJA6oUEiXgPCQKQZvoCc4mjHYQdmTqtV28kKbEAo2Jf SPm89KoFCC3CJccV7GlkxJ6pyr10FzuRFlcMK9DnVIVodb7/LPyMakU4L9rBAtHT9ALI T431+UqF70hiIwCYOFl1SUf/MbAYnm3Mp030Vt3OoLGRSlXK/qVP/+hagSBXAa6L17Dr YfUQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=raIJEbfo; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id z66si23617365ede.301.2021.10.19.08.17.55; Tue, 19 Oct 2021 08:18:20 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=raIJEbfo; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S233392AbhJSPQJ (ORCPT + 99 others); Tue, 19 Oct 2021 11:16:09 -0400 Received: from mail.kernel.org ([198.145.29.99]:35028 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232601AbhJSPQI (ORCPT ); Tue, 19 Oct 2021 11:16:08 -0400 Received: by mail.kernel.org (Postfix) with ESMTPSA id 7FCF460FDA; Tue, 19 Oct 2021 15:13:55 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1634656435; bh=9XoFczMc+6tFEnlfl7hf3gXyyeGcNpPKujV8wE7uMk0=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=raIJEbfogEXYYugcM4ZZnVr7nSCS8Le0kLTOHNQh/D1aqCahzkGvvuchWQJbtrZn7 Cf9uAAjcs9hr6veVBSde8LCLbiP5GvsFqAYAcgBTs8eJ0LJQ0jxV8+ZIsP4/V/NDQq PxtLTNK3nJaBjQUT5KNi/eJN9P/N67JzAmRDfPd5wy2HbbsGwBpRAnEJCkBswqxAaU un4oKRaPXm8BOMs1KDO56vsjy/DystQINPYVs+w+zNqtR2mxkwYZ+uyhXH9L5kbvJ5 u6fiN0Vis2UkjeA7JgGtwPbgjY8R56eOM3WEyOXyTw/SGeT7ezkRSgAjO4gstD1Sah CyJ/DdlpePzuQ== Received: by quaco.ghostprotocols.net (Postfix, from userid 1000) id DC5C5410A1; Tue, 19 Oct 2021 12:13:52 -0300 (-03) Date: Tue, 19 Oct 2021 12:13:52 -0300 From: Arnaldo Carvalho de Melo To: Ian Rogers Cc: Andi Kleen , Jiri Olsa , Jin Yao , Namhyung Kim , John Garry , Kajol Jain , "Paul A . Clarke" , Riccardo Mancini , Kan Liang , Peter Zijlstra , Ingo Molnar , Mark Rutland , Alexander Shishkin , Kees Cook , Sami Tolvanen , Nick Desaulniers , Andrew Morton , Jacob Keller , Zhen Lei , ToastC , Joakim Zhang , Felix Fietkau , Jiapeng Chong , Song Liu , Fabian Hemmer , Alexander Antonov , Nicholas Fraser , Adrian Hunter , Denys Zagorui , Wan Jiabing , Thomas Richter , Sumanth Korikkar , Heiko Carstens , Changbin Du , linux-kernel@vger.kernel.org, linux-perf-users@vger.kernel.org, Andrew Kilroy , Stephane Eranian Subject: Re: [PATCH v2 21/21] perf metric: Allow modifiers on metrics. Message-ID: References: <20211015172132.1162559-1-irogers@google.com> <20211015172132.1162559-22-irogers@google.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: X-Url: http://acmel.wordpress.com Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Em Tue, Oct 19, 2021 at 12:06:17PM -0300, Arnaldo Carvalho de Melo escreveu: > Em Fri, Oct 15, 2021 at 10:21:32AM -0700, Ian Rogers escreveu: > > By allowing modifiers on metrics we can, for example, gather the > > same metric for kernel and user mode. On a SkylakeX with > > TopDownL1 this gives: > > > > $ perf stat -M TopDownL1:u,TopDownL1:k -a sleep 2 > > > > Performance counter stats for 'system wide': > > Hi Ian, can you please take a look on this? this is on my perf/core > branch. I processed the first version of this series, reviewed by Andi, can you please submit the diff from one to the other? - Arnaldo > - Arnaldo > > [root@five ~]# perf test -vv "for cgroups" > 70: Event expansion for cgroups : > --- start --- > test child forked, pid 992568 > Using CPUID AuthenticAMD-25-21-0 > libpfm was not enabled > metric expr 1 / IPC for CPI > parsing metric: 1 / IPC > metric expr instructions / cycles for IPC > parsing metric: instructions / cycles > found event instructions > found event cycles > Parsing metric events '{instructions/metric-id=instructions/,cycles/metric-id=cycles/}:W' > copying metric event for cgroup 'A': instructions (idx=0) > copying metric event for cgroup 'B': instructions (idx=0) > copying metric event for cgroup 'C': instructions (idx=0) > free(): double free detected in tcache 2 > test child interrupted > ---- end ---- > Event expansion for cgroups: FAILED! > [root@five ~]# > > ⬢[acme@toolbox perf]$ git bisect bad > c067335fcbfc67c36663dabdb9ccaa96badf9359 is the first bad commit > commit c067335fcbfc67c36663dabdb9ccaa96badf9359 > Author: Ian Rogers > Date: Thu Oct 7 09:56:47 2021 -0700 > > perf metric: Allow modifiers on metrics. > > By allowing modifiers on metrics we can, for example, gather the > same metric for kernel and user mode. On a SkylakeX with > TopDownL1 this gives: > -- - Arnaldo