• Zhang Rui's avatar
    thermal: intel: pch: improve the cooling delay log · bd30d075
    Zhang Rui authored
    Previously, during suspend, intel_pch_thermal driver logs for every
    cooling iteration, about the current PCH temperature and number of cooling
    iterations that have been tried, like below
    
    [  100.955526] intel_pch_thermal 0000:00:14.2: CPU-PCH current temp [53C] higher than the threshold temp [50C], sleep 1 times for 100 ms duration
    [  101.064156] intel_pch_thermal 0000:00:14.2: CPU-PCH current temp [53C] higher than the threshold temp [50C], sleep 2 times for 100 ms duration
    
    After changing the default delay_cnt to 600, in practice, it is common to
    see tens of the above messages if the system is suspended when PCH
    overheats. Thus, change this log message from dev_warn to dev_dbg because
    it is only useful when we want to check the temperature trend.
    
    At the same time, there is always a one-line message given by the driver
    with the patch applied, with below four possibilities.
    
    1. PCH is cool, no cooling delay needed
    [ 1791.902853] intel_pch_thermal 0000:00:12.0: CPU-PCH is cool [48C]
    
    2. PCH overheats and becomes cool after the cooling delays
    [ 1475.511617] intel_pch_thermal 0000:00:12.0: CPU-PCH is cool [49C] after 30700 ms delay
    
    3. PCH still overheats after the overall cooling timeout
    [ 2250.157487] intel_pch_thermal 0000:00:12.0: CPU-PCH is hot [60C] after 60000 ms delay. S0ix might fail
    
    4. PCH aborts cooling because of wakeup event detected during the delay
    [ 1933.639509] intel_pch_thermal 0000:00:12.0: Wakeup event detected, abort cooling
    Signed-off-by: default avatarZhang Rui <rui.zhang@intel.com>
    Tested-by: default avatarSumeet Pawnikar <sumeet.r.pawnikar@intel.com>
    Signed-off-by: default avatarRafael J. Wysocki <rafael.j.wysocki@intel.com>
    bd30d075
intel_pch_thermal.c 14 KB