Re: [PATCH v3 0/3] update infiniband uverbs documentation

2019-02-03 Thread Joel Nider
Jonathan Corbet wrote on 02/02/2019 01:15:33 AM: > Subject: Re: [PATCH v3 0/3] update infiniband uverbs documentation > > On Fri, 1 Feb 2019 09:52:52 -0700 > Jason Gunthorpe wrote: > > > Doc folks, what is the feedback on these patches? Should I take > > them through the rdma tree? > > If

Re: [PATCH v3 0/3] update infiniband uverbs documentation

2019-01-30 Thread Joel Nider
Hi Jon, Have you had a chance to review this patchset? Thanks, Joel Nider/Haifa/IBM@IBMIL wrote on 01/22/2019 12:00:32 PM: > From: Joel Nider/Haifa/IBM@IBMIL > To: "Jonathan Corbet" > Cc: "Jason Gunthorpe" , "Leon Romanovsky" , > "Doug Ledfor

Re: [PATCH 5/5] RDMA/uverbs: add UVERBS_METHOD_REG_REMOTE_MR

2019-01-30 Thread Joel Nider
linux-rdma-ow...@vger.kernel.org wrote on 01/29/2019 07:04:06 PM: > On Tue, Jan 29, 2019 at 03:26:26PM +0200, Joel Nider wrote: > > Add a new handler for new uverb reg_remote_mr. The purpose is to register > > a memory region in a different address space (i.e. process) than

Re: [PATCH 0/5] RDMA: reg_remote_mr

2019-01-30 Thread Joel Nider
Steve Wise wrote on 01/29/2019 06:44:48 PM: > > On 1/29/2019 7:26 AM, Joel Nider wrote: > > As discussed at LPC'18, there is a need to be able to register a memory > > region (MR) on behalf of another process. One example is the case of > > post-copy container

[PATCH 5/5] RDMA/uverbs: add UVERBS_METHOD_REG_REMOTE_MR

2019-01-29 Thread Joel Nider
the migration. Signed-off-by: Joel Nider --- drivers/infiniband/core/uverbs_std_types_mr.c | 129 +- include/rdma/ib_verbs.h | 8 ++ include/uapi/rdma/ib_user_ioctl_cmds.h| 13 +++ 3 files changed, 149 insertions(+), 1 deletion(-) diff --git

[PATCH 0/5] RDMA: reg_remote_mr

2019-01-29 Thread Joel Nider
. In this case, we want all RDMA READ requests to be served by the address space of the migration process directly (not by CRIU). This patchset implements a new uverbs command which allows an application to register a memory region in the address space of another process. Joel Nider (5): mm: add

[PATCH 2/5] RDMA/uverbs: add owner parameter to reg_user_mr

2019-01-29 Thread Joel Nider
for the caller and registree to be different processes, which is required for the reg_remote_mr verb implemented in the following patches. No functional changes in these files. Signed-off-by: Joel Nider --- drivers/infiniband/core/uverbs_cmd.c | 2 +- drivers/infiniband/hw/bnxt_re/ib_verbs.c

[PATCH 4/5] RDMA/uverbs: add owner parameter to ib_umem_odp_get

2019-01-29 Thread Joel Nider
a page fault from ODP. Signed-off-by: Joel Nider --- drivers/infiniband/core/umem.c | 4 +-- drivers/infiniband/core/umem_odp.c | 50 ++ drivers/infiniband/hw/mlx5/odp.c | 6 - include/rdma/ib_umem_odp.h | 6 +++-- 4 files changed, 35

[PATCH 1/5] mm: add get_user_pages_remote_longterm function

2019-01-29 Thread Joel Nider
(get_user_pages_longterm) and remote pages (get_user_pages_remote) already exist - this new function combines the functionality of both of them. Signed-off-by: Joel Nider Reviewed-by: Mike Rapoport --- include/linux/mm.h | 28 +--- mm/gup.c | 15 ++- 2 files changed, 35

[PATCH 3/5] RDMA/uverbs: add owner parameter to ib_umem_get

2019-01-29 Thread Joel Nider
that call this function are also updated, but the default behaviour is to keep backwards compatibility by assuming the current process is the owner when the 'owner' parameter is NULL. Signed-off-by: Joel Nider --- drivers/infiniband/core/umem.c| 26

[PATCH v3 1/3] docs-rst: Convert user verbs doc to rst

2019-01-22 Thread Joel Nider
. Since this documents a userspace API, its home should be with the other userspace API docs. Signed-off-by: Joel Nider --- Documentation/infiniband/user_verbs.txt | 69 Documentation/userspace-api/index.rst | 1 + Documentation/userspace-api

[PATCH v3 3/3] MAINTAINERS: add new RDMA/Infiniband documentation

2019-01-22 Thread Joel Nider
with the responsibility of the new files. Signed-off-by: Joel Nider --- MAINTAINERS | 2 ++ 1 file changed, 2 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 2cf9c1c..977b83d 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -7487,6 +7487,8 @@ T:git git://git.kernel.org/pub/scm/linux/kernel

[PATCH v3 2/3] docs-rst: driver-api: Add infiniband interface documentation

2019-01-22 Thread Joel Nider
A short document regarding the user verbs interface implementation on the kernel side. Also, the corresponding index entry in the documentation tree. Signed-off-by: Joel Nider --- Documentation/driver-api/index.rst | 1 + Documentation/driver-api/infiniband.rst | 73

[PATCH v3 0/3] update infiniband uverbs documentation

2019-01-22 Thread Joel Nider
and Jason: The location of the new content should be driver-api The location of the old (converted) content should be userspace-api MAINTAINERS file must be updated Joel Nider (3): docs-rst: Convert user verbs doc to rst docs-rst: driver-api: Add infiniband interface documentation

Re: [PATCH v2 2/2] docs-rst: userspace: update verbs API details

2019-01-16 Thread Joel Nider
Jonathan Corbet wrote on 01/15/2019 11:38:59 PM: > We are not attempting to duplicate the man pages; there's been occasional > talk of bringing them into the kernel tree, but enthusiasm for that is > scarce for a number of good reasons. But there's a lot of information > about the user-space

Re: [PATCH v2 1/2] docs-rst: Convert user verbs doc to rst

2019-01-15 Thread Joel Nider
Jonathan Corbet wrote on 01/15/2019 08:02:18 PM: > From: Jonathan Corbet > To: Jason Gunthorpe > Cc: Joel Nider , Leon Romanovsky , Doug > Ledford , Mike Rapoport , linux- > d...@vger.kernel.org, linux-kernel@vger.kernel.org > Date: 01/15/2019 08:14 PM > Subject: Re: [P

Re: [PATCH v2 2/2] docs-rst: userspace: update verbs API details

2019-01-15 Thread Joel Nider
Jonathan Corbet wrote on 01/15/2019 08:08:54 PM: > From: Jonathan Corbet > To: "Joel Nider" > Cc: Matthew Wilcox , Doug Ledford , > Jason Gunthorpe , Leon Romanovsky , linux- > d...@vger.kernel.org, linux-kernel@vger.kernel.org, Mike Rapoport > Date: 01/15

Re: [PATCH v2 2/2] docs-rst: userspace: update verbs API details

2019-01-15 Thread Joel Nider
Matthew Wilcox wrote on 01/15/2019 05:31:28 PM: > From: Matthew Wilcox > To: Joel Nider > Cc: Jonathan Corbet , Jason Gunthorpe , Leon > Romanovsky , Doug Ledford , Mike > Rapoport , linux-...@vger.kernel.org, linux-kernel@vger.kernel.org > Date: 01/15/2019 05:31 PM >

[PATCH v2 2/2] docs-rst: userspace: update verbs API details

2019-01-15 Thread Joel Nider
the stable parts so as to avoid incorrect information since documentation changes tend to lag behind code changes. Signed-off-by: Joel Nider --- Documentation/userspace-api/user_verbs.rst | 69 +- 1 file changed, 68 insertions(+), 1 deletion(-) diff --git

docs-rst: update infiniband uverbs doc

2019-01-15 Thread Joel Nider
A small patchset to update the verbs API documentation with some information regarding the ioctl syscall. First patch converts the file format to ReST, since this is the new preferred format, moves the file to Documentation/userspace-api, and updates the index. The 2nd patch adds the new content,

[PATCH v2 1/2] docs-rst: Convert user verbs doc to rst

2019-01-15 Thread Joel Nider
this documents a userspace API, its home should be with the other userspace API docs. This is in preparation for updating the content in a subsequent patch. Signed-off-by: Joel Nider --- Documentation/infiniband/user_verbs.txt| 69 - Documentation/userspace-api

Re: [PATCH 1/3] docs-rst: infiniband: Convert user verbs doc to rst

2019-01-14 Thread Joel Nider
Jonathan Corbet wrote on 01/14/2019 07:34:21 PM: > From: Jonathan Corbet > To: Jason Gunthorpe > Cc: Joel Nider , Leon Romanovsky , Doug > Ledford , Mike Rapoport , linux- > d...@vger.kernel.org, linux-kernel@vger.kernel.org > Date: 01/14/2019 07:37 PM > Subject: Re:

[PATCH 3/3] docs-rst: infiniband: update verbs API details

2019-01-14 Thread Joel Nider
the stable parts so as to avoid incorrect information since documentation changes tend to lag behind code changes. Signed-off-by: Joel Nider --- Documentation/infiniband/user_verbs.rst | 69 - 1 file changed, 68 insertions(+), 1 deletion(-) diff --git

[PATCH 2/3] docs-rst: update index file with infiniband docs

2019-01-14 Thread Joel Nider
Link the previously converted Documentation/infiniband/user_verbs.rst to the main index by creating a new subsystem (Infiniband) under the root document. This manifests as a new section under "Kernel API Documentation" in the index.html, as well as a new section in the table of contents pane.

[PATCH 1/3] docs-rst: infiniband: Convert user verbs doc to rst

2019-01-14 Thread Joel Nider
-by: Joel Nider --- Documentation/infiniband/user_verbs.rst | 70 + Documentation/infiniband/user_verbs.txt | 69 2 files changed, 70 insertions(+), 69 deletions(-) create mode 100644 Documentation/infiniband/user_verbs.rst delete

updating user verbs documentation

2019-01-14 Thread Joel Nider
A small patchset to update the verbs API documentation with some information regarding the ioctl syscall. First patch converts the file format to ReST, since this is the new preferred format. 2nd patch links this file to the main index so we can actually find it by browsing (search will work in

[PATCH v2] docs-rst: doc-guide: Minor grammar fixes

2019-01-13 Thread Joel Nider
While using this guide to learn the new documentation method, I saw a few phrases that I felt could be improved. These small changes improve the grammar and choice of words to further enhance the installation instructions. Signed-off-by: Joel Nider Acked-by: Matthew Wilcox --- v2: address

[PATCH] docs-rst: doc-guide: Minor grammar fixes

2019-01-13 Thread Joel Nider
While using this guide to learn the new documentation method, I saw a few phrases that I felt could be improved. These small changes improve the grammar and choice of words to further enhance the installation instructions. Signed-off-by: Joel Nider --- Documentation/doc-guide/sphinx.rst | 12