All of lore.kernel.org
 help / color / mirror / Atom feed
From: Dennis Dalessandro <dennis.dalessandro-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
To: Moni Shoua <monis-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
Cc: Doug Ledford <dledford-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>,
	linux-rdma <linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>
Subject: Re: [PATCH 0/8] IB/rdmavt: Add functions for cq, qp, post send and recv
Date: Tue, 19 Jan 2016 13:53:00 -0500	[thread overview]
Message-ID: <20160119185300.GB29112@phlsvsds.ph.intel.com> (raw)
In-Reply-To: <CAG9sBKMNm1bMtjoUsGjj+FVL3ZmBw--tK-6iXYMW+0N5j2VE2A-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>

On Tue, Jan 19, 2016 at 06:44:11PM +0200, Moni Shoua wrote:
>On Sat, Jan 9, 2016 at 4:59 PM, Dennis Dalessandro
><dennis.dalessandro-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org> wrote:
>> This series continues adding support for verbs into rdmavt. This brings in
>> completion queue, queue pair, and post send/recv functionality.
>>
>> It also introduces event tracing and moves some more of the hfi1/qib
>> registration functionality.
>>
>> This patch set applies on top of the "Start to rely on rdmavt for qp support"
>> series previously submitted.
>>
>> ---
>>
>> Dennis Dalessandro (8):
>>       IB/rdmavt: Add completion queue functions
>>       IB/rdmavt: Add post send to rdmavt
>>       IB/rdmavt: Add support for tracing events
>>       IB/rdmavt: Add modify qp
>>       IB/rdmavt: Add destroy qp verb
>>       IB/rdmavt: Add post receive to rdmavt
>>       IB/rdmavt: Add multicast functions
>>       IB/rdmavt: Add misc dev register functionality
>
>I didn't see anywhere an API for RVT that handles received packets
>Did I miss it?
>If not, what is expected from the lower driver to do when a packet is received?

Similar to my response for the send side, this is the first step. When a 
packet is received it is handled by the driver. As to the details of just 
what moves into rdmavt and what stays in the driver is a matter of debate, 
and a decision which will have to take into account performance aspects. A 
perfect design from a software engineering standpoint may not always be 
possible, these are high performance devices and we can't sacrifice that 
performance.

At any rate, I'd like to focus on the send side next with the recv to 
follow.

-Denny
--
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

      parent reply	other threads:[~2016-01-19 18:53 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-01-09 14:59 [PATCH 0/8] IB/rdmavt: Add functions for cq, qp, post send and recv Dennis Dalessandro
     [not found] ` <20160109145731.8585.48926.stgit-9QXIwq+3FY+1XWohqUldA0EOCMrvLtNR@public.gmane.org>
2016-01-09 15:00   ` [PATCH 1/8] IB/rdmavt: Add completion queue functions Dennis Dalessandro
2016-01-09 15:00   ` [PATCH 2/8] IB/rdmavt: Add post send to rdmavt Dennis Dalessandro
     [not found]     ` <20160109150007.8585.76575.stgit-9QXIwq+3FY+1XWohqUldA0EOCMrvLtNR@public.gmane.org>
2016-01-19 16:23       ` Moni Shoua
     [not found]         ` <CAG9sBKOpDfxOaPU+dJ+M42JrbCVEoNAORi8jk1KJLay-KDhSSA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2016-01-19 18:46           ` Dennis Dalessandro
2016-01-09 15:00   ` [PATCH 3/8] IB/rdmavt: Add support for tracing events Dennis Dalessandro
2016-01-09 15:00   ` [PATCH 4/8] IB/rdmavt: Add modify qp Dennis Dalessandro
2016-01-09 15:00   ` [PATCH 5/8] IB/rdmavt: Add destroy qp verb Dennis Dalessandro
2016-01-09 15:00   ` [PATCH 6/8] IB/rdmavt: Add post receive to rdmavt Dennis Dalessandro
2016-01-09 15:00   ` [PATCH 7/8] IB/rdmavt: Add multicast functions Dennis Dalessandro
2016-01-09 15:00   ` [PATCH 8/8] IB/rdmavt: Add misc dev register functionality Dennis Dalessandro
2016-01-19 16:44   ` [PATCH 0/8] IB/rdmavt: Add functions for cq, qp, post send and recv Moni Shoua
     [not found]     ` <CAG9sBKMNm1bMtjoUsGjj+FVL3ZmBw--tK-6iXYMW+0N5j2VE2A-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2016-01-19 18:53       ` Dennis Dalessandro [this message]

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=20160119185300.GB29112@phlsvsds.ph.intel.com \
    --to=dennis.dalessandro-ral2jqcrhueavxtiumwx3w@public.gmane.org \
    --cc=dledford-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org \
    --cc=linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=monis-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 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.