Commit 162add8c authored by Parav Pandit's avatar Parav Pandit Committed by David S. Miller
Browse files

net/mlx5e: Use devlink virtual flavour for VF devlink port



Use newly introduce 'virtual' port flavour for devlink
port of PCI VF devlink device in non-representors mode.

While at it, remove recently introduced empty lines at end of the file.

Reviewed-by: default avatarJiri Pirko <jiri@mellanox.com>
Signed-off-by: default avatarParav Pandit <parav@mellanox.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent acf1ee44
Loading
Loading
Loading
Loading
+1 −1
Original line number Diff line number Diff line
@@ -880,7 +880,7 @@ struct mlx5e_priv {
#endif
	struct devlink_health_reporter *tx_reporter;
	struct devlink_health_reporter *rx_reporter;
	struct devlink_port             dl_phy_port;
	struct devlink_port            dl_port;
	struct mlx5e_xsk           xsk;
#if IS_ENABLED(CONFIG_PCI_HYPERV_INTERFACE)
	struct mlx5e_hv_vhca_stats_agent stats_agent;
+23 −16
Original line number Diff line number Diff line
@@ -3,36 +3,43 @@

#include "en/devlink.h"

int mlx5e_devlink_phy_port_register(struct net_device *dev)
int mlx5e_devlink_port_register(struct net_device *netdev)
{
	struct mlx5_core_dev *dev;
	struct mlx5e_priv *priv;
	struct devlink *devlink;
	int err;

	priv = netdev_priv(dev);
	devlink = priv_to_devlink(priv->mdev);
	priv = netdev_priv(netdev);
	dev = priv->mdev;

	devlink_port_attrs_set(&priv->dl_phy_port,
	if (mlx5_core_is_pf(dev))
		devlink_port_attrs_set(&priv->dl_port,
				       DEVLINK_PORT_FLAVOUR_PHYSICAL,
			       PCI_FUNC(priv->mdev->pdev->devfn),
				       PCI_FUNC(dev->pdev->devfn),
				       false, 0,
				       NULL, 0);
	err = devlink_port_register(devlink, &priv->dl_phy_port, 1);
	else
		devlink_port_attrs_set(&priv->dl_port,
				       DEVLINK_PORT_FLAVOUR_VIRTUAL,
				       0, false, 0, NULL, 0);

	devlink = priv_to_devlink(dev);
	err = devlink_port_register(devlink, &priv->dl_port, 1);
	if (err)
		return err;
	devlink_port_type_eth_set(&priv->dl_phy_port, dev);
	devlink_port_type_eth_set(&priv->dl_port, netdev);
	return 0;
}

void mlx5e_devlink_phy_port_unregister(struct mlx5e_priv *priv)
void mlx5e_devlink_port_unregister(struct mlx5e_priv *priv)
{
	devlink_port_unregister(&priv->dl_phy_port);
	devlink_port_unregister(&priv->dl_port);
}

struct devlink_port *mlx5e_get_devlink_phy_port(struct net_device *dev)
struct devlink_port *mlx5e_get_devlink_port(struct net_device *dev)
{
	struct mlx5e_priv *priv = netdev_priv(dev);

	return &priv->dl_phy_port;
	return &priv->dl_port;
}
+3 −4
Original line number Diff line number Diff line
@@ -7,9 +7,8 @@
#include <net/devlink.h>
#include "en.h"

int mlx5e_devlink_phy_port_register(struct net_device *dev);
void mlx5e_devlink_phy_port_unregister(struct mlx5e_priv *priv);
struct devlink_port *mlx5e_get_devlink_phy_port(struct net_device *dev);
int mlx5e_devlink_port_register(struct net_device *dev);
void mlx5e_devlink_port_unregister(struct mlx5e_priv *priv);
struct devlink_port *mlx5e_get_devlink_port(struct net_device *dev);

#endif
+3 −3
Original line number Diff line number Diff line
@@ -4606,7 +4606,7 @@ const struct net_device_ops mlx5e_netdev_ops = {
	.ndo_set_vf_link_state   = mlx5e_set_vf_link_state,
	.ndo_get_vf_stats        = mlx5e_get_vf_stats,
#endif
	.ndo_get_devlink_port    = mlx5e_get_devlink_phy_port,
	.ndo_get_devlink_port    = mlx5e_get_devlink_port,
};

static int mlx5e_check_required_hca_cap(struct mlx5_core_dev *mdev)
@@ -5473,7 +5473,7 @@ static void *mlx5e_add(struct mlx5_core_dev *mdev)
		goto err_detach;
	}

	err = mlx5e_devlink_phy_port_register(netdev);
	err = mlx5e_devlink_port_register(netdev);
	if (err) {
		mlx5_core_err(mdev, "mlx5e_devlink_phy_port_register failed, %d\n", err);
		goto err_unregister_netdev;
@@ -5507,7 +5507,7 @@ static void mlx5e_remove(struct mlx5_core_dev *mdev, void *vpriv)
#ifdef CONFIG_MLX5_CORE_EN_DCB
	mlx5e_dcbnl_delete_app(priv);
#endif
	mlx5e_devlink_phy_port_unregister(priv);
	mlx5e_devlink_port_unregister(priv);
	unregister_netdev(priv->netdev);
	mlx5e_detach(mdev, vpriv);
	mlx5e_destroy_netdev(priv);