diff options
author | Albert Cheng <acheng@hdfgroup.org> | 2008-05-20 18:34:00 (GMT) |
---|---|---|
committer | Albert Cheng <acheng@hdfgroup.org> | 2008-05-20 18:34:00 (GMT) |
commit | e3180b57c03e9b91d7d2765351cd8f6f71d0f1c6 (patch) | |
tree | 11f6b48c9cb29170118649febc8e0d8aef547ba6 | |
parent | e6715b970433b2d9087dbce96740352aadbbc0ca (diff) | |
download | hdf5-e3180b57c03e9b91d7d2765351cd8f6f71d0f1c6.zip hdf5-e3180b57c03e9b91d7d2765351cd8f6f71d0f1c6.tar.gz hdf5-e3180b57c03e9b91d7d2765351cd8f6f71d0f1c6.tar.bz2 |
[svn-r15043] Bug fix.
Description:
perf would crash at the end as it complained it was doing MPI calls after
MPI_Finalize() is called. I suspect the HDF5 library is making MPI calls
in the at_exit area. This should not happened.
Solution:
Added the H5exit() right before MPI_Finalize() fixed the problem.
But it needs to find out why HDF5 library is making MPI calls in
the at_exit() area. It should not.
Also, changed the temporay data file name from /tmp/test.out to
perftest.out, both to avoid multiple instance of this may use the
same /tmp/xxx name and also make the file name less generic.
Tested:
Kagiso PP.
-rw-r--r-- | perform/perf.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/perform/perf.c b/perform/perf.c index 33f7433..d0c49f6 100644 --- a/perform/perf.c +++ b/perform/perf.c @@ -77,8 +77,8 @@ int opt_iter = 1; int opt_stripe = -1; int opt_correct = 0; int amode = O_RDWR | O_CREAT; -char opt_file[256] = "/tmp/test.out\0"; -char opt_pvfstab[256] = "notset\0"; +char opt_file[256] = "perftest.out"; +char opt_pvfstab[256] = "notset"; int opt_pvfstab_set = 0; /* function prototypes */ @@ -372,6 +372,8 @@ die_jar_jar_die: free(tmp); if (opt_correct) free(tmp2); + /* Close down the HDF5 library before MPI_Finalize. */ + H5close(); MPI_Finalize(); return(0); } |