2021-04-27 13:52:47 |
Dan Streetman |
bug |
|
|
added bug |
2021-04-27 19:44:11 |
Heather Lemon |
freeipmi (Ubuntu): assignee |
|
Heather Lemon (hypothetical-lemon) |
|
2021-05-18 20:08:37 |
Heather Lemon |
nominated for series |
|
Ubuntu Groovy |
|
2021-05-18 20:08:37 |
Heather Lemon |
bug task added |
|
freeipmi (Ubuntu Groovy) |
|
2021-05-18 20:08:37 |
Heather Lemon |
nominated for series |
|
Ubuntu Hirsute |
|
2021-05-18 20:08:37 |
Heather Lemon |
bug task added |
|
freeipmi (Ubuntu Hirsute) |
|
2021-05-18 20:08:37 |
Heather Lemon |
nominated for series |
|
Ubuntu Bionic |
|
2021-05-18 20:08:37 |
Heather Lemon |
bug task added |
|
freeipmi (Ubuntu Bionic) |
|
2021-05-18 20:08:37 |
Heather Lemon |
nominated for series |
|
Ubuntu Impish |
|
2021-05-18 20:08:37 |
Heather Lemon |
bug task added |
|
freeipmi (Ubuntu Impish) |
|
2021-05-18 20:08:37 |
Heather Lemon |
nominated for series |
|
Ubuntu Focal |
|
2021-05-18 20:08:37 |
Heather Lemon |
bug task added |
|
freeipmi (Ubuntu Focal) |
|
2021-05-18 20:34:16 |
Heather Lemon |
freeipmi (Ubuntu Hirsute): assignee |
|
Heather Lemon (hypothetical-lemon) |
|
2021-05-18 20:34:18 |
Heather Lemon |
freeipmi (Ubuntu Groovy): assignee |
|
Heather Lemon (hypothetical-lemon) |
|
2021-05-18 20:34:20 |
Heather Lemon |
freeipmi (Ubuntu Focal): assignee |
|
Heather Lemon (hypothetical-lemon) |
|
2021-05-18 20:34:22 |
Heather Lemon |
freeipmi (Ubuntu Bionic): assignee |
|
Heather Lemon (hypothetical-lemon) |
|
2021-05-20 21:39:28 |
Heather Lemon |
attachment added |
|
hirsute-1926299.debdiff https://bugs.launchpad.net/ubuntu/+source/freeipmi/+bug/1926299/+attachment/5499113/+files/hirsute-1926299.debdiff |
|
2021-05-20 21:39:43 |
Heather Lemon |
freeipmi (Ubuntu Bionic): status |
New |
In Progress |
|
2021-05-20 21:39:46 |
Heather Lemon |
freeipmi (Ubuntu Focal): status |
New |
In Progress |
|
2021-05-20 21:39:49 |
Heather Lemon |
freeipmi (Ubuntu Groovy): status |
New |
In Progress |
|
2021-05-20 21:39:51 |
Heather Lemon |
freeipmi (Ubuntu Hirsute): status |
New |
In Progress |
|
2021-05-20 21:42:19 |
Heather Lemon |
freeipmi (Ubuntu Hirsute): importance |
Undecided |
Low |
|
2021-05-20 21:42:21 |
Heather Lemon |
freeipmi (Ubuntu Groovy): importance |
Undecided |
Low |
|
2021-05-20 21:42:24 |
Heather Lemon |
freeipmi (Ubuntu Focal): importance |
Undecided |
Low |
|
2021-05-20 21:42:26 |
Heather Lemon |
freeipmi (Ubuntu Bionic): importance |
Undecided |
Low |
|
2021-05-20 22:20:28 |
Heather Lemon |
attachment added |
|
groovy-1926299.debdiff https://bugs.launchpad.net/ubuntu/+source/freeipmi/+bug/1926299/+attachment/5499121/+files/groovy-1926299.debdiff |
|
2021-05-20 22:20:46 |
Heather Lemon |
attachment removed |
hirsute-1926299.debdiff https://bugs.launchpad.net/ubuntu/+source/freeipmi/+bug/1926299/+attachment/5499113/+files/hirsute-1926299.debdiff |
|
|
2021-05-20 22:28:10 |
Heather Lemon |
attachment added |
|
hirsute-1926299.debdiff https://bugs.launchpad.net/ubuntu/+source/freeipmi/+bug/1926299/+attachment/5499122/+files/hirsute-1926299.debdiff |
|
2021-05-21 04:28:10 |
Ubuntu Foundations Team Bug Bot |
tags |
|
patch |
|
2021-05-21 04:28:20 |
Ubuntu Foundations Team Bug Bot |
bug |
|
|
added subscriber Ubuntu Sponsors Team |
2021-05-21 13:41:47 |
Heather Lemon |
attachment removed |
groovy-1926299.debdiff https://bugs.launchpad.net/ubuntu/+source/freeipmi/+bug/1926299/+attachment/5499121/+files/groovy-1926299.debdiff |
|
|
2021-05-21 13:42:17 |
Heather Lemon |
attachment added |
|
groovy-1926299.debdiff https://bugs.launchpad.net/ubuntu/+source/freeipmi/+bug/1926299/+attachment/5499271/+files/groovy-1926299.debdiff |
|
2021-05-21 13:45:15 |
Heather Lemon |
attachment removed |
groovy-1926299.debdiff https://bugs.launchpad.net/ubuntu/+source/freeipmi/+bug/1926299/+attachment/5499271/+files/groovy-1926299.debdiff |
|
|
2021-05-21 13:46:02 |
Heather Lemon |
attachment added |
|
groovy-1926299.debdiff https://bugs.launchpad.net/ubuntu/+source/freeipmi/+bug/1926299/+attachment/5499272/+files/groovy-1926299.debdiff |
|
2021-05-21 13:55:44 |
Heather Lemon |
attachment added |
|
focal-1926299.debdiff https://bugs.launchpad.net/ubuntu/+source/freeipmi/+bug/1926299/+attachment/5499274/+files/focal-1926299.debdiff |
|
2021-05-21 14:01:17 |
Dan Streetman |
freeipmi (Ubuntu Impish): status |
New |
In Progress |
|
2021-05-21 14:01:20 |
Dan Streetman |
freeipmi (Ubuntu Impish): importance |
Undecided |
Low |
|
2021-05-21 14:08:46 |
Heather Lemon |
attachment added |
|
bionic-1926299.debdiff https://bugs.launchpad.net/ubuntu/+source/freeipmi/+bug/1926299/+attachment/5499275/+files/bionic-1926299.debdiff |
|
2021-05-21 14:10:38 |
Heather Lemon |
tags |
patch |
patch sts-sponsor verification-needed-bionic verification-needed-focal verification-needed-groovy verification-needed-hirsute |
|
2021-05-21 14:48:15 |
Dan Streetman |
description |
[impact]
ipmi-sensors reads the wrong sensor on a Lenovo BMC (although the problem may happen on other BMCs as well)
[test case]
run ipmi-sensors over the network using ipmi 1.5 or 2 protocol, and check the sensor output. Specifically for this case, the server is a Lenovo SR665 with only 1 cpu socket populated, and dimms 1-16 are connected to cpu 1 while dimms 17-32 are connected to cpu2, so dimms 17-32 should not return any values.
$ ipmi-sensors -u $USERNAME -p $PASSWD -D LAN_2_0 -l USER -h $BMCADDR | grep DIMM | grep Temp
110 | DIMM 1 Temp | Temperature | N/A | C | N/A
113 | DIMM 2 Temp | Temperature | N/A | C | N/A
116 | DIMM 3 Temp | Temperature | N/A | C | N/A
119 | DIMM 4 Temp | Temperature | N/A | C | N/A
122 | DIMM 5 Temp | Temperature | N/A | C | N/A
125 | DIMM 6 Temp | Temperature | N/A | C | N/A
128 | DIMM 7 Temp | Temperature | N/A | C | N/A
131 | DIMM 8 Temp | Temperature | N/A | C | N/A
134 | DIMM 9 Temp | Temperature | 52.00 | C | 'OK'
137 | DIMM 10 Temp | Temperature | 42.00 | C | 'OK'
140 | DIMM 11 Temp | Temperature | N/A | C | N/A
143 | DIMM 12 Temp | Temperature | N/A | C | N/A
146 | DIMM 13 Temp | Temperature | 27.00 | C | 'OK'
149 | DIMM 14 Temp | Temperature | 26.00 | C | 'OK'
152 | DIMM 15 Temp | Temperature | 24.00 | C | 'OK'
155 | DIMM 16 Temp | Temperature | 24.00 | C | 'OK'
158 | DIMM 17 Temp | Temperature | 218.00 | C | 'OK'
161 | DIMM 18 Temp | Temperature | 212.00 | C | 'OK'
165 | DIMM 19 Temp | Temperature | 212.00 | C | 'OK'
173 | DIMM 20 Temp | Temperature | N/A | C | N/A
176 | DIMM 21 Temp | Temperature | N/A | C | N/A
179 | DIMM 22 Temp | Temperature | N/A | C | N/A
182 | DIMM 23 Temp | Temperature | N/A | C | N/A
185 | DIMM 24 Temp | Temperature | N/A | C | N/A
188 | DIMM 25 Temp | Temperature | 0.00 | C | 'At or Below (<=) Lower Non-Critical Threshold'
191 | DIMM 26 Temp | Temperature | 0.00 | C | 'At or Below (<=) Lower Non-Critical Threshold'
194 | DIMM 27 Temp | Temperature | 0.00 | C | 'At or Below (<=) Lower Non-Critical Threshold'
197 | DIMM 28 Temp | Temperature | 0.00 | C | 'At or Below (<=) Lower Non-Critical Threshold'
200 | DIMM 29 Temp | Temperature | 0.00 | C | 'At or Below (<=) Lower Non-Critical Threshold'
203 | DIMM 30 Temp | Temperature | 0.00 | C | 'At or Below (<=) Lower Non-Critical Threshold'
206 | DIMM 31 Temp | Temperature | 0.00 | C | 'At or Below (<=) Lower Non-Critical Threshold'
209 | DIMM 32 Temp | Temperature | N/A | C | N/A
[regression potential]
any regression would likely result in incorrect values shown for some/all sensor readings
[scope]
this is not fixed upstream, so it needed in all releases
[other info]
the problem is that this particular BMC has chosen to use a non-zero 'LUN' number for these sensors; almost all BMCs use the standard LUN number 0, as the spec does state in section 19.3:
"Unless otherwise specified, commands that are listed as mandatory must be accessed via LUN 00b"
However the spec does allow for implementations to use alternate LUN numbers, specifically section 5.4 describes how the BMC would report the LUN number to the requesting software, and section 7.2 clarifies that while LUN 00b is reserved for devices beloning to the BMC itself and 10b is reserved for SMS messages, LUN numbers 01b and 11b are reserved for 'OEM' use. So it does appear valid for the BMC implementation to place sensor(s) under LUN 01b instead of the default 00b.
The cause of the invalid sensor readings in this particular case is because the BMC also defines sensors with the exact same sensor id number, but with LUN 00b. So when freeipmi attempts to read e.g. 'DIMM 17 Temp' sensor, it actually reads the current value for a completely different sensor, which (in this case) isn't even a temperature sensor, resulting in invalid temperature reading shown for the sensor, instead of showing it as N/A (since the DIMM slot isn't populated).
Note that ipmitool correctly uses the alternate LUN number so this bug does not exist with that tool, this affects only freeipmi tooling. |
[impact]
ipmi-sensors reads the wrong sensor on a Lenovo BMC (although the problem may happen on other BMCs as well)
[test case]
run ipmi-sensors over the network using ipmi 1.5 or 2 protocol, and check the sensor output. Specifically for this case, the server is a Lenovo SR665 with only 1 cpu socket populated, and dimms 1-16 are connected to cpu 1 while dimms 17-32 are connected to cpu2, so dimms 17-32 should not return any values.
$ ipmi-sensors -u $USERNAME -p $PASSWD -D LAN_2_0 -l USER -h $BMCADDR | grep DIMM | grep Temp
110 | DIMM 1 Temp | Temperature | N/A | C | N/A
113 | DIMM 2 Temp | Temperature | N/A | C | N/A
116 | DIMM 3 Temp | Temperature | N/A | C | N/A
119 | DIMM 4 Temp | Temperature | N/A | C | N/A
122 | DIMM 5 Temp | Temperature | N/A | C | N/A
125 | DIMM 6 Temp | Temperature | N/A | C | N/A
128 | DIMM 7 Temp | Temperature | N/A | C | N/A
131 | DIMM 8 Temp | Temperature | N/A | C | N/A
134 | DIMM 9 Temp | Temperature | 52.00 | C | 'OK'
137 | DIMM 10 Temp | Temperature | 42.00 | C | 'OK'
140 | DIMM 11 Temp | Temperature | N/A | C | N/A
143 | DIMM 12 Temp | Temperature | N/A | C | N/A
146 | DIMM 13 Temp | Temperature | 27.00 | C | 'OK'
149 | DIMM 14 Temp | Temperature | 26.00 | C | 'OK'
152 | DIMM 15 Temp | Temperature | 24.00 | C | 'OK'
155 | DIMM 16 Temp | Temperature | 24.00 | C | 'OK'
158 | DIMM 17 Temp | Temperature | 218.00 | C | 'OK'
161 | DIMM 18 Temp | Temperature | 212.00 | C | 'OK'
165 | DIMM 19 Temp | Temperature | 212.00 | C | 'OK'
173 | DIMM 20 Temp | Temperature | N/A | C | N/A
176 | DIMM 21 Temp | Temperature | N/A | C | N/A
179 | DIMM 22 Temp | Temperature | N/A | C | N/A
182 | DIMM 23 Temp | Temperature | N/A | C | N/A
185 | DIMM 24 Temp | Temperature | N/A | C | N/A
188 | DIMM 25 Temp | Temperature | 0.00 | C | 'At or Below (<=) Lower Non-Critical Threshold'
191 | DIMM 26 Temp | Temperature | 0.00 | C | 'At or Below (<=) Lower Non-Critical Threshold'
194 | DIMM 27 Temp | Temperature | 0.00 | C | 'At or Below (<=) Lower Non-Critical Threshold'
197 | DIMM 28 Temp | Temperature | 0.00 | C | 'At or Below (<=) Lower Non-Critical Threshold'
200 | DIMM 29 Temp | Temperature | 0.00 | C | 'At or Below (<=) Lower Non-Critical Threshold'
203 | DIMM 30 Temp | Temperature | 0.00 | C | 'At or Below (<=) Lower Non-Critical Threshold'
206 | DIMM 31 Temp | Temperature | 0.00 | C | 'At or Below (<=) Lower Non-Critical Threshold'
209 | DIMM 32 Temp | Temperature | N/A | C | N/A
[regression potential]
any regression would likely result in incorrect values shown for some/all sensor readings
[scope]
this was recently fixed upstream, so it needed in all releases
https://github.com/chu11/freeipmi-mirror/pull/43
a new upstream release was made including the fix, so debian will pick it up in their next devel cycle
[other info]
the problem is that this particular BMC has chosen to use a non-zero 'LUN' number for these sensors; almost all BMCs use the standard LUN number 0, as the spec does state in section 19.3:
"Unless otherwise specified, commands that are listed as mandatory must be accessed via LUN 00b"
However the spec does allow for implementations to use alternate LUN numbers, specifically section 5.4 describes how the BMC would report the LUN number to the requesting software, and section 7.2 clarifies that while LUN 00b is reserved for devices beloning to the BMC itself and 10b is reserved for SMS messages, LUN numbers 01b and 11b are reserved for 'OEM' use. So it does appear valid for the BMC implementation to place sensor(s) under LUN 01b instead of the default 00b.
The cause of the invalid sensor readings in this particular case is because the BMC also defines sensors with the exact same sensor id number, but with LUN 00b. So when freeipmi attempts to read e.g. 'DIMM 17 Temp' sensor, it actually reads the current value for a completely different sensor, which (in this case) isn't even a temperature sensor, resulting in invalid temperature reading shown for the sensor, instead of showing it as N/A (since the DIMM slot isn't populated).
Note that ipmitool correctly uses the alternate LUN number so this bug does not exist with that tool, this affects only freeipmi tooling. |
|
2021-05-22 05:07:33 |
Launchpad Janitor |
freeipmi (Ubuntu Impish): status |
In Progress |
Fix Released |
|
2021-06-01 21:42:38 |
Brian Murray |
freeipmi (Ubuntu Hirsute): status |
In Progress |
Fix Committed |
|
2021-06-01 21:42:40 |
Brian Murray |
bug |
|
|
added subscriber Ubuntu Stable Release Updates Team |
2021-06-01 21:42:44 |
Brian Murray |
bug |
|
|
added subscriber SRU Verification |
2021-06-01 21:42:50 |
Brian Murray |
tags |
patch sts-sponsor verification-needed-bionic verification-needed-focal verification-needed-groovy verification-needed-hirsute |
patch sts-sponsor verification-needed verification-needed-bionic verification-needed-focal verification-needed-groovy verification-needed-hirsute |
|
2021-06-01 21:46:43 |
Brian Murray |
removed subscriber Ubuntu Sponsors Team |
|
|
|
2021-06-01 21:46:54 |
Brian Murray |
freeipmi (Ubuntu Groovy): status |
In Progress |
Fix Committed |
|
2021-06-01 21:52:32 |
Brian Murray |
freeipmi (Ubuntu Focal): status |
In Progress |
Fix Committed |
|
2021-06-01 22:01:29 |
Brian Murray |
freeipmi (Ubuntu Bionic): status |
In Progress |
Fix Committed |
|
2021-07-26 19:46:18 |
Dan Streetman |
freeipmi (Ubuntu Groovy): status |
Fix Committed |
Won't Fix |
|
2021-09-04 18:02:25 |
Mathew Hodson |
tags |
patch sts-sponsor verification-needed verification-needed-bionic verification-needed-focal verification-needed-groovy verification-needed-hirsute |
patch sts-sponsor verification-needed-bionic verification-needed-focal verification-needed-hirsute |
|
2021-11-25 12:56:35 |
Eric Desrochers |
tags |
patch sts-sponsor verification-needed-bionic verification-needed-focal verification-needed-hirsute |
patch sts-sponsor verification-failed-bionic verification-failed-focal verification-failed-hirsute |
|
2022-01-26 21:59:10 |
Brian Murray |
freeipmi (Ubuntu Hirsute): status |
Fix Committed |
Won't Fix |
|