OSDN Git Service

thermal: intel: powerclamp: Fix cur_state for multi package system
authorSrinivas Pandruvada <srinivas.pandruvada@linux.intel.com>
Wed, 1 Feb 2023 20:39:41 +0000 (12:39 -0800)
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>
Thu, 2 Feb 2023 20:08:32 +0000 (21:08 +0100)
commit8e47363588377e1bdb65e2b020b409cfb44dd260
treec5cb02a2d9ceb0bb74585b198f3bf413e79dd6a8
parent2153a87ff9ef5537b8a96e7c94a9d79a78c7a30c
thermal: intel: powerclamp: Fix cur_state for multi package system

The powerclamp cooling device cur_state shows actual idle observed by
package C-state idle counters. But the implementation is not sufficient
for multi package or multi die system. The cur_state value is incorrect.
On these systems, these counters must be read from each package/die and
somehow aggregate them. But there is no good method for aggregation.

It was not a problem when explicit CPU model addition was required to
enable intel powerclamp. In this way certain CPU models could have
been avoided. But with the removal of CPU model check with the
availability of Package C-state counters, the driver is loaded on most
of the recent systems.

For multi package/die systems, just show the actual target idle state,
the system is trying to achieve. In powerclamp this is the user set
state minus one.

Also there is no use of starting a worker thread for polling package
C-state counters and applying any compensation for multiple package
or multiple die systems.

Fixes: b721ca0d1927 ("thermal/powerclamp: remove cpu whitelist")
Signed-off-by: Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com>
Cc: 4.14+ <stable@vger.kernel.org> # 4.14+
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
drivers/thermal/intel/intel_powerclamp.c