linux-rdma.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
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 --]

      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).