Add user space dmabuf support #895
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
j-xiong
force-pushed
the
v5.10-dmabuf
branch
6 times, most recently
from
af24517
to
370c1db
Compare
EdwardSro
suggested changes
Nov 25, 2020
EdwardSro
suggested changes
Nov 25, 2020
j-xiong
force-pushed
the
v5.10-dmabuf
branch
3 times, most recently
from
a68d210
to
e95e9b6
Compare
|
PR has been updated to address all the existing review comments. |
|
This needs rebasing. Can you mark all the fixed things as resolved in the github GUI? Note the package version is now 34 so the debian/control needs updating during the rebase |
|
Also fix all of the CI errors and warnings - click on checks and follow it through |
jgunthorpe
requested changes
Jan 20, 2021
j-xiong
force-pushed
the
v5.10-dmabuf
branch
2 times, most recently
from
6afef59
to
f715059
Compare
|
PR updated to address review comments from @jgunthorpe. |
|
Please fix conflct, symbols and update kernel-headers with kernel-header/update script so we will be able to merge this PR. Thanks |
To commit bfe0cc6eb249 ("RDMA/uverbs: Add uverbs command for dma-buf based
MR registration").
Signed-off-by: Jianxin Xiong <jianxin.xiong@intel.com>
Add new API function and new provider method for registering dma-buf based memory region. Update the man page and bump the API version. Signed-off-by: Jianxin Xiong <jianxin.xiong@intel.com>
Implement the new provider method for registering dma-buf based memory regions. Signed-off-by: Jianxin Xiong <jianxin.xiong@intel.com>
Define a new sub-class of 'MR' that uses dma-buf object for the memory region. Define a new class 'DmaBuf' as a wrapper for dma-buf allocation mechanism implemented in C. Update the cmake function for cython modules to allow building modules with mixed cython and c source files. Signed-off-by: Jianxin Xiong <jianxin.xiong@intel.com>
Define a set of unit tests similar to regular MR tests and a set of tests for send/recv and rdma traffic using dma-buf MRs. Add a utility function to generate access flags for dma-buf based MRs because the set of supported flags is smaller. Signed-off-by: Jianxin Xiong <jianxin.xiong@intel.com>
The filter definition is wrong and causes get_access_flags() always returning empty list. As the result the MR tests using this function are effectively skipped (but report success). Signed-off-by: Jianxin Xiong <jianxin.xiong@intel.com>
|
PR rebased to resolve conflicts and symbols. Re-ran kernel-header/update against linux-rdma/for-next. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This is the user space counter-part of the kernel patch set to add
dma-buf support to the RDMA subsystem. This patch series adds user
space API for registering dma-buf based memory regions, updates
pyverbs with the new API, and adds new tests.
This series consists of six patches. The first patch updates the
kernel headers for dma-buf support. Patch 2 adds the new API function
and updates the man pages. Patch 3 implements the new API in the mlx5
provider. Patch 4 adds new class definitions to pyverbs for the new API.
Patch 5 adds a set of new tests for the new API. Patch 6 fixes bug in
the utility code of the tests.