Open
Conversation
Replace minimal TLS/TCB header Don't copy back TCB when jump back, add comments Use SET_LOWER_HALF_FS_CONTEXT in all wrappers WIP: Add fortran MPI_Ibarrier test case WIP: debug TLS/TCB Increase TLS size to 0x4a0 Tmp changes Make it can enable LOGGING C/R crash debug Track TLS/TCB changes
jungan
force-pushed
the
master
branch
2 times, most recently
from
93c8409 to
4e53142
Compare
jungan
pushed a commit
that referenced
this pull request
Apr 29, 2022
There is race between two phase algorithm preSuspendBarrier() and two phase commit commit_begin(). T1 thread #1: setCkptPending() T2 thread mpickpt#2: checks isCkptPending() then stop(comm) T3 thread mpickpt#2: stop(comm) sets state to PHASE_1 T4 thread mpickpt#2: while isCkptPending() loop T5 thread #1 calls waitForNewStateAfter(PHASE_1) This change adds a timeout when waiting state change. When the preSuspend wait for PHASE_1 state change times out, it would set response to FREE_PASS. Co-authored-by: Dahong Li <root@cori.nersc.gov>
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
Replace minimal TLS/TCB header
Don't copy back TCB when jump back, add comments
Use SET_LOWER_HALF_FS_CONTEXT in all wrappers
WIP: Add fortran MPI_Ibarrier test case
WIP: debug TLS/TCB
Increase TLS size to 0x4a0
Tmp changes
Make it can enable LOGGING
C/R crash debug
Track TLS/TCB changes