RDMA/core: Use dev_name instead of ibdev->name
authorJason Gunthorpe <jgg@mellanox.com>
Thu, 20 Sep 2018 22:42:25 +0000 (16:42 -0600)
committerJason Gunthorpe <jgg@mellanox.com>
Wed, 26 Sep 2018 19:51:48 +0000 (13:51 -0600)
These return the same thing but dev_name is a more conventional use of the
kernel API.

Signed-off-by: Jason Gunthorpe <jgg@mellanox.com>
Reviewed-by: Steve Wise <swise@opengridcomputing.com>
Reviewed-by: Leon Romanovsky <leonro@mellanox.com>
Reviewed-by: Dennis Dalessandro <dennis.dalessandro@intel.com>
drivers/infiniband/core/cm.c
drivers/infiniband/core/cma_configfs.c
drivers/infiniband/core/device.c
drivers/infiniband/core/fmr_pool.c
drivers/infiniband/core/iwcm.c
drivers/infiniband/core/nldev.c
drivers/infiniband/core/sa_query.c
drivers/infiniband/core/security.c
drivers/infiniband/core/user_mad.c
drivers/infiniband/core/uverbs_main.c

index 6e39c27dca8ec1cf8205d4180609ea994f8d88ed..a6a20603ccea5192d9e4648bcda303d062110b5b 100644 (file)
@@ -4367,7 +4367,7 @@ static void cm_add_one(struct ib_device *ib_device)
        cm_dev->going_down = 0;
        cm_dev->device = device_create(&cm_class, &ib_device->dev,
                                       MKDEV(0, 0), NULL,
-                                      "%s", ib_device->name);
+                                      "%s", dev_name(&ib_device->dev));
        if (IS_ERR(cm_dev->device)) {
                kfree(cm_dev);
                return;
index eee38b40be99041e69b28cd77c6322848e5663b2..8c2dfb3e294ec9043b89685adbac24af1c44db8d 100644 (file)
@@ -65,7 +65,7 @@ static struct cma_dev_port_group *to_dev_port_group(struct config_item *item)
 
 static bool filter_by_name(struct ib_device *ib_dev, void *cookie)
 {
-       return !strcmp(ib_dev->name, cookie);
+       return !strcmp(dev_name(&ib_dev->dev), cookie);
 }
 
 static int cma_configfs_params_get(struct config_item *item,
index 7c3ff43092fd9f43a76057a6136c8c2404f82a8d..d105b9b2d11864f2128e2c9a67e0681be903ac09 100644 (file)
@@ -165,7 +165,7 @@ static struct ib_device *__ib_device_get_by_name(const char *name)
        struct ib_device *device;
 
        list_for_each_entry(device, &device_list, core_list)
-               if (!strncmp(name, device->name, IB_DEVICE_NAME_MAX))
+               if (!strcmp(name, dev_name(&device->dev)))
                        return device;
 
        return NULL;
@@ -184,7 +184,7 @@ static int alloc_name(struct ib_device *ibdev, const char *name)
        list_for_each_entry(device, &device_list, core_list) {
                char buf[IB_DEVICE_NAME_MAX];
 
-               if (sscanf(device->name, name, &i) != 1)
+               if (sscanf(dev_name(&device->dev), name, &i) != 1)
                        continue;
                if (i < 0 || i >= PAGE_SIZE * 8)
                        continue;
@@ -219,9 +219,7 @@ static void ib_device_release(struct device *device)
 static int ib_device_uevent(struct device *device,
                            struct kobj_uevent_env *env)
 {
-       struct ib_device *dev = container_of(device, struct ib_device, dev);
-
-       if (add_uevent_var(env, "NAME=%s", dev->name))
+       if (add_uevent_var(env, "NAME=%s", dev_name(device)))
                return -ENOMEM;
 
        /*
index d1f2eee7d5daa87d1516d5e8365f07b921d5458b..83ba0068e8bb97621e76736cb56c3da75065efe1 100644 (file)
@@ -257,7 +257,8 @@ struct ib_fmr_pool *ib_create_fmr_pool(struct ib_pd             *pd,
        atomic_set(&pool->flush_ser, 0);
        init_waitqueue_head(&pool->force_wait);
 
-       pool->worker = kthread_create_worker(0, "ib_fmr(%s)", device->name);
+       pool->worker =
+               kthread_create_worker(0, "ib_fmr(%s)", dev_name(&device->dev));
        if (IS_ERR(pool->worker)) {
                pr_warn(PFX "couldn't start cleanup kthread worker\n");
                ret = PTR_ERR(pool->worker);
index 5d676cff41f496ce519f4dc000eda17f6fd43999..ba668d49c751d64c2161b85f58644207acadceda 100644 (file)
@@ -509,7 +509,7 @@ static int iw_cm_map(struct iw_cm_id *cm_id, bool active)
        cm_id->m_local_addr = cm_id->local_addr;
        cm_id->m_remote_addr = cm_id->remote_addr;
 
-       memcpy(pm_reg_msg.dev_name, cm_id->device->name,
+       memcpy(pm_reg_msg.dev_name, dev_name(&cm_id->device->dev),
               sizeof(pm_reg_msg.dev_name));
        memcpy(pm_reg_msg.if_name, cm_id->device->iwcm->ifname,
               sizeof(pm_reg_msg.if_name));
index 0385ab4383208b40a692f60cced8429d8486b528..ba5403fbcd88bf87e4a017e1df2675790ca6e5d5 100644 (file)
@@ -179,7 +179,8 @@ static int fill_nldev_handle(struct sk_buff *msg, struct ib_device *device)
 {
        if (nla_put_u32(msg, RDMA_NLDEV_ATTR_DEV_INDEX, device->index))
                return -EMSGSIZE;
-       if (nla_put_string(msg, RDMA_NLDEV_ATTR_DEV_NAME, device->name))
+       if (nla_put_string(msg, RDMA_NLDEV_ATTR_DEV_NAME,
+                          dev_name(&device->dev)))
                return -EMSGSIZE;
 
        return 0;
index d3d6275b3b7efa2320d4faefaa745f953bcdf37e..19e1833e13fc100e5608031a3cb4d041314c0d7c 100644 (file)
@@ -761,7 +761,7 @@ static void ib_nl_set_path_rec_attrs(struct sk_buff *skb,
 
        /* Construct the family header first */
        header = skb_put(skb, NLMSG_ALIGN(sizeof(*header)));
-       memcpy(header->device_name, query->port->agent->device->name,
+       memcpy(header->device_name, dev_name(&query->port->agent->device->dev),
               LS_DEVICE_NAME_MAX);
        header->port_num = query->port->port_num;
 
index 9b0bea8303e073c8983d2327d5fffc144b50e7ae..1143c0448666847699939f327c1a6d99581e90a5 100644 (file)
@@ -685,9 +685,8 @@ static int ib_mad_agent_security_change(struct notifier_block *nb,
        if (event != LSM_POLICY_CHANGE)
                return NOTIFY_DONE;
 
-       ag->smp_allowed = !security_ib_endport_manage_subnet(ag->security,
-                                                            ag->device->name,
-                                                            ag->port_num);
+       ag->smp_allowed = !security_ib_endport_manage_subnet(
+               ag->security, dev_name(&ag->device->dev), ag->port_num);
 
        return NOTIFY_OK;
 }
@@ -708,7 +707,7 @@ int ib_mad_agent_security_setup(struct ib_mad_agent *agent,
                return 0;
 
        ret = security_ib_endport_manage_subnet(agent->security,
-                                               agent->device->name,
+                                               dev_name(&agent->device->dev),
                                                agent->port_num);
        if (ret)
                return ret;
index c34a6852d691f666fb1d8deac965335c4f7c0840..9961859da06a59dee7f4cd391814e054645603fd 100644 (file)
@@ -1132,7 +1132,7 @@ static ssize_t show_ibdev(struct device *dev, struct device_attribute *attr,
        if (!port)
                return -ENODEV;
 
-       return sprintf(buf, "%s\n", port->ib_dev->name);
+       return sprintf(buf, "%s\n", dev_name(&port->ib_dev->dev));
 }
 static DEVICE_ATTR(ibdev, S_IRUGO, show_ibdev, NULL);
 
index 8d56773aac564e1fada6a1594e29f4d9d59cc44b..12d8f809757429149a47c2833c2000078b0bfe74 100644 (file)
@@ -1179,7 +1179,7 @@ static ssize_t ibdev_show(struct device *device, struct device_attribute *attr,
        srcu_key = srcu_read_lock(&dev->disassociate_srcu);
        ib_dev = srcu_dereference(dev->ib_dev, &dev->disassociate_srcu);
        if (ib_dev)
-               ret = sprintf(buf, "%s\n", ib_dev->name);
+               ret = sprintf(buf, "%s\n", dev_name(&ib_dev->dev));
        srcu_read_unlock(&dev->disassociate_srcu, srcu_key);
 
        return ret;