mlxbf_gige: fix devm_ioremap error handling
Bug #1991403 reported by
David Thompson
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
linux-bluefield (Ubuntu) |
Invalid
|
Undecided
|
Unassigned | ||
Jammy |
Fix Committed
|
Medium
|
Unassigned |
Bug Description
SRU Justification:
[Impact]
The following upstream commit was recently added to Jammy:
3a1a274e933f mlxbf_gige: compute MDIO period based on i1clk
This patch added driver logic that includes a "devm_ioremap"
call, but the error handling logic pertaining to this call
is incorrect.
[Fix]
The "devm_ioremap" function returns NULL on error, it doesn't
return error pointers. The error handling logic has been
fixed accordingly.
[Test Case]
Boot platform and bring up "oob_net0" interface properly
Test that network traffic works properly
[Regression Potential]
Low, as the patch was reviewed upstream
[Other]
None
Changed in linux-bluefield (Ubuntu Jammy): | |
importance: | Undecided → Medium |
status: | New → In Progress |
Changed in linux-bluefield (Ubuntu): | |
status: | New → Invalid |
Changed in linux-bluefield (Ubuntu Jammy): | |
status: | In Progress → Fix Committed |
tags: |
added: verification-done-jammy removed: verification-needed-jammy |
To post a comment you must log in.
This bug is awaiting verification that the linux-bluefield /5.15.0- 1009.11 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- jammy' to 'verification- done-jammy' . If the problem still exists, change the tag 'verification- needed- jammy' to 'verification- failed- jammy'.
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!