-
Notifications
You must be signed in to change notification settings - Fork 8
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
WIP - NNFDM implementation in AXL #131
Draft
mcfadden8
wants to merge
39
commits into
main
Choose a base branch
from
mcfadden8/add-dm-client
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Changes from 1 commit
Commits
Show all changes
39 commits
Select commit
Hold shift + click to select a range
6951d3a
Prototype for NNFDM implementation in AXL
mcfadden8 ffb4dcc
Use new HPE C++ NNFDM library
mcfadden8 18b0cfb
Leave NNFDM off by default for now
mcfadden8 2dc695c
Add NNFDM Cancel
mcfadden8 bae2a8d
Turn HPE data mover back on
mcfadden8 d52d9a9
make nnfdm inclusion automatic
mcfadden8 6d8cf6c
Merge branch 'main' into mcfadden8/add-dm-client
mcfadden8 e719849
Added basic test of nnf-dm library
mcfadden8 0277f59
Checkpoint: SCR now successfully is using nnfdm to copy files. The n…
mcfadden8 13bdf5f
Initial working version with single rank
mcfadden8 e64691e
Removed unused code
mcfadden8 bc0f07f
Minor debug statement updates
mcfadden8 2f7479b
Updates to build with latest version of nnfdm library
mcfadden8 5c2d38d
Update signatures to latest nnfdm library
mcfadden8 9f60116
Updates from main
mcfadden8 e14bd94
Added find dependency for NNFDM if/when NNFDM is included
mcfadden8 830cee7
Add nnfdm to list of external static libs for AXL
mcfadden8 2958a74
Added nnf user container example
mcfadden8 45a0b4a
Updates
mcfadden8 fdb967d
Working set of hpe containers for rabbits
mcfadden8 6d2b38b
Added documentation for building Dockerfile
mcfadden8 a2e3e45
Initial container created
mcfadden8 638f969
Updates
mcfadden8 d911cf6
Added container for Hari
mcfadden8 5755781
Added container for Cameron
mcfadden8 e1552c6
Updates
mcfadden8 a1064ce
Updates
mcfadden8 da4b28c
Another update
mcfadden8 0f9b660
Separate nnf containers for hetchy and tioga
mcfadden8 b040ff6
Make NNFDM inclusion optional
mcfadden8 b1a372b
Merge branch 'main' into mcfadden8/add-dm-client
mcfadden8 33bb280
Updates: run some mpiGraph tests on rabbits
mcfadden8 87068a9
Updates for v0.0.6 version of library
mcfadden8 da33014
Added new profile parameter for nnfdm::CreateResponse API
mcfadden8 28922bd
Improved error handling around nnfdm create
mcfadden8 ac58578
Added more debugging capabilities
mcfadden8 2608e22
Use SCR data movement profile
mcfadden8 e59fc12
Work with nnfdm SCR profile
mcfadden8 6598792
axl: Update to latest v0.1.7 nnf-dm version
mcfadden8 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,41 @@ | ||
# - Try to find libnnfdm | ||
# | ||
# Once done this will define | ||
# NNFDM_FOUND - System has libdatawarp | ||
# NNFDM_INCLUDE_DIRS - The libdatawarp include directories | ||
# NNFDM_LIBRARIES - The libraries needed to use libdatawarp | ||
# | ||
# This is early days for this library. For now, we assume that there is | ||
# a the following directory and contents pointed to by WITH_NNFDM_PREFIX: | ||
# | ||
# path/nnfdm/ | ||
# lib64/ | ||
# libnnfdm.a | ||
# include/ | ||
# datamovement.pb-c.h | ||
# nnfdm.h | ||
# | ||
# So, the following cmake line will cause this to be found assuming the prefix exists: | ||
# cmake -DWITH_NNFDM_PREFIX="/usr/WS2/martymcf/scr/dm/nnfdm" -DMPI=ON .. | ||
|
||
FIND_LIBRARY(NNFDM_LIBRARIES | ||
NAMES nnfdm | ||
HINTS ${WITH_NNFDM_PREFIX}/lib64 | ||
) | ||
|
||
FIND_PATH(NNFDM_INCLUDE_DIRS | ||
NAMES nnfdm.h | ||
HINTS ${WITH_NNFDM_PREFIX}/include | ||
) | ||
|
||
INCLUDE(FindPackageHandleStandardArgs) | ||
FIND_PACKAGE_HANDLE_STANDARD_ARGS(NNFDM DEFAULT_MSG | ||
NNFDM_LIBRARIES | ||
NNFDM_INCLUDE_DIRS | ||
) | ||
|
||
# Hide these vars from ccmake GUI | ||
MARK_AS_ADVANCED( | ||
NNFDM_LIBRARIES | ||
NNFDM_INCLUDE_DIRS | ||
) |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,5 +1,6 @@ | ||
// Machine Specific Libs | ||
#cmakedefine HAVE_PTHREADS | ||
#cmakedefine HAVE_DATAWARP | ||
#cmakedefine HAVE_NNFDM | ||
#cmakedefine HAVE_BBAPI | ||
#cmakedefine HAVE_BBAPI_FALLBACK |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's include a comment here about why we don't have to do anything.