ipmb_host.c: Fix slow transactions
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
linux-bluefield (Ubuntu) |
Invalid
|
Undecided
|
Unassigned | ||
Focal |
Fix Released
|
Medium
|
Asmaa Mnebhi |
Bug Description
SRU Justification:
[Impact]
* The previous ipmb_host patch broke the customer tests. They requested to make the ipmb_host response time as long as before. In the case where the I2C bus is made very busy, the ipmb_host driver just drops slow/delayed responses. This fix elongates the timeout of the response.
* Fixes a crash which occurs after powercycling certain BlueField-2 systems.
* The crash is due to the handshake which takes too long to wait for a response at boot time.
[Fix]
* Before executing the IPMB handshake which loads the ipmb_host driver, detect if there is a slave detected and make sure that the slave can respond.
* Free current_msg pointer right after sending the message so that sent transactions are independent of the response transactions.
[Test Case]
* soft reset
* powercycle
* run ipmitool commands
* remove and load module manually
[Regression Potential]
* The regression can be considered as low, since:
* it restores previous stable code for the customer
* and fixes a crash occurring after powercycling certain BlueField-2 systems.
CVE References
Changed in linux-bluefield (Ubuntu Focal): | |
assignee: | nobody → Asmaa Mnebhi (asmaam) |
importance: | Undecided → Medium |
status: | New → In Progress |
Changed in linux-bluefield (Ubuntu): | |
status: | New → Invalid |
Changed in linux-bluefield (Ubuntu Focal): | |
status: | In Progress → Fix Committed |
This bug is awaiting verification that the kernel in -proposed solves the problem. Please test the kernel and update this bug with the results. If the problem is solved, change the tag 'verification- needed- focal' to 'verification- done-focal' . If the problem still exists, change the tag 'verification- needed- focal' to 'verification- failed- focal'.
If verification is not done by 5 working days from today, this fix will be dropped from the source code, and this bug will be closed.
See https:/ /wiki.ubuntu. com/Testing/ EnableProposed for documentation how to enable and use -proposed. Thank you!