All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jason Gunthorpe <jgunthorpe-ePGOBjL8dl3ta4EC/59zMFaTQe2KTcn/@public.gmane.org>
To: Artemy Kovalyov <artemyko-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
Cc: dledford-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org,
	linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	linux-mm-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	leon-Dj0/KMMU01E@public.gmane.org,
	haggaie-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org,
	sagig-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org
Subject: Re: [RFC 0/7] Peer-direct memory
Date: Thu, 11 Feb 2016 12:18:38 -0700	[thread overview]
Message-ID: <20160211191838.GA23675@obsidianresearch.com> (raw)
In-Reply-To: <1455207177-11949-1-git-send-email-artemyko-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>

On Thu, Feb 11, 2016 at 06:12:50PM +0200, Artemy Kovalyov wrote:

> Recently introduced ZONE_DEVICE patch [1] allows to register devices as
> providers of "device memory" regions, making RDMA operation with them
> transparantly available. This patch is intended for scenarios which not fit
> into ZONE_DEVICE infrastrcture, but device still want to exposure it's
> IO regions to RDMA access.

Most of this stuff (eg the whole peer_memory_client thing) has no
buisness being part of the RDMA stack. We can't ask other drivers to
call IB functions just because their devices have ZONE_DIRECT memory.

Resubmit those parts under the mm subsystem, or another more
appropriate place.

This is the same comment as last time this was submitted.

If you want to make some incremental progress then implement the
existing ZONE_DEVICE API for the IB core and add the invalidate stuff
later, once you've negotiated a common API for that with linux-mm.

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

  parent reply	other threads:[~2016-02-11 19:18 UTC|newest]

Thread overview: 33+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-02-11 16:12 [RFC 0/7] Peer-direct memory Artemy Kovalyov
     [not found] ` <1455207177-11949-1-git-send-email-artemyko-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
2016-02-11 16:12   ` [RFC 1/7] IB/core: Introduce peer client interface Artemy Kovalyov
2016-02-11 16:12   ` [RFC 2/7] IB/core: Get/put peer memory client Artemy Kovalyov
2016-02-11 16:12   ` [RFC 3/7] IB/core: Umem tunneling peer memory APIs Artemy Kovalyov
2016-02-11 16:12   ` [RFC 4/7] IB/core: Infrastructure to manage peer core context Artemy Kovalyov
2016-02-11 16:12   ` [RFC 5/7] IB/core: Invalidation support for peer memory Artemy Kovalyov
2016-02-11 16:12   ` [RFC 6/7] IB/core: Peer memory client for IO memory Artemy Kovalyov
2016-02-11 16:12   ` [RFC 7/7] IB/mlx5: Invalidation support for MR over peer memory Artemy Kovalyov
2016-02-11 19:18   ` Jason Gunthorpe [this message]
     [not found]     ` <20160211191838.GA23675-ePGOBjL8dl3ta4EC/59zMFaTQe2KTcn/@public.gmane.org>
2016-02-12 20:13       ` [RFC 0/7] Peer-direct memory Christoph Hellwig
     [not found]         ` <20160212201328.GA14122-wEGCiKHe2LqWVfeAwA7xHQ@public.gmane.org>
2016-02-12 20:36           ` Jason Gunthorpe
     [not found]             ` <20160212203649.GA10540-ePGOBjL8dl3ta4EC/59zMFaTQe2KTcn/@public.gmane.org>
2016-02-14 15:25               ` Sagi Grimberg
     [not found]                 ` <56C09C7E.4060808-LDSdmyG8hGV8YrgS2mwiifqBs+8SCbDb@public.gmane.org>
2016-02-18 14:44                   ` Stephen Bates
2016-02-21  9:06                     ` Haggai Eran
     [not found]                       ` <56C97E13.9090101-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
2016-02-24 23:45                         ` Stephen Bates
2016-02-24 23:45                           ` Stephen Bates
2016-02-25 11:27                           ` Haggai Eran
2016-02-14 14:09           ` Haggai Eran
2016-02-14 14:05       ` Haggai Eran
2016-02-14 14:27     ` Haggai Eran
2016-02-16 18:22       ` Jason Gunthorpe
2016-02-17  4:03         ` davide rossetti
     [not found]           ` <CAPSaadxbFCOcKV=c3yX7eGw9Wqzn3jvPRZe2LMWYmiQcijT4nw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2016-02-17  4:13             ` davide rossetti
2016-02-17  4:13               ` davide rossetti
2016-02-17  4:44               ` Jason Gunthorpe
2016-02-17  8:49                 ` Christoph Hellwig
     [not found]                   ` <20160217084959.GB13616-wEGCiKHe2LqWVfeAwA7xHQ@public.gmane.org>
2016-02-18 17:12                     ` Jason Gunthorpe
2016-02-18 17:12                       ` Jason Gunthorpe
     [not found]               ` <CAPSaadx3vNBSxoWuvjrTp2n8_-DVqofttFGZRR+X8zdWwV86nw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2016-02-17  8:44                 ` Christoph Hellwig
2016-02-17  8:44                   ` Christoph Hellwig
2016-02-17 15:25                   ` Haggai Eran
     [not found]                     ` <56C490DF.1090100-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
2016-02-19 18:54                       ` Dan Williams
2016-02-19 18:54                         ` Dan Williams

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=20160211191838.GA23675@obsidianresearch.com \
    --to=jgunthorpe-epgobjl8dl3ta4ec/59zmfatqe2ktcn/@public.gmane.org \
    --cc=artemyko-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org \
    --cc=dledford-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org \
    --cc=haggaie-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org \
    --cc=leon-Dj0/KMMU01E@public.gmane.org \
    --cc=linux-mm-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=sagig-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.