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.

Sign up for GitHub

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

Jump to bottom

prov/efa: Two cleanup changes #9947

Merged
merged 2 commits into from
Mar 29, 2024
Merged

prov/efa: Two cleanup changes #9947

merged 2 commits into from
Mar 29, 2024

Conversation

shijin-aws
Copy link
Contributor

Two cleanup changes

  • Remove unused cq_attr in efa_rdm_ep_open
  • Move the declaration of efa_rdm_cq_poll_ibv_cq from efa_cq.h to efa_rdm_cq.h because it is specific for RDM endpoint

@shijin-aws
prov/efa: remove unused cq_attr
3299864 
Signed-off-by: Shi Jin <[email protected]>
@shijin-aws
prov/efa: Move efa_rdm_cq_poll_ibv_cq to efa_rdm_cq.h
1403c51 
This function is specific for RDM endpoint.

Signed-off-by: Shi Jin <[email protected]>
@shijin-aws shijin-aws requested a review from a team March 28, 2024 21:51
@shijin-aws shijin-aws merged commit 9a049b8 into ofiwg:main Mar 29, 2024
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@wenduwan wenduwan wenduwan approved these changes

@a-szegel a-szegel a-szegel approved these changes

@darrylabbate darrylabbate darrylabbate approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@shijin-aws @darrylabbate @a-szegel @wenduwan