Bug 611144

Summary: lm_sensors doesn't load coretemp module automatically
Product: Red Hat Enterprise Linux 6 Reporter: David Kovalsky <dkovalsk>
Component: lm_sensorsAssignee: Nikola Pajkovsky <npajkovs>
Status: CLOSED DUPLICATE QA Contact: BaseOS QE - Apps <qe-baseos-apps>
Severity: medium Docs Contact:
Priority: medium    
Version: 6.0CC: benl, dhoward, ovasik
Target Milestone: rc   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
: 623572 (view as bug list) Environment:
Last Closed: 2010-08-12 08:24:11 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
Bug Depends On:    
Bug Blocks: 623572, 623587    
Attachments:
Description Flags
/proc/cpuinfo
none
proposed patch none

Description David Kovalsky 2010-07-03 22:49:22 UTC
Created attachment 429308 [details]
/proc/cpuinfo

For Core i7 processors and running sensors-detect, coretemp module is not loaded automatically and temperatures not shown. If I manually modprobe coretemp, I see the output.

[test@kovinek ~]$ sensors
thinkpad-isa-0000
Adapter: ISA adapter
fan1:       3563 RPM
temp1:       +78.0°C                                    
temp2:        +0.0°C                                    
temp3:        +0.0°C                                    
temp4:        +0.0°C                                    
temp5:        +0.0°C                                    
temp6:        +0.0°C                                    
temp7:        +0.0°C                                    
temp8:        +0.0°C                                    

(coretemp module which I don't see without manually modprobing coretemp)

coretemp-isa-0000
Adapter: ISA adapter
Core 0:      +76.0°C  (high = +84.0°C, crit = +100.0°C)  

coretemp-isa-0001
Adapter: ISA adapter
Core 1:      +76.0°C  (high = +84.0°C, crit = +100.0°C)  

coretemp-isa-0002
Adapter: ISA adapter
Core 2:      +77.0°C  (high = +84.0°C, crit = +100.0°C)  

coretemp-isa-0003
Adapter: ISA adapter
Core 3:      +77.0°C  (high = +84.0°C, crit = +100.0°C)  

coretemp-isa-0004
Adapter: ISA adapter
Core 4:      +78.0°C  (high = +84.0°C, crit = +100.0°C)  

coretemp-isa-0005
Adapter: ISA adapter
Core 5:      +78.0°C  (high = +84.0°C, crit = +100.0°C)  

coretemp-isa-0006
Adapter: ISA adapter
Core 6:      +77.0°C  (high = +84.0°C, crit = +100.0°C)  

coretemp-isa-0007
Adapter: ISA adapter
Core 7:      +77.0°C  (high = +84.0°C, crit = +100.0°C)  


[test@kovinek ~]$ cat /etc/sysconfig/lm_sensors 
# Generated by sensors-detect on Thu Jul  1 13:38:56 2010
# This file is sourced by /etc/init.d/lm_sensors and defines the modules to
# be loaded/unloaded.
#
# The format of this file is a shell script that simply defines variables:
# HWMON_MODULES for hardware monitoring driver modules, and optionally
# BUS_MODULES for any required bus driver module (for example for I2C or SPI).

HWMON_MODULES="sbs"

# For compatibility reasons, modules are also listed individually as variables
#    MODULE_0, MODULE_1, MODULE_2, etc.
# You should use BUS_MODULES and HWMON_MODULES instead if possible.

MODULE_0=sbs



lm_sensors-3.1.1-8.el6.x86_64

Comment 2 Nikola Pajkovsky 2010-07-07 08:28:11 UTC
Do you have installed lm_sensors-sensord?

Comment 3 David Kovalsky 2010-07-08 12:40:02 UTC
Yes, lm_sensors-sensord-3.1.1-8.el6.x86_64

Comment 4 David Kovalsky 2010-07-08 13:53:53 UTC
Furthermore, even if I set 

HWMON_MODULES="coretemp"
MODULE_0=coretemp

in /etc/sysconfig/lm_sensors

service sensord start doesn't load the module.

Checking the initscript, only sensord is sourced from /etc/sysconfig/. 

It seems like 2 bugs to me:
1) sensors-detect doesn't create a proper config file with coretemp module specified

2) lm_sensors ship a file that isn't used anywhere - /etc/sysconfig/lm_sensors

Comment 5 David Kovalsky 2010-07-08 13:54:41 UTC
/etc/sysconfig/sensord:

# configuration for harware sensors monitoring daemon
# in intevrals use suffix "m" for minutes, "s" for seconds, "h" for hours
# 0 means turning facility off

# interval between scanning alarms
INTERVAL=1m

# interval between logging
LOG_INTERVAL=20m

# interval between RRD logging
# RRD_INTERVAL=1m

# RRD db location
# RRD_LOGFILE=/var/log/sensors.rrd

Comment 6 David Kovalsky 2010-07-08 14:01:00 UTC
If it makes a difference, I ran sensors-detect before installing lm_sensors-sensord.

Issues mentioned above don't seem to be limited only to Core i7, raising priority.

Comment 7 Nikola Pajkovsky 2010-07-21 15:36:36 UTC
Created attachment 433450 [details]
proposed patch

Damn, I forgot add a few lines into init script.

Comment 8 Nikola Pajkovsky 2010-08-12 08:24:11 UTC

*** This bug has been marked as a duplicate of bug 616760 ***