All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Weiny, Ira" <ira.weiny-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
To: Or Gerlitz <or.gerlitz-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>,
	"Hefty,
	Sean" <sean.hefty-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
Cc: "roland-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org"
	<roland-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>,
	"linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org"
	<linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
	"shlomop-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org"
	<shlomop-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>,
	Tzahi Oved <tzahio-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
Subject: RE: [PATCH V4 for-next 1/5] IB/core: Add RSS and TSS QP groups
Date: Mon, 15 Apr 2013 00:58:22 +0000	[thread overview]
Message-ID: <2807E5FD2F6FDA4886F6618EAC48510E0158709F@CRSMSX102.amr.corp.intel.com> (raw)
In-Reply-To: <CAJZOPZKt-R=Y_gxHidCi78u4xAB52tiT2MkQ5-5Yo7pR8po3iA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>

> -----Original Message-----
> From: linux-rdma-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org [mailto:linux-rdma-
> owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org] On Behalf Of Or Gerlitz
> Sent: Thursday, April 11, 2013 4:09 PM
> To: Hefty, Sean
> Cc: roland-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org; linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org;
> shlomop-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org; Tzahi Oved
> Subject: Re: [PATCH V4 for-next 1/5] IB/core: Add RSS and TSS QP groups
> 
> On Thu, Apr 11, 2013 at 5:45 PM, Hefty, Sean <sean.hefty-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org> wrote:
> [...]
> >> but lets get there after hopefully agreeing what is RSS QP group.
> 
> > So far, this is what I think it is:
> > - a collection of related receive queues
> > - each queue is configured somewhat separately - i.e. sized, CQ, sge size,
> etc.
> > - receives are posted to the queues separately
> > - the queues are allocated together
> >
> > This is where it gets confusing.  They're allocated together, but
> > through separate API calls.
> > I'm not sure if they share states or not.  Can all of them but one go
> > into the error state > and still receive data?  Do packets get routed to
> whichever queue actually works, or do > packets sometimes get dropped,
> but sometimes don't, depending on some local rules > which have been
> programmed into the HCA?  Can the queues really be destroyed
> independently?
> 
> We only require (== implemented that) for the verbs level to mandate for
> the RSS parent not to be destroyed before ANY of the RSS childs is destroyed
> and be placed to action only after ALL RSS childs are created. The queues
> (RSS childs can be destroyed independently after the parent is destroyed,
> yes.
> 
> RSS child QPs are plain UD or RAW Packet QPs that only have consecutive
> QPNs which is common requirement of HW for configuring the RSS parent
> which in networking is called the RSS indirection or dispatching QP. You can
> send and receive on them.

How do you ensure that the QPN's are consecutive?

> 
> If an RSS child goes to the error state it will not receive data.

If you transition it back to RTS would it start working again?

Could you remove it and add a new one?  (I guess not because the new QPN would likely not be consecutive.)

> 
> Packets are routed to RSS childs only per the hash function output, not per
> the state of that child.

So if the QP chosen by the hash is in error state the packets get lost?  Above you said they would not receive data.

Ira

> 
> This design doesn't allow for an app to do DoS attack on the HW either if they
> try that out or just have a bug, but does require them to think out their code
> (design/review/test) - fair enough. RSS exists in almost any Ethernet NIC you
> take from the shelf, and works in the manner I explained here, e.g if one of
> the Ethernet RSS child queues isn't functional packets hashed to it will not be
> received
> 
> > Is it even necessary to treat the receive queues as being independent?
> What happens > if they're allocated, destroyed, and modified as a single
> entity?
> [...]
> 
> can you elaborate/explain the question a little more?
> 
> Or.
> --
> 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
--
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:[~2013-04-15  0:58 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-03-24 12:43 [PATCH V4 for-next 0/5] IB/IPoIB: Add multi-queue TSS and RSS support Or Gerlitz
     [not found] ` <1364129012-12198-1-git-send-email-ogerlitz-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
2013-03-24 12:43   ` [PATCH V4 for-next 1/5] IB/core: Add RSS and TSS QP groups Or Gerlitz
     [not found]     ` <1364129012-12198-2-git-send-email-ogerlitz-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
2013-04-09 21:43       ` Or Gerlitz
     [not found]         ` <CAJZOPZKBns8kPH+ayzd1k--6K0NXFx=1Pez_bfSg9rQz-fvmHg-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2013-04-11  4:16           ` Or Gerlitz
2013-04-11 14:45           ` Hefty, Sean
     [not found]             ` <1828884A29C6694DAF28B7E6B8A823736F36DCF9-P5GAC/sN6hmkrb+BlOpmy7fspsVTdybXVpNB7YpNyf8@public.gmane.org>
2013-04-11 23:08               ` Or Gerlitz
     [not found]                 ` <CAJZOPZKt-R=Y_gxHidCi78u4xAB52tiT2MkQ5-5Yo7pR8po3iA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2013-04-15  0:58                   ` Weiny, Ira [this message]
2013-03-24 12:43   ` [PATCH V4 for-next 2/5] IB/mlx4: Add support for " Or Gerlitz
2013-03-24 12:43   ` [PATCH V4 for-next 3/5] IB/ipoib: Move to multi-queue device Or Gerlitz
2013-03-24 12:43   ` [PATCH V4 for-next 4/5] IB/ipoib: Add RSS and TSS support for datagram mode Or Gerlitz
2013-03-24 12:43   ` [PATCH V4 for-next 5/5] IB/ipoib: Support changing the number of RX/TX rings with ethtool 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=2807E5FD2F6FDA4886F6618EAC48510E0158709F@CRSMSX102.amr.corp.intel.com \
    --to=ira.weiny-ral2jqcrhueavxtiumwx3w@public.gmane.org \
    --cc=linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=or.gerlitz-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \
    --cc=roland-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org \
    --cc=sean.hefty-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org \
    --cc=shlomop-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org \
    --cc=tzahio-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.