Skip to content

osc-rdma: fix datatype lower bound errors in ompi_osc_rdma_master() #2197

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

Merged

Conversation

tkordenbrock
Copy link
Member

Instead of ompi_datatype_get_extent(), use ompi_datatype_get_true_extent()
to get the local and remote lower bound. For derived types like
subarray, true_lb is the correct offset for RDMA operations.

@tkordenbrock tkordenbrock added this to the v2.x milestone Oct 8, 2016
@tkordenbrock
Copy link
Member Author

@hjelmn Discovered this with the epochtest from the ANL mpi3test test suite.

Instead of ompi_datatype_get_extent(), use ompi_datatype_get_true_extent()
to get the local and remote lower bound.  For derived types like
subarray, true_lb is the correct offset for RDMA operations.
@tkordenbrock tkordenbrock force-pushed the topic/master/osc-rdma.put.use.true_extent branch from 0177a9d to 05f86b5 Compare October 10, 2016 11:46
@tkordenbrock tkordenbrock modified the milestones: v2.1.0, v2.x Oct 10, 2016
@tkordenbrock
Copy link
Member Author

bot:mellanox:retest

@hjelmn
Copy link
Member

hjelmn commented Oct 11, 2016

Looks good. Please file a PR for 2.0.2 and 2.1.0.

@hjelmn hjelmn merged commit 432d790 into open-mpi:master Oct 11, 2016
clementFoyer pushed a commit to bosilca/ompi that referenced this pull request Nov 4, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants