Hi Ricardo,


On 11/6/21 1:33 AM, Ricardo Neri wrote:

From: Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com>

Add a new netlink event to notify change in CPU capabilities in terms of
performance and efficiency.

Is this going to be handled by some 'generic' tools? If yes, maybe
the values for 'performance' might be aligned with capacity
[0,1024] ? Or are they completely not related so the mapping is
simply impossible?

Firmware may change CPU capabilities as a result of thermal events in the
system or to account for changes in the TDP (thermal design power) level.

This notification type will allow user space to avoid running workloads
on certain CPUs or proactively adjust power limits to avoid future events.

Cc: Andi Kleen <redacted>
Cc: Aubrey Li <redacted>
Cc: Tim Chen <redacted>
Cc: "Ravi V. Shankar" <redacted>
Reviewed-by: Len Brown <redacted>
Signed-off-by: Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com>
---
  drivers/thermal/thermal_netlink.c | 52 +++++++++++++++++++++++++++++++
  drivers/thermal/thermal_netlink.h | 13 ++++++++
  include/uapi/linux/thermal.h      |  6 +++-
  3 files changed, 70 insertions(+), 1 deletion(-)

[snip]

  
+struct cpu_capability {
+	int cpu;
+	int perf;
+	int eff;

Why not use the full names, instead of thse shortcuts? We use full
naming e.g. in cpufreq framework such as 'frequency' not 'freq'.
The 'eff' is really not meaningful ('perf' a bit less but it has
to meanings in kernel).

Regards,
Lukasz