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 vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 7C259C61DF7 for ; Thu, 23 Nov 2023 14:42:25 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1345860AbjKWOmQ (ORCPT ); Thu, 23 Nov 2023 09:42:16 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:57336 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1345772AbjKWOmP (ORCPT ); Thu, 23 Nov 2023 09:42:15 -0500 Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id F0A9E1A8 for ; Thu, 23 Nov 2023 06:42:21 -0800 (PST) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 6870BC433C8; Thu, 23 Nov 2023 14:42:21 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1700750541; bh=mfaBvRhfODG3G//JXKw40tGCkxSfBY+Q43wy4Fkk4fI=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=Q6AkV1egQOR4iwp/oOQIOVMdXVAh6VnMf44ZFxdWRxYcQPdOSem5/kDFtj4RqqbrR DLGrlfnWEXE26WGZheblSS7JJKLf4n0QhwRzEDx05eY7gPTUcgfR+6Q+BfX6sC0avk gSKAI4hu6OxQ/vheUP97V32TMhd0fdhKR38u/S4vEUHX2XmZT37oyM1SWjNddY+BTn 9cj7eSax0g9aTJRSYcnGHJV8PSFGcZMhmxfH/ejo7t3gB38ZWyrYg/cGJ9pymjx21p mq6jPFox0f17VWk1o9s8Ohx/vhJCQ/kVoKATYEw7m+zMzyYzdskPKOau1FpBlIFeUb 1LKCcDV2IHuaw== Received: by quaco.ghostprotocols.net (Postfix, from userid 1000) id 925D740094; Thu, 23 Nov 2023 11:42:18 -0300 (-03) Date: Thu, 23 Nov 2023 11:42:18 -0300 From: Arnaldo Carvalho de Melo To: Leo Yan Cc: Adrian Hunter , James Clark , Suzuki K Poulose , Mike Leach , John Garry , Will Deacon , Peter Zijlstra , Ingo Molnar , Mark Rutland , Alexander Shishkin , Jiri Olsa , Namhyung Kim , Ian Rogers , coresight@lists.linaro.org, linux-arm-kernel@lists.infradead.org, linux-perf-users@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v1 1/2] perf auxtrace: Add 'T' itrace option for timestamp trace Message-ID: References: <20231014074513.1668000-1-leo.yan@linaro.org> <20231014074513.1668000-2-leo.yan@linaro.org> <8a2ea58f-f835-4d1a-8bd6-3a63b3b0db94@intel.com> <2940af18-8bcd-4456-a8ed-e77cade1b160@intel.com> <20231107094852.GA656098@leoy-yangtze.lan> <6fc8bc4c-abc3-45d5-a8c8-045f91adfebf@intel.com> <20231107141808.GD656098@leoy-yangtze.lan> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20231107141808.GD656098@leoy-yangtze.lan> X-Url: http://acmel.wordpress.com Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Em Tue, Nov 07, 2023 at 10:18:08PM +0800, Leo Yan escreveu: > On Tue, Nov 07, 2023 at 12:16:25PM +0200, Adrian Hunter wrote: > > [...] > > > >>>> "If users know" <- how would users know? Could the kernel > > >>>> or tools also figure it out? > > >>> > > >>> Adrian, I'm trying to go all the outstanding patches, do you still have > > >>> any issues with this series? > > >> > > >> No, although the question wasn't actually answered. I presume users > > >> just have to try the 'T' option and see if it helps. > > > > > > Sometimes, users are software developers in SoC companies, they can > > > know well for the hardware design but are confused why current > > > implementation cannot use timestamp trace. This is the main reason > > > I sent this patch set. > > > > > > An example hardware platform is DB410c [1], we know its CoreSight can > > > support timestamp trace, but if without this adding option 'T', we > > > have no chance to use it due to it its CPU arch is prior to Armv8.4. > > > > perf config might be better than an itrace option, but you decide. > > I understand perf config is a better approach due to users don't need > to bother inputting options after set it once. I will look at it and > respin new patch set. > > Thanks for suggestion! Thanks, applied. - Arnaldo