Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add RDMA write support to EFA provider #1317

Merged
merged 3 commits into from May 4, 2023
Merged

Conversation

YonatanNachum
Copy link
Contributor

EFA devices are now support RDMA write, make minor refactoring of EFA send inner function for creating wqe, set remote address and set immediate, and add RDMA write support.

@gal-pressman
Copy link
Contributor

Where is the kernel side?

@YonatanNachum
Copy link
Contributor Author

Kernel side is dependent on data polling patch, will be glad to get reviews for both:
PR
kernel patch

thanks

@jgunthorpe
Copy link
Member

This still seems like the kernel side hasn't been posted, I don't see it on patchworks.

@mrgolin
Copy link
Contributor

mrgolin commented Apr 2, 2023

We've found another needed change, will sent kernel patch soon.

@YonatanNachum
Copy link
Contributor Author

kernel patch

Thanks.

@jgunthorpe
Copy link
Member

This needs to be rebased

Refactor send functions (messages and RDMA) to be more modular, add two
new functions: set imm and set remote addr so send functions (messages
and RDMA) can use those functions if needed.

Signed-off-by: Yonatan Nachum <ynachum@amazon.com>
@YonatanNachum YonatanNachum force-pushed the rdma_write branch 2 times, most recently from 7f2ff12 to f866ad7 Compare April 22, 2023 06:03
@YonatanNachum
Copy link
Contributor Author

@jgunthorpe
Rebased branch and updated kernel headers.

Thanks.

To commit 531094dc7164 ("RDMA/efa: Add rdma write capability to device caps").

Signed-off-by: Yonatan Nachum <ynachum@amazon.com>
Add rdma write functions (with and without imm) to qp extended api and
expose capability bit using query device direct verb.

Signed-off-by: Yonatan Nachum <ynachum@amazon.com>
@@ -76,6 +76,9 @@ struct efadv_device_attr {
Reading source address (SGID) from receive completion descriptors is supported.
Valid only for unknown AH.

EFADV_DEVICE_ATTR_CAPS_RDMA_WRITE:
RDMA write is supported

*max_rdma_size*
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does this apply to both RDMA read and write now?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey,
No this does not apply for RDMA read. For RDMA read you must get also EFADV_DEVICE_ATTR_CAPS_RDMA_READ to know it is supported.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm referring to max_rdma_size.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey, sorry.
Yes, the max RDMA size is identical for read and write.

@YonatanNachum
Copy link
Contributor Author

@jgunthorpe
Hey, kind reminder for this patch.

Thank you.

@jgunthorpe jgunthorpe merged commit eb47f08 into linux-rdma:master May 4, 2023
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants