[Bug 231209] New: OpenIPMI bug
https://bugzilla.novell.com/show_bug.cgi?id=231209 Summary: OpenIPMI bug Product: openSUSE 10.2 Version: RC 5 Platform: Other OS/Version: Other Status: NEW Severity: Normal Priority: P5 - None Component: Kernel AssignedTo: kernel-maintainers@forge.provo.novell.com ReportedBy: zvika@imagine-com.com QAContact: qa@suse.de I am using the libipmi plugin. The IPMI driver is up and running along with the openhpid daemon. When I use ipmitool to retrieve the sensors I get all the sensors (about 28 of them). When I used the openhpi clients (such as hpisensor etc), the count is 5. After turning on the debugging in the ipmi plugin, here is what I noticed... There are 28 entries of Entities ADDED (which matches with what ipmitool returns). However, there are *ONLY* 5 entries of Entities which are marked PRESENT. During ipmi_discover_resources, events will be sent to the infrastructure queue for *ONLY* the entities that are marked *PRESENT*. Hence, hpisensor or any of the hpi clients or even for that matter the apps that use SaHPI apis return only 5 as these are the only ones that are inserted into the HPI data model. I started looking at the openIPMI code (mainly entity.c and sensor.c) and noticed the following: 1) ent_detect_presence is called to detect the presence of the entity (following the IPMI spec). If neither the Presence Sensor exists nor the Presence Bit sensor is set, detect_no_presence_sensor_presence is called to try and see if the sensors (try_presence_sensors), controls(try_presence_controls) detect something. 2) In try_presence_sensors, sensor_detect_send is called. 3) sensor_detect_send checks if ignore_if_no_entity bit is set in the sensor SDR. If this bit is SET, we do not ping the sensor. If the whole point is to detect the presence of the entity and probing the sensor is required in order to do that - should we still be checking this BIT ? I removed this check and now I am able to retrieve all the sensors even with hpisensor client. But, want to make sure that I am not missing something here... -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug, or are watching someone who is.
https://bugzilla.novell.com/show_bug.cgi?id=231209 gregkh@novell.com changed: What |Removed |Added ---------------------------------------------------------------------------- AssignedTo|kernel- |bnc-team-screening@forge.provo.novell.com |maintainers@forge.provo.nove| |ll.com | Component|Kernel |Basesystem ------- Comment #1 from gregkh@novell.com 2007-01-11 20:59 MST ------- Not a kernel issue, this is a userspace package, reassigning... -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug, or are watching someone who is.
https://bugzilla.novell.com/show_bug.cgi?id=231209 chrubis@novell.com changed: What |Removed |Added ---------------------------------------------------------------------------- AssignedTo|bnc-team- |duwe@novell.com |screening@forge.provo.novell| |.com | -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug, or are watching someone who is.
https://bugzilla.novell.com/show_bug.cgi?id=231209 duwe@novell.com changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |NEEDINFO Info Provider| |zvika@imagine-com.com ------- Comment #2 from duwe@novell.com 2007-01-16 04:39 MST ------- 2 Questions: * Do the other 23 sensor readings return sane values, e.g. _are_ the sensors present? * Where's the patch ;-) -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug, or are watching someone who is.
https://bugzilla.novell.com/show_bug.cgi?id=231209#c3
Stephan Kulow
participants (1)
-
bugzilla_noreply@novell.com