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

Feature: Migratablelun #3589

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Feature: Migratablelun #3589

wants to merge 3 commits into from

Conversation

ericjbohm
Copy link
Contributor

! Handle the details of migrating LUNs across ranks
! basic idea:
MigratableLUN is a utility to simplify migration for FORTRAN I/O

To use it the application creates the registry using createRegistry()

Application registers each LUN with the API using the same parameters it would use to open the file using registerLUN().

Application calls closeRegisteredLUNSForMigration() in about_to_migrate

Application calls reopenRegisteredLUNs() in just_migrated

Thats is all there is to it, otherwise the normal AMPI rules apply

Implementation assumes it is being used for LUNs associated with files.
YMMV if you need to migrate other LUNs.

MigratableLUN does not solve LUN uniqueness across virtual ranks. It is compatible with CK_LUN, which can be used to address that.

Does a simple factorial computation differentiated by rank.  Triggers LB.
Does more iterations, then has each rank read the output of the next
higher rank and check that the values are correct.

So if something went wrong with the output process during migration, we
hope to trip over it by checking every line of every output file.
Add AMPI_LUN_open as wrapper for register+open
the AMPI_LUN function returns the virtualized version of any LUN

see virtluntest.f90 for an example of usage, as it replaces
the manual LUN virtualization scheme used in luntest.f90
@evan-charmworks
Copy link
Contributor

I rebased this from the pieglobals-testing branch onto the main branch for review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
AMPI Issues affecting AMPI ease of use migration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants