From: Doug Ledford <dledford-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
To: Matan Barak <matanb-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
Cc: linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
Or Gerlitz <ogerlitz-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>,
Jason Gunthorpe
<jgunthorpe-ePGOBjL8dl3ta4EC/59zMFaTQe2KTcn/@public.gmane.org>,
Eran Ben Elisha <eranbe-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>,
Somnath Kotur
<Somnath.Kotur-idTK6quXuVSLFuii7jzJGg@public.gmane.org>
Subject: Re: [PATCH for-next v2 00/10] Add RoCE GID cache usage in verbs/cma
Date: Wed, 21 Oct 2015 23:52:25 -0400 [thread overview]
Message-ID: <56285D79.8080405@redhat.com> (raw)
In-Reply-To: <1444923534-10882-1-git-send-email-matanb-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
[-- Attachment #1: Type: text/plain, Size: 2012 bytes --]
On 10/15/2015 11:38 AM, Matan Barak wrote:
> Hi Doug,
>
> This purpose of this series is to add usage of the GID cache to
> the CMA and IB stack. Instead of passing Ethernet L2 attributes
> via QP attributes, we could just use the GID cache that's already
> points to a ndev and thus to all required L2 attributes.
>
> The first five patches query the GID table to find only GIDs which
> are related to the bounded net-device of the specific used port. This
> extra information is carried via ib_sa_path_rec (extending it to
> include the namespace and ifindex). Querying the ndev and port
> is achieved by adding a gid_attr argument to ib_query_gid, add a
> ndev argument to ib_find_cached_gid and add a new
> ib_find_cached_gid_by_port. This usage of the GID table replaces
> the usage of QP attributes.
>
> The sixth patch adds an ib_cache_gid_find_by_filter function.
> This allows the user to query the cache by a specific filter.
> ib_cache_gid_find_by_filter is used by the seventh patch. Instead
> of storing the smac and vid on the AH, we could just resolve
> them from the net-device which the sgid index is assigned to.
> This change means that instead of resolving all L2 attributes,
> the server, we get the dgid and vlan and searches the GID cache
> in order to find a matching GID index.
>
> The last three patches removed unused fields and attributes.
>
> Matan
>
> Changes from V1:
> - Rebased patches against Doug's latest k.o/for-4.4 tree.
Most of this code seemed like pretty straight forward prepatory word for
the follow on patch series. However, I did note that this patch set
extends the per-element locking of the GID tables. I've taken this
patch set in spite of that fact. Please prioritize changing that
locking to something more reasonable. I would prefer if those changes
landed prior to the 4.4 merge window closing.
--
Doug Ledford <dledford-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
GPG KeyID: 0E572FDD
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 884 bytes --]
prev parent reply other threads:[~2015-10-22 3:52 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-10-15 15:38 [PATCH for-next v2 00/10] Add RoCE GID cache usage in verbs/cma Matan Barak
[not found] ` <1444923534-10882-1-git-send-email-matanb-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
2015-10-15 15:38 ` [PATCH for-next v2 01/10] IB/core: Add netdev and gid attributes paramteres to cache Matan Barak
[not found] ` <1444923534-10882-2-git-send-email-matanb-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
2015-10-18 15:57 ` Devesh Sharma
2015-10-15 15:38 ` [PATCH for-next v2 02/10] IB/core: Expose and rename ib_find_cached_gid_by_port cache API Matan Barak
[not found] ` <1444923534-10882-3-git-send-email-matanb-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
2015-10-18 15:51 ` Devesh Sharma
2015-10-15 15:38 ` [PATCH for-next v2 03/10] IB/core: Add netdev to path record Matan Barak
[not found] ` <1444923534-10882-4-git-send-email-matanb-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
2015-10-18 16:03 ` Devesh Sharma
2015-10-15 15:38 ` [PATCH for-next v2 04/10] IB/cm: cm_init_av_by_path should find a GID by its netdevice Matan Barak
[not found] ` <1444923534-10882-5-git-send-email-matanb-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
2015-10-18 16:05 ` Devesh Sharma
2015-10-15 15:38 ` [PATCH for-next v2 05/10] IB/cma: cma_validate_port should verify the port and netdevice Matan Barak
[not found] ` <1444923534-10882-6-git-send-email-matanb-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
2015-10-18 16:10 ` Devesh Sharma
2015-10-15 15:38 ` [PATCH for-next v2 06/10] IB/cache: Add ib_find_gid_by_filter cache API Matan Barak
[not found] ` <1444923534-10882-7-git-send-email-matanb-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
2015-10-18 16:57 ` Devesh Sharma
2015-10-15 15:38 ` [PATCH for-next v2 07/10] IB/core: Use GID table in AH creation and dmac resolution Matan Barak
2015-10-15 15:38 ` [PATCH for-next v2 08/10] IB/cm: Remove the usage of smac and vid of qp_attr and cm_av Matan Barak
[not found] ` <1444923534-10882-9-git-send-email-matanb-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
2015-10-18 17:22 ` Devesh Sharma
2015-10-15 15:38 ` [PATCH for-next v2 09/10] IB/core: Remove smac and vlan id from qp_attr and ah_attr Matan Barak
[not found] ` <1444923534-10882-10-git-send-email-matanb-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
2015-10-18 17:24 ` Devesh Sharma
2015-10-15 15:38 ` [PATCH for-next v2 10/10] IB/core: Remove smac and vlan id from path record Matan Barak
[not found] ` <1444923534-10882-11-git-send-email-matanb-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
2015-10-18 17:26 ` Devesh Sharma
2015-10-22 3:52 ` Doug Ledford [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=56285D79.8080405@redhat.com \
--to=dledford-h+wxahxf7alqt0dzr+alfa@public.gmane.org \
--cc=Somnath.Kotur-idTK6quXuVSLFuii7jzJGg@public.gmane.org \
--cc=eranbe-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org \
--cc=jgunthorpe-ePGOBjL8dl3ta4EC/59zMFaTQe2KTcn/@public.gmane.org \
--cc=linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=matanb-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org \
--cc=ogerlitz-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).