Re: [PATCH v3 4/7] thermal/drivers/tegra: Add driver for Tegra30 thermal sensor

From
Vincent Guittot <vincent.guittot@linaro.org>
Date
2021-06-16 08:30:58
Also in
linux-devicetree, linux-tegra, lkml
Reply to
Viresh Kumar, 2021-06-16
On Wed, 16 Jun 2021 at 10:03, Viresh Kumar [off-list ref] wrote:
+Vincent.

On 15-06-21, 22:32, Dmitry Osipenko wrote:
quoted
IIUC, the cpufreq already should be prepared for the case where firmware
may override frequency. Viresh, could you please clarify what are the
possible implications of the frequency overriding?
The only implication is software would think hardware is running at
some other frequency, while it is not. Not sure if something may break
as a result of this.

The scheduler's view of CPUs will not be same though, i.e. scheduler
will see capacity as X, while in reality it has changed to Y.
thermal_pressure is used by scheduler to balance the load between CPUs
according to the actual max frequency. If the thermal pressure doesn't
reflect reality, scheduler will end up enqueuing too many  tasks on a
throttle CPU.
--
viresh
Thread (22 messages) 22 messages, 6 authors, last reply 2021-06-16
Keyboard shortcuts
hback out one level
jnext message in thread / next series-diff position
kprevious message in thread / previous series-diff position
ldrill in
Escclose help / fold thread tree
?toggle this help