Message ID | 20230302031146.2801588-1-changbin.du@huawei.com |
---|---|
Headers |
Return-Path: <linux-kernel-owner@vger.kernel.org> Delivered-To: ouuuleilei@gmail.com Received: by 2002:a5d:5915:0:0:0:0:0 with SMTP id v21csp4013952wrd; Wed, 1 Mar 2023 19:17:59 -0800 (PST) X-Google-Smtp-Source: AK7set/6d+MMrEoguuhw1wbUi/KacJjG4J44MNDRU4DSJ+9jkybJlqqXVJ9nP/sudVhtpov4MgtK X-Received: by 2002:a17:906:dac7:b0:8f4:809e:faee with SMTP id xi7-20020a170906dac700b008f4809efaeemr867713ejb.19.1677727078925; Wed, 01 Mar 2023 19:17:58 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1677727078; cv=none; d=google.com; s=arc-20160816; b=ISiYSm4MTnspGxO0ao+Qa0XqUL5s4LdWYNDgNnClw0UEDqY64Cy9bV2Y0tp8Z+hRPk fqRgse/Cs6tDOiU7dvdcYwCKyxCnBCp8gsKtSEV7hhVXxyMDrovhsQIcIic30bzv/6xx OlhosPEfZF4fKA5M5jY43nqq+LJScbLt8orxCPxDo4g+I+U6NZ7KhP0xzC5CoLWGYuqI bFlRFZWNY/Qz3DEInShD3cGkHzrzllchjJyBSWCSL0MBEm+MWAwJoGmmzN2sYJSgCieE pGmLQmT7KhPYSbi9E5HmzjD0nlz1afNuFqmlS2tECAnOfXYbYLGW02zDQeG8os2IaFZX AFmg== 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:date:subject:cc:to:from; bh=ylWX//OhEX2WnwZfdtIWq6uGYCh/XDo////XjD2rCgM=; b=V7PVgnRFNhnHN3LVgLi1o0LXGZPjfQcK+fWgVafdUlokNHlvy+eRJWd6PZRu7+NwbI omsGZxMFBaIYrlYD6lxy4Qf3ua9ILpKiUdRHn4ws51GiRXgiil8Sf+sLRe59tPABYiOl 6S+o3VvKHFNVKm3txsfXOvaxE1vDFm2KMYK81OY+AyfNBPhuVjddfMjlZaBYUXEjoQw7 zxW578f1n511z+RnDa6SN+AOMLGiuEe57XNkXaM8Tn3ENo4ELu62k8jnXeLZ7RJdqMWy bzUzRQqS0Cft1+gz96jO6yboEYdfzBqp8wf3ZwfvkyVbCvkQ3MGqQzYsraCrWGJK74Dl I8YA== ARC-Authentication-Results: i=1; mx.google.com; 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=fail (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=huawei.com Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id d2-20020a1709064c4200b008b17e6683besi16321825ejw.418.2023.03.01.19.17.35; Wed, 01 Mar 2023 19:17:58 -0800 (PST) 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; 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=fail (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=huawei.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229705AbjCBDMY (ORCPT <rfc822;davidbtadokoro@gmail.com> + 99 others); Wed, 1 Mar 2023 22:12:24 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:56646 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229462AbjCBDMX (ORCPT <rfc822;linux-kernel@vger.kernel.org>); Wed, 1 Mar 2023 22:12:23 -0500 Received: from szxga08-in.huawei.com (szxga08-in.huawei.com [45.249.212.255]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 0B8A44C6F2; Wed, 1 Mar 2023 19:12:22 -0800 (PST) Received: from kwepemi500013.china.huawei.com (unknown [172.30.72.55]) by szxga08-in.huawei.com (SkyGuard) with ESMTP id 4PRx0g07dfz16P0k; Thu, 2 Mar 2023 11:09:38 +0800 (CST) Received: from M910t.huawei.com (10.110.54.157) by kwepemi500013.china.huawei.com (7.221.188.120) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.21; Thu, 2 Mar 2023 11:12:17 +0800 From: Changbin Du <changbin.du@huawei.com> To: Arnaldo Carvalho de Melo <acme@kernel.org>, Peter Zijlstra <peterz@infradead.org>, Ingo Molnar <mingo@redhat.com> CC: Mark Rutland <mark.rutland@arm.com>, Alexander Shishkin <alexander.shishkin@linux.intel.com>, Jiri Olsa <jolsa@kernel.org>, Namhyung Kim <namhyung@kernel.org>, <linux-perf-users@vger.kernel.org>, <linux-kernel@vger.kernel.org>, Hui Wang <hw.huiwang@huawei.com>, Changbin Du <changbin.du@huawei.com> Subject: [PATCH v2 0/3] perf : fix counting when initial delay configured Date: Thu, 2 Mar 2023 11:11:43 +0800 Message-ID: <20230302031146.2801588-1-changbin.du@huawei.com> X-Mailer: git-send-email 2.25.1 MIME-Version: 1.0 Content-Transfer-Encoding: 7BIT Content-Type: text/plain; charset=US-ASCII X-Originating-IP: [10.110.54.157] X-ClientProxiedBy: dggems706-chm.china.huawei.com (10.3.19.183) To kwepemi500013.china.huawei.com (7.221.188.120) X-CFilter-Loop: Reflected X-Spam-Status: No, score=-4.2 required=5.0 tests=BAYES_00,RCVD_IN_DNSWL_MED, RCVD_IN_MSPIKE_H2,SPF_HELO_NONE,SPF_PASS 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: <linux-kernel.vger.kernel.org> X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: =?utf-8?q?INBOX?= X-GMAIL-THRID: =?utf-8?q?1759224349371935970?= X-GMAIL-MSGID: =?utf-8?q?1759224349371935970?= |
Series |
perf : fix counting when initial delay configured
|
|
Message
Changbin Du
March 2, 2023, 3:11 a.m. UTC
The first one fixes the problem that counters are never enabled when initial delay configured. The remaining two reuse the common field target::initial_delay for record/ftrace/trace subcommands. v2: - introduce common filed target::initial_delay Changbin Du (3): perf stat: fix counting when initial delay configured perf record: reuse target::initial_delay perf: ftrace: reuse target::initial_delay tools/perf/builtin-ftrace.c | 10 +++++----- tools/perf/builtin-record.c | 12 ++++++------ tools/perf/builtin-stat.c | 15 +++++---------- tools/perf/builtin-trace.c | 8 ++++---- tools/perf/util/evlist.c | 6 +++--- tools/perf/util/evsel.c | 2 +- tools/perf/util/ftrace.h | 1 - tools/perf/util/record.h | 1 - tools/perf/util/stat.c | 6 +----- tools/perf/util/stat.h | 1 - tools/perf/util/target.h | 12 ++++++++++++ 11 files changed, 37 insertions(+), 37 deletions(-)
Comments
Hello, On Wed, Mar 1, 2023 at 7:12 PM Changbin Du <changbin.du@huawei.com> wrote: > > The first one fixes the problem that counters are never enabled when initial > delay configured. > The remaining two reuse the common field target::initial_delay for > record/ftrace/trace subcommands. > > v2: > - introduce common filed target::initial_delay > > Changbin Du (3): > perf stat: fix counting when initial delay configured > perf record: reuse target::initial_delay > perf: ftrace: reuse target::initial_delay Acked-by: Namhyung Kim <namhyung@kernel.org> Thanks, Namhyung > > tools/perf/builtin-ftrace.c | 10 +++++----- > tools/perf/builtin-record.c | 12 ++++++------ > tools/perf/builtin-stat.c | 15 +++++---------- > tools/perf/builtin-trace.c | 8 ++++---- > tools/perf/util/evlist.c | 6 +++--- > tools/perf/util/evsel.c | 2 +- > tools/perf/util/ftrace.h | 1 - > tools/perf/util/record.h | 1 - > tools/perf/util/stat.c | 6 +----- > tools/perf/util/stat.h | 1 - > tools/perf/util/target.h | 12 ++++++++++++ > 11 files changed, 37 insertions(+), 37 deletions(-) > > -- > 2.25.1 >
Em Thu, Mar 02, 2023 at 11:22:37AM -0800, Namhyung Kim escreveu: > Hello, > > On Wed, Mar 1, 2023 at 7:12 PM Changbin Du <changbin.du@huawei.com> wrote: > > > > The first one fixes the problem that counters are never enabled when initial > > delay configured. > > The remaining two reuse the common field target::initial_delay for > > record/ftrace/trace subcommands. > > > > v2: > > - introduce common filed target::initial_delay > > > > Changbin Du (3): > > perf stat: fix counting when initial delay configured > > perf record: reuse target::initial_delay > > perf: ftrace: reuse target::initial_delay > > Acked-by: Namhyung Kim <namhyung@kernel.org> Thanks, applying the first to perf-tools (old perf/urgent) and the rest to perf-tools-next (old perf/core). - Arnaldo > Thanks, > Namhyung > > > > > tools/perf/builtin-ftrace.c | 10 +++++----- > > tools/perf/builtin-record.c | 12 ++++++------ > > tools/perf/builtin-stat.c | 15 +++++---------- > > tools/perf/builtin-trace.c | 8 ++++---- > > tools/perf/util/evlist.c | 6 +++--- > > tools/perf/util/evsel.c | 2 +- > > tools/perf/util/ftrace.h | 1 - > > tools/perf/util/record.h | 1 - > > tools/perf/util/stat.c | 6 +----- > > tools/perf/util/stat.h | 1 - > > tools/perf/util/target.h | 12 ++++++++++++ > > 11 files changed, 37 insertions(+), 37 deletions(-) > > > > -- > > 2.25.1 > >
On Thu, Mar 02, 2023 at 05:40:16PM -0300, Arnaldo Carvalho de Melo wrote: > Em Thu, Mar 02, 2023 at 11:22:37AM -0800, Namhyung Kim escreveu: > > Hello, > > > > On Wed, Mar 1, 2023 at 7:12 PM Changbin Du <changbin.du@huawei.com> wrote: > > > > > > The first one fixes the problem that counters are never enabled when initial > > > delay configured. > > > The remaining two reuse the common field target::initial_delay for > > > record/ftrace/trace subcommands. > > > > > > v2: > > > - introduce common filed target::initial_delay > > > > > > Changbin Du (3): > > > perf stat: fix counting when initial delay configured > > > perf record: reuse target::initial_delay > > > perf: ftrace: reuse target::initial_delay > > > > Acked-by: Namhyung Kim <namhyung@kernel.org> > > Thanks, applying the first to perf-tools (old perf/urgent) and the rest > to perf-tools-next (old perf/core). > > - Arnaldo > Hi Arnaldo, it seems only the first one is applied. The remaining two patches are missed in your tree. - perf record: reuse target::initial_delay - perf: ftrace: reuse target::initial_delay
Em Fri, Mar 10, 2023 at 10:37:14AM +0800, Changbin Du escreveu: > On Thu, Mar 02, 2023 at 05:40:16PM -0300, Arnaldo Carvalho de Melo wrote: > > Em Thu, Mar 02, 2023 at 11:22:37AM -0800, Namhyung Kim escreveu: > > > Hello, > > > > > > On Wed, Mar 1, 2023 at 7:12 PM Changbin Du <changbin.du@huawei.com> wrote: > > > > > > > > The first one fixes the problem that counters are never enabled when initial > > > > delay configured. > > > > The remaining two reuse the common field target::initial_delay for > > > > record/ftrace/trace subcommands. > > > > > > > > v2: > > > > - introduce common filed target::initial_delay > > > > > > > > Changbin Du (3): > > > > perf stat: fix counting when initial delay configured > > > > perf record: reuse target::initial_delay > > > > perf: ftrace: reuse target::initial_delay > > > > > > Acked-by: Namhyung Kim <namhyung@kernel.org> > > > > Thanks, applying the first to perf-tools (old perf/urgent) and the rest > > to perf-tools-next (old perf/core). > > > > - Arnaldo > > > Hi Arnaldo, it seems only the first one is applied. The remaining two patches > are missed in your tree. > - perf record: reuse target::initial_delay > - perf: ftrace: reuse target::initial_delay Those are not fixes, right? I mentioned that I would apply it after the first is merged. - Arnaldo
On Fri, Mar 10, 2023 at 12:05:38PM -0300, Arnaldo Carvalho de Melo wrote: > Em Fri, Mar 10, 2023 at 10:37:14AM +0800, Changbin Du escreveu: > > On Thu, Mar 02, 2023 at 05:40:16PM -0300, Arnaldo Carvalho de Melo wrote: > > > Em Thu, Mar 02, 2023 at 11:22:37AM -0800, Namhyung Kim escreveu: > > > > Hello, > > > > > > > > On Wed, Mar 1, 2023 at 7:12 PM Changbin Du <changbin.du@huawei.com> wrote: > > > > > > > > > > The first one fixes the problem that counters are never enabled when initial > > > > > delay configured. > > > > > The remaining two reuse the common field target::initial_delay for > > > > > record/ftrace/trace subcommands. > > > > > > > > > > v2: > > > > > - introduce common filed target::initial_delay > > > > > > > > > > Changbin Du (3): > > > > > perf stat: fix counting when initial delay configured > > > > > perf record: reuse target::initial_delay > > > > > perf: ftrace: reuse target::initial_delay > > > > > > > > Acked-by: Namhyung Kim <namhyung@kernel.org> > > > > > > Thanks, applying the first to perf-tools (old perf/urgent) and the rest > > > to perf-tools-next (old perf/core). > > > > > > - Arnaldo > > > > > Hi Arnaldo, it seems only the first one is applied. The remaining two patches > > are missed in your tree. > > - perf record: reuse target::initial_delay > > - perf: ftrace: reuse target::initial_delay > > Those are not fixes, right? I mentioned that I would apply it after the > first is merged. > Sorry, I misunderstood your previous reply :) > - Arnaldo