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

Improve EFA work completion statuses #1456

Merged
merged 3 commits into from
May 8, 2024
Merged

Conversation

mrgolin
Copy link
Contributor

@mrgolin mrgolin commented May 2, 2024

Improve device work completion status enum values to make them more accurate and clear, and adjust translation to IBV work completion status where needed.

dkkranz and others added 3 commits May 2, 2024 18:17
EFA returns bad address completion status when RKEY is not registered or
does not match remote IOVA. Fix the status description and match it to
the appropriate remote access verbs error.

Reviewed-by: Michael Margolin <mrgolin@amazon.com>
Reviewed-by: Yonatan Nachum <ynachum@amazon.com>
Signed-off-by: Daniel Kranzdorf <dkkranzd@amazon.com>
Signed-off-by: Michael Margolin <mrgolin@amazon.com>
EFA supports rdma operations only when a valid AH exists.
Add a new completion status for this remote error.

Reviewed-by: Michael Margolin <mrgolin@amazon.com>
Reviewed-by: Yonatan Nachum <ynachum@amazon.com>
Signed-off-by: Daniel Kranzdorf <dkkranzd@amazon.com>
Signed-off-by: Michael Margolin <mrgolin@amazon.com>
Align to device interface on work completion with unsupported operation
error. This terminology is more accurate since this error can be related
not only to an incorrect opcode.

Reviewed-by: Daniel Kranzdorf <dkkranzd@amazon.com>
Reviewed-by: Youssef Youssef <ussef@amazon.com>
Signed-off-by: Michael Margolin <mrgolin@amazon.com>
@rleon rleon merged commit df84cc4 into linux-rdma:master May 8, 2024
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
Development

Successfully merging this pull request may close these issues.

3 participants