From: "John B. Wyatt IV" <jwyatt@redhat.com>
To: Shuah Khan <skhan@linuxfoundation.org>,
Thomas Renninger <trenn@suse.com>
Cc: linux-pm@vger.kernel.org,
Aboorva Devarajan <aboorvad@linux.ibm.com>,
Shuah Khan <shuah@kernel.org>,
"Rafael J. Wysocki" <rafael@kernel.org>,
linux-kernel@vger.kernel.org, John Kacur <jkacur@redhat.com>,
"John B. Wyatt IV" <sageofredondo@gmail.com>
Subject: Re: [PATCH] pm: cpupower: Add missing residency header changes in cpuidle.h to SWIG
Date: Mon, 13 Jan 2025 09:04:10 -0500 [thread overview]
Message-ID: <Z4UdWpRie1ZsJ4tk@thinkpad2024> (raw)
In-Reply-To: <20250108221852.30771-1-jwyatt@redhat.com>
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@redhat.com>
> Signed-off-by: "John B. Wyatt IV" <sageofredondo@gmail.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
next prev parent reply other threads:[~2025-01-13 14:04 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-01-08 22:18 [PATCH] pm: cpupower: Add missing residency header changes in cpuidle.h to SWIG John B. Wyatt IV
2025-01-13 14:04 ` John B. Wyatt IV [this message]
2025-01-13 16:36 ` Shuah Khan
2025-01-13 23:17 ` Shuah Khan
2025-01-14 20:39 ` John B. Wyatt IV
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=Z4UdWpRie1ZsJ4tk@thinkpad2024 \
--to=jwyatt@redhat.com \
--cc=aboorvad@linux.ibm.com \
--cc=jkacur@redhat.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pm@vger.kernel.org \
--cc=rafael@kernel.org \
--cc=sageofredondo@gmail.com \
--cc=shuah@kernel.org \
--cc=skhan@linuxfoundation.org \
--cc=trenn@suse.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.