diff options
author | Jason Gunthorpe <jgg@mellanox.com> | 2019-01-12 02:42:42 +0000 |
---|---|---|
committer | Jason Gunthorpe <jgg@mellanox.com> | 2019-01-18 13:05:58 -0700 |
commit | 344684e6d02ff21ff2fce8cf1b2de3fd3cafcac7 (patch) | |
tree | 6782ef8c0446bb99955672b189510917b8cf6ee2 | |
parent | de77503a59403e7045c18c6bb0a10c245a99b648 (diff) | |
download | linux-344684e6d02ff21ff2fce8cf1b2de3fd3cafcac7.tar.bz2 |
RDMA/device: Use __ib_device_get_by_name() in ib_device_rename()
No reason to open code this loop.
Signed-off-by: Jason Gunthorpe <jgg@mellanox.com>
Reviewed-by: Steve Wise <swise@opengridcomputing.com>
-rw-r--r-- | drivers/infiniband/core/device.c | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/drivers/infiniband/core/device.c b/drivers/infiniband/core/device.c index f8180cf1a004..4a9aa6d10c5e 100644 --- a/drivers/infiniband/core/device.c +++ b/drivers/infiniband/core/device.c @@ -183,18 +183,15 @@ static struct ib_device *__ib_device_get_by_name(const char *name) int ib_device_rename(struct ib_device *ibdev, const char *name) { - struct ib_device *device; int ret = 0; if (!strcmp(name, dev_name(&ibdev->dev))) return ret; mutex_lock(&device_mutex); - list_for_each_entry(device, &device_list, core_list) { - if (!strcmp(name, dev_name(&device->dev))) { - ret = -EEXIST; - goto out; - } + if (__ib_device_get_by_name(name)) { + ret = -EEXIST; + goto out; } ret = device_rename(&ibdev->dev, name); |