From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id CC801C05027 for ; Fri, 20 Jan 2023 14:43:45 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:Content-Type: Content-Transfer-Encoding:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:In-Reply-To:From:References:Cc:To:Subject: MIME-Version:Date:Message-ID:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=W8e3FxPPODbAvIV9cnkCcmT9nFyEDYITQtCKAFgGWwU=; b=XmI5WvqZ5BanpC THVkTeXO5Kv50zrBK0B8p/hJCybKQoaHGNGJ/yXYRvBeJnq5AZdl33BXIF97CnRDRDD89eYvJ1TSv 1B9v2gj7n6vfKWoNZgPKIFHmwu3N1FCGuckHoa1e1+V+r3WUBYTJK7dyh2mSofst3+QVo2B5VqcP2 TsoI7eHYoRr4h3k64QpJpwIKSTVLZR3YuD37JSRXJ4XsUJGzFaeHqsy+14jxt12HzXCgRkUAB/Kzq uezH9QpafG3LsJnqJB40HfU80scPfwkBK6nY3L5AQi8PbYYrsnTRFyHMGv5nXDbRc3UzoxzDLo1Yf SdJArse2mFJVQ/yhfreg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1pIsbI-00An2Q-0G; Fri, 20 Jan 2023 14:42:41 +0000 Received: from foss.arm.com ([217.140.110.172]) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1pIsbA-00AmzK-IH for linux-arm-kernel@lists.infradead.org; Fri, 20 Jan 2023 14:42:34 +0000 Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id B80621515; Fri, 20 Jan 2023 06:43:10 -0800 (PST) Received: from [10.1.197.1] (ewhatever.cambridge.arm.com [10.1.197.1]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id D1EE63F67D; Fri, 20 Jan 2023 06:42:25 -0800 (PST) Message-ID: Date: Fri, 20 Jan 2023 14:42:24 +0000 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.4.2 Subject: Re: [PATCH v5 0/8] perf cs_etm: Basic support for virtual/kernel timestamps Content-Language: en-US To: James Clark , linux-perf-users@vger.kernel.org, tanmay@marvell.com, leo.yan@linaro.org, mike.leach@linaro.org Cc: sgoutham@marvell.com, gcherian@marvell.com, lcherian@marvell.com, bbhushan2@marvell.com, Mathieu Poirier , John Garry , Will Deacon , Peter Zijlstra , Ingo Molnar , Arnaldo Carvalho de Melo , Mark Rutland , Alexander Shishkin , Jiri Olsa , Namhyung Kim , coresight@lists.linaro.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org References: <20230120143702.4035046-1-james.clark@arm.com> From: Suzuki K Poulose In-Reply-To: <20230120143702.4035046-1-james.clark@arm.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230120_064232_767089_4DC416B0 X-CRM114-Status: GOOD ( 16.62 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset="us-ascii"; Format="flowed" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On 20/01/2023 14:36, James Clark wrote: > Changes since v4: > > * Rebase onto perf/core > * Convert new perf_pmu__cpu_slots_per_cycle() function to use > new helper functions > > =========================== > Changes since v3: > > * Scale time estimates by INSTR_PER_NS, rather than assuming 1 > instruction = 1ns > * Add a new commit that fixes some issues around timestamps going > backwards > * Use nanoseconds inside cs-etm-decoder.c, rather than storing the > raw time values and converting when a sample is synthesized. This > simplifies some of the code like estimating the first timestamp. > > =========================== > Changes since v2: > > * Remove const to non-const change and copy strings where needed > instead. > * Use sizeof() instead of PATH_MAX > * Append "will not be set accurately." to new error message > * Remove unneeded stat() call > * Rebase on perf/core > > ========================== > > Changes since v1: > > * Add 3 refactor commits for sysfs reading around pmu.c as suggested > by Arnaldo here [1] > * The dependency on [2] has now reached mainline so is no longer > blocking > * Rebase on perf/core > > [1]: https://lore.kernel.org/all/YnqVqq5QW%2Fb14oPZ@kernel.org/ > [2]: https://lore.kernel.org/all/20220503123537.1003035-1-german.gomez@arm.com/ > > German Gomez (4): > perf pmu: Add function to check if a pmu file exists > perf cs_etm: Keep separate symbols for ETMv4 and ETE parameters > perf cs_etm: Record ts_source in AUXTRACE_INFO for ETMv4 and ETE > perf cs_etm: Set the time field in the synthetic samples > > James Clark (4): > perf: Remove duplication around EVENT_SOURCE_DEVICE_PATH > perf: Use perf_pmu__open_file() and perf_pmu__scan_file() > perf: Remove remaining duplication of bus/event_source/devices/... > perf: cs-etm: Ensure that Coresight timestamps don't go backwards > > tools/perf/arch/arm/util/auxtrace.c | 5 +- > tools/perf/arch/arm/util/cs-etm.c | 91 ++++++++- > tools/perf/arch/arm64/util/pmu.c | 4 +- > tools/perf/arch/x86/util/pmu.c | 12 +- > tools/perf/util/cputopo.c | 9 +- > tools/perf/util/cs-etm-base.c | 34 +++- > .../perf/util/cs-etm-decoder/cs-etm-decoder.c | 68 +++++-- > tools/perf/util/cs-etm.c | 95 +++++++++- > tools/perf/util/cs-etm.h | 16 +- > tools/perf/util/pmu-hybrid.c | 27 +-- > tools/perf/util/pmu.c | 177 +++++++----------- > tools/perf/util/pmu.h | 10 +- > 12 files changed, 351 insertions(+), 197 deletions(-) For the series, Acked-by: Suzuki K Poulose > > > base-commit: 1962ab6f6e0b39e4216206205bda14aff87705f3 > prerequisite-patch-id: 9722bf86e3e6d16d177ff9a1411992a795a7dcbd > prerequisite-patch-id: b05dbef439c2ea8465f3321532257b0ca29f21f9 > prerequisite-patch-id: 92680a4781cbcf010fcb007e6ea030f59e9eaefc > prerequisite-patch-id: 8e3a73a04e4b89b503377b5fac1d89d551159393 > prerequisite-patch-id: 09980d8fedcdaa70b220a7802428109f48448a58 _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel