aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorParav Pandit <parav@mellanox.com>2017-02-02 00:14:08 -0500
committerDoug Ledford <dledford@redhat.com>2017-02-15 09:51:28 -0500
commit590396084039b6504097b371cda6474dfcb83648 (patch)
tree026b5c3b15ebaffb2e6fdfb9061e857ca33dc2be
parent2b0841766a898aba84630fb723989a77a9d3b4e6 (diff)
IB/core: Remove pointer casting from void to net_device
This patch avoids unnecessary type casting from void to net_device. CC: Yuval Shaia <yuval.shaia@oracle.com> Signed-off-by: Parav Pandit <parav@mellanox.com> Signed-off-by: Leon Romanovsky <leon@kernel.org> Reviewed-by: Yuval Shaia <yuval.shaia@oracle.com> Signed-off-by: Doug Ledford <dledford@redhat.com>
-rw-r--r--drivers/infiniband/core/roce_gid_mgmt.c28
1 files changed, 10 insertions, 18 deletions
diff --git a/drivers/infiniband/core/roce_gid_mgmt.c b/drivers/infiniband/core/roce_gid_mgmt.c
index 0621f4455732..db958d3207ef 100644
--- a/drivers/infiniband/core/roce_gid_mgmt.c
+++ b/drivers/infiniband/core/roce_gid_mgmt.c
@@ -144,7 +144,6 @@ static enum bonding_slave_state is_eth_active_slave_of_bonding_rcu(struct net_de
144static int is_eth_port_of_netdev(struct ib_device *ib_dev, u8 port, 144static int is_eth_port_of_netdev(struct ib_device *ib_dev, u8 port,
145 struct net_device *rdma_ndev, void *cookie) 145 struct net_device *rdma_ndev, void *cookie)
146{ 146{
147 struct net_device *event_ndev = (struct net_device *)cookie;
148 struct net_device *real_dev; 147 struct net_device *real_dev;
149 int res; 148 int res;
150 149
@@ -152,11 +151,11 @@ static int is_eth_port_of_netdev(struct ib_device *ib_dev, u8 port,
152 return 0; 151 return 0;
153 152
154 rcu_read_lock(); 153 rcu_read_lock();
155 real_dev = rdma_vlan_dev_real_dev(event_ndev); 154 real_dev = rdma_vlan_dev_real_dev(cookie);
156 if (!real_dev) 155 if (!real_dev)
157 real_dev = event_ndev; 156 real_dev = cookie;
158 157
159 res = ((rdma_is_upper_dev_rcu(rdma_ndev, event_ndev) && 158 res = ((rdma_is_upper_dev_rcu(rdma_ndev, cookie) &&
160 (is_eth_active_slave_of_bonding_rcu(rdma_ndev, real_dev) & 159 (is_eth_active_slave_of_bonding_rcu(rdma_ndev, real_dev) &
161 REQUIRED_BOND_STATES)) || 160 REQUIRED_BOND_STATES)) ||
162 real_dev == rdma_ndev); 161 real_dev == rdma_ndev);
@@ -192,17 +191,16 @@ static int pass_all_filter(struct ib_device *ib_dev, u8 port,
192static int upper_device_filter(struct ib_device *ib_dev, u8 port, 191static int upper_device_filter(struct ib_device *ib_dev, u8 port,
193 struct net_device *rdma_ndev, void *cookie) 192 struct net_device *rdma_ndev, void *cookie)
194{ 193{
195 struct net_device *event_ndev = (struct net_device *)cookie;
196 int res; 194 int res;
197 195
198 if (!rdma_ndev) 196 if (!rdma_ndev)
199 return 0; 197 return 0;
200 198
201 if (rdma_ndev == event_ndev) 199 if (rdma_ndev == cookie)
202 return 1; 200 return 1;
203 201
204 rcu_read_lock(); 202 rcu_read_lock();
205 res = rdma_is_upper_dev_rcu(rdma_ndev, event_ndev); 203 res = rdma_is_upper_dev_rcu(rdma_ndev, cookie);
206 rcu_read_unlock(); 204 rcu_read_unlock();
207 205
208 return res; 206 return res;
@@ -379,18 +377,14 @@ static void _add_netdev_ips(struct ib_device *ib_dev, u8 port,
379static void add_netdev_ips(struct ib_device *ib_dev, u8 port, 377static void add_netdev_ips(struct ib_device *ib_dev, u8 port,
380 struct net_device *rdma_ndev, void *cookie) 378 struct net_device *rdma_ndev, void *cookie)
381{ 379{
382 struct net_device *event_ndev = (struct net_device *)cookie; 380 enum_netdev_default_gids(ib_dev, port, cookie, rdma_ndev);
383 381 _add_netdev_ips(ib_dev, port, cookie);
384 enum_netdev_default_gids(ib_dev, port, event_ndev, rdma_ndev);
385 _add_netdev_ips(ib_dev, port, event_ndev);
386} 382}
387 383
388static void del_netdev_ips(struct ib_device *ib_dev, u8 port, 384static void del_netdev_ips(struct ib_device *ib_dev, u8 port,
389 struct net_device *rdma_ndev, void *cookie) 385 struct net_device *rdma_ndev, void *cookie)
390{ 386{
391 struct net_device *event_ndev = (struct net_device *)cookie; 387 ib_cache_gid_del_all_netdev_gids(ib_dev, port, cookie);
392
393 ib_cache_gid_del_all_netdev_gids(ib_dev, port, event_ndev);
394} 388}
395 389
396static void enum_all_gids_of_dev_cb(struct ib_device *ib_dev, 390static void enum_all_gids_of_dev_cb(struct ib_device *ib_dev,
@@ -460,7 +454,7 @@ static void handle_netdev_upper(struct ib_device *ib_dev, u8 port,
460 u8 port, 454 u8 port,
461 struct net_device *ndev)) 455 struct net_device *ndev))
462{ 456{
463 struct net_device *ndev = (struct net_device *)cookie; 457 struct net_device *ndev = cookie;
464 struct upper_list *upper_iter; 458 struct upper_list *upper_iter;
465 struct upper_list *upper_temp; 459 struct upper_list *upper_temp;
466 LIST_HEAD(upper_list); 460 LIST_HEAD(upper_list);
@@ -519,9 +513,7 @@ static void del_netdev_default_ips_join(struct ib_device *ib_dev, u8 port,
519static void del_netdev_default_ips(struct ib_device *ib_dev, u8 port, 513static void del_netdev_default_ips(struct ib_device *ib_dev, u8 port,
520 struct net_device *rdma_ndev, void *cookie) 514 struct net_device *rdma_ndev, void *cookie)
521{ 515{
522 struct net_device *event_ndev = (struct net_device *)cookie; 516 bond_delete_netdev_default_gids(ib_dev, port, cookie, rdma_ndev);
523
524 bond_delete_netdev_default_gids(ib_dev, port, event_ndev, rdma_ndev);
525} 517}
526 518
527/* The following functions operate on all IB devices. netdevice_event and 519/* The following functions operate on all IB devices. netdevice_event and