diff options
| author |   Janakarajan Natarajan <Janakarajan.Natarajan@amd.com> | 2019-11-05 17:16:57 +0000 |
|---|---|---|
| committer |   Shuah Khan <skhan@linuxfoundation.org> | 2019-11-05 17:23:23 -0700 |
| commit | 4611a4fb0cce3973dce8c9d74e5d6261ffa4210f (patch) | |
| tree | 581e1cf7f8b366c3aeec28a0adb493ea0f5a43af /tools/power/cpupower/ToDo | |
| parent | cpupower: mperf_monitor: Update cpupower to use the RDPRU instruction (diff) | |
| download | linux-dev-4611a4fb0cce3973dce8c9d74e5d6261ffa4210f.tar.xz linux-dev-4611a4fb0cce3973dce8c9d74e5d6261ffa4210f.zip | |
cpupower: ToDo: Update ToDo with ideas for per_cpu_schedule handling
Based on Thomas Renninger's feedback/ideas. Re-structure the code
to better handle the per_cpu_schedule mechanism which was introduced
when adding support for AMD Zen based processors.
Signed-off-by: Janakarajan Natarajan <Janakarajan.Natarajan@amd.com>
Acked-by: Thomas Renninger <trenn@suse.de>
Signed-off-by: Shuah Khan <skhan@linuxfoundation.org>
Diffstat (limited to 'tools/power/cpupower/ToDo')
| -rw-r--r-- | tools/power/cpupower/ToDo | 14 |
1 files changed, 14 insertions, 0 deletions
diff --git a/tools/power/cpupower/ToDo b/tools/power/cpupower/ToDo index 6e8b89f282e6..b196a139a3e4 100644 --- a/tools/power/cpupower/ToDo +++ b/tools/power/cpupower/ToDo @@ -8,3 +8,17 @@ ToDos sorted by priority: - Add another c1e debug idle monitor -> Is by design racy with BIOS, but could be added with a --force option and some "be careful" messages +- Add cpu_start()/cpu_stop() callbacks for monitor + -> This is to move the per_cpu logic from inside the + monitor to outside it. This can be given higher + priority in fork_it. +- Fork as many processes as there are CPUs in case the + per_cpu_schedule flag is set. + -> Bind forked process to each cpu. + -> Execute start measures via the forked processes on + each cpu. + -> Run test executable in a forked process. + -> Execute stop measures via the forked processes on + each cpu. + This would be ideal as it will not introduce noise in the + tested executable. |