* [lm-sensors] Fwd: [Bug 16755] [lm_sensors] ASSIGNED: [PATCH] Bad
2005-07-11 11:19 [lm-sensors] Fwd: [Bug 16755] [lm_sensors] ASSIGNED: [PATCH] Bad Michel Bouissou
@ 2005-07-11 11:49 ` Jean Delvare
2005-07-11 12:19 ` Michel Bouissou
` (3 subsequent siblings)
4 siblings, 0 replies; 6+ messages in thread
From: Jean Delvare @ 2005-07-11 11:49 UTC (permalink / raw)
To: lm-sensors
Salut Michel,
On 2005-07-11, Michel Bouissou wrote:
> So they asked me to bug you ;-))
And quite rightly so, this is indeed not acceptable that sensors -s has
to be ran twice.
Unfortunately your report isn't as complete as it would need to be for
us to help you. We would need to know:
* Which version of the Linux kernel you are using.
* If this kernel has patches applied which could affect the i2c subsystem
or the it87 driver, we need to know which they are and what they do.
* Which version of "sensors" and "libsensors" you are using.
* If lm_sensors has patches applied, we need a list.
* What is your it87-compatible chip? IT8705? IT87012F? Which revision, if
you happen to know? Is it accessed through the SMBus or as an ISA device?
* The complete sensors initialization script used.
* Which limits exactly don't get set by the first "sensors -s"? All?
Please provide the output of "sensors" before the first "sensors
-s", after the first "sensors -s" and after the second "sensors -s".
* Anything in the logs?
Please keep in mind that this is *expected* that reported alarms won't
disappear immediately after changing the limits. You will always see
them once after that, per chip design (you must not miss once-only
alarms).
> (BTW the sensors@stimpy.netroedge.com email address mentioned on the
> http://www2.lm-sensors.nu/~lm78/support.html web page bounces with "User
> unknown"...)
Correct, I just asked our administrator to fix it. Thanks for reporting.
--
Jean Delvare
^ permalink raw reply [flat|nested] 6+ messages in thread* [lm-sensors] Fwd: [Bug 16755] [lm_sensors] ASSIGNED: [PATCH] Bad
2005-07-11 11:19 [lm-sensors] Fwd: [Bug 16755] [lm_sensors] ASSIGNED: [PATCH] Bad Michel Bouissou
2005-07-11 11:49 ` Jean Delvare
@ 2005-07-11 12:19 ` Michel Bouissou
2005-07-11 12:48 ` Jean Delvare
` (2 subsequent siblings)
4 siblings, 0 replies; 6+ messages in thread
From: Michel Bouissou @ 2005-07-11 12:19 UTC (permalink / raw)
To: lm-sensors
Le Lundi 11 Juillet 2005 11:41, Jean Delvare a ?crit :
> Salut Michel,
Salut :-)
> Unfortunately your report isn't as complete as it would need to be for
> us to help you. We would need to know:
So I'll try my best ;-)
> * Which version of the Linux kernel you are using.
That's a 2.6.12-3mib3v ;-) which means a custom compilation of the Mandrake's
cooker 2.6.12-3
Let me add immediately that I have always had the same lm_sensors "double
initialization needed problem" since I have this board (Gigabyte GA7-VAXP),
and thru each and every version of 2.4.x or 2.6.x kernels, and every version
of lm_sensors I ran on it, for a long time... I had just been too lazy to
report the problem before, but this problem is not related to this precise
kernel...
> * If this kernel has patches applied which could affect the i2c subsystem
> or the it87 driver, we need to know which they are and what they do.
Well, being a Mandrake kernel, it includes a number of patches from Mdk, which
complete list I'm not aware of...
> * Which version of "sensors" and "libsensors" you are using.
These are Mandrake RPMs, I suppose the version is accurate...:
[root@totor etc]# rpm -qa | grep -i sensors
liblm_sensors3-devel-2.9.0-4mdk
lm_sensors-2.9.0-4mdk
liblm_sensors3-2.9.0-4mdk
> * If lm_sensors has patches applied, we need a list.
Uh... I'll try to get one from the source RPMs if you think it's absolutely
necessary ;-))
> * What is your it87-compatible chip? IT8705? IT87012F? Which revision, if
> you happen to know? Is it accessed through the SMBus or as an ISA device?
ITE - IT8705F is written on the chip itself. lm_sensors accesses it thru the
ISA bus.
> * The complete sensors initialization script used.
Please see attached. It's Mandrake's one... Just with my patch to call
"sensors -s" a 2nd time if there are alarms.
> * Which limits exactly don't get set by the first "sensors -s"? All?
As far as I noticed, only the fans RPM thresholds are concerned.
> Please provide the output of "sensors" before the first "sensors
> -s", after the first "sensors -s" and after the second "sensors -s".
I don't have them on hand, but what I can tell from memory is that after the
1st "sensors -s", the fans RPM thresholds are set to 5100+ something RPMs,
which doesn't correspond at all to what I have in sensors.conf.
After the 2nd "sensors -s", they are set properly.
Here's the output of "sensors" with everything set properly here:
[root@totor etc]# sensors
lm90-i2c-0-4c
Adapter: SMBus Via Pro adapter at 5000
LM90 T?: +48?C (low = +15?C, high = +60?C)
Athlon T?: +50.8?C (low = +15.0?C, high = +70.0?C)
LM90 Crit: +70?C (hyst = +60?C)
CPU Crit: +80?C (hyst = +70?C)
it87-isa-0290
Adapter: ISA adapter
CPU core: +1.74 V (min = +1.66 V, max = +1.84 V)
DDRAM: +1.22 V (min = +1.17 V, max = +1.33 V)
3v3 I/O: +3.23 V (min = +3.14 V, max = +3.47 V)
+5V: +5.04 V (min = +4.76 V, max = +5.26 V)
+12V: +11.97 V (min = +11.39 V, max = +12.61 V)
-12V: -11.11 V (min = -7.17 V, max = -16.76 V)
-5V: -6.28 V (min = -2.48 V, max = -10.02 V)
Stdby: +5.09 V (min = +4.76 V, max = +5.26 V)
CPU Fan: 2057 RPM (min = 1599 RPM, div = 4)
BOX Fan: 1318 RPM (min = 897 RPM, div = 8)
> * Anything in the logs?
Nothing special is visible when starting lm_sensors, but my kernel logs the
following during its boot sequence:
it87: Found IT8705F chip at 0x290, revision 2
it87 1-0290: Detected broken BIOS defaults, disabling PWM interface
(What does this mean, btw ?)
> Please keep in mind that this is *expected* that reported alarms won't
> disappear immediately after changing the limits. You will always see
> them once after that, per chip design (you must not miss once-only
> alarms).
I understand, but what I observe prove that it's the thresholds that don't get
set correctly as first "sensors -s" call, so it's not just an alarm
problem...
Cheers.
--
Michel Bouissou <michel@bouissou.net> OpenPGP ID 0xDDE8AC6E
-------------- next part --------------
A non-text attachment was scrubbed...
Name: lm_sensors.gz
Type: application/x-gzip
Size: 1168 bytes
Desc: not available
Url : http://lists.lm-sensors.org/pipermail/lm-sensors/attachments/20050711/5bfcebc3/lm_sensors.gz
^ permalink raw reply [flat|nested] 6+ messages in thread* [lm-sensors] Fwd: [Bug 16755] [lm_sensors] ASSIGNED: [PATCH] Bad
2005-07-11 11:19 [lm-sensors] Fwd: [Bug 16755] [lm_sensors] ASSIGNED: [PATCH] Bad Michel Bouissou
2005-07-11 11:49 ` Jean Delvare
2005-07-11 12:19 ` Michel Bouissou
@ 2005-07-11 12:48 ` Jean Delvare
2005-07-11 12:57 ` Michel Bouissou
2005-07-11 13:12 ` Michel Bouissou
4 siblings, 0 replies; 6+ messages in thread
From: Jean Delvare @ 2005-07-11 12:48 UTC (permalink / raw)
To: lm-sensors
Hi Michel,
> > * Which limits exactly don't get set by the first "sensors -s"? All?
>
> As far as I noticed, only the fans RPM thresholds are concerned.
This could be easily explained then. In your configuration file there is:
> set fan1_min 1600
> set fan1_div 4
> # set fan2_min 1500
> set fan2_min 900
> set fan2_div 8
You set the min limit before the clock divider, while a comment at the
top of the default sensors.conf file explicitely suggests NOT to do so:
> # A 'set fan1_div' statement must go before a 'set fan1_min' statement,
> # because the driver uses the divisor in calculating the minimum.
> # Also, one should set vrm prior to using vid in any formula.
Just try swapping the lines, and then your second "sensors -s" may no
more be necessary.
> > Please provide the output of "sensors" before the first "sensors
> > -s", after the first "sensors -s" and after the second "sensors -s".
>
> I don't have them on hand, but what I can tell from memory is that after
> the 1st "sensors -s", the fans RPM thresholds are set to 5100+ something
> RPMs, which doesn't correspond at all to what I have in sensors.conf.
Aren't there by any chance multiples of what you asked for? Or
compeletely different?
> it87: Found IT8705F chip at 0x290, revision 2
> it87 1-0290: Detected broken BIOS defaults, disabling PWM interface
>
> (What does this mean, btw ?)
That Gigabyte did not properly intialize your chip for PWM operation (fan
control). If the driver would let you use the PWM interface right now,
you'd certainly burn your system because the fans have inverted
polarity (you'd stop the fan if asking it to go full speed, and
vice-versa). So for safety we had the driver disable the interface
altogether. You can reconfigure the chip and re-enable it using the
module parameter "fix_pwm_polarity". Use at your own risk though. I'd
suggest you first complain to Gigabyte and ask them for an updated BIOS
before resorting to this.
Do you have any fan control options in your BIOS setup screens?
> > Please keep in mind that this is *expected* that reported alarms won't
> > disappear immediately after changing the limits. You will always see
> > them once after that, per chip design (you must not miss once-only
> > alarms).
>
> I understand, but what I observe prove that it's the thresholds that don't
> get set correctly as first "sensors -s" call, so it's not just an alarm
> problem...
What I meant was that your test (grep -q ALARMS) is fundamentally broken.
The second "sensors -s" is likely to run on almost all systems,
whether required or not, because it would take a second call to sensors
after the caching delay (typically 1 to 2 seconds) before all remnants
alarms wear off. But hopefully we'll find a real fix anyway, so it
doesn't really matter anymore.
So please try swapping the fan_min and fan_div lines in your
configuration file and report success or failure.
I have quite a few pending patches for that driver BTW, I was just
waiting for a bigger patch to be applied before (the one moving all
driver files around).
Thanks,
--
Jean Delvare
^ permalink raw reply [flat|nested] 6+ messages in thread
* [lm-sensors] Fwd: [Bug 16755] [lm_sensors] ASSIGNED: [PATCH] Bad
2005-07-11 11:19 [lm-sensors] Fwd: [Bug 16755] [lm_sensors] ASSIGNED: [PATCH] Bad Michel Bouissou
` (2 preceding siblings ...)
2005-07-11 12:48 ` Jean Delvare
@ 2005-07-11 12:57 ` Michel Bouissou
2005-07-11 13:12 ` Michel Bouissou
4 siblings, 0 replies; 6+ messages in thread
From: Michel Bouissou @ 2005-07-11 12:57 UTC (permalink / raw)
To: lm-sensors
Le Lundi 11 Juillet 2005 12:42, Jean Delvare a ?crit :
>
> This could be easily explained then. In your configuration file there is:
> > set fan1_min 1600
> > set fan1_div 4
> > # set fan2_min 1500
> > set fan2_min 900
> > set fan2_div 8
>
> You set the min limit before the clock divider, while a comment at the
> top of the default sensors.conf file explicitely suggests NOT to do so:
OOOops !!!
I usually reads the docs thoroughly (yes I do ;-) but I had missed this one...
I'm not familar with the order of parameters being of any importance in
config files...
> Just try swapping the lines, and then your second "sensors -s" may no
> more be necessary.
I'll check this.
> > it87: Found IT8705F chip at 0x290, revision 2
> > it87 1-0290: Detected broken BIOS defaults, disabling PWM interface
> >
> > (What does this mean, btw ?)
>
> That Gigabyte did not properly intialize your chip for PWM operation (fan
> control). If the driver would let you use the PWM interface right now,
> you'd certainly burn your system because the fans have inverted
> polarity (you'd stop the fan if asking it to go full speed, and
> vice-versa). So for safety we had the driver disable the interface
> altogether. You can reconfigure the chip and re-enable it using the
> module parameter "fix_pwm_polarity". Use at your own risk though. I'd
> suggest you first complain to Gigabyte and ask them for an updated BIOS
> before resorting to this.
>
> Do you have any fan control options in your BIOS setup screens?
No. AFAIK, my board is capable of reporting the fans speeds, but doesn't have
active control over them. That's probably why these BIOS defaults are
"broken".
My CPU fan is controlled via a manual potentiometer, and my box fans are under
control of my power supply (Antec) that reports their speed to the MB thru a
specific cable.
Thanks for the help !
--
Michel Bouissou <michel@bouissou.net> OpenPGP ID 0xDDE8AC6E
^ permalink raw reply [flat|nested] 6+ messages in thread* [lm-sensors] Fwd: [Bug 16755] [lm_sensors] ASSIGNED: [PATCH] Bad
2005-07-11 11:19 [lm-sensors] Fwd: [Bug 16755] [lm_sensors] ASSIGNED: [PATCH] Bad Michel Bouissou
` (3 preceding siblings ...)
2005-07-11 12:57 ` Michel Bouissou
@ 2005-07-11 13:12 ` Michel Bouissou
4 siblings, 0 replies; 6+ messages in thread
From: Michel Bouissou @ 2005-07-11 13:12 UTC (permalink / raw)
To: lm-sensors
Le Lundi 11 Juillet 2005 12:42, Jean Delvare a ?crit :
> Just try swapping the lines, and then your second "sensors -s" may no
> more be necessary.
You were right, shame on me :-(
This parameter ordering issue got me :-/
--
Michel Bouissou <michel@bouissou.net> OpenPGP ID 0xDDE8AC6E
^ permalink raw reply [flat|nested] 6+ messages in thread