Re: [PATCHv4 1/1] Hwmon: Add core/pkg Threshold Support to Coretemp

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Jean,

[snip] 
> I am still suspicious... Does this mean that the thermal interrupt
> mechanism is disabled by default?
[snip]

As you rightly pointed out, the Interrupts for the thresholds are disabled,
by default. We have to enable them if we decide to do so.
If we enable them, we will get interrupts when the input temperature
Crosses temp1_max and temp1_max_hyst in either directions. 

The temp1_max is supposed to be more than temp1_input and
temp1_max_hyst is supposed to be lesser than temp1_input.
By default the temp1_max_hyst is left at 0.

I wanted to use the ABI names temp1_thresh1 and temp1_thresh0.
But Guenter pointed out those are new ABI names, which nobody
is aware of. So, I reused the existing ABI.

The basic idea is this:
Consider, the input temperature is 40. 
We configure the max to 45 and max_hyst to 35.
Case 1:
------
After a while, the temperature reaches 45. We get an interrupt,
and throttle the factors that cause an increase in temperature.
(For example, reduce CPU frequency)
Case 2:
------
After a while, the temperature reaches 35. We get an interrupt,
and release the throttle (if we had any) or increase the
performance.

Our coretemp driver, on getting these interrupts, will
notify the user space (possibly through Uevent or something)
Then the user space will take appropriate actions.

This is what I had in mind, when I submitted this patch.
I am willing to add code to coretemp that will do this.

Kindly let me know your thoughts on this.

Thanks,
Durga

_______________________________________________
lm-sensors mailing list
lm-sensors@xxxxxxxxxxxxxx
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors


[Index of Archives]     [Linux Kernel]     [Linux Hardware Monitoring]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]

  Powered by Linux