Received: by 2002:a05:6a10:5bc5:0:0:0:0 with SMTP id os5csp556770pxb; Tue, 19 Oct 2021 08:19:54 -0700 (PDT) X-Google-Smtp-Source: ABdhPJx6CxxOEqVwHNvSfBV9r4VV/jIpm3YFFu2osT8ynWVGkQ1+teGFMhykNkYJYtAP4G+d6AOL X-Received: by 2002:a17:90a:940d:: with SMTP id r13mr435308pjo.65.1634656794829; Tue, 19 Oct 2021 08:19:54 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1634656794; cv=none; d=google.com; s=arc-20160816; b=0TRUM1aghWCANCmR26I7/s95zKVh5f/C/OKKjRcyEi7pcyOWOxsSzeeyD/EkelmmuE t/fi6o9BOhb3XDUQbY9DcnToLMi49bkXkgnwPS/1SAa9dOLRHcerdgSZkVxZ8r9OoXVQ mEKtmmeh7XvxOGO3yTDOC7YfVfiYc3eVga4+/tLHaCpj0Ly4QARa3FgR2SgPlP2NG9og 70hjrnnH7vvBOY7JHcQxZLzrQNeo6VkZOo5Y9IipUQrXrF4yrRCCpAf0uyIlPYlEO1Zl SFCAdZYbL9sREIfNy7ZoZa+fgOnG8kOPlgSNINJp4k2p1tS9xLtNFTnQpO9df7Z6jRxT +38Q== 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=k0JUy0C2LRDzMYdanjIwhU+dSjS2gLzeYujsvBbY064=; b=S76lqMRpPUbJP33nOpDLcp1PU9JbLykWCzPpcSP3WEqH9ixLWZX33LtxnuMNX+CuDK jZV649ZG9QMGiiP9rC0BWN0P8SV0xamPxdU9HGArFx6fSnXuVXWjTypbsiQaOYUjKJJL uKT5W0wB8eEHRbSSx2k7QjOqWVEfTTMLhdsEbjhp9737eZILStIKpyDbD5/oMt6QF4DQ Lai7yD4/nPOJtAj3VK2zn16V111/gGaS0NtJmlzvbtCe6IMyHhijNfM0yJaJ5VuJpYO0 SVFNfbqxPKPo1tzQc+kArnorcfZGvoaQwgej3VmWq78pFjBE43KthwLwD+Byf4GJ0s3g 6yIA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=MBZ5cd5e; 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 j6si22389082pld.185.2021.10.19.08.19.41; Tue, 19 Oct 2021 08:19:54 -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=MBZ5cd5e; 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 S232764AbhJSPUb (ORCPT + 99 others); Tue, 19 Oct 2021 11:20:31 -0400 Received: from mail.kernel.org ([198.145.29.99]:39032 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S233552AbhJSPUa (ORCPT ); Tue, 19 Oct 2021 11:20:30 -0400 Received: by mail.kernel.org (Postfix) with ESMTPSA id 18FE161074; Tue, 19 Oct 2021 15:18:15 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1634656695; bh=iOMPFD5BW/TglrD5jW0xYpdvnZD0AjgFVyQTSJS7yRM=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=MBZ5cd5eOOEBgiOJsfQjDnR9OpFCU8PWsCJYjlNDixpB5VxLDJSLxb911McnfPUDy qPVDoQaObA98kq+Naw1y8z4JlfC6koPz8HPluR2G1ZW/1C7ZmXqsfbTXWX+boMw+rr XV3ZUnAu2VNhLCRCyyhFL2TEz83zcrWaQRqvvm0t51i7ypUeMADe0DkMCU/SSv1SX7 Wqxmnj5BdgdfieQqp0VsfsFNYBzYb/SN1iZUeYwnnc9ZU7BoxxOQhsEt7t8PBUabwj eVPpJeFe6GyOFmf2JjshfqQH34fREa8DZNFCyMQJE1FQtXaKs7pLWunl7fw2BB+PWk 656qEx2sOyKcw== Received: by quaco.ghostprotocols.net (Postfix, from userid 1000) id 7E98B410A1; Tue, 19 Oct 2021 12:18:12 -0300 (-03) Date: Tue, 19 Oct 2021 12:18:12 -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:17:10PM -0300, Arnaldo Carvalho de Melo escreveu: > Em Tue, Oct 19, 2021 at 12:13:52PM -0300, Arnaldo Carvalho de Melo escreveu: > > 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? > > The interdiff from the 21st patch on the first batch versus on the v2 > batch is below, but it doesn't apply to my current perf/core branch, > lemme push it to tmp.perf/core... It was there already, what I have locally is what is in tmp.perf/core. > - Arnaldo > > ⬢[acme@toolbox perf]$ interdiff ~/wb/old.patch ~/wb/new.patch > diff -u b/tools/perf/util/metricgroup.c b/tools/perf/util/metricgroup.c > --- b/tools/perf/util/metricgroup.c > +++ b/tools/perf/util/metricgroup.c > @@ -1308,8 +1308,7 @@ > int ret; > > *out_evlist = NULL; > - ret = metricgroup__build_event_string(&events, ids, modifier, > - has_constraint); > + ret = metricgroup__build_event_string(&events, ids, has_constraint); > if (ret) > return ret; > > @@ -1324,7 +1323,8 @@ > > ids__insert(ids->ids, tmp); > } > - ret = metricgroup__build_event_string(&events, ids, has_constraint); > + ret = metricgroup__build_event_string(&events, ids, modifier, > + has_constraint); > if (ret) > return ret; > > @@ -1568,7 +1568,10 @@ > return -ENOMEM; > > new_expr->metric_expr = old_expr->metric_expr; > - new_expr->metric_name = old_expr->metric_name; > + new_expr->metric_name = strdup(old_expr->metric_name); > + if (!new_expr->metric_name) > + return -ENOMEM; > + > new_expr->metric_unit = old_expr->metric_unit; > new_expr->runtime = old_expr->runtime; > > ⬢[acme@toolbox perf]$ -- - Arnaldo