[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Freeipmi-devel] sensor output issue
From: |
Anand Babu |
Subject: |
Re: [Freeipmi-devel] sensor output issue |
Date: |
Wed, 24 Mar 2004 16:46:37 -0800 |
User-agent: |
Gnus/5.1006 (Gnus v5.10.6) Emacs/21.3 (gnu/linux) |
,----
| > | Do you agree on this??
| > `----
| > Table 36.1, Event/Reading Type Code Ranges classifies "0x6F" as
| > "Sensor-specific discrete" type. Such types *should only* refer
| > "Table 36-3, Sensor Type Codes".
|
| So are you saying "HSC Drv X Pres (Drive Slot)" should still not be
| displayed??
`----
Yes. "HSC Drv X Pres (Drive Slot)" should also be ignored.
-ab
----- Original Message -----
From: Anand Babu <address@hidden>
Date: Wednesday, March 24, 2004 3:53 pm
Subject: Re: [Freeipmi-devel] sensor output issue
> ,----[ Albert Chu <address@hidden> ]
> | Hey AB,
> |
> | The following sensors are not output unless the -vv option is
> passed to
> | sensors.
> |
> | HSC Drv X Status (Drive Slot)
> | HSC Drv X Pres (Drive Slot)
> |
> | HSC Drv X Status has an event/read type code of 0x6F. Table 36-3
> of the
> | IPMI Spec doesn't list any info about Drive Slots. So I believe
> | that one can be ignored.
> `----
> Yes you are correct.
>
> ,----
> | But, HSC Drv X Pres has an event code of 0x08. So this sensor
> should| be output with either "Device Present" or "Device Absent"
> (Table| 36-2).
> |
> | I think the bug lies in the function
> ipmi_is_sensor_reading_available().| Currently, the function only
> checks if the sensor type is
> | IPMI_SENSOR_TYPE_DRIVE_SLOT. Instead, the function should check
> if the
> | sensor type is IPMI_SENSOR_TYPE_DRIVE_SLOT and that the
> event/read type
> | is == 0x6Fh.
> |
> | Do you agree on this??
> `----
> Table 36.1, Event/Reading Type Code Ranges classifies "0x6F" as
> "Sensor-specific discrete" type. Such types *should only* refer
> "Table 36-3, Sensor Type Codes".
>
> Happy Hacking,
> --
> _.|_
> (_||_)
> Free as in Freedom <www.gnu.org>
>
--
_.|_
(_||_)
Free as in Freedom <www.gnu.org>