diff options
| author | Mohamad Chaarawi <chaarawi@hdfgroup.org> | 2014-01-06 17:56:45 (GMT) |
|---|---|---|
| committer | Mohamad Chaarawi <chaarawi@hdfgroup.org> | 2014-01-06 17:56:45 (GMT) |
| commit | 6ceb9711b8f891229cf4206417a824ffb7850c3d (patch) | |
| tree | 7b837f45ff5605abd1a51d918b98790dfa8488ca /testpar/t_mdset.c | |
| parent | 1febc8a4531e221c468c55a1f367b110d15094b9 (diff) | |
| download | hdf5-6ceb9711b8f891229cf4206417a824ffb7850c3d.zip hdf5-6ceb9711b8f891229cf4206417a824ffb7850c3d.tar.gz hdf5-6ceb9711b8f891229cf4206417a824ffb7850c3d.tar.bz2 | |
[svn-r24612] fix bugs in parallel tests exposed in corner cases when running with 1
or 2 processes.
First bug is in testpar/t_mdset.c, where the test reports an error in
addition to skipping the test if there are less than three procs. Fix
to just skip the test.
Second bug is in testpar/t_dset.c in actual_io_mode tests, where
incorrect expected value for IO mode was set if the number of procs
running the test is 1.
tested with h5committest.
Diffstat (limited to 'testpar/t_mdset.c')
| -rw-r--r-- | testpar/t_mdset.c | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/testpar/t_mdset.c b/testpar/t_mdset.c index 8fc739e..516cc2f 100644 --- a/testpar/t_mdset.c +++ b/testpar/t_mdset.c @@ -1779,12 +1779,9 @@ void rr_obj_hdr_flush_confusion(void) */ MPI_Comm_rank(MPI_COMM_WORLD, &mpi_rank); MPI_Comm_size(MPI_COMM_WORLD, &mpi_size); - if (mpi_size < 3){ - HDfprintf(stdout, "%s needs at least 3 processes to run. Terminated.\n", - fcn_name); - nerrors++; - return; - } + + HDassert(mpi_size > 2); + is_reader = mpi_rank % 2; mrc = MPI_Comm_split(MPI_COMM_WORLD, is_reader, mpi_rank, &comm); VRFY((mrc==MPI_SUCCESS), "MPI_Comm_split"); |