Received: by 2002:a05:6a10:5bc5:0:0:0:0 with SMTP id os5csp977802pxb; Tue, 19 Oct 2021 17:39:36 -0700 (PDT) X-Google-Smtp-Source: ABdhPJzno4mAniyCFmapn7I9Z7CrJnrYr/PpStJF0kXFtKgoYvcgVqhTgkgP3awkbNs+JKjjrQ6n X-Received: by 2002:a17:906:2412:: with SMTP id z18mr41121646eja.56.1634690376603; Tue, 19 Oct 2021 17:39:36 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1634690376; cv=none; d=google.com; s=arc-20160816; b=LRSAZXjlixg2pQRt34yBQKBZLwmngzO5OnsDfJqmjuVb69HCNZDWpvzI0OGJNsRREj CB8Jcw1pabA5kSiADwITfBJQ/aebZ95aTggQ8k7VObJN8O7Q1m471zneycgfaN2YeE49 tnOg08c8ET3z6TWCj0hPqR6Mvw1NM4W3Jj2YEQdIL3pxCNLhz4oJ4lanK6skcfrJe67M qbILJ7GHJztYg5zSw/fy2YP+4irkJmENGihTnhV1PSGCNg7+CisUc4kjKX5bBHdAPwYb MNuNNEdCLt0DTG9KBHAp/Rn22UymrG9m6/8X2rsHJEpv4Y2aWaty+73betj18IMmW1mj zn2w== 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 :message-id:references:in-reply-to:user-agent:subject:cc:to:from :date:dkim-signature; bh=DBHktv6tWnHaAPhvn/ATHNr/sBbAryr3v0f/Ync7WK0=; b=pPyq19JIql+b2dbl8KiUUFFQ0Cs/p3jhZL+VX9NjTFNc4Kx2/avw46bjsSatWZaDJN /FLA3iaePBDe+/JCLAiU8w4RxyN6urv21K+3R/e8vd+A5TzQGynFnZ9GkTfspsA4TsJG vRBShmb8tvrfZUC8nDSU2r456DFWC981Y/qHRpGM1ypACPXn1+qbva8yKkeUHfJWKppD 2O3I5tRInaH3ymx9L2RuqzqE37tb2RIAXC5e6d21gjxWSxvmkM4YEI8y9U7JJoZ3qiJQ RpJ5G53D2cWJB9dAFMIFUBfgl/ptBCS2MwLTGSU7GwtHbcj7LchO/ZCWkX6qC121W32S aY7Q== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20210112 header.b=a3rUdqTZ; 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=QUARANTINE dis=NONE) header.from=gmail.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id g23si598222ejk.552.2021.10.19.17.39.12; Tue, 19 Oct 2021 17:39:36 -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=@gmail.com header.s=20210112 header.b=a3rUdqTZ; 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=QUARANTINE dis=NONE) header.from=gmail.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229910AbhJTAhd (ORCPT + 99 others); Tue, 19 Oct 2021 20:37:33 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:33462 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229627AbhJTAhd (ORCPT ); Tue, 19 Oct 2021 20:37:33 -0400 Received: from mail-ua1-x936.google.com (mail-ua1-x936.google.com [IPv6:2607:f8b0:4864:20::936]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id B0921C06161C; Tue, 19 Oct 2021 17:35:19 -0700 (PDT) Received: by mail-ua1-x936.google.com with SMTP id h4so3533710uaw.1; Tue, 19 Oct 2021 17:35:19 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=date:from:to:cc:subject:user-agent:in-reply-to:references :message-id:mime-version:content-transfer-encoding; bh=DBHktv6tWnHaAPhvn/ATHNr/sBbAryr3v0f/Ync7WK0=; b=a3rUdqTZqUyD0z9UFNRkvnwBUK3seVDZfsy5S9vUFigOsHNLRQJliep0ocNoGq0O7S 1gxnUtcVrC6foGEr107JoDmJf9gmtaaEMxWY+8ppIfzuY4aEw7Jzva2TX173tyQIQS6r 2b5xA9/OLAkGEzMuFmvf8ADZA9XOPCKkIkxpeaBGdabYhjRZO2PNOV9wJeIQAYSylLrF qxGxOvmfmknh2STXs1mTsi9F8hU7RQen2TQLM6m7KNvXFHh6AjGfWKDEubQt4vSM/YxA F06B7qaQ1+9R61F9AmEMIK6KljwCjETvqNGA64p/FOD6H6EQql8jhVH/8m+AF2IwT8X9 AG/Q== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:date:from:to:cc:subject:user-agent:in-reply-to :references:message-id:mime-version:content-transfer-encoding; bh=DBHktv6tWnHaAPhvn/ATHNr/sBbAryr3v0f/Ync7WK0=; b=D8Tarsl8rik/HO6naH8QNHQSUSFResxsaSszGZs+JWABmzFK9MYc7SZjiAjuj6l5Yx 5gYihlsG+vUV9llnMaDsg64vmd/VQi7BzywfqtLfDurGFDIBf0hJFXHgW/4j3WnCz2rH U798ITnJJ0y5d+wpCyI5yDLxxhP0yx51i4ddFudFdS45g3cSayyYTWy2DE/uVAEUvpwB u6amIqxxUxtaE0ThfKI4UIoY+o3e/+T6GE9MoNxTbd7HnVr0JTxTPSI6HLFaMnyqKG+F Zy0D0AeqP/fHG74UrAHkUGpxRrn/gxEFCEQc6HlvZpspsgVNNSOK0yhC2PPDy2yYCHbH 7oAA== X-Gm-Message-State: AOAM533D3ok/Us/PabG/YnTYn08FiSBwLdPY5vUx4Qs4wzsYiVVuYcaX p+xoyBzQu8uEUxh0nyctph8= X-Received: by 2002:a67:e0c2:: with SMTP id m2mr39976209vsl.2.1634690118818; Tue, 19 Oct 2021 17:35:18 -0700 (PDT) Received: from [127.0.0.1] ([179.97.37.151]) by smtp.gmail.com with ESMTPSA id w128sm420145vka.27.2021.10.19.17.35.17 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Tue, 19 Oct 2021 17:35:18 -0700 (PDT) Date: Tue, 19 Oct 2021 21:35:13 -0300 From: Arnaldo Carvalho de Melo To: Ian Rogers , Arnaldo Carvalho de Melo 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. User-Agent: K-9 Mail for Android In-Reply-To: References: <20211015172132.1162559-1-irogers@google.com> <20211015172132.1162559-22-irogers@google.com> Message-ID: MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On October 19, 2021 5:00:23 PM GMT-03:00, Ian Rogers wrote: >On Tue, Oct 19, 2021 at 8:18 AM Arnaldo Carvalho de Melo > wrote: >> >> Em Tue, Oct 19, 2021 at 12:17:10PM -0300, Arnaldo Carvalho de Melo escr= eveu: >> > Em Tue, Oct 19, 2021 at 12:13:52PM -0300, Arnaldo Carvalho de Melo es= creveu: >> > > 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 th= e >> > > > > same metric for kernel and user mode=2E 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/co= re >> > > > branch=2E >> > > >> > > 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=2Eperf/core=2E=2E=2E >> >> It was there already, what I have locally is what is in tmp=2Eperf/core= =2E > >Hi Arnaldo, > >The last change I see in tmp=2Eperf/core is: >https://git=2Ekernel=2Eorg/pub/scm/linux/kernel/git/acme/linux=2Egit/comm= it/?h=3Dtmp=2Eperf/core&id=3D5f43061b84d815c0f6477c1a8836bf0c6fec15a2 > >The difference between v2 and v1 which fixes this bug is: > >+@@ -1500,7 +1568,10 @@ int metricgroup__copy_metric_events(struct >evlist *evlist, struct cgroup *cgrp, >+ return -ENOMEM; >+ >+ new_expr->metric_expr =3D old_expr->metric_expr; >+- new_expr->metric_name =3D old_expr->metric_name; >++ new_expr->metric_name =3D strdup(old_expr->metric= _name); >++ if (!new_expr->metric_name) >++ return -ENOMEM; >++ >+ new_expr->metric_unit =3D old_expr->metric_unit; >+ new_expr->runtime =3D old_expr->runtime; >+ > >I also cleaned up some checkpatch line length warnings in v2, which I >think is the reason for the other changes=2E Ideally I'd prefer the v2 >patch set over the v1, but they are largely identical=2E Both were based >on acme/perf/core=2E Let me know how I can help=2E > I'm trying not to rewrite the perf/core branch after it's made public, so = the best at this point is to add a new patch fixing the problem, if you can= please send it, that would help=2E If it was just on tmp=2Eperf/core, then it would be ok to fix it=2E - Arnaldo