Skip to content

IB/core: an improved infrastructure for uverbs commands

Yann Droneaud linux-rdma 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v2 Latest4 patches 6 repliesIn review

Patches in v2 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/4IB/core: extended command: an improved infrastructure for uverbs commandsdrivers/infiniband/core/uverbs.h drivers/infiniband/core/uverbs_cmd.c drivers/infiniband/core/uverbs_main.c +3 more+155 -672 replies
2/4IB/core: extended command: move comp_mask to the extended headerdrivers/infiniband/core/uverbs.h drivers/infiniband/core/uverbs_cmd.c drivers/infiniband/core/uverbs_main.c +1 more+19 -110 replies
3/4IB/core: extended command: enforce command sizedrivers/infiniband/core/uverbs_cmd.c+6 -00 replies
4/4IB/core: extended command: add a common extended response headerdrivers/infiniband/core/uverbs.h drivers/infiniband/core/uverbs_cmd.c drivers/infiniband/core/uverbs_main.c +2 more+87 -164 replies

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v2 yet.

Read the v2 thread

Cover letter [PATCH v2 0/4] IB/core: an improved infrastructure for uverbs commands

Hi,

Please find a follow up to my previous proposal for a revised extensible command infrastructure [1].

With this new patchset, I'm trying to improve it further.

The main patch, which was sent previously, was cleaned up.

A second patch tightly related to the previous one make "comp_mask" part of the extended command header, as suggested.

The third patch check extended command size: extend commands should take care to not accept more data than the "comp_mask" is describing.

The latest patch is more experimental: it tries to add a common response header to hold the returned "comp_mask" and size of uverbs (eg. core/) and provider (eg. hw/) responses. I'm not really happy of the shape of this one. It's complicated and makes me uncomfortable. But please have a look at it.

Regards.

Links:

[1]: http://marc.info/?i=1380039392-15480-1-git-send-email-ydroneaud-RlY5vtjFyJ3QT0dZR+AlfA@public.gmane.org

     http://mid.gmane.org/1380039392-15480-1-git-send-email-ydroneaud-RlY5vtjFyJ3QT0dZR+AlfA@public.gmane.org

Yann Droneaud (4):
IB/core: extended command: an improved infrastructure for uverbs
commands
IB/core: extended command: move comp_mask to the extended header IB/core: extended command: enforce command size IB/core: extended command: add a common extended response header

 drivers/infiniband/core/uverbs.h      |  19 +++-
 drivers/infiniband/core/uverbs_cmd.c  |  76 +++++++++-------
 drivers/infiniband/core/uverbs_main.c | 161 ++++++++++++++++++++++++++++------
 drivers/infiniband/hw/mlx4/main.c     |   6 +-
 include/rdma/ib_verbs.h               |  23 +++++
 include/uapi/rdma/ib_user_verbs.h     |  34 ++++---
 6 files changed, 246 insertions(+), 73 deletions(-)
Signature · 6 lines
1.8.3.1

--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html