[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <Z4UdWpRie1ZsJ4tk@thinkpad2024>
Date: Mon, 13 Jan 2025 09:04:10 -0500
From: "John B. Wyatt IV" <jwyatt@...hat.com>
To: Shuah Khan <skhan@...uxfoundation.org>,
Thomas Renninger <trenn@...e.com>
Cc: linux-pm@...r.kernel.org, Aboorva Devarajan <aboorvad@...ux.ibm.com>,
Shuah Khan <shuah@...nel.org>,
"Rafael J. Wysocki" <rafael@...nel.org>,
linux-kernel@...r.kernel.org, John Kacur <jkacur@...hat.com>,
"John B. Wyatt IV" <sageofredondo@...il.com>
Subject: Re: [PATCH] pm: cpupower: Add missing residency header changes in
cpuidle.h to SWIG
Hello Shuah,
Did you have a chance to look at this? This is the second header change
that was missing from the bindings.
On Wed, Jan 08, 2025 at 05:18:44PM -0500, John B. Wyatt IV wrote:
> "tools/cpupower: display residency value in idle-info" added a new
> function to cpuidle.h. This patch adds them to the bindings.
>
> Link: https://lore.kernel.org/linux-pm/20240809083728.266697-1-aboorvad@linux.ibm.com/
>
> Tested by compiling both libcpupower and the headers; running the test
> script that does not use the functions as a basic sanity test.
>
> Signed-off-by: "John B. Wyatt IV" <jwyatt@...hat.com>
> Signed-off-by: "John B. Wyatt IV" <sageofredondo@...il.com>
> ---
> tools/power/cpupower/bindings/python/raw_pylibcpupower.swg | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/tools/power/cpupower/bindings/python/raw_pylibcpupower.swg b/tools/power/cpupower/bindings/python/raw_pylibcpupower.swg
> index a8226c79cfea..d82af6fa93c3 100644
> --- a/tools/power/cpupower/bindings/python/raw_pylibcpupower.swg
> +++ b/tools/power/cpupower/bindings/python/raw_pylibcpupower.swg
> @@ -163,6 +163,8 @@ int cpuidle_state_disable(unsigned int cpu, unsigned int idlestate,
> unsigned int disable);
> unsigned long cpuidle_state_latency(unsigned int cpu,
> unsigned int idlestate);
> +unsigned long cpuidle_state_residency(unsigned int cpu,
> + unsigned int idlestate);
> unsigned long cpuidle_state_usage(unsigned int cpu,
> unsigned int idlestate);
> unsigned long long cpuidle_state_time(unsigned int cpu,
> --
> 2.47.1
>
--
Sincerely,
John Wyatt
Software Engineer, Core Kernel
Red Hat
Powered by blists - more mailing lists