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=-8.2 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED,USER_AGENT_SANE_1 autolearn=unavailable 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 29381C55ABD for ; Fri, 13 Nov 2020 20:04: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 906BB21D7F for ; Fri, 13 Nov 2020 20:04:29 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="lQQTEnIx" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 906BB21D7F Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=arm.com 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=ag3Ev/09wZVzojhSG6M+A5iu95V1loM0susRXxa3ae8=; b=lQQTEnIx6iJpwFDe/Ad+AIceD ziqyS5/lgt1S/VrSc5zWs+yCNzB7Al5Lq/a9/8s8QBmXqFrWSYv+NkI6nEOGMJdPkoysPj/Kkb/Nq LED/t10uT0ANcox3gMd5YUVoBdmisqrk6uH+WKgdKO7WvyJ+aw3o0VxGD26yhj75EPK+9pFErtAcX wTqxACcL5aY/uK/jXJIjrj9WYOH9cGm7G5J8SwIt2O9PdoHhLzTR6fdnge3MjqkfHuD3wbgxTJX1R 3AS9w6CuP7xGopV8q/7DOjQYc6Wf0fVfYCaSOGL9Br8gVqzF3N8cSjUu1umVvliYMuEWKvvrvYYen e6Tf4yQ/w==; Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1kdfIc-0003mK-Uz; Fri, 13 Nov 2020 20:03:58 +0000 Received: from mail.kernel.org ([198.145.29.99]) by merlin.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1kdfIZ-0003m0-Rg for linux-arm-kernel@lists.infradead.org; Fri, 13 Nov 2020 20:03:56 +0000 Received: from gaia (unknown [2.26.170.190]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id E316122242; Fri, 13 Nov 2020 20:03:53 +0000 (UTC) Date: Fri, 13 Nov 2020 20:03:51 +0000 From: Catalin Marinas To: Ionela Voinescu Subject: Re: [PATCH v4 3/3] arm64: implement CPPC FFH support using AMUs Message-ID: <20201113200351.GB20519@gaia> References: <20201106125334.21570-1-ionela.voinescu@arm.com> <20201106125334.21570-4-ionela.voinescu@arm.com> <20201113141658.v2oq47nzerx3abga@bogus> <20201113163712.GA5058@arm.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20201113163712.GA5058@arm.com> User-Agent: Mutt/1.10.1 (2018-07-13) X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20201113_150356_018025_12982169 X-CRM114-Status: GOOD ( 26.75 ) 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@arm.com, linux-kernel@vger.kernel.org, Sudeep Holla , will@kernel.org, morten.rasmussen@arm.com, 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 On Fri, Nov 13, 2020 at 04:37:12PM +0000, Ionela Voinescu wrote: > On Friday 13 Nov 2020 at 14:16:58 (+0000), Sudeep Holla wrote: > [..] > > > diff --git a/arch/arm64/kernel/topology.c b/arch/arm64/kernel/topology.c > > > index b8cb16e3a2cc..7c9b6a0ecd6a 100644 > > > --- a/arch/arm64/kernel/topology.c > > > +++ b/arch/arm64/kernel/topology.c > > > @@ -147,6 +147,9 @@ void update_freq_counters_refs(void) > > > > > > static inline bool freq_counters_valid(int cpu) > > > { > > > + if ((cpu >= nr_cpu_ids) || !cpumask_test_cpu(cpu, cpu_present_mask)) > > > + return false; > > > + > > > if (!cpu_has_amu_feat(cpu)) { > > > pr_debug("CPU%d: counters are not supported.\n", cpu); > > > return false; > > > @@ -323,3 +326,64 @@ void topology_scale_freq_tick(void) > > > this_cpu_write(arch_core_cycles_prev, core_cnt); > > > this_cpu_write(arch_const_cycles_prev, const_cnt); > > > } > > > + > > > +#ifdef CONFIG_ACPI_CPPC_LIB > > > +#include > > > > Not sure what arm64 maintainers prefer, but this code has nothing to do > > with topolopy strictly speaking. I wonder if we can put it in separate > > Yes, you are correct. I am/was wondering the same for all the > counters/AMU related functions, but given they were only used for > topology_scale_freq_tick() *until now*, it was okay to keep them in > topology.c. > > But I might soon have at least one additional (to FIE and FFH) small > usecase for them in the implementation of arch_freq_get_on_cpu(), so all > these functions might be better off in a separate file as well. > > Side note: I don't think frequency invariance is strictly speaking > related to topology either. Nether are other functions in the > arch_topology driver. It's likely we got used to placing all > arch function implementation in either the arch_topology driver or the > /kernel/topology.c. Yeah, it looks like these topology files became a dumping ground for whatever power related ;). I'm ok with these patches as they are for now but it would be good to do some refactoring on top and maybe move them to an amu.c file (it's not urgent, it can be for 5.12 or when you plan to add more stuff next). I don't have an opinion for arch_topology.c, so far it doesn't seem to have any AMU stuff in it. -- Catalin _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel