From: jgg@ziepe.ca (Jason Gunthorpe)
Subject: [v3,1/9] RDMA/core: Add implicit per-device completion queue pools
Date: Mon, 11 Dec 2017 16:50:44 -0700 [thread overview]
Message-ID: <20171211235044.GA32331@ziepe.ca> (raw)
In-Reply-To: <20171108095742.25365-2-sagi@grimberg.me>
On Wed, Nov 08, 2017@11:57:34AM +0200, Sagi Grimberg wrote:
> Allow a ULP to ask the core to implicitly assign a completion
> queue to a queue-pair based on a least-used search on a per-device
> cq pools. The device CQ pools grow in a lazy fashion with every
> QP creation.
>
> In addition, expose an affinity hint for a queue pair creation.
> If passed, the core will attempt to attach a CQ with a completion
> vector that is directed to the cpu core as the affinity hint
> provided.
>
> Signed-off-by: Sagi Grimberg <sagi at grimberg.me>
Sagi, Bart,
Did we reach a conclusion on this? Is v3 the series to take, and
should it all go through the RDMA tree? It looks like there are some
missing acks for that??
I think there was also an unapplied comment from bart in the
patchworks notes...
Could you please add some commit messages when you resend it? Not sure
I should be accepting such large commits with empty messages???
Thanks,
Jason
WARNING: multiple messages have this Message-ID (diff)
From: Jason Gunthorpe <jgg-uk2M96/98Pc@public.gmane.org>
To: Sagi Grimberg <sagi-NQWnxTmZq1alnMjI0IkVqw@public.gmane.org>,
Bart Van Assche <Bart.VanAssche-Sjgp3cTcYWE@public.gmane.org>
Cc: linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
linux-nvme-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org,
Christoph Hellwig <hch-jcswGhMUV9g@public.gmane.org>,
Max Gurtuvoy <maxg-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
Subject: Re: [v3,1/9] RDMA/core: Add implicit per-device completion queue pools
Date: Mon, 11 Dec 2017 16:50:44 -0700 [thread overview]
Message-ID: <20171211235044.GA32331@ziepe.ca> (raw)
In-Reply-To: <20171108095742.25365-2-sagi-NQWnxTmZq1alnMjI0IkVqw@public.gmane.org>
On Wed, Nov 08, 2017 at 11:57:34AM +0200, Sagi Grimberg wrote:
> Allow a ULP to ask the core to implicitly assign a completion
> queue to a queue-pair based on a least-used search on a per-device
> cq pools. The device CQ pools grow in a lazy fashion with every
> QP creation.
>
> In addition, expose an affinity hint for a queue pair creation.
> If passed, the core will attempt to attach a CQ with a completion
> vector that is directed to the cpu core as the affinity hint
> provided.
>
> Signed-off-by: Sagi Grimberg <sagi-NQWnxTmZq1alnMjI0IkVqw@public.gmane.org>
Sagi, Bart,
Did we reach a conclusion on this? Is v3 the series to take, and
should it all go through the RDMA tree? It looks like there are some
missing acks for that??
I think there was also an unapplied comment from bart in the
patchworks notes...
Could you please add some commit messages when you resend it? Not sure
I should be accepting such large commits with empty messages???
Thanks,
Jason
--
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
next prev parent reply other threads:[~2017-12-11 23:50 UTC|newest]
Thread overview: 92+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-11-08 9:57 [PATCH v3 0/9] Introduce per-device completion queue pools Sagi Grimberg
2017-11-08 9:57 ` Sagi Grimberg
2017-11-08 9:57 ` [PATCH v3 1/9] RDMA/core: Add implicit " Sagi Grimberg
2017-11-08 9:57 ` Sagi Grimberg
2017-11-09 10:45 ` Max Gurtovoy
2017-11-09 10:45 ` Max Gurtovoy
2017-11-09 17:31 ` Sagi Grimberg
2017-11-09 17:31 ` Sagi Grimberg
2017-11-09 17:33 ` Bart Van Assche
2017-11-09 17:33 ` Bart Van Assche
2017-11-13 20:28 ` Sagi Grimberg
2017-11-13 20:28 ` Sagi Grimberg
2017-11-14 16:28 ` Bart Van Assche
2017-11-14 16:28 ` Bart Van Assche
2017-11-20 12:31 ` Sagi Grimberg
2017-11-20 12:31 ` Sagi Grimberg
2017-12-11 23:50 ` Jason Gunthorpe [this message]
2017-12-11 23:50 ` [v3,1/9] " Jason Gunthorpe
2018-01-03 17:47 ` Jason Gunthorpe
2018-01-03 17:47 ` Jason Gunthorpe
2017-11-08 9:57 ` [PATCH v3 2/9] IB/isert: use implicit CQ allocation Sagi Grimberg
2017-11-08 9:57 ` Sagi Grimberg
2017-11-08 10:27 ` Nicholas A. Bellinger
2017-11-08 10:27 ` Nicholas A. Bellinger
2017-11-08 10:27 ` Nicholas A. Bellinger
2017-11-14 9:14 ` Max Gurtovoy
2017-11-14 9:14 ` Max Gurtovoy
2017-11-08 9:57 ` [PATCH v3 3/9] IB/iser: " Sagi Grimberg
2017-11-08 9:57 ` Sagi Grimberg
2017-11-08 10:25 ` Nicholas A. Bellinger
2017-11-08 10:25 ` Nicholas A. Bellinger
2017-11-08 10:25 ` Nicholas A. Bellinger
2017-11-08 9:57 ` [PATCH v3 4/9] IB/srpt: " Sagi Grimberg
2017-11-08 9:57 ` Sagi Grimberg
2017-11-08 9:57 ` [PATCH v3 5/9] svcrdma: Use RDMA core " Sagi Grimberg
2017-11-08 9:57 ` Sagi Grimberg
2017-11-08 9:57 ` [PATCH v3 6/9] nvme-rdma: use " Sagi Grimberg
2017-11-08 9:57 ` Sagi Grimberg
2017-11-08 9:57 ` [PATCH v3 7/9] nvmet-rdma: " Sagi Grimberg
2017-11-08 9:57 ` Sagi Grimberg
2017-11-08 9:57 ` [PATCH v3 8/9] nvmet: allow assignment of a cpulist for each nvmet port Sagi Grimberg
2017-11-08 9:57 ` Sagi Grimberg
2017-11-08 9:57 ` [PATCH v3 9/9] nvmet-rdma: assign cq completion vector based on the port allowed cpus Sagi Grimberg
2017-11-08 9:57 ` Sagi Grimberg
2017-11-08 16:42 ` [PATCH v3 0/9] Introduce per-device completion queue pools Chuck Lever
2017-11-08 16:42 ` Chuck Lever
2017-11-09 17:06 ` Sagi Grimberg
2017-11-09 17:06 ` Sagi Grimberg
2017-11-10 19:27 ` Chuck Lever
2017-11-10 19:27 ` Chuck Lever
2017-11-13 20:47 ` Sagi Grimberg
2017-11-13 20:47 ` Sagi Grimberg
2017-11-13 22:15 ` Chuck Lever
2017-11-13 22:15 ` Chuck Lever
2017-11-20 12:08 ` Sagi Grimberg
2017-11-20 12:08 ` Sagi Grimberg
2017-11-20 15:54 ` Chuck Lever
2017-11-20 15:54 ` Chuck Lever
2017-11-09 16:42 ` Bart Van Assche
2017-11-09 16:42 ` Bart Van Assche
2017-11-09 17:22 ` Sagi Grimberg
2017-11-09 17:22 ` Sagi Grimberg
2017-11-09 17:31 ` Bart Van Assche
2017-11-09 17:31 ` Bart Van Assche
2017-11-13 20:31 ` Sagi Grimberg
2017-11-13 20:31 ` Sagi Grimberg
2017-11-13 20:34 ` Jason Gunthorpe
2017-11-13 20:34 ` Jason Gunthorpe
2017-11-13 20:48 ` Sagi Grimberg
2017-11-13 20:48 ` Sagi Grimberg
2017-11-14 2:48 ` Jason Gunthorpe
2017-11-14 2:48 ` Jason Gunthorpe
2017-11-20 12:10 ` Sagi Grimberg
2017-11-20 12:10 ` Sagi Grimberg
2017-11-20 19:24 ` Jason Gunthorpe
2017-11-20 19:24 ` Jason Gunthorpe
2017-11-20 21:29 ` Bart Van Assche
2017-11-20 21:29 ` Bart Van Assche
2017-11-14 16:21 ` Bart Van Assche
2017-11-14 16:21 ` Bart Van Assche
2017-11-20 12:26 ` Sagi Grimberg
2017-11-20 12:26 ` Sagi Grimberg
2017-11-14 10:06 ` Max Gurtovoy
2017-11-14 10:06 ` Max Gurtovoy
2017-11-20 12:20 ` Sagi Grimberg
2017-11-20 12:20 ` Sagi Grimberg
2017-11-09 18:52 ` Leon Romanovsky
2017-11-09 18:52 ` Leon Romanovsky
2017-11-13 20:32 ` Sagi Grimberg
2017-11-13 20:32 ` Sagi Grimberg
2017-11-13 22:11 ` Doug Ledford
2017-11-13 22:11 ` 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=20171211235044.GA32331@ziepe.ca \
--to=jgg@ziepe.ca \
/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.