This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "MPICH primary repository". The branch, master has been updated via 0b126663cc56320edc34d1cd9a6d50bc765a600c (commit) from a5412fec1e0e2f15b0d38369c2f489686e90d36c (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log ----------------------------------------------------------------- http://git.mpich.org/mpich.git/commitdiff/0b126663cc56320edc34d1cd9a6d50bc76... commit 0b126663cc56320edc34d1cd9a6d50bc765a600c Author: Xin Zhao <[email protected]> Date: Thu Nov 14 23:57:32 2013 -0600 Fix ticket-1960: delete redundant MPIU_Assert. MPIU_Assert at L2311 checks if rma_ops_list is empty before exiting MPIDI_Win_flush. It causes /test/mpi/threads/rma/multirma to fail because while one thread is executing the loop of poking progress engine at L2293 ~ L2302, another thread may enqueue new RMA operations to rma_ops_list. rma_ops_list has already been checked for empty before exiting MPIDI_CH3I_Do_passive_target_rma (L2724) to ensure that all enqueued operations are issued out, therefore it does not need to be checked again here. Signed-off-by: Wesley Bland <[email protected]> diff --git a/src/mpid/ch3/src/ch3u_rma_sync.c b/src/mpid/ch3/src/ch3u_rma_sync.c index beb5a6c..7a2553a 100644 --- a/src/mpid/ch3/src/ch3u_rma_sync.c +++ b/src/mpid/ch3/src/ch3u_rma_sync.c @@ -2308,7 +2308,6 @@ int MPIDI_Win_flush(int rank, MPID_Win *win_ptr) } fn_exit: - MPIU_Assert(MPIDI_CH3I_RMA_Ops_isempty(&win_ptr->targets[rank].rma_ops_list)); MPIDI_RMA_FUNC_EXIT(MPID_STATE_MPIDI_WIN_FLUSH); return mpi_errno; /* --BEGIN ERROR HANDLING-- */ ----------------------------------------------------------------------- Summary of changes: src/mpid/ch3/src/ch3u_rma_sync.c | 1 - 1 files changed, 0 insertions(+), 1 deletions(-) hooks/post-receive -- MPICH primary repository