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 5029FC0032E for ; Sun, 22 Oct 2023 09:10:48 +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:MIME-Version:References: Message-ID:Subject:CC: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=F2xxPTTe9yUMSyqGzFEUJFTc135E1DrN+dnb0ttpD0Y=; b=o9Z9J5Wz45UcKy 0wflX3vhzFJNfaFEnViFE2l86hRZ0BwkCBuqYJ5kyxDYfHQzNZoL3BScyMgNsJGSnxuVNJdJVEjCa OiTyDoh5TbpiZPm8XdJaEBiSj837AuZVIDW1SdLUKawAuTjmhsJPULzL8v9ugMsgV7LHnPNicV48P pqo6/fJckkxlYttj1hGj62jszEAFSnJIDnLcDBjBh0UrKq1wh/Er3OZU10q+aJQyjOFAtdgUWwKlb sf92dhhykKT0Uxp0F/1mJIa1g+G/1iVTDt1DeJ+zSxkS5osI4BLDdHW8IvadYNCSBCulwLd1f6Gvc 5T4X8MSgozkEtczgbLaA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1quUTP-0057DY-35; Sun, 22 Oct 2023 09:10:15 +0000 Received: from 60-248-80-70.hinet-ip.hinet.net ([60.248.80.70] helo=Atcsqr.andestech.com) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1quUTL-0057Cx-0q; Sun, 22 Oct 2023 09:10:13 +0000 Received: from mail.andestech.com (ATCPCS16.andestech.com [10.0.1.222]) by Atcsqr.andestech.com with ESMTP id 39M99FBx063388; Sun, 22 Oct 2023 17:09:15 +0800 (+08) (envelope-from peterlin@andestech.com) Received: from APC323 (10.0.12.98) by ATCPCS16.andestech.com (10.0.1.222) with Microsoft SMTP Server id 14.3.498.0; Sun, 22 Oct 2023 17:09:12 +0800 Date: Sun, 22 Oct 2023 17:09:09 +0800 From: Yu-Chien Peter Lin To: Conor Dooley CC: , , , , , , , , , , , , , , , , , , , , , , , , Subject: Re: [RFC PATCH v2 07/10] perf: RISC-V: Move T-Head PMU to CPU feature alternative framework Message-ID: References: <20231019140119.3659651-1-peterlin@andestech.com> <20231019-predator-quartet-e56f43d5aa8d@spud> <20231020-snippet-diffusive-1a6052d52aae@spud> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20231020-snippet-diffusive-1a6052d52aae@spud> User-Agent: Mutt/2.2.10 (2023-03-25) X-Originating-IP: [10.0.12.98] X-DNSRBL: X-MAIL: Atcsqr.andestech.com 39M99FBx063388 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20231022_021012_080823_7ED5880C X-CRM114-Status: GOOD ( 40.74 ) 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 Hi Conor, On Fri, Oct 20, 2023 at 10:05:20AM +0100, Conor Dooley wrote: > On Fri, Oct 20, 2023 at 04:54:58PM +0800, Yu-Chien Peter Lin wrote: > > On Thu, Oct 19, 2023 at 05:13:00PM +0100, Conor Dooley wrote: > > > On Thu, Oct 19, 2023 at 10:01:19PM +0800, Yu Chien Peter Lin wrote: > > > > > > $subject: perf: RISC-V: Move T-Head PMU to CPU feature alternative framework > > > > > > IMO, this should be "RISC-V, perf:" or just "RISC-V" as the changes > > > being made to the arch code are far more meaningful than those > > > elsewhere. > > > > OK will update the subject to "RISC-V:" > > > > > > The custom PMU extension was developed to support perf event sampling > > > > prior to the ratification of Sscofpmf. Instead of utilizing the standard > > > > bits and CSR of Sscofpmf, a set of custom CSRs is added. So we may > > > > consider it as a CPU feature rather than an erratum. > > > > > > > > T-Head cores need to append "xtheadpmu" to the riscv,isa-extensions > > > > for each cpu node in device tree, and enable CONFIG_THEAD_CUSTOM_PMU > > > > for proper functioning as of this commit. > > > > > > And in doing so, you regress break perf for existing DTs :( > > > You didn't add the property to existing DTS in-kernel either, so if this > > > series was applied, perf would just entirely stop working, no? > > > > Only `perf record/top` stop working I think. > > > > There are too many users out there, and don't have the boards to > > test, so leave those DTS unchanged, it would be great if T-Head > > community could help to check/update their DTS. > > So, there are too many users to add xtheadpmu to the devicetrees, but > not too many users to make changes that will cause a regression? > I'm not following the logic here, sorry. humm, I'll try. I assume that the sun20i-d1s.dtsi is all I need to update for T-Head PMU. > > > > Signed-off-by: Yu Chien Peter Lin > > > > --- > > > > Hi All, > > > > > > > > This is in preparation for introducing other PMU alternative. > > > > We follow Conor's suggestion [1] to use cpu feature alternative > > > > framework rather than errta, if you want to stick with errata > > > > alternative or have other issues, please let me know. Thanks. > > > > > > Personally, I like this conversion, but it is going to regress support > > > for perf on any T-Head cores which may be a bitter pill to get people to > > > actually accept... > > > Perhaps we could add this "improved" detection in parallel, and > > > eventually remove the m*id based stuff in the future. > > > > > > > [1] https://patchwork.kernel.org/project/linux-riscv/patch/20230907021635.1002738-4-peterlin@andestech.com/#25503860 > > > > > > > > Changes v1 -> v2: > > > > - New patch > > > > --- > > > > > @@ -805,7 +816,8 @@ static int pmu_sbi_setup_irqs(struct riscv_pmu *pmu, struct platform_device *pde > > > > if (riscv_isa_extension_available(NULL, SSCOFPMF)) { > > > > riscv_pmu_irq_num = RV_IRQ_PMU; > > > > riscv_pmu_use_irq = true; > > > > - } else if (IS_ENABLED(CONFIG_ERRATA_THEAD_PMU) && > > > > + } else if (riscv_isa_extension_available(NULL, XTHEADPMU) && > > > > + IS_ENABLED(CONFIG_THEAD_CUSTOM_PMU) && > > > > riscv_cached_mvendorid(0) == THEAD_VENDOR_ID && > > > > riscv_cached_marchid(0) == 0 && > > > > riscv_cached_mimpid(0) == 0) { > > > > > > Can all of the m*id checks be removed, since the firmware is now > > > explicitly telling us that the T-Head PMU is supported? > > > > I can only comfirm that boards with "allwinner,sun20i-d1" compatible > > string uses the T-Head PMU device callbacks. > > I'm not sure how that is an answer to my question. Sorry for that unclear answer. Yes, I agree we no longer need to check the m*id here. In OpenSBI, it appears that allwinner D1 is the only platform that has T-Head PMU support, the other T-Head platforms need to ensure that the callbacks [1] are registered in order to work with SBI PMU driver in kernel. Regards, Peter Lin [1] https://github.com/riscv-software-src/opensbi/blob/v1.3.1/platform/generic/allwinner/sun20i-d1.c#L263-L272 > Thanks, > Conor. _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel