All of lore.kernel.org
 help / color / mirror / Atom feed
From: Yann Droneaud <ydroneaud-RlY5vtjFyJ3QT0dZR+AlfA@public.gmane.org>
To: Roland Dreier <roland-BHEL68pLQRGGvPXPguhicg@public.gmane.org>,
	Igor Ivanov <Igor.Ivanov-wN0M4riKYwLQT0dZR+AlfA@public.gmane.org>,
	Hadar Hen Zion <hadarh-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>,
	Or Gerlitz <ogerlitz-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>,
	Matan Barak <matanb-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
Cc: linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	Yann Droneaud <ydroneaud-RlY5vtjFyJ3QT0dZR+AlfA@public.gmane.org>
Subject: [PATCH v2 0/4] IB/core: an improved infrastructure for uverbs commands
Date: Mon,  7 Oct 2013 22:49:13 +0200	[thread overview]
Message-ID: <cover.1381177342.git.ydroneaud@opteya.com> (raw)

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(-)

-- 
1.8.3.1

--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

             reply	other threads:[~2013-10-07 20:49 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-10-07 20:49 Yann Droneaud [this message]
     [not found] ` <cover.1381177342.git.ydroneaud-RlY5vtjFyJ3QT0dZR+AlfA@public.gmane.org>
2013-10-07 20:49   ` [PATCH v2 1/4] IB/core: extended command: an improved infrastructure for uverbs commands Yann Droneaud
     [not found]     ` <70486466a70aae3e3facf5a12c1d0bb960a9a462.1381177342.git.ydroneaud-RlY5vtjFyJ3QT0dZR+AlfA@public.gmane.org>
2013-10-09 11:38       ` Or Gerlitz
     [not found]         ` <5255402F.4000806-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
2013-10-09 12:42           ` Yann Droneaud
2013-10-07 20:49   ` [PATCH v2 2/4] IB/core: extended command: move comp_mask to the extended header Yann Droneaud
2013-10-07 20:49   ` [PATCH v2 3/4] IB/core: extended command: enforce command size Yann Droneaud
2013-10-07 20:49   ` [PATCH v2 4/4] IB/core: extended command: add a common extended response header Yann Droneaud
     [not found]     ` <9e111d8215c1fd86b75db17f2c890f0f28e0d076.1381177342.git.ydroneaud-RlY5vtjFyJ3QT0dZR+AlfA@public.gmane.org>
2013-10-08 11:58       ` Matan Barak
     [not found]         ` <5253F37C.90508-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
2013-10-09 10:30           ` Or Gerlitz
     [not found]             ` <52553050.8050501-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
2013-10-09 11:29               ` Yann Droneaud
     [not found]                 ` <0d695256ac989e7b1bcccd3a2bfafcbf-zgzEX58YAwA@public.gmane.org>
2013-10-09 11:33                   ` Or Gerlitz

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=cover.1381177342.git.ydroneaud@opteya.com \
    --to=ydroneaud-rly5vtjfyj3qt0dzr+alfa@public.gmane.org \
    --cc=Igor.Ivanov-wN0M4riKYwLQT0dZR+AlfA@public.gmane.org \
    --cc=hadarh-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org \
    --cc=linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=matanb-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org \
    --cc=ogerlitz-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org \
    --cc=roland-BHEL68pLQRGGvPXPguhicg@public.gmane.org \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.