Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 295044 - sys-power/cpufreqd-2.3.4-r1 doesn't see CPU temperature sensors via lm_sensors3
Summary: sys-power/cpufreqd-2.3.4-r1 doesn't see CPU temperature sensors via lm_sensors3
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: AMD64 Linux
: High major (vote)
Assignee: Mobile Herd (OBSOLETE)
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-11-29 13:53 UTC by Vladimir
Modified: 2010-01-10 23:35 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
cpufreqd-2.3.4-lm_sensors-3.patch (cpufreqd-2.3.4-lm_sensors-3.patch,5.32 KB, patch)
2010-01-10 05:42 UTC, Francois Guimond
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Vladimir 2009-11-29 13:53:42 UTC
cpufreqd-2.3.4-r1 doesn't see my CPU temperature sensors via sensors-plugin.
(I have lm_sensors-3 installed.)
sensors-plugin parses config file entries correctly, but it seems to me, that lm_sensors give no sensors for my chip. But if I type "sensors -u", I get the message:
"Core0 Temp:
    temp1_input:  45

Core1 Temp:
    temp3_input: -49"

So I can clearly see, that there ARE temperature sensors on my chip (temp1_input), but cpufreqd ignores the "sensor=temp1_input:61-100", for example, in my cpufreqd.conf file.

Reproducible: Always

Steps to Reproduce:
1. emerge sys-apps/lm_sensors-3.1.1 sys-power/cpufreqd-2.3.4-r1
2. type "sensors -u" and see the sensors information.
3. start cpufreqd and see the /va/log/messages file to see the "Discarding" messages from cpufreqd.

Actual Results:  
Cpufreqd ignores the temperature sensors.

Expected Results:  
Cpufrqd shouldn't ignore temperature sensors.
Comment 1 Francois Guimond 2010-01-10 05:42:25 UTC
Created attachment 215896 [details, diff]
cpufreqd-2.3.4-lm_sensors-3.patch

Had the same problem, found out there is a bug in the patch in portage that was added in bug #233481.  When it browses the chips, it uses the same iterator for the features than for the chips, so it doesn't see the "temp1" feature on Intel Core 2 chips (btw, you also need to change your config to be "sensor=temp1:61-100").

Here's the fixed patch to replace the other one.
Comment 2 Vladimir 2010-01-10 12:27:44 UTC
(In reply to comment #1)
> ...
> Here's the fixed patch to replace the other one.
> 

  Thank you very much for this patch!!! :) I am happy to see my cpufreqd daemon working properly again on my laptop!
  Thank you!

  I have also reported about your patch to upstream developer. I have reported him about this bug earlier, but looks like he hasn't yet figured out what the problem was. Now I hope he will accept this patch and fix this problem upstream.

  Here's the link to this topic on SourceForge:
http://sourceforge.net/tracker/?func=detail&aid=2920219&group_id=58904&atid=489239
Comment 3 Thilo Bangert (RETIRED) (RETIRED) gentoo-dev 2010-01-10 23:35:31 UTC
cpufreqd-2.3.4-r2 has the proposed patch.
thanks for the report and for the solution.