From: Yishai Hadas <yishaih@mellanox.com>
To: linux-rdma@vger.kernel.org
Cc: jgg@mellanox.com, dledford@redhat.com, yishaih@mellanox.com,
maorg@mellanox.com, michaelgur@mellanox.com
Subject: [PATCH rdma-core 3/7] verbs: Move alloc_context to ioctl
Date: Thu, 9 Jan 2020 16:04:32 +0200 [thread overview]
Message-ID: <1578578676-752-4-git-send-email-yishaih@mellanox.com> (raw)
In-Reply-To: <1578578676-752-1-git-send-email-yishaih@mellanox.com>
From: Michael Guralnik <michaelgur@mellanox.com>
Execute alloc_context using ioctl mechanism with fallback to write
method.
In the ioctl flow split the aync_fd allocation to a new ioctl call and
get the 'core_support' field returned from the kernel.
Signed-off-by: Michael Guralnik <michaelgur@mellanox.com>
Signed-off-by: Yishai Hadas <yishaih@mellanox.com>
---
libibverbs/cmd.c | 18 -----------
libibverbs/cmd_device.c | 79 +++++++++++++++++++++++++++++++++++++++++++++++++
libibverbs/driver.h | 1 +
3 files changed, 80 insertions(+), 18 deletions(-)
diff --git a/libibverbs/cmd.c b/libibverbs/cmd.c
index 26eaa47..b68f3e3 100644
--- a/libibverbs/cmd.c
+++ b/libibverbs/cmd.c
@@ -47,24 +47,6 @@
bool verbs_allow_disassociate_destroy;
-int ibv_cmd_get_context(struct verbs_context *context_ex,
- struct ibv_get_context *cmd, size_t cmd_size,
- struct ib_uverbs_get_context_resp *resp, size_t resp_size)
-{
- int ret;
-
- ret = execute_cmd_write(&context_ex->context,
- IB_USER_VERBS_CMD_GET_CONTEXT, cmd, cmd_size,
- resp, resp_size);
- if (ret)
- return ret;
-
- context_ex->context.async_fd = resp->async_fd;
- context_ex->context.num_comp_vectors = resp->num_comp_vectors;
-
- return 0;
-}
-
static void copy_query_dev_fields(struct ibv_device_attr *device_attr,
struct ib_uverbs_query_device_resp *resp,
uint64_t *raw_fw_ver)
diff --git a/libibverbs/cmd_device.c b/libibverbs/cmd_device.c
index d806351..4de59c0 100644
--- a/libibverbs/cmd_device.c
+++ b/libibverbs/cmd_device.c
@@ -99,3 +99,82 @@ int ibv_cmd_query_port(struct ibv_context *context, uint8_t port_num,
return 0;
}
+static int cmd_alloc_async_fd(struct ibv_context *context)
+{
+ DECLARE_COMMAND_BUFFER(cmdb, UVERBS_OBJECT_ASYNC_EVENT,
+ UVERBS_METHOD_ASYNC_EVENT_ALLOC, 1);
+ struct ib_uverbs_attr *handle;
+ int ret;
+
+ handle = fill_attr_out_fd(cmdb, UVERBS_ATTR_ASYNC_EVENT_ALLOC_FD_HANDLE,
+ 0);
+
+ ret = execute_ioctl(context, cmdb);
+ if (ret)
+ return ret;
+
+ context->async_fd =
+ read_attr_fd(UVERBS_ATTR_ASYNC_EVENT_ALLOC_FD_HANDLE, handle);
+ return 0;
+}
+
+static int cmd_get_context(struct verbs_context *context_ex,
+ struct ibv_command_buffer *link)
+{
+ DECLARE_FBCMD_BUFFER(cmdb, UVERBS_OBJECT_DEVICE,
+ UVERBS_METHOD_GET_CONTEXT, 2, link);
+
+ struct ibv_context *context = &context_ex->context;
+ struct verbs_device *verbs_device;
+ uint64_t core_support;
+ uint32_t num_comp_vectors;
+ int ret;
+
+ fill_attr_out_ptr(cmdb, UVERBS_ATTR_GET_CONTEXT_NUM_COMP_VECTORS,
+ &num_comp_vectors);
+ fill_attr_out_ptr(cmdb, UVERBS_ATTR_GET_CONTEXT_CORE_SUPPORT,
+ &core_support);
+
+ /* Using free_context cmd_name as alloc context is not in
+ * verbs_context_ops while free_context is and doesn't use ioctl
+ */
+ switch (execute_ioctl_fallback(context, free_context, cmdb, &ret)) {
+ case TRY_WRITE: {
+ DECLARE_LEGACY_UHW_BUFS(link, IB_USER_VERBS_CMD_GET_CONTEXT);
+
+ ret = execute_write_bufs(context, IB_USER_VERBS_CMD_GET_CONTEXT,
+ req, resp);
+ if (ret)
+ return ret;
+
+ context->async_fd = resp->async_fd;
+ context->num_comp_vectors = resp->num_comp_vectors;
+
+ return 0;
+ }
+ case SUCCESS:
+ ret = cmd_alloc_async_fd(context);
+ if (ret)
+ return ret;
+ break;
+ default:
+ return ret;
+ };
+
+ context->num_comp_vectors = num_comp_vectors;
+ verbs_device = verbs_get_device(context->device);
+ verbs_device->core_support = core_support;
+ return 0;
+}
+
+int ibv_cmd_get_context(struct verbs_context *context_ex,
+ struct ibv_get_context *cmd, size_t cmd_size,
+ struct ib_uverbs_get_context_resp *resp,
+ size_t resp_size)
+{
+ DECLARE_CMD_BUFFER_COMPAT(cmdb, UVERBS_OBJECT_DEVICE,
+ UVERBS_METHOD_GET_CONTEXT, cmd, cmd_size,
+ resp, resp_size);
+
+ return cmd_get_context(context_ex, cmdb);
+}
diff --git a/libibverbs/driver.h b/libibverbs/driver.h
index 88603ce..09974d9 100644
--- a/libibverbs/driver.h
+++ b/libibverbs/driver.h
@@ -230,6 +230,7 @@ struct verbs_device {
atomic_int refcount;
struct list_node entry;
struct verbs_sysfs_dev *sysfs;
+ uint64_t core_support;
};
struct verbs_counters {
--
1.8.3.1
next prev parent reply other threads:[~2020-01-09 14:05 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-01-09 14:04 [PATCH rdma-core 0/7] verbs: Relaxed ordering memory regions Yishai Hadas
2020-01-09 14:04 ` [PATCH rdma-core 1/7] Update kernel headers Yishai Hadas
2020-01-09 14:04 ` [PATCH rdma-core 2/7] verbs: Move free_context from verbs_device_ops to verbs_context_ops Yishai Hadas
2020-01-09 14:04 ` Yishai Hadas [this message]
2020-01-09 14:04 ` [PATCH rdma-core 4/7] verbs: Relaxed ordering memory regions Yishai Hadas
2020-01-09 14:04 ` [PATCH rdma-core 5/7] mlx5: Add optional access flags range to DM Yishai Hadas
2020-01-09 14:04 ` [PATCH rdma-core 6/7] pyverbs: Add relaxed ordering access flag Yishai Hadas
2020-01-09 14:04 ` [PATCH rdma-core 7/7] tests: Add relaxed ordering access test Yishai Hadas
2020-01-22 9:50 ` [PATCH rdma-core 0/7] verbs: Relaxed ordering memory regions Yishai Hadas
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1578578676-752-4-git-send-email-yishaih@mellanox.com \
--to=yishaih@mellanox.com \
--cc=dledford@redhat.com \
--cc=jgg@mellanox.com \
--cc=linux-rdma@vger.kernel.org \
--cc=maorg@mellanox.com \
--cc=michaelgur@mellanox.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).