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 X-Spam-Level: X-Spam-Status: No, score=-11.3 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH, MAILING_LIST_MULTI,SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED, USER_AGENT_SANE_1 autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id F2DBBC4363A for ; Fri, 30 Oct 2020 02:46:30 +0000 (UTC) Received: from merlin.infradead.org (merlin.infradead.org [205.233.59.134]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 77AFA206BE for ; Fri, 30 Oct 2020 02:46:30 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="DR9TtqPw"; dkim=fail reason="signature verification failed" (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="THGZF/2+" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 77AFA206BE Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=linaro.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=merlin.20170209; h=Sender:Content-Transfer-Encoding: Content-Type:Cc:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References:Message-ID: Subject:To:From:Date:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=/Mm868ICKNkkb2MQF+GLZVcyrY93Rs0FoDZLVllfdHc=; b=DR9TtqPw8N+Tg0LxaCHTsImWY RItBB4UYnSXHXYJnX/WQHm8y4WIE2+xgv4Cic6Jy5GvWgWYA8WVVHwgP4mkkhaZtv6BxHzvE8eYUU Z4bWXOmYI6UsumVCR+QIbjodQ188BwWbJc9g7ijuOP4IQ1itO/qPNS8FreqNO82VxZZtIf9E1BqKR eEshVPIfacBsvoThhSJWs4Yr7bBAX0jFPxGLChaeb/f90jEyZU2J0l6iVcWZLqIB/J4TdoDvgpV7s Jvho/oSkUCDuwjl3Gk1j4WKmW0iwqd8TFemJ2E4LRbMBrWDUyDbMOa+CkLohccInVmdMl9p9WyWpE 0EnLg62SA==; Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1kYJXv-0004Ou-SE; Fri, 30 Oct 2020 01:49:39 +0000 Received: from mail-pf1-x442.google.com ([2607:f8b0:4864:20::442]) by merlin.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1kYJXr-0004Nr-Vp for linux-arm-kernel@lists.infradead.org; Fri, 30 Oct 2020 01:49:37 +0000 Received: by mail-pf1-x442.google.com with SMTP id w65so3926880pfd.3 for ; Thu, 29 Oct 2020 18:49:35 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to:user-agent; bh=7hL4sw2f+Ivb3IounfMGG3eIBIiM36uU9xeGb4tykPg=; b=THGZF/2+WvMOVeVXUUAgjrNJJh7aY0Jzs+xE+IoeMpId+D14ma7KaA2ONflJXuTGUb mHBvKH7pYppLrCpWp+t14e5yOKI3lvJZp66TcuRzJ+FdemCI19HPMVPwXQxDPlXjhhxw 7FeoxxzHlo0C5tRhuYeXqE/b7oPSHeF5WOzceO2qgyfksuH1vy4ThVRyJrUAU3kg8oRN 2s+d2cCRaIpo2E9z1cNn89JP4csGbs+TocXL1wGIwm/BazEnvudTqq1hUkS31f+NNhhF Od9XRQRXEiDwHv+DUskL1SWR9PS/jHorzFUgFtJFyEQ7N1pcRdZlTml4n1p2EYuGzcMV vjsQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to:user-agent; bh=7hL4sw2f+Ivb3IounfMGG3eIBIiM36uU9xeGb4tykPg=; b=Kr8bz3UL9ojbuZmtxu/AzIzKwkwkdtFKZHiaTXtA9ovyRe2/fWAAAqlvvduumLsRKp 7gACcHZ0usT5MCUV9xeAGll7jre5PWPGuvRMSgdHsVh6JzrkzeFfGDdZ7cYTBXkLtG8V csnNkD78AgcEVHT1Joxnv9KAags6Wzrj768HuC5Ze/inHvssSEQtQyW+St8t7uXh9YkJ P3AxtARMIS+LXoIapfpFAsPzielNrnjGrf+b7wEY7HxSjXe0RCKftnef8bBEphL3yhL7 ZXidvvA+Vgib5qrqhHXi5Pn/Ce/aEzhBCk2138GOyIA4I24A/dvv3jLAM0Lr3bLydxuZ tkkQ== X-Gm-Message-State: AOAM533K+ErlOblPb4qWMkOY5DlVub1cQFb6nGF/xTgf6TsIUOtdefnn N6tJ+n4bAzAnBJoxtTEbHYkuaw== X-Google-Smtp-Source: ABdhPJzRxca4tIWAeMP1wIIPAXLWARs3P9tjhDoamuXeTUxVmlzcDkDjwpb3wsN/h1zzFrY19nCeOA== X-Received: by 2002:a05:6a00:8ca:b029:15d:7af3:8360 with SMTP id s10-20020a056a0008cab029015d7af38360mr7079165pfu.12.1604022573470; Thu, 29 Oct 2020 18:49:33 -0700 (PDT) Received: from leoy-ThinkPad-X240s ([103.141.182.112]) by smtp.gmail.com with ESMTPSA id x22sm4316365pfp.181.2020.10.29.18.49.27 (version=TLS1_2 cipher=ECDHE-ECDSA-CHACHA20-POLY1305 bits=256/256); Thu, 29 Oct 2020 18:49:32 -0700 (PDT) Date: Fri, 30 Oct 2020 09:49:23 +0800 From: Leo Yan To: Ian Rogers Subject: Re: [PATCH v3 9/9] perf mem: Document event type 'ldst' Message-ID: <20201030014923.GK16862@leoy-ThinkPad-X240s> References: <20201028063813.8562-1-leo.yan@linaro.org> <20201028063813.8562-10-leo.yan@linaro.org> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.9.4 (2018-02-28) X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20201029_214936_407463_0EEC538D X-CRM114-Status: GOOD ( 22.60 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Mark Rutland , Al Grant , Mathieu Poirier , Kemeng Shi , Peter Zijlstra , Will Deacon , John Garry , LKML , Arnaldo Carvalho de Melo , Alexander Shishkin , Ingo Molnar , James Clark , =?iso-8859-1?Q?Andr=E9?= Przywara , Namhyung Kim , "Naveen N. Rao" , Jiri Olsa , Linux ARM , Wei Li Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Hi Ian, On Thu, Oct 29, 2020 at 06:10:40PM -0700, Ian Rogers wrote: > On Tue, Oct 27, 2020 at 11:39 PM Leo Yan wrote: > > > > The event type 'ldst' is added for recording both load and store memory > > operations, this patch documents for the new event type. > > > > Signed-off-by: Leo Yan > > --- > > tools/perf/Documentation/perf-mem.txt | 6 +++++- > > 1 file changed, 5 insertions(+), 1 deletion(-) > > > > diff --git a/tools/perf/Documentation/perf-mem.txt b/tools/perf/Documentation/perf-mem.txt > > index 199ea0f0a6c0..c50ef37dba72 100644 > > --- a/tools/perf/Documentation/perf-mem.txt > > +++ b/tools/perf/Documentation/perf-mem.txt > > @@ -38,7 +38,11 @@ OPTIONS > > > > -t:: > > --type=:: > > - Select the memory operation type: load or store (default: load,store) > > + Select the memory operation type: load, store, ldst (default: load,store). > > + The type 'ldst' means the single event can record both for load and store > > + operations; Intel and PowerPC support the types 'load' and 'store' but > > + 'ldst' cannot be used; on Arm64, it uses Arm SPE as memory events and > > + user needs to specify one of these three types. > > Naive question, could the type remain load,store for mem record and > the tool internally change it to ldst for ARM SPE? Good point. I understand you are suggesting to avoid complex options for usage, looked again at the code, my conclusion is it's feabile to convert 'load,store' to Arm SPE's ldst type. If find any block issue when I experiment this idea, will come back to update it; otherwise, I will apply the suggestion in next spin. Thanks for suggestion, Leo _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel