Skip to content

Stream_df_cholesky_peek: Guarantee data is copied back to shared memory#7

Open
nmnobre wants to merge 1 commit intomasterfrom
cholesky_sync_fix
Open

Stream_df_cholesky_peek: Guarantee data is copied back to shared memory#7
nmnobre wants to merge 1 commit intomasterfrom
cholesky_sync_fix

Conversation

@nmnobre
Copy link
Member

@nmnobre nmnobre commented Jan 10, 2020

Previously, the control program could proceed to halt the timer and
enter the verification stage w/o guaranteeing the matrix was fully
unloaded from the streams to shared memory. This caused verification
to fail on single-threaded runs. This was not a problem in
multi-threaded runs, presumably because the other threads would be free
to finish these copies before the copy of the last block (which the
control programs always waited for) happened.

Previously, the control program could proceed to halt the timer and
enter the verification stage w/o guaranteeing the matrix was fully
unloaded from the streams to shared memory. This caused verification
to fail on single-threaded runs. This was not a problem in
multi-threaded runs, presumably because the other threads would be free
to finish these copies before the copy of the last block (which the
control programs always waited for) happened.
@nmnobre nmnobre requested a review from andidr January 10, 2020 18:28
@nmnobre nmnobre self-assigned this Jan 10, 2020
@IgWod IgWod added benchmark Benchmark related issue bug Something isn't working labels Jun 8, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

benchmark Benchmark related issue bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants