mirror of
https://github.com/torvalds/linux.git
synced 2025-12-07 20:06:24 +00:00
net/mlx5: Use common mlx5_same_hw_devs function
Refactor duplicate hardware device comparison code to use the common mlx5_same_hw_devs() function instead of reimplementing system GUID comparison logic in multiple places. This cleanup eliminates code duplication in: - Bridge representor device comparison. - TC hardware device comparison. Using the centralized function improves maintainability and ensures consistent behavior across the driver. Signed-off-by: Mark Bloch <mbloch@nvidia.com> Reviewed-by: Shay Drori <shayd@nvidia.com> Signed-off-by: Tariq Toukan <tariqt@nvidia.com> Link: https://patch.msgid.link/1761211020-925651-2-git-send-email-tariqt@nvidia.com Signed-off-by: Paolo Abeni <pabeni@redhat.com>
This commit is contained in:
@@ -30,15 +30,11 @@ static bool mlx5_esw_bridge_dev_same_hw(struct net_device *dev, struct mlx5_eswi
|
||||
{
|
||||
struct mlx5e_priv *priv = netdev_priv(dev);
|
||||
struct mlx5_core_dev *mdev, *esw_mdev;
|
||||
u64 system_guid, esw_system_guid;
|
||||
|
||||
mdev = priv->mdev;
|
||||
esw_mdev = esw->dev;
|
||||
|
||||
system_guid = mlx5_query_nic_system_image_guid(mdev);
|
||||
esw_system_guid = mlx5_query_nic_system_image_guid(esw_mdev);
|
||||
|
||||
return system_guid == esw_system_guid;
|
||||
return mlx5_same_hw_devs(mdev, esw_mdev);
|
||||
}
|
||||
|
||||
static struct net_device *
|
||||
|
||||
@@ -3614,15 +3614,11 @@ static bool same_port_devs(struct mlx5e_priv *priv, struct mlx5e_priv *peer_priv
|
||||
bool mlx5e_same_hw_devs(struct mlx5e_priv *priv, struct mlx5e_priv *peer_priv)
|
||||
{
|
||||
struct mlx5_core_dev *fmdev, *pmdev;
|
||||
u64 fsystem_guid, psystem_guid;
|
||||
|
||||
fmdev = priv->mdev;
|
||||
pmdev = peer_priv->mdev;
|
||||
|
||||
fsystem_guid = mlx5_query_nic_system_image_guid(fmdev);
|
||||
psystem_guid = mlx5_query_nic_system_image_guid(pmdev);
|
||||
|
||||
return (fsystem_guid == psystem_guid);
|
||||
return mlx5_same_hw_devs(fmdev, pmdev);
|
||||
}
|
||||
|
||||
static int
|
||||
|
||||
Reference in New Issue
Block a user