diff options
| author | Jedrzej Jagielski <[email protected]> | 2025-07-03 10:41:15 +0000 |
|---|---|---|
| committer | Tony Nguyen <[email protected]> | 2025-08-12 20:24:09 +0000 |
| commit | e67a0bc3ed4fd8ee1697cb6d937e2b294ec13b5e (patch) | |
| tree | 281054d818895fc7d7fb7ce75c3d847cffe10f9b | |
| parent | devlink: let driver opt out of automatic phys_port_name generation (diff) | |
| download | kernel-e67a0bc3ed4fd8ee1697cb6d937e2b294ec13b5e.tar.gz kernel-e67a0bc3ed4fd8ee1697cb6d937e2b294ec13b5e.zip | |
ixgbe: prevent from unwanted interface name changes
Users of the ixgbe driver report that after adding devlink support by
the commit a0285236ab93 ("ixgbe: add initial devlink support") their
configs got broken due to unwanted changes of interface names. It's
caused by automatic phys_port_name generation during devlink port
initialization flow.
To prevent from that set no_phys_port_name flag for ixgbe devlink ports.
Reported-by: David Howells <[email protected]>
Closes: https://lore.kernel.org/netdev/[email protected]/
Reported-by: David Kaplan <[email protected]>
Closes: https://lore.kernel.org/netdev/LV3PR12MB92658474624CCF60220157199470A@LV3PR12MB9265.namprd12.prod.outlook.com/
Fixes: a0285236ab93 ("ixgbe: add initial devlink support")
Signed-off-by: Jedrzej Jagielski <[email protected]>
Tested-by: Rinitha S <[email protected]> (A Contingent worker at Intel)
Signed-off-by: Tony Nguyen <[email protected]>
| -rw-r--r-- | drivers/net/ethernet/intel/ixgbe/devlink/devlink.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/net/ethernet/intel/ixgbe/devlink/devlink.c b/drivers/net/ethernet/intel/ixgbe/devlink/devlink.c index 54f1b83dfe42..d227f4d2a2d1 100644 --- a/drivers/net/ethernet/intel/ixgbe/devlink/devlink.c +++ b/drivers/net/ethernet/intel/ixgbe/devlink/devlink.c @@ -543,6 +543,7 @@ int ixgbe_devlink_register_port(struct ixgbe_adapter *adapter) attrs.flavour = DEVLINK_PORT_FLAVOUR_PHYSICAL; attrs.phys.port_number = adapter->hw.bus.func; + attrs.no_phys_port_name = 1; ixgbe_devlink_set_switch_id(adapter, &attrs.switch_id); devlink_port_attrs_set(devlink_port, &attrs); |