| Message ID | 20241031073914.2368421-1-yanjun.zhu@linux.dev (mailing list archive) |
|---|---|
| State | Changes Requested |
| Headers | show |
| Series | [1/1] RDMA/mlx4: Use IB get_netdev functions and remove get_netdev callback | expand |
On Thu, Oct 31, 2024 at 08:39:14AM +0100, Zhu Yanjun wrote: > In the commit 8d159eb2117b ("RDMA/mlx5: Use IB set_netdev and get_netdev > functions") removed the get_netdev callback from > mlx5_ib_dev_common_roce_ops, in mlx4, get_netdev callback should also > be removed. It wasn't simple remove in mlx5, but an end result of long series to set right netdev in LAG mode. The similar thing needs to be done to mlx4, where you should set right netdev in bond mode. Thanks > > Signed-off-by: Zhu Yanjun <yanjun.zhu@linux.dev> > --- > compile successfully only > --- > --- > drivers/infiniband/hw/mlx4/main.c | 35 ------------------------------- > 1 file changed, 35 deletions(-) > > diff --git a/drivers/infiniband/hw/mlx4/main.c b/drivers/infiniband/hw/mlx4/main.c > index 529db874d67c..cf34d92de7b1 100644 > --- a/drivers/infiniband/hw/mlx4/main.c > +++ b/drivers/infiniband/hw/mlx4/main.c > @@ -123,40 +123,6 @@ static int num_ib_ports(struct mlx4_dev *dev) > return ib_ports; > } > > -static struct net_device *mlx4_ib_get_netdev(struct ib_device *device, > - u32 port_num) > -{ > - struct mlx4_ib_dev *ibdev = to_mdev(device); > - struct net_device *dev, *ret = NULL; > - > - rcu_read_lock(); > - for_each_netdev_rcu(&init_net, dev) { > - if (dev->dev.parent != ibdev->ib_dev.dev.parent || > - dev->dev_port + 1 != port_num) > - continue; > - > - if (mlx4_is_bonded(ibdev->dev)) { > - struct net_device *upper; > - > - upper = netdev_master_upper_dev_get_rcu(dev); > - if (upper) { > - struct net_device *active; > - > - active = bond_option_active_slave_get_rcu(netdev_priv(upper)); > - if (active) > - dev = active; > - } > - } > - > - dev_hold(dev); > - ret = dev; > - break; > - } > - > - rcu_read_unlock(); > - return ret; > -} > - > static int mlx4_ib_update_gids_v1(struct gid_entry *gids, > struct mlx4_ib_dev *ibdev, > u32 port_num) > @@ -2544,7 +2510,6 @@ static const struct ib_device_ops mlx4_ib_dev_ops = { > .get_dev_fw_str = get_fw_ver_str, > .get_dma_mr = mlx4_ib_get_dma_mr, > .get_link_layer = mlx4_ib_port_link_layer, > - .get_netdev = mlx4_ib_get_netdev, > .get_port_immutable = mlx4_port_immutable, > .map_mr_sg = mlx4_ib_map_mr_sg, > .mmap = mlx4_ib_mmap, > -- > 2.34.1 > >
diff --git a/drivers/infiniband/hw/mlx4/main.c b/drivers/infiniband/hw/mlx4/main.c index 529db874d67c..cf34d92de7b1 100644 --- a/drivers/infiniband/hw/mlx4/main.c +++ b/drivers/infiniband/hw/mlx4/main.c @@ -123,40 +123,6 @@ static int num_ib_ports(struct mlx4_dev *dev) return ib_ports; } -static struct net_device *mlx4_ib_get_netdev(struct ib_device *device, - u32 port_num) -{ - struct mlx4_ib_dev *ibdev = to_mdev(device); - struct net_device *dev, *ret = NULL; - - rcu_read_lock(); - for_each_netdev_rcu(&init_net, dev) { - if (dev->dev.parent != ibdev->ib_dev.dev.parent || - dev->dev_port + 1 != port_num) - continue; - - if (mlx4_is_bonded(ibdev->dev)) { - struct net_device *upper; - - upper = netdev_master_upper_dev_get_rcu(dev); - if (upper) { - struct net_device *active; - - active = bond_option_active_slave_get_rcu(netdev_priv(upper)); - if (active) - dev = active; - } - } - - dev_hold(dev); - ret = dev; - break; - } - - rcu_read_unlock(); - return ret; -} - static int mlx4_ib_update_gids_v1(struct gid_entry *gids, struct mlx4_ib_dev *ibdev, u32 port_num) @@ -2544,7 +2510,6 @@ static const struct ib_device_ops mlx4_ib_dev_ops = { .get_dev_fw_str = get_fw_ver_str, .get_dma_mr = mlx4_ib_get_dma_mr, .get_link_layer = mlx4_ib_port_link_layer, - .get_netdev = mlx4_ib_get_netdev, .get_port_immutable = mlx4_port_immutable, .map_mr_sg = mlx4_ib_map_mr_sg, .mmap = mlx4_ib_mmap,
In the commit 8d159eb2117b ("RDMA/mlx5: Use IB set_netdev and get_netdev functions") removed the get_netdev callback from mlx5_ib_dev_common_roce_ops, in mlx4, get_netdev callback should also be removed. Signed-off-by: Zhu Yanjun <yanjun.zhu@linux.dev> --- compile successfully only --- --- drivers/infiniband/hw/mlx4/main.c | 35 ------------------------------- 1 file changed, 35 deletions(-)