[sr #110963] ipmi-sensors does not return fan rpm on Lenovo SR650 v3 but ipmitool does

[Albert Chu]

Follow-up Comment #15, sr #110963 (project freeipmi):

I looked through the ipmitool code and one chunk looks very concerning in
`ipmi_sdr_read_sensor_value()`

```
        if (rsp->ccode) {                                                     
                                                             
                if ( !((sr.full    && rsp->ccode == 0xcb) ||                  
                                                             
                       (sr.compact && rsp->ccode == 0xcd)) ) {                
                                                             
                        lprintf(LOG_DEBUG,                                    
                                                             
                                "Error reading sensor %s (#%02x): %s",
sr.s_id,                                                             
                                sensor->keys.sensor_num,                      
                                                             
                                val2str(rsp->ccode, completion_code_vals));   
                                                             
                }                                                             
                                                             
                return &sr;                                                   
                                                             
        }    
```

It seems to suggest that if an error occurs during the sensor reading, output
a debug message (if necessary), and then return the sensor reading as though
it were successful.

So this may be a bug in ipmitool in which it gets an error, but ignores the
error.  

What confidence do you have in the RPM values actually being correct?

The RPM may simply be junk data being output.




    _______________________________________________________

Reply to this item at:

  <https://savannah.gnu.org/support/?110963>

_______________________________________________
Message sent via Savannah
https://savannah.gnu.org/