[Ubuntu 21.04] net/mlx5: Fix HW spec violation configuring uplink
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Ubuntu on IBM z Systems |
Fix Released
|
High
|
Skipper Bug Screeners | ||
linux (Ubuntu) |
Invalid
|
High
|
Skipper Bug Screeners | ||
Focal |
Fix Released
|
Medium
|
Unassigned | ||
Groovy |
Fix Released
|
Medium
|
Unassigned | ||
Hirsute |
Fix Released
|
Medium
|
Skipper Bug Screeners |
Bug Description
SRU Justification:
==================
[Impact]
* In addition to 9c9be85f6b59 "net/mlx5e: Add missing capability check for uplink follow" (handled in LP#1921104) another fix 1a73704c82ed "Fix HW spec violation configuring uplink" (handled in this bug, LP#1925452) is needed to fix issues that were introduced with commit 7d0314b11cdd "net/mlx5e: Modify uplink state on interface up/down".
* Commit 1a73704c82ed "Fix HW spec violation configuring uplink" fixes a regression for mlx5 adapters required to operate in switchdev mode.
* This fix makes sure that the uplink port is modified to follow only if the uplink_follow capability if it's set as required by the hw specification.
* Failure cause traffic to the uplink representer net device to cease after switching to switchdev mode.
[Fix]
* upstream fix (upstream with v5.12-rc7)
1a73704c82ed4ee
* can be cleanly cherry picked from hirsute master-next.
* a backport for groovy:
https:/
* a backport for focal:
https:/
[Test Case]
* Two servers, installed with Ubuntu Server 20.04 or 20.10 are needed.
* Each server needs to have a Mellanox ConnectX4/5 adapter, attached to the same switch
* Adapters must be running adapter firmware level 16.29.1006 or earlier.
* enable SRIOV and switchdev mode on one adapter:
echo 0 > /sys/bus/
echo 0 > /sys/bus/
echo 64 > /sys/bus/
echo 64 > /sys/bus/
devlink dev eswitch set pci/0100:00:00.0 mode switchdev
devlink dev eswitch set pci/0100:00:00.1 mode switchdev
* Assign an IP address to the physical function device of the adapters on both systems
* IP communication will fail
* With the fix, IP communication can be established.
[Regression Potential]
* There is always at least some potential for regression. In this case the new code can go wrong (or might become worse than before) in case the new if statement is wrong.
* It checks for the condition of "MLX5_CAP_GEN(mdev, uplink_follow)" and in case MLX5_CAP_GEN is calculated erroneous or mdev is other than expected, the mlx5_modify_
* But since only the If clause was added, the changes are pretty minimal and therefore well traceable.
[Other]
* Since the patch/commit is upstream with 5.12-rc7, it will be in Impish.
__________
With the 5.4.0-48 update for Ubuntu 20.04, the commit
"net/mlx5e: Modify uplink state on interface up/down"
was integrated which contains a regression for mlx5 adapters with older adapter firmware.
This is the second bugfix to repair the regression, required to operate the adapter in switchdev mode.
The first part was integrated via Bug 192185 - LP1921104.
This is the upstream fix required to be backported to Ubuntu 20.04:
https:/
---Additional Hardware Info---
Mellanox ConnectX network adapter
---uname output---
5.4.0-73-generic
---Steps to Reproduce---
enable switchdev mode
try to ping external host
Backports need to be provide for 21.04, 20.10 and 20.04 (if not cleanly applyable)!
tags: | added: architecture-s3903164 bugnameltc-192501 severity-high targetmilestone-inin2104 |
Changed in ubuntu: | |
assignee: | nobody → Skipper Bug Screeners (skipper-screen-team) |
affects: | ubuntu → linux (Ubuntu) |
Changed in ubuntu-z-systems: | |
status: | New → Triaged |
importance: | Undecided → High |
assignee: | nobody → Skipper Bug Screeners (skipper-screen-team) |
description: | updated |
description: | updated |
summary: |
- [Ubuntu 21.04]: net/mlx5: Fix HW spec violation configuring uplink - (mandatory for 20.04) + [Ubuntu 21.04] net/mlx5: Fix HW spec violation configuring uplink |
description: | updated |
Changed in linux (Ubuntu Groovy): | |
status: | New → In Progress |
Changed in linux (Ubuntu Focal): | |
status: | New → In Progress |
Changed in ubuntu-z-systems: | |
status: | Triaged → In Progress |
Changed in ubuntu-z-systems: | |
status: | In Progress → Fix Committed |
------- Comment From <email address hidden> 2021-04-22 05:16 EDT-------
More info will follow.