[v5.0.x] btl/base: push operation->hdr to am_rdma_respond for queued operation #10529
Conversation
This file contains hidden or 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
Currently, when calling am_rdma_respond() for a queued operation, amd_rdma_retry_operation() pass NULL for the hdr argument. The idea is that hdr is only used for allocating operation->descriptor. A queued operation should already have a descriptor, therefore does not need hdr. This missed the possibility that the allocation of descriptor in am_rdma_respond() can fail, which will lead to the operation to be queued without a descriptor. This patch make retry_operation() to pass operation->hdr to am_rdma_repsond() to address the issue. It also added an assertion in am_rdma_repsond() about hdr must not be NULL before hdr is being used. Signed-off-by: Wei Zhang <wzam@amazon.com> (cherry picked from commit 1758e3d)
|
backport #10463 to v5.0.x branch |
|
prepend with v5.0.x? |
bwbarrett
approved these changes
Jun 29, 2022
wzamazon
changed the title
I added v5.0.x to PR title, but did not add to commit. because previous back port commits does not have it. |
Thanks - we actually prefer it this way. It makes diffing the release branch with main easier. |
|
Is there anything else needed to get it merged? |
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.
Currently, when calling am_rdma_respond() for a queued
operation, amd_rdma_retry_operation() pass NULL for the hdr argument.
The idea is that hdr is only used for allocating operation->descriptor.
A queued operation should already have a descriptor, therefore does
not need hdr.
This missed the possibility that the allocation of descriptor
in am_rdma_respond() can fail, which will lead to the operation
to be queued without a descriptor.
This patch make retry_operation() to pass operation->hdr to
am_rdma_repsond() to address the issue.
It also added an assertion in am_rdma_repsond() about hdr must
not be NULL before hdr is being used.
Signed-off-by: Wei Zhang wzam@amazon.com
(cherry picked from commit 1758e3d)