[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <7c583e2e-14b6-40e5-8e12-01584b817e4c@arm.com>
Date: Fri, 25 Apr 2025 17:31:01 +0100
From: Christian Loehle <christian.loehle@....com>
To: Ricardo Neri <ricardo.neri-calderon@...ux.intel.com>,
Sudeep Holla <sudeep.holla@....com>, "Rafael J. Wysocki"
<rafael@...nel.org>, Danilo Krummrich <dakr@...nel.org>,
Srinivas Pandruvada <srinivas.pandruvada@...ux.intel.com>,
Len Brown <len.brown@...el.com>
Cc: linux-kernel@...r.kernel.org, linux-pm@...r.kernel.org,
Ricardo Neri <ricardo.neri@...el.com>
Subject: Re: [PATCH 1/2] arch_topology: Relocate cpu_scale to topology.[h|c]
On 4/19/25 03:55, Ricardo Neri wrote:
> arch_topology.c provides functionality to parse and scale CPU capacity. It
> also provides a corresponding sysfs interface. Some architectures parse
> and scale CPU capacity differently as per their own needs. On Intel
> processors, for instance, it is responsibility of the Intel P-state driver.
>
> Relocate the implementation of that interface to a common location in
> topology.c. Architectures can use the interface and populate it using their
> own mechanisms.
>
> An alternative approach would be to compile arch_topology.c even if not
> needed only to get this interface. This approach would create duplicated
> and conflicting functionality and data structures.
>
> Signed-off-by: Ricardo Neri <ricardo.neri-calderon@...ux.intel.com>
Maybe an FYI for the non-x86 folks, this doesn't break anything on the
usual arm64 setup:
Tested-by: Christian Loehle <christian.loehle@....com>
Powered by blists - more mailing lists