linux-rdma.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Steve Wise <swise-7bPotxP6k4+P2YhJcF5u+p6NamaJ0bNTAL8bYrjMMd8@public.gmane.org>
To: linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
Cc: bart.vanassche-XdAiOPVOjttBDgjK7y7TUQ@public.gmane.org,
	ogerlitz-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org,
	sagig-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org,
	roid-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org
Subject: [PATCH v4 0/4] new ib_drain_qp() API
Date: Wed, 17 Feb 2016 12:35:36 -0800	[thread overview]
Message-ID: <cover.1455741336.git.swise@chelsio.com> (raw)

This series creates new helper API functions for draining a queue pair.
It is a rework of an original patch created by Christoph Hellwig as
part of the CQ API rework and was dropped to be resubmitted by me with
iw_cxgb4 support.

Original thread: http://www.spinics.net/lists/linux-rdma/msg30296.html


Changes since v3:

- WARN_ONCE() if the CQ is using IB_POLL_DIRECT context.

- added iser patch to use ib_drain_sq().

- fixed up function comments to specify caller requirements

- checkpatched


Changes since v2:

- created 3 drain API functions: ib_drain_rq(), ib_drain_sq(), and
ib_drain_qp()

- add provider-specific drain function pointers for the sq and rq

- refactored the code a bit

- support for IB_DIRECT_POLL CQs


Changes since v1:

- added comments to the ib_drain_qp() function header specifying the
consumer requirements

- in __ib_drain_qp(), if the ib_post_send() fails, still wait for the
recv wr to drain since we already posted it.

- CC the SRP maintainer, bart.vanassche-XdAiOPVOjtvowKkBSvOlow@public.gmane.org

---

Steve Wise (4):
  IB: new common API for draining queues
  iw_cxgb4: add queue drain functions
  IB/srp: Use ib_drain_rq()
  IB/iser: Use ib_drain_sq()

 drivers/infiniband/core/verbs.c              | 164 +++++++++++++++++++++++++++
 drivers/infiniband/hw/cxgb4/cq.c             |   9 +-
 drivers/infiniband/hw/cxgb4/iw_cxgb4.h       |   4 +
 drivers/infiniband/hw/cxgb4/provider.c       |   2 +
 drivers/infiniband/hw/cxgb4/qp.c             |  16 +++
 drivers/infiniband/ulp/iser/iscsi_iser.h     |   7 --
 drivers/infiniband/ulp/iser/iser_initiator.c |   7 --
 drivers/infiniband/ulp/iser/iser_verbs.c     |  15 +--
 drivers/infiniband/ulp/srp/ib_srp.c          |  40 +------
 include/rdma/ib_verbs.h                      |   5 +
 10 files changed, 205 insertions(+), 64 deletions(-)

-- 
2.7.0

--
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:[~2016-02-17 20:35 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-02-17 20:35 Steve Wise [this message]
     [not found] ` <cover.1455741336.git.swise-ut6Up61K2wZBDgjK7y7TUQ@public.gmane.org>
2016-02-17 16:15   ` [PATCH v4 1/4] IB: new common API for draining queues Steve Wise
     [not found]     ` <5b4799151723516ec440191b3f0ae98a46b356f0.1455741336.git.swise-ut6Up61K2wZBDgjK7y7TUQ@public.gmane.org>
2016-02-17 23:35       ` Bart Van Assche
     [not found]         ` <56C503D0.7060402-XdAiOPVOjttBDgjK7y7TUQ@public.gmane.org>
2016-02-17 23:39           ` Steve Wise
2016-02-21 15:01       ` Sagi Grimberg
     [not found]         ` <56C9D14B.9010307-LDSdmyG8hGV8YrgS2mwiifqBs+8SCbDb@public.gmane.org>
2016-02-22  5:39           ` Devesh Sharma
2016-02-17 16:15   ` [PATCH v4 2/4] iw_cxgb4: add queue drain functions Steve Wise
     [not found]     ` <162a5429342dfd5e16881f4360c0ef858fd39cbd.1455741336.git.swise-ut6Up61K2wZBDgjK7y7TUQ@public.gmane.org>
2016-02-21 15:01       ` Sagi Grimberg
2016-03-09 12:53       ` Leon Romanovsky
     [not found]         ` <20160309125349.GA1977-2ukJVAZIZ/Y@public.gmane.org>
2016-03-09 15:23           ` Steve Wise
2016-03-09 15:28             ` Sagi Grimberg
     [not found]               ` <56E0412F.4000602-LDSdmyG8hGV8YrgS2mwiifqBs+8SCbDb@public.gmane.org>
2016-03-10  5:05                 ` Leon Romanovsky
     [not found]                   ` <20160310050558.GD1977-2ukJVAZIZ/Y@public.gmane.org>
2016-03-24 10:46                     ` Leon Romanovsky
     [not found]                       ` <20160324104659.GC11949-2ukJVAZIZ/Y@public.gmane.org>
2016-03-24 14:36                         ` Faisal Latif
2016-02-17 16:15   ` [PATCH v4 3/4] IB/srp: Use ib_drain_rq() Steve Wise
     [not found]     ` <8a0a024ce256ac8c9be5e46a05f158d6e74a3cb9.1455741336.git.swise-ut6Up61K2wZBDgjK7y7TUQ@public.gmane.org>
2016-02-21 15:02       ` Sagi Grimberg
2016-02-17 16:17   ` [PATCH v4 4/4] IB/iser: Use ib_drain_sq() Steve Wise
     [not found]     ` <269ab4c6de9c30bb6beba249302d5f1d1db9b924.1455741337.git.swise-ut6Up61K2wZBDgjK7y7TUQ@public.gmane.org>
2016-02-21 15:02       ` Sagi Grimberg
2016-03-03 10:10       ` Sagi Grimberg
     [not found]         ` <56D80D8F.7070700-LDSdmyG8hGV8YrgS2mwiifqBs+8SCbDb@public.gmane.org>
2016-03-03 15:17           ` Steve Wise
2016-02-17 21:09   ` [PATCH v4 0/4] new ib_drain_qp() API Doug Ledford

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.1455741336.git.swise@chelsio.com \
    --to=swise-7bpotxp6k4+p2yhjcf5u+p6namaj0bntal8byrjmmd8@public.gmane.org \
    --cc=bart.vanassche-XdAiOPVOjttBDgjK7y7TUQ@public.gmane.org \
    --cc=linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=ogerlitz-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org \
    --cc=roid-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org \
    --cc=sagig-VPRAkNaXOzVWk0Htik3J/w@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 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).