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

[5.0.x] osc/rdma: fix compiler warnings in osc_rdma_accumulate.c #12155

Conversation

devreal
Copy link
Contributor

@devreal devreal commented Dec 8, 2023

Ensure const-correctness of the compare_addr argument to atomic CAS.

Copy link

github-actions bot commented Dec 8, 2023

Hello! The Git Commit Checker CI bot found a few problems with this PR:

34e8f25: osc/rdma: fix compiler warnings in osc_rdma_accumu...

  • check_cherry_pick: does not include a cherry pick message (did you need to bot:notacherrypick?)

Please fix these problems and, if necessary, force-push new commits back up to the PR branch. Thanks!

@janjust
Copy link
Contributor

janjust commented Dec 11, 2023

@devreal could you add the cherry-pick message for CI to pass?

@janjust
Copy link
Contributor

janjust commented Dec 11, 2023

oh I see - the main isn't merged yet, can you merge it please - I'd like to pick this up in v5.0 before rc goes out (hopefully today)

@wenduwan
Copy link
Contributor

Related to #12130

@wenduwan
Copy link
Contributor

Change on main #12154

Copy link
Member

@jsquyres jsquyres left a comment

Choose a reason for hiding this comment

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

@devreal Can you please add the cherry-pick message to the git commit message?

@devreal devreal force-pushed the fix-osc-rdma-accumulate-compiler-warnings-v5.0.x branch from 34e8f25 to f1085b4 Compare December 11, 2023 23:03
Ensure const-correctness of the compare_addr argument to atomic CAS.

Signed-off-by: Joseph Schuchart <[email protected]>
(cherry picked from commit 1360be4)
@devreal devreal force-pushed the fix-osc-rdma-accumulate-compiler-warnings-v5.0.x branch from f1085b4 to ca56ceb Compare December 11, 2023 23:07
@devreal
Copy link
Contributor Author

devreal commented Dec 11, 2023

Done, sorry I missed the bot message. Rebased and with proper cherry-pick message now.

@jsquyres
Copy link
Member

@devreal CI will still fail until #12154 is merged (this prevents us from merging a cherry pick until the commit it is cherry-picking from actually exists up on main). Can you merge your main PR? Thanks!

@janjust janjust merged commit 10c7bd6 into open-mpi:v5.0.x Dec 12, 2023
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants