Commit 908d6460 authored by Daniel Jurgens's avatar Daniel Jurgens Committed by Jason Gunthorpe

IB/core: Change roce_rescan_device to return void

It always returns 0. Change return type to void.
Signed-off-by: default avatarDaniel Jurgens <danielj@mellanox.com>
Reviewed-by: default avatarParav Pandit <parav@mellanox.com>
Signed-off-by: default avatarLeon Romanovsky <leon@kernel.org>
Signed-off-by: default avatarJason Gunthorpe <jgg@mellanox.com>
parent 8737f818
...@@ -821,12 +821,7 @@ static int gid_table_setup_one(struct ib_device *ib_dev) ...@@ -821,12 +821,7 @@ static int gid_table_setup_one(struct ib_device *ib_dev)
if (err) if (err)
return err; return err;
err = roce_rescan_device(ib_dev); roce_rescan_device(ib_dev);
if (err) {
gid_table_cleanup_one(ib_dev);
gid_table_release_one(ib_dev);
}
return err; return err;
} }
......
...@@ -137,7 +137,7 @@ int ib_cache_gid_del_all_netdev_gids(struct ib_device *ib_dev, u8 port, ...@@ -137,7 +137,7 @@ int ib_cache_gid_del_all_netdev_gids(struct ib_device *ib_dev, u8 port,
int roce_gid_mgmt_init(void); int roce_gid_mgmt_init(void);
void roce_gid_mgmt_cleanup(void); void roce_gid_mgmt_cleanup(void);
int roce_rescan_device(struct ib_device *ib_dev); void roce_rescan_device(struct ib_device *ib_dev);
unsigned long roce_gid_type_mask_support(struct ib_device *ib_dev, u8 port); unsigned long roce_gid_type_mask_support(struct ib_device *ib_dev, u8 port);
int ib_cache_setup_one(struct ib_device *device); int ib_cache_setup_one(struct ib_device *device);
......
...@@ -412,12 +412,10 @@ static void enum_all_gids_of_dev_cb(struct ib_device *ib_dev, ...@@ -412,12 +412,10 @@ static void enum_all_gids_of_dev_cb(struct ib_device *ib_dev,
/* This function will rescan all of the network devices in the system /* This function will rescan all of the network devices in the system
* and add their gids, as needed, to the relevant RoCE devices. */ * and add their gids, as needed, to the relevant RoCE devices. */
int roce_rescan_device(struct ib_device *ib_dev) void roce_rescan_device(struct ib_device *ib_dev)
{ {
ib_enum_roce_netdev(ib_dev, pass_all_filter, NULL, ib_enum_roce_netdev(ib_dev, pass_all_filter, NULL,
enum_all_gids_of_dev_cb, NULL); enum_all_gids_of_dev_cb, NULL);
return 0;
} }
static void callback_for_addr_gid_device_scan(struct ib_device *device, static void callback_for_addr_gid_device_scan(struct ib_device *device,
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment