Skip to content

new fbtl using direct I/O #1290

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
Closed
wants to merge 6 commits into from
Closed

new fbtl using direct I/O #1290

wants to merge 6 commits into from

Conversation

edgargabriel
Copy link
Member

No description provided.

@lanl-ompi
Copy link
Contributor

Test FAILed.

@lanl-ompi
Copy link
Contributor

Test FAILed.

jsquyres pushed a commit to jsquyres/ompi that referenced this pull request Jan 8, 2016
@hjelmn
osc/rdma: fix bugs when running more than one process per node
0de9445 
A previous commit updated the one-sided code to register the state
region only once. This created an issue when using the scratch lock
with fetching atomics. In this case on any rank that isn't local rank
0 the module->state_handle is NULL. This commit fixes the issue by
removing the scratch lock and using a fragment pointer instead.

Fixes open-mpi#1290

Signed-off-by: Nathan Hjelm <hjelmn@lanl.gov>
@jsquyres
Copy link
Member

jsquyres commented Jan 8, 2016

@edgargabriel Looks like Travis actually found an error here in the OS X build. You can see the build error at the bottom of the full / raw build log: https://s3.amazonaws.com/archive.travis-ci.org/jobs/101080685/log.txt

@edgargabriel
Copy link
Member Author

hm, the error that I see does not come from directio fbtl, but from
somewhere else. The directio seems to compile correctly, and
monitoring_prof seems to fail.
Nevertheless, I am still working on the directio component, have a
couple of other smaller items, will not merge it yet.

On 1/8/2016 10:05 AM, Jeff Squyres wrote:

@edgargabriel https://github.com/edgargabriel Looks like Travis
actually found an error here in the OS X build. You can see the build
error at the bottom of the full / raw build log:
https://s3.amazonaws.com/archive.travis-ci.org/jobs/101080685/log.txt

—
Reply to this email directly or view it on GitHub
#1290 (comment).

Edgar Gabriel
Associate Professor
Parallel Software Technologies Lab http://pstl.cs.uh.edu
Department of Computer Science University of Houston
Philip G. Hoffman Hall, Room 524 Houston, TX-77204, USA

Tel: +1 (713) 743-3857 Fax: +1 (713) 743-3335

@edgargabriel
Copy link
Member Author

Not merging the branch, the performance of the direct I/O component is disappointing. Keeping the code in the branch however around.

jsquyres pushed a commit to jsquyres/ompi that referenced this pull request Aug 23, 2016
@hppritcha
Merge pull request open-mpi#1290 from rivis/pr/v2.x/mpisync
e5a8612 
v2.x: mpisync: Fix a compilation error.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@edgargabriel @lanl-ompi @jsquyres