From patchwork Tue Jun 6 15:57:54 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Beata Michalska X-Patchwork-Id: 13269409 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 AC7F6C7EE24 for ; Tue, 6 Jun 2023 15:58:45 +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:MIME-Version:Message-Id:Date:Subject:Cc :To:From:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:In-Reply-To:References: List-Owner; bh=PukIc9rdKFi8ZwJpZg5GrAIQikxvhxMsNdgga+xtu0k=; b=t1SsAZvV7cYA5O yRt8/EpLRDwoUioWxkoyLtDPQI4+yIqCoAzPaZxfs6AhjRS4qGj3dRitNVBJ40eBp0rxvhgcRP6bB ysEypChuoFyu5nErHHFOP8/wJcxXuxTZONB8sjJBSeMJ2omytfaKdka/ZI3ArWMyVG1blsNcx0T5Z D2nbDf6eWPOobLDVi5k5DFMn0hsGAmCZFfQoG3YR0S7mysR9FXD1eOqfLn7qyEWPi24zneKm4jv8k /ohXS4LhDntw1FKnamZzEc8wJaiNElVKyG+bMZjMxBl240bWUQTxBHUYvAlsPdvQosUf9FQ0suYSF NA9+I9Rf63tdADYY3sTA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1q6Z4Y-002KfJ-0D; Tue, 06 Jun 2023 15:58:14 +0000 Received: from foss.arm.com ([217.140.110.172]) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1q6Z4U-002Kbo-1s for linux-arm-kernel@lists.infradead.org; Tue, 06 Jun 2023 15:58:12 +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 1E3642F4; Tue, 6 Jun 2023 08:58:50 -0700 (PDT) Received: from e125905.cambridge.arm.com (e125905.cambridge.arm.com [10.1.198.22]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPA id 026C73F6C4; Tue, 6 Jun 2023 08:58:02 -0700 (PDT) From: Beata Michalska To: linux-kernel@vger.kernel.org, linux-pm@vger.kernel.org, linux-arm-kernel@lists.infradead.org Cc: catalin.marinas@arm.com, mark.rutland@arm.com, will@kernel.org, rafael@kernel.org, viresh.kumar@linaro.org, sudeep.holla@arm.com, ionela.voinescu@arm.com, sumitg@nvidia.com, yang@os.amperecomputing.com Subject: [PATCH] arm64: Provide an AMU-based version of arch_freq_get_on_cpu Date: Tue, 6 Jun 2023 16:57:54 +0100 Message-Id: <20230606155754.245998-1-beata.michalska@arm.com> X-Mailer: git-send-email 2.25.1 MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230606_085810_731936_3F07FC2E X-CRM114-Status: GOOD ( 15.43 ) 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: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org With the Frequency Invariance Engine (FIE) being already wired up with sched tick and making use of relevant (core counter and constant counter) AMU counters, getting the current frequency for a given CPU on supported platforms, can be achieved by utilizing the frequency scale factor which reflects an average CPU frequency for the last tick period length. With that at hand, arch_freq_get_on_cpu dedicated implementation gets enrolled into cpuinfo_cur_freq policy sysfs attribute handler, which is expected to represent the current frequency of a given CPU, as obtained by the hardware. This is exactly the type of feedback that cycle counters provide. In order to avoid calling arch_freq_get_on_cpu from the scaling_cur_freq attribute handler for platforms that do provide cpuinfo_cur_freq, and yet keeping things intact for those platform that do not, its use gets conditioned on the presence of cpufreq_driver (*get) callback (which also seems to be the case for creating cpuinfo_cur_freq attribute). Suggested-by: Ionela Voinescu Signed-off-by: Beata Michalska Reviewed-by: Sudeep Holla --- arch/arm64/kernel/topology.c | 34 ++++++++++++++++++++++++++++++++++ drivers/cpufreq/cpufreq.c | 9 +++++++-- 2 files changed, 41 insertions(+), 2 deletions(-) diff --git a/arch/arm64/kernel/topology.c b/arch/arm64/kernel/topology.c index 817d788cd866..00a1aa421ec2 100644 --- a/arch/arm64/kernel/topology.c +++ b/arch/arm64/kernel/topology.c @@ -17,6 +17,7 @@ #include #include #include +#include #include #include @@ -251,6 +252,39 @@ static int __init init_amu_fie(void) } core_initcall(init_amu_fie); +unsigned int arch_freq_get_on_cpu(int cpu) +{ + unsigned int freq; + u64 scale; + + if (!cpumask_test_cpu(cpu, amu_fie_cpus)) + return 0; + + if (!housekeeping_cpu(cpu, HK_TYPE_TICK)) { + struct cpufreq_policy *policy = cpufreq_cpu_get(cpu); + int ref_cpu = nr_cpu_ids; + + if (cpumask_intersects(housekeeping_cpumask(HK_TYPE_TICK), + policy->cpus)) + ref_cpu = cpumask_nth_and(cpu, policy->cpus, + housekeeping_cpumask(HK_TYPE_TICK)); + cpufreq_cpu_put(policy); + if (ref_cpu >= nr_cpu_ids) + return 0; + cpu = ref_cpu; + } + /* + * Reversed computation to the one used to determine + * the arch_freq_scale value + * (see amu_scale_freq_tick for details) + */ + scale = per_cpu(arch_freq_scale, cpu); + scale *= cpufreq_get_hw_max_freq(cpu); + freq = scale >> SCHED_CAPACITY_SHIFT; + + return freq; +} + #ifdef CONFIG_ACPI_CPPC_LIB #include diff --git a/drivers/cpufreq/cpufreq.c b/drivers/cpufreq/cpufreq.c index 6b52ebe5a890..9f2cf45bf190 100644 --- a/drivers/cpufreq/cpufreq.c +++ b/drivers/cpufreq/cpufreq.c @@ -710,7 +710,8 @@ static ssize_t show_scaling_cur_freq(struct cpufreq_policy *policy, char *buf) ssize_t ret; unsigned int freq; - freq = arch_freq_get_on_cpu(policy->cpu); + freq = !cpufreq_driver->get ? arch_freq_get_on_cpu(policy->cpu) + : 0; if (freq) ret = sprintf(buf, "%u\n", freq); else if (cpufreq_driver->setpolicy && cpufreq_driver->get) @@ -747,7 +748,11 @@ store_one(scaling_max_freq, max); static ssize_t show_cpuinfo_cur_freq(struct cpufreq_policy *policy, char *buf) { - unsigned int cur_freq = __cpufreq_get(policy); + unsigned int cur_freq; + + cur_freq = arch_freq_get_on_cpu(policy->cpu); + if (!cur_freq) + cur_freq = __cpufreq_get(policy); if (cur_freq) return sprintf(buf, "%u\n", cur_freq);