driver/broadcom/bnxt: invalid dev_port

Bug #1931245 reported by Thibaut Collet
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
linux (Ubuntu)
In Progress
Medium
Krzysztof Kozlowski
Focal
In Progress
Medium
Krzysztof Kozlowski

Bug Description

[SRU Justification]

[Impact]

With broadcom dual port NIC naming convention between port 1 and port 2 can be inconsistent

For example, if network port 1 of a device is named enp1s0f0, port 2 can end up with the name enp1s0f1d1 rather than enp1s0f1

Issue is due to an inconsistent port_dev and any scripts based on it can have unexpected behavior

This has been fixed in v5.6 by the following upstream commit
 - 1d86859fdf31("bnxt_en: Remove the setting of dev_port")

https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=1d86859fdf31

The bug exists since the beginning of the driver.

== Fix ==

Backport the requested patches to Focal (5.4)

== Risk of Regression ==

The patch is trivial, the potential regressions are low.

Daniel Manrique (roadmr)
affects: canonical-identity-provider → linux (Ubuntu)
Revision history for this message
Krzysztof Kozlowski (krzk) wrote :

For the risk of regression - this paragraph should explain possible failures, for example if something goes wrong. Even trivial patch can cause some effects. This one as well.

Revision history for this message
Krzysztof Kozlowski (krzk) wrote :

Correction: bug was fixed in v5.6.

Changed in linux (Ubuntu):
assignee: nobody → Krzysztof Kozlowski (krzk)
Changed in linux (Ubuntu Focal):
assignee: nobody → Krzysztof Kozlowski (krzk)
Changed in linux (Ubuntu Hirsute):
assignee: nobody → Krzysztof Kozlowski (krzk)
Changed in linux (Ubuntu Groovy):
assignee: nobody → Krzysztof Kozlowski (krzk)
Changed in linux (Ubuntu Impish):
status: New → In Progress
description: updated
Revision history for this message
Krzysztof Kozlowski (krzk) wrote (last edit ):

Sent backport for v5.4 stable. Greg replied, that backport is queued for v4.9, v4.14, v4.19 and v5.4, so Focal will get it via regular stable update

Changed in linux (Ubuntu Impish):
status: In Progress → Invalid
Changed in linux (Ubuntu Hirsute):
status: New → Invalid
Changed in linux (Ubuntu Groovy):
status: New → Invalid
Changed in linux (Ubuntu Focal):
status: New → In Progress
no longer affects: linux (Ubuntu Groovy)
no longer affects: linux (Ubuntu Hirsute)
no longer affects: linux (Ubuntu Impish)
Changed in linux (Ubuntu):
status: Invalid → In Progress
Changed in linux (Ubuntu Focal):
importance: Undecided → Medium
Changed in linux (Ubuntu):
importance: Undecided → Medium
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.