Skip to content

Conversation

@jsquyres
Copy link
Member

@jsquyres jsquyres commented May 9, 2017

This change has the side effect of improving the performance of all atomic data structures (in addition to making the code correct under a certain interpretation of the volatile usage).

Original issue: #3450
Master PR: #3468
Fixes #3478 (v3.0.x tracking issue)

Has been approved by @hjelmn @bwbarrett @bosilca on master.

Signed-off-by: George Bosilca bosilca@icl.utk.edu
(cherry picked from commit d7ebcca)
Signed-off-by: Jeff Squyres jsquyres@cisco.com

This change has the side effect of improving the performance of all
atomic data structures (in addition to making the code crrect under a
certain interpretation of the volatile usage).
This commit fixes open-mpi#3450.

Signed-off-by: George Bosilca <bosilca@icl.utk.edu>(cherry picked from commit d7ebcca)
Signed-off-by: Jeff Squyres <jsquyres@cisco.com>
@jsquyres jsquyres added this to the v3.0.0 milestone May 9, 2017
@jsquyres jsquyres requested a review from hppritcha May 9, 2017 15:20
@jsquyres jsquyres changed the title v3.0.x: Add volatile to the pointer in the list_item structure. (#3468) v3.0.x: Add volatile to the pointer in the list_item structure May 9, 2017
@bwbarrett bwbarrett merged commit a491f63 into open-mpi:v3.x May 19, 2017
@jsquyres jsquyres deleted the pr/v3.0.x/opal-list-item-volatile branch July 17, 2022 12:41
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