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=-6.7 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI, SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED,USER_AGENT_SANE_1 autolearn=no 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 ADACCC64E7B for ; Thu, 3 Dec 2020 06:41:11 +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 483BC206B6 for ; Thu, 3 Dec 2020 06:41:11 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 483BC206B6 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=fbTc2qKqykG6CNCAJLBH4B7OzntoSMM1O0nDXvNN01s=; b=zPjdA+qiQzy9xEbj46uyTxy5V UKGMuq7DPD/1uxAoiKiIMe1t7yUfqfkLodSEBJv0jRR44TTvt9WyZLJWULQFVBj3ZI2WQhx6Pppd0 6Las8CCjh/r0c9egM9NsIOWxbQ5M17OJoq5yU3hdvWfWiiREkiR2szcoPQxkFGvrmTti2SEErSJfd iFZXPl4frLvdp9ZLi/qkjRMnsA51u5ieyI+Y0Nseg8V9QeSo3R0cpoCW7fdfLH/iHl0RmMiRRryZF OS9c9DN2Ll1wEUJ6isxR+IiMzJZqjpYavGxnEd+H6KYXRzsirHbP93iLkOWO0aAXk9r8cxLDo4pCP E6bk7Zrdg==; Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1kkiHU-0008HQ-Ix; Thu, 03 Dec 2020 06:39:56 +0000 Received: from mail-pj1-x102a.google.com ([2607:f8b0:4864:20::102a]) by merlin.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1kkiHQ-0008Gy-Ij for linux-arm-kernel@lists.infradead.org; Thu, 03 Dec 2020 06:39:54 +0000 Received: by mail-pj1-x102a.google.com with SMTP id z12so565483pjn.1 for ; Wed, 02 Dec 2020 22:39:50 -0800 (PST) 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=p90ZTrcODaMIh1PUcjXTtSOF3aHh5Iqv2c9lQCLd5uw=; b=mQlNuueE54XSMWq6SP8fFDIEQPOEDPETEey3dENLqhob4IP0rwSDaMWufXPbwQrTy5 DfOrJG0MoN015zDuWdAoZ4n0MZyPwbYHfBWIKxRgyAUBU8n1e0wiQO1R+V7ZpQmTwAEx Dv4qJX6AA5LkzCGWlZJt2qIot4zzQBc8GU68zXp4UDW6ZK01adsFGAIjkUQMLlNpuBRr eO3/jXso0XELDb0aW38cbsWgQVhK175ctbxvqRBykQYaE45dzfpt/UoIGjViFAY404Dq XzE3KsYbpByBOeAmpCl4EtATdsHpDQQcly+6UBNE16+Jp/Q2yirUSTKopr+zqgNnwDY5 cTng== 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=p90ZTrcODaMIh1PUcjXTtSOF3aHh5Iqv2c9lQCLd5uw=; b=gFDoMKYlNysji9GPn8/BM3YY4c8GyLE3y/vVCJvA/vaIDTVmZed/jRDt9CrzdhQXnx Uh6HRx+TIr0fPd6A96lpvWsHX+Fxnh4DJMIWuc5l4n5sLuFQDvYFjN/ScLjfHJkgJoSt QRNWM9TXlps3EcNlYPRPelUzx0D7c+24vgGwiO/EYH+w/EYVKR8v2P2kTbVR24RalgJ6 +FABixoKgDvaxLOT1t4858B4rJ0QhgJZgbHemYtoDPgnHpSnEgYUVzO0eaRnIOHQRQa8 2lH8959vR5fxDvwzkyNBkz8aATnpN6Nm4c2Bo+mENEZmwX+zAeAVJcl28XpPgqNOEs6b lMJg== X-Gm-Message-State: AOAM530BnqFbXJbPmI96XLvXWFHkrO7iTufsFa/PlBlUsIfjXiR4AnJ8 T4m0e4BCnkbEZj6E3K68LD7mIg== X-Google-Smtp-Source: ABdhPJyOoElyxwiAEuD0omkoyT6H/99s9lIaMAXnW6c09Gjnn8aUkR6r+ju4a9dxwk4XRIAiskjs6A== X-Received: by 2002:a17:902:b498:b029:da:84a7:be94 with SMTP id y24-20020a170902b498b02900da84a7be94mr1731127plr.52.1606977588537; Wed, 02 Dec 2020 22:39:48 -0800 (PST) Received: from leoy-ThinkPad-X240s ([202.155.204.36]) by smtp.gmail.com with ESMTPSA id j10sm306546pji.29.2020.12.02.22.39.45 (version=TLS1_2 cipher=ECDHE-ECDSA-CHACHA20-POLY1305 bits=256/256); Wed, 02 Dec 2020 22:39:47 -0800 (PST) Date: Thu, 3 Dec 2020 14:39:41 +0800 From: Leo Yan To: Will Deacon Subject: Re: [PATCH] drivers/perf: Enable PID_IN_CONTEXTIDR with SPE Message-ID: <20201203063941.GE28939@leoy-ThinkPad-X240s> References: <20201130162454.28255-1-james.clark@arm.com> <20201130164650.GA25187@willie-the-truck> <20201201041040.GC28939@leoy-ThinkPad-X240s> <20201201230935.GD28496@willie-the-truck> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20201201230935.GD28496@willie-the-truck> 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-20201203_013952_946968_E6160C6F X-CRM114-Status: GOOD ( 32.04 ) 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 , Suzuki K Poulose , John Garry , linux-kernel@vger.kernel.org, linux-perf-users@vger.kernel.org, James Clark , linux-arm-kernel@lists.infradead.org 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 Will, [ + Mathieu ] On Tue, Dec 01, 2020 at 11:09:36PM +0000, Will Deacon wrote: > On Tue, Dec 01, 2020 at 12:10:40PM +0800, Leo Yan wrote: > > On Mon, Nov 30, 2020 at 04:46:51PM +0000, Will Deacon wrote: > > > On Mon, Nov 30, 2020 at 06:24:54PM +0200, James Clark wrote: > > > > Enable PID_IN_CONTEXTIDR by default when Arm SPE is enabled. > > > > This flag is required to get PID data in the SPE trace. Without > > > > it the perf tool will report 0 for PID which isn't very useful, > > > > especially when doing system wide profiling or profiling > > > > applications that fork. > > > > > > Can perf not figure out the pid some other way? (e.g. by tracing context > > > switches and correlating that with the SPE data?). > > > > For perf 'per-thread' mode, we can use context switch trace event as > > assisted info to select thread context. But for "system wide" mode and > > "snapshot" mode in perf tool, since the trace data is continuous, I > > think we cannot use context switch trace event to correlate the SPE > > trace data. > > Is there no way to correlate them with something like CNTVCT? Good point. Yes, we can convert CNTVCT to system time; I read the code in the perf's intel-pt.c and found the timestamp is used to correlate the auxtrace heap. I think it's better to dig more for detailed implementation. > > > Also, how does this work with pid namespaces? > > > > Here we are studying the implemetation of Intel-PT and Arm CoreSight. > > > > The context ID is stored into the hardware trace data when record; > > afterwards when perf tool decodes the trace data and detects the > > packet for context ID, it will select the machine's thread context in > > perf [1]. Since the perf tool gathers all the threads infomation in > > perf data file, based on the context ID, it can find the corresponding > > thread pointer with function machine__find_thread() [2]. > > > > Since your question is for "pid namespace", to be honest, I don't know > > how perf tool to handle any confliction for differrent processes share > > the same PID, and I am not sure if you are asking CGroup related stuff > > or not. If this cannot answer your question, please let me know. > > My point was that the pid value written to CONTEXTIDR is a global pid > and does not take namespacing into account. If perf is run inside a pid > namespace, it will therefore not work. Understand now. The perf events PERF_RECORD_ITRACE_START/PERF_RECORD_SWITCH/ PERF_RECORD_SWITCH_CPU_WIDE can be used to set pid/tid in perf. So this would be a safe way for perf tool running in pid namespace. Loop in Mathieu, this is a common issue for both Arm SPE and CoreSight (IIRC, though CoreSight's timestamp is not strictly attaching to Arm arch timer counter, the trend is to unify this for using arch timer counter). I think James could continue to upstream a new patch by following your suggestion for enabling PID_IN_CONTEXTIDR, eventually, it's a feature for Arm SPE to record CONTEXTIDR in its packet. Your questions inspired me, thanks! Leo _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel