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 B3F28C87FDC for ; Fri, 9 Jun 2023 04:30:46 +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-Transfer-Encoding:Content-Type: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=JQ75U5tcGVb4CrBQ6ZqgWDSLuet40elJw04qSdw0qtg=; b=IRZolH+Mjl/9q+ ceaWr56DwdSnnjAjY4ETyEIZ+AjA++JvlZmWm17NOVD5g8yNKf+qLq5Vup6BSo02UseJj0QpmO1FU 0nAaDfWyN+PiAY2WpEMcEGlNGYdmxsvZVHa2L/fLQJAnnMONVnd4r1HZ0fJffWjOotf7e87/6aibx vcszaryx4yEWmipdH6Wbi6Sa43XkRRtfiM4PDPX7+MQNz1ajrSrSzJywAKy1JJzcYhrUVrJqxaXBU oMV50DXjdEET4V+B120F0Zj4r9CYPWdvmm+4xOfPIbCu+W60UN4IuNr/axtAw3L13tffKSSRTgMnj 1LVChhCPUGNlwWbCuZsA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1q7TlX-00BbG4-00; Fri, 09 Jun 2023 04:30:23 +0000 Received: from foss.arm.com ([217.140.110.172]) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1q7TlT-00BbF4-2b for linux-arm-kernel@lists.infradead.org; Fri, 09 Jun 2023 04:30:20 +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 9F847AB6; Thu, 8 Jun 2023 21:31:02 -0700 (PDT) Received: from [10.163.44.201] (unknown [10.163.44.201]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 988423F587; Thu, 8 Jun 2023 21:30:12 -0700 (PDT) Message-ID: <006be03c-43da-6482-6bc4-83fe65dcd706@arm.com> Date: Fri, 9 Jun 2023 10:00:09 +0530 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.11.0 Subject: Re: [PATCH V11 06/10] arm64/perf: Enable branch stack events via FEAT_BRBE Content-Language: en-US To: Mark Rutland Cc: linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, will@kernel.org, catalin.marinas@arm.com, Mark Brown , James Clark , Rob Herring , Marc Zyngier , Suzuki Poulose , Peter Zijlstra , Ingo Molnar , Arnaldo Carvalho de Melo , linux-perf-users@vger.kernel.org References: <20230531040428.501523-1-anshuman.khandual@arm.com> <20230531040428.501523-7-anshuman.khandual@arm.com> From: Anshuman Khandual In-Reply-To: X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230608_213019_889895_1097C941 X-CRM114-Status: GOOD ( 15.97 ) 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-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 On 6/5/23 19:13, Mark Rutland wrote: >> +static u64 branch_type_to_brbcr(int branch_type) >> +{ >> + u64 brbcr = BRBCR_EL1_DEFAULT_TS; >> + >> + /* >> + * BRBE need not be paused on PMU interrupt while tracing only >> + * the user space, bcause it will automatically be inside the >> + * prohibited region. But even after PMU overflow occurs, the >> + * interrupt could still take much more cycles, before it can >> + * be taken and by that time BRBE will have been overwritten. >> + * Let's enable pause on PMU interrupt mechanism even for user >> + * only traces. >> + */ >> + brbcr |= BRBCR_EL1_FZP; > I think this is trying to say that we *should* use FZP when sampling the > kernel (due to IRQ latency), and *can* safely use it when sampling userspace, > so it would be good to explain it that way around. Agreed, following updated comment explains why we should enable FZP when sampling kernel, otherwise BRBE will capture unwanted records. It also explains why we should enable FZP even when sampling user space due to IRQ latency. /* * BRBE should be paused on PMU interrupt while tracing kernel * space to stop capturing further branch records. Otherwise * interrupt handler branch records might get into the samples * which is not desired. * * BRBE need not be paused on PMU interrupt while tracing only * the user space, because it will automatically be inside the * prohibited region. But even after PMU overflow occurs, the * interrupt could still take much more cycles, before it can * be taken and by that time BRBE will have been overwritten. * Hence enable pause on PMU interrupt mechanism even for user * only traces as well. */ brbcr |= BRBCR_EL1_FZP; > > It's a bit unfortunate, because where this matters we'll always be losing some > branches either way, but I guess we don't have much say in the matter. _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel