IB/uverbs: Do not pass struct ib_device to the ioctl methods

This does the same as the patch before, except for ioctl. The rules are
the same, but for the ioctl methods the core code handles setting up the
uobject.

- Retrieve the ib_dev from the uobject->context->device. This is
  safe under ioctl as the core has already done rdma_alloc_begin_uobject
  and so CREATE calls are entirely protected by the rwsem.
- Retrieve the ib_dev from uobject->object
- Call ib_uverbs_get_ucontext()

Signed-off-by: Jason Gunthorpe <jgg@mellanox.com>

diff --git a/drivers/infiniband/core/uverbs_ioctl.c b/drivers/infiniband/core/uverbs_ioctl.c
index f3776f909ca58b439380cf0090df352abf449f26..f0655a84f9d9c999b8fa91905011946ae08011a5 100644 (file)
--- a/drivers/infiniband/core/uverbs_ioctl.c
+++ b/drivers/infiniband/core/uverbs_ioctl.c
@@ -354,7 +354,7 @@ static int uverbs_handle_method(struct ib_uverbs_attr __user *uattr_ptr,
                        goto cleanup;
        }
 
-       ret = method_spec->handler(ibdev, ufile, attr_bundle);
+       ret = method_spec->handler(ufile, attr_bundle);
 
        if (destroy_attr) {
                uobj_put_destroy(destroy_attr->uobject);

diff --git a/drivers/infiniband/core/uverbs_std_types.c b/drivers/infiniband/core/uverbs_std_types.c
index c1e0492cc78a74cd2632b33ecc1ed8e4c5cf3c84..3aa7c7deac749a5ef37951d81bc42990bed6165d 100644 (file)
--- a/drivers/infiniband/core/uverbs_std_types.c
+++ b/drivers/infiniband/core/uverbs_std_types.c
@@ -210,8 +210,7 @@ static int uverbs_hot_unplug_completion_event_file(struct ib_uobject *uobj,
        return 0;
 };
 
-int uverbs_destroy_def_handler(struct ib_device *ib_dev,
-                              struct ib_uverbs_file *file,
+int uverbs_destroy_def_handler(struct ib_uverbs_file *file,
                               struct uverbs_attr_bundle *attrs)
 {
        return 0;

diff --git a/drivers/infiniband/core/uverbs_std_types_counters.c b/drivers/infiniband/core/uverbs_std_types_counters.c
index 34589799f4461a0c805f0ee533bbfa86ee63786f..dfacc9e833995d5390e3cae1130edd9d619743bb 100644 (file)
--- a/drivers/infiniband/core/uverbs_std_types_counters.c
+++ b/drivers/infiniband/core/uverbs_std_types_counters.c
@@ -47,12 +47,13 @@ static int uverbs_free_counters(struct ib_uobject *uobject,
        return counters->device->destroy_counters(counters);
 }
 
-static int UVERBS_HANDLER(UVERBS_METHOD_COUNTERS_CREATE)(struct ib_device *ib_dev,
-                                                        struct ib_uverbs_file *file,
-                                                        struct uverbs_attr_bundle *attrs)
+static int UVERBS_HANDLER(UVERBS_METHOD_COUNTERS_CREATE)(
+       struct ib_uverbs_file *file, struct uverbs_attr_bundle *attrs)
 {
+       struct ib_uobject *uobj = uverbs_attr_get_uobject(
+               attrs, UVERBS_ATTR_CREATE_COUNTERS_HANDLE);
+       struct ib_device *ib_dev = uobj->context->device;
        struct ib_counters *counters;
-       struct ib_uobject *uobj;
        int ret;
 
        /*
@@ -63,7 +64,6 @@ static int UVERBS_HANDLER(UVERBS_METHOD_COUNTERS_CREATE)(struct ib_device *ib_de
        if (!ib_dev->create_counters)
                return -EOPNOTSUPP;
 
-       uobj = uverbs_attr_get_uobject(attrs, UVERBS_ATTR_CREATE_COUNTERS_HANDLE);
        counters = ib_dev->create_counters(ib_dev, attrs);
        if (IS_ERR(counters)) {
                ret = PTR_ERR(counters);
@@ -81,9 +81,8 @@ err_create_counters:
        return ret;
 }
 
-static int UVERBS_HANDLER(UVERBS_METHOD_COUNTERS_READ)(struct ib_device *ib_dev,
-                                                      struct ib_uverbs_file *file,
-                                                      struct uverbs_attr_bundle *attrs)
+static int UVERBS_HANDLER(UVERBS_METHOD_COUNTERS_READ)(
+       struct ib_uverbs_file *file, struct uverbs_attr_bundle *attrs)
 {
        struct ib_counters_read_attr read_attr = {};
        const struct uverbs_attr *uattr;
@@ -91,7 +90,7 @@ static int UVERBS_HANDLER(UVERBS_METHOD_COUNTERS_READ)(struct ib_device *ib_dev,
                uverbs_attr_get_obj(attrs, UVERBS_ATTR_READ_COUNTERS_HANDLE);
        int ret;
 
-       if (!ib_dev->read_counters)
+       if (!counters->device->read_counters)
                return -EOPNOTSUPP;
 
        if (!atomic_read(&counters->usecnt))
@@ -110,9 +109,7 @@ static int UVERBS_HANDLER(UVERBS_METHOD_COUNTERS_READ)(struct ib_device *ib_dev,
        if (!read_attr.counters_buff)
                return -ENOMEM;
 
-       ret = ib_dev->read_counters(counters,
-                                   &read_attr,
-                                   attrs);
+       ret = counters->device->read_counters(counters, &read_attr, attrs);
        if (ret)
                goto err_read;
 

diff --git a/drivers/infiniband/core/uverbs_std_types_cq.c b/drivers/infiniband/core/uverbs_std_types_cq.c
index 68c86e6e932e11372aeeadc4695830b5ee6c3768..5b5f2052cd5216c64270391b0db78e847cc2d475 100644 (file)
--- a/drivers/infiniband/core/uverbs_std_types_cq.c
+++ b/drivers/infiniband/core/uverbs_std_types_cq.c
@@ -57,11 +57,13 @@ static int uverbs_free_cq(struct ib_uobject *uobject,
        return ret;
 }
 
-static int UVERBS_HANDLER(UVERBS_METHOD_CQ_CREATE)(struct ib_device *ib_dev,
-                                                  struct ib_uverbs_file *file,
-                                                  struct uverbs_attr_bundle *attrs)
+static int UVERBS_HANDLER(UVERBS_METHOD_CQ_CREATE)(
+       struct ib_uverbs_file *file, struct uverbs_attr_bundle *attrs)
 {
-       struct ib_ucq_object           *obj;
+       struct ib_ucq_object *obj = container_of(
+               uverbs_attr_get_uobject(attrs, UVERBS_ATTR_CREATE_CQ_HANDLE),
+               typeof(*obj), uobject);
+       struct ib_device *ib_dev = obj->uobject.context->device;
        struct ib_udata uhw;
        int ret;
        u64 user_handle;
@@ -104,9 +106,6 @@ static int UVERBS_HANDLER(UVERBS_METHOD_CQ_CREATE)(struct ib_device *ib_dev,
                goto err_event_file;
        }
 
-       obj = container_of(uverbs_attr_get_uobject(attrs,
-                                                  UVERBS_ATTR_CREATE_CQ_HANDLE),
-                          typeof(*obj), uobject);
        obj->comp_events_reported  = 0;
        obj->async_events_reported = 0;
        INIT_LIST_HEAD(&obj->comp_list);
@@ -173,9 +172,8 @@ DECLARE_UVERBS_NAMED_METHOD(
                            UA_MANDATORY),
        UVERBS_ATTR_UHW());
 
-static int UVERBS_HANDLER(UVERBS_METHOD_CQ_DESTROY)(struct ib_device *ib_dev,
-                                                   struct ib_uverbs_file *file,
-                                                   struct uverbs_attr_bundle *attrs)
+static int UVERBS_HANDLER(UVERBS_METHOD_CQ_DESTROY)(
+       struct ib_uverbs_file *file, struct uverbs_attr_bundle *attrs)
 {
        struct ib_uobject *uobj =
                uverbs_attr_get_uobject(attrs, UVERBS_ATTR_DESTROY_CQ_HANDLE);

diff --git a/drivers/infiniband/core/uverbs_std_types_dm.c b/drivers/infiniband/core/uverbs_std_types_dm.c
index c90efa4b99f4e26c7168a7c062a36c6a1622221b..edc3ff7733d47d38e6457c924efc85926a6c22ae 100644 (file)
--- a/drivers/infiniband/core/uverbs_std_types_dm.c
+++ b/drivers/infiniband/core/uverbs_std_types_dm.c
@@ -46,12 +46,15 @@ static int uverbs_free_dm(struct ib_uobject *uobject,
        return dm->device->dealloc_dm(dm);
 }
 
-static int UVERBS_HANDLER(UVERBS_METHOD_DM_ALLOC)(struct ib_device *ib_dev,
-                                                 struct ib_uverbs_file *file,
-                                                 struct uverbs_attr_bundle *attrs)
+static int
+UVERBS_HANDLER(UVERBS_METHOD_DM_ALLOC)(struct ib_uverbs_file *file,
+                                      struct uverbs_attr_bundle *attrs)
 {
        struct ib_dm_alloc_attr attr = {};
-       struct ib_uobject *uobj;
+       struct ib_uobject *uobj =
+               uverbs_attr_get(attrs, UVERBS_ATTR_ALLOC_DM_HANDLE)
+                       ->obj_attr.uobject;
+       struct ib_device *ib_dev = uobj->context->device;
        struct ib_dm *dm;
        int ret;
 
@@ -68,8 +71,6 @@ static int UVERBS_HANDLER(UVERBS_METHOD_DM_ALLOC)(struct ib_device *ib_dev,
        if (ret)
                return ret;
 
-       uobj = uverbs_attr_get(attrs, UVERBS_ATTR_ALLOC_DM_HANDLE)->obj_attr.uobject;
-
        dm = ib_dev->alloc_dm(ib_dev, uobj->context, &attr, attrs);
        if (IS_ERR(dm))
                return PTR_ERR(dm);

diff --git a/drivers/infiniband/core/uverbs_std_types_flow_action.c b/drivers/infiniband/core/uverbs_std_types_flow_action.c
index adb9209c47105e0e51d86744e166ab92014e0a38..d8cfafe23bd9cd9e1bff887d176bed47d1c6ae24 100644 (file)
--- a/drivers/infiniband/core/uverbs_std_types_flow_action.c
+++ b/drivers/infiniband/core/uverbs_std_types_flow_action.c
@@ -304,12 +304,13 @@ static int parse_flow_action_esp(struct ib_device *ib_dev,
        return 0;
 }
 
-static int UVERBS_HANDLER(UVERBS_METHOD_FLOW_ACTION_ESP_CREATE)(struct ib_device *ib_dev,
-                                                               struct ib_uverbs_file *file,
-                                                               struct uverbs_attr_bundle *attrs)
+static int UVERBS_HANDLER(UVERBS_METHOD_FLOW_ACTION_ESP_CREATE)(
+       struct ib_uverbs_file *file, struct uverbs_attr_bundle *attrs)
 {
+       struct ib_uobject *uobj = uverbs_attr_get_uobject(
+               attrs, UVERBS_ATTR_CREATE_FLOW_ACTION_ESP_HANDLE);
+       struct ib_device *ib_dev = uobj->context->device;
        int                               ret;
-       struct ib_uobject                 *uobj;
        struct ib_flow_action             *action;
        struct ib_flow_action_esp_attr    esp_attr = {};
 
@@ -321,8 +322,6 @@ static int UVERBS_HANDLER(UVERBS_METHOD_FLOW_ACTION_ESP_CREATE)(struct ib_device
                return ret;
 
        /* No need to check as this attribute is marked as MANDATORY */
-       uobj = uverbs_attr_get_uobject(
-               attrs, UVERBS_ATTR_CREATE_FLOW_ACTION_ESP_HANDLE);
        action = ib_dev->create_flow_action_esp(ib_dev, &esp_attr.hdr, attrs);
        if (IS_ERR(action))
                return PTR_ERR(action);
@@ -336,32 +335,28 @@ static int UVERBS_HANDLER(UVERBS_METHOD_FLOW_ACTION_ESP_CREATE)(struct ib_device
        return 0;
 }
 
-static int UVERBS_HANDLER(UVERBS_METHOD_FLOW_ACTION_ESP_MODIFY)(struct ib_device *ib_dev,
-                                                               struct ib_uverbs_file *file,
-                                                               struct uverbs_attr_bundle *attrs)
+static int UVERBS_HANDLER(UVERBS_METHOD_FLOW_ACTION_ESP_MODIFY)(
+       struct ib_uverbs_file *file, struct uverbs_attr_bundle *attrs)
 {
+       struct ib_uobject *uobj = uverbs_attr_get_uobject(
+               attrs, UVERBS_ATTR_MODIFY_FLOW_ACTION_ESP_HANDLE);
+       struct ib_flow_action *action = uobj->object;
        int                               ret;
-       struct ib_uobject                 *uobj;
-       struct ib_flow_action             *action;
        struct ib_flow_action_esp_attr    esp_attr = {};
 
-       if (!ib_dev->modify_flow_action_esp)
+       if (!action->device->modify_flow_action_esp)
                return -EOPNOTSUPP;
 
-       ret = parse_flow_action_esp(ib_dev, file, attrs, &esp_attr, true);
+       ret = parse_flow_action_esp(action->device, file, attrs, &esp_attr,
+                                   true);
        if (ret)
                return ret;
 
-       uobj = uverbs_attr_get_uobject(
-               attrs, UVERBS_ATTR_MODIFY_FLOW_ACTION_ESP_HANDLE);
-       action = uobj->object;
-
        if (action->type != IB_FLOW_ACTION_ESP)
                return -EINVAL;
 
-       return ib_dev->modify_flow_action_esp(action,
-                                             &esp_attr.hdr,
-                                             attrs);
+       return action->device->modify_flow_action_esp(action, &esp_attr.hdr,
+                                                     attrs);
 }
 
 static const struct uverbs_attr_spec uverbs_flow_action_esp_keymat[] = {

diff --git a/drivers/infiniband/core/uverbs_std_types_mr.c b/drivers/infiniband/core/uverbs_std_types_mr.c
index d63da0c2a8c175a10baf48b7c50a8391b0da6baf..cf02e774303e3b2d4dfac0d73acd1ae6f5bf86c1 100644 (file)
--- a/drivers/infiniband/core/uverbs_std_types_mr.c
+++ b/drivers/infiniband/core/uverbs_std_types_mr.c
@@ -39,14 +39,18 @@ static int uverbs_free_mr(struct ib_uobject *uobject,
        return ib_dereg_mr((struct ib_mr *)uobject->object);
 }
 
-static int UVERBS_HANDLER(UVERBS_METHOD_DM_MR_REG)(struct ib_device *ib_dev,
-                                                  struct ib_uverbs_file *file,
-                                                  struct uverbs_attr_bundle *attrs)
+static int UVERBS_HANDLER(UVERBS_METHOD_DM_MR_REG)(
+       struct ib_uverbs_file *file, struct uverbs_attr_bundle *attrs)
 {
        struct ib_dm_mr_attr attr = {};
-       struct ib_uobject *uobj;
-       struct ib_dm *dm;
-       struct ib_pd *pd;
+       struct ib_uobject *uobj =
+               uverbs_attr_get_uobject(attrs, UVERBS_ATTR_REG_DM_MR_HANDLE);
+       struct ib_dm *dm =
+               uverbs_attr_get_obj(attrs, UVERBS_ATTR_REG_DM_MR_DM_HANDLE);
+       struct ib_pd *pd =
+               uverbs_attr_get_obj(attrs, UVERBS_ATTR_REG_DM_MR_PD_HANDLE);
+       struct ib_device *ib_dev = pd->device;
+
        struct ib_mr *mr;
        int ret;
 
@@ -75,12 +79,6 @@ static int UVERBS_HANDLER(UVERBS_METHOD_DM_MR_REG)(struct ib_device *ib_dev,
        if (ret)
                return ret;
 
-       pd = uverbs_attr_get_obj(attrs, UVERBS_ATTR_REG_DM_MR_PD_HANDLE);
-
-       dm = uverbs_attr_get_obj(attrs, UVERBS_ATTR_REG_DM_MR_DM_HANDLE);
-
-       uobj = uverbs_attr_get(attrs, UVERBS_ATTR_REG_DM_MR_HANDLE)->obj_attr.uobject;
-
        if (attr.offset > dm->length || attr.length > dm->length ||
            attr.length > dm->length - attr.offset)
                return -EINVAL;

diff --git a/drivers/infiniband/hw/mlx5/devx.c b/drivers/infiniband/hw/mlx5/devx.c
index c9a7a12a8c13e3416d537c85d274ed6c848c4cc0..29c6883723902cfaf2ddf8283700cca61fe03794 100644 (file)
--- a/drivers/infiniband/hw/mlx5/devx.c
+++ b/drivers/infiniband/hw/mlx5/devx.c
@@ -409,11 +409,11 @@ static bool devx_is_general_cmd(void *in)
        }
 }
 
-static int UVERBS_HANDLER(MLX5_IB_METHOD_DEVX_QUERY_EQN)(struct ib_device *ib_dev,
-                                 struct ib_uverbs_file *file,
-                                 struct uverbs_attr_bundle *attrs)
+static int UVERBS_HANDLER(MLX5_IB_METHOD_DEVX_QUERY_EQN)(
+       struct ib_uverbs_file *file, struct uverbs_attr_bundle *attrs)
 {
-       struct mlx5_ib_dev *dev = to_mdev(ib_dev);
+       struct mlx5_ib_ucontext *c;
+       struct mlx5_ib_dev *dev;
        int user_vector;
        int dev_eqn;
        unsigned int irqn;
@@ -423,6 +423,11 @@ static int UVERBS_HANDLER(MLX5_IB_METHOD_DEVX_QUERY_EQN)(struct ib_device *ib_de
                             MLX5_IB_ATTR_DEVX_QUERY_EQN_USER_VEC))
                return -EFAULT;
 
+       c = devx_ufile2uctx(file);
+       if (IS_ERR(c))
+               return PTR_ERR(c);
+       dev = to_mdev(c->ibucontext.device);
+
        err = mlx5_vector2eqn(dev->mdev, user_vector, &dev_eqn, &irqn);
        if (err < 0)
                return err;
@@ -454,9 +459,8 @@ static int UVERBS_HANDLER(MLX5_IB_METHOD_DEVX_QUERY_EQN)(struct ib_device *ib_de
  * of the buggy user for execution (just insert it to the hardware schedule
  * queue or arm its CQ for event generation), no further harm is expected.
  */
-static int UVERBS_HANDLER(MLX5_IB_METHOD_DEVX_QUERY_UAR)(struct ib_device *ib_dev,
-                                 struct ib_uverbs_file *file,
-                                 struct uverbs_attr_bundle *attrs)
+static int UVERBS_HANDLER(MLX5_IB_METHOD_DEVX_QUERY_UAR)(
+       struct ib_uverbs_file *file, struct uverbs_attr_bundle *attrs)
 {
        struct mlx5_ib_ucontext *c;
        struct mlx5_ib_dev *dev;
@@ -483,9 +487,8 @@ static int UVERBS_HANDLER(MLX5_IB_METHOD_DEVX_QUERY_UAR)(struct ib_device *ib_de
        return 0;
 }
 
-static int UVERBS_HANDLER(MLX5_IB_METHOD_DEVX_OTHER)(struct ib_device *ib_dev,
-                                 struct ib_uverbs_file *file,
-                                 struct uverbs_attr_bundle *attrs)
+static int UVERBS_HANDLER(MLX5_IB_METHOD_DEVX_OTHER)(
+       struct ib_uverbs_file *file, struct uverbs_attr_bundle *attrs)
 {
        struct mlx5_ib_ucontext *c;
        struct mlx5_ib_dev *dev;
@@ -712,9 +715,8 @@ static int devx_obj_cleanup(struct ib_uobject *uobject,
        return ret;
 }
 
-static int UVERBS_HANDLER(MLX5_IB_METHOD_DEVX_OBJ_CREATE)(struct ib_device *ib_dev,
-                                  struct ib_uverbs_file *file,
-                                  struct uverbs_attr_bundle *attrs)
+static int UVERBS_HANDLER(MLX5_IB_METHOD_DEVX_OBJ_CREATE)(
+       struct ib_uverbs_file *file, struct uverbs_attr_bundle *attrs)
 {
        void *cmd_in = uverbs_attr_get_alloced_ptr(attrs, MLX5_IB_ATTR_DEVX_OBJ_CREATE_CMD_IN);
        int cmd_out_len =  uverbs_attr_get_len(attrs,
@@ -769,9 +771,8 @@ obj_free:
        return err;
 }
 
-static int UVERBS_HANDLER(MLX5_IB_METHOD_DEVX_OBJ_MODIFY)(struct ib_device *ib_dev,
-                                  struct ib_uverbs_file *file,
-                                  struct uverbs_attr_bundle *attrs)
+static int UVERBS_HANDLER(MLX5_IB_METHOD_DEVX_OBJ_MODIFY)(
+       struct ib_uverbs_file *file, struct uverbs_attr_bundle *attrs)
 {
        void *cmd_in = uverbs_attr_get_alloced_ptr(attrs, MLX5_IB_ATTR_DEVX_OBJ_MODIFY_CMD_IN);
        int cmd_out_len = uverbs_attr_get_len(attrs,
@@ -811,9 +812,8 @@ other_cmd_free:
        return err;
 }
 
-static int UVERBS_HANDLER(MLX5_IB_METHOD_DEVX_OBJ_QUERY)(struct ib_device *ib_dev,
-                                  struct ib_uverbs_file *file,
-                                  struct uverbs_attr_bundle *attrs)
+static int UVERBS_HANDLER(MLX5_IB_METHOD_DEVX_OBJ_QUERY)(
+       struct ib_uverbs_file *file, struct uverbs_attr_bundle *attrs)
 {
        void *cmd_in = uverbs_attr_get_alloced_ptr(attrs, MLX5_IB_ATTR_DEVX_OBJ_QUERY_CMD_IN);
        int cmd_out_len = uverbs_attr_get_len(attrs,
@@ -931,9 +931,8 @@ static void devx_umem_reg_cmd_build(struct mlx5_ib_dev *dev,
                             MLX5_IB_MTT_READ);
 }
 
-static int UVERBS_HANDLER(MLX5_IB_METHOD_DEVX_UMEM_REG)(struct ib_device *ib_dev,
-                                struct ib_uverbs_file *file,
-                                struct uverbs_attr_bundle *attrs)
+static int UVERBS_HANDLER(MLX5_IB_METHOD_DEVX_UMEM_REG)(
+       struct ib_uverbs_file *file, struct uverbs_attr_bundle *attrs)
 {
        struct devx_umem_reg_cmd cmd;
        struct devx_umem *obj;

diff --git a/drivers/infiniband/hw/mlx5/flow.c b/drivers/infiniband/hw/mlx5/flow.c
index ee398a9b5f26b0c32002d5f2e171ab098f4475bc..1a29f47f836e5abb076aa30ae388fb13f457cc04 100644 (file)
--- a/drivers/infiniband/hw/mlx5/flow.c
+++ b/drivers/infiniband/hw/mlx5/flow.c
@@ -39,8 +39,7 @@ static const struct uverbs_attr_spec mlx5_ib_flow_type[] = {
 };
 
 static int UVERBS_HANDLER(MLX5_IB_METHOD_CREATE_FLOW)(
-       struct ib_device *ib_dev, struct ib_uverbs_file *file,
-       struct uverbs_attr_bundle *attrs)
+       struct ib_uverbs_file *file, struct uverbs_attr_bundle *attrs)
 {
        struct mlx5_ib_flow_handler *flow_handler;
        struct mlx5_ib_flow_matcher *fs_matcher;
@@ -109,7 +108,7 @@ static int UVERBS_HANDLER(MLX5_IB_METHOD_CREATE_FLOW)(
        if (IS_ERR(flow_handler))
                return PTR_ERR(flow_handler);
 
-       ib_set_flow(uobj, &flow_handler->ibflow, qp, ib_dev);
+       ib_set_flow(uobj, &flow_handler->ibflow, qp, &dev->ib_dev);
 
        return 0;
 }
@@ -129,8 +128,7 @@ static int flow_matcher_cleanup(struct ib_uobject *uobject,
 }
 
 static int UVERBS_HANDLER(MLX5_IB_METHOD_FLOW_MATCHER_CREATE)(
-       struct ib_device *ib_dev, struct ib_uverbs_file *file,
-       struct uverbs_attr_bundle *attrs)
+       struct ib_uverbs_file *file, struct uverbs_attr_bundle *attrs)
 {
        struct ib_uobject *uobj = uverbs_attr_get_uobject(
                attrs, MLX5_IB_ATTR_FLOW_MATCHER_CREATE_HANDLE);

diff --git a/include/rdma/ib_verbs.h b/include/rdma/ib_verbs.h
index be208421f7d3e1f1482c8c500b51ee7032b3518f..dea770e5b9ae1b52f1674ce7eb0f57bedb0fdce6 100644 (file)
--- a/include/rdma/ib_verbs.h
+++ b/include/rdma/ib_verbs.h
@@ -4170,7 +4170,6 @@ void rdma_roce_rescan_device(struct ib_device *ibdev);
 
 struct ib_ucontext *ib_uverbs_get_ucontext(struct ib_uverbs_file *ufile);
 
-int uverbs_destroy_def_handler(struct ib_device *ib_dev,
-                              struct ib_uverbs_file *file,
+int uverbs_destroy_def_handler(struct ib_uverbs_file *file,
                               struct uverbs_attr_bundle *attrs);
 #endif /* IB_VERBS_H */

diff --git a/include/rdma/uverbs_ioctl.h b/include/rdma/uverbs_ioctl.h
index 5e6d0569d97c0968f2756cd70674501ba81a09c0..8d71b7a7f1472a7a565ad96b914c10eca2777135 100644 (file)
--- a/include/rdma/uverbs_ioctl.h
+++ b/include/rdma/uverbs_ioctl.h
@@ -128,7 +128,7 @@ struct uverbs_method_spec {
        u32                                             flags;
        size_t                                          num_buckets;
        size_t                                          num_child_attrs;
-       int (*handler)(struct ib_device *ib_dev, struct ib_uverbs_file *ufile,
+       int (*handler)(struct ib_uverbs_file *ufile,
                       struct uverbs_attr_bundle *ctx);
        struct uverbs_attr_spec_hash            *attr_buckets[0];
 };
@@ -171,7 +171,7 @@ struct uverbs_method_def {
        u32                                  flags;
        size_t                               num_attrs;
        const struct uverbs_attr_def * const (*attrs)[];
-       int (*handler)(struct ib_device *ib_dev, struct ib_uverbs_file *ufile,
+       int (*handler)(struct ib_uverbs_file *ufile,
                       struct uverbs_attr_bundle *ctx);
 };