#1772 Several sensor bugs with Intel baseboards

3.2.1
closed-fixed
5
2013-06-20
2012-10-25
Andy Cress
No

In further testing with openhpi-3.2.0 and the ipmidirect plugin, these sensor bugs were noticed:
1) For Intel ME-based sensors, the GetSensorReading returns an error because it attempts to use the wrong slave address. (e.g. MTT, NM Capabilities)
2) For any sensors, it fails to detect if the IPMI sensor is in Init state, meaning that the reading is invalid.
3) For Intel Romley baseboards, they should be included in the IntelRms registration list
4) For some ME sensors (e.g. MTT) it sometimes can take longer than 1 second, and the reading returns with c3 (timeout).

Discussion

  • Andy Cress

    Andy Cress - 2012-10-25

    Attached is a patch for these issues for review (oh_sensorbugs.patch).

     
  • Andy Cress

    Andy Cress - 2012-10-25
    • assigned_to: larswetzel --> arcress
     
  • Andy Cress

    Andy Cress - 2012-10-25

    One more:
    5) The FindSensor/FindRdr logic needs to handle different slave addresses, as in case 1 above.

     
  • Andy Cress

    Andy Cress - 2012-10-25

    oh_sensorbugs.patch

     
  • Andy Cress

    Andy Cress - 2012-10-25
    • assigned_to: arcress --> larswetzel
     
  • Andy Cress

    Andy Cress - 2012-10-25

    Lars,
    Please review this patch. The implementation is targeted to the specific problems and should not affect other systems adversely.
    Andy

     
  • Lars Wetzel

    Lars Wetzel - 2012-10-25

    Hi Andy
    commit it.
    Regards
    Lars

     
  • Lars Wetzel

    Lars Wetzel - 2012-10-25
    • assigned_to: larswetzel --> arcress
     
  • Andy Cress

    Andy Cress - 2012-10-25
    • status: open --> pending-fixed
     
  • Andy Cress

    Andy Cress - 2012-10-25

    committed to the trunk with revision 7520.

     
  • Andy Cress

    Andy Cress - 2013-05-29
    • status: pending-fixed --> closed-fixed
     
  • dr_mohan

    dr_mohan - 2013-06-20
    • Group: 3.2.x --> 3.2.1
     
  • Tariq Shureih

    Tariq Shureih - 2013-06-20

    *ATTENTION**
    This account is disabled and is no longer accessed by the recipient.
    Please remove it from your address book.

    Thanks

     

Log in to post a comment.

Get latest updates about Open Source Projects, Conferences and News.

Sign up for the SourceForge newsletter:





No, thanks