summaryrefslogtreecommitdiffstats
path: root/testpar/t_mpi.c
Commit message (Collapse)AuthorAgeFilesLines
* Update Copyright headers. (#2534)Larry Knox2023-03-131-1/+0
|
* clang 13 format #1933 (#1939)Allen Byrne2022-07-271-1/+1
|
* [1.12 Merge]Hdf5 1 12 warnings fixes (#1715)jhendersonHDF2022-05-031-12/+22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Warnings fixes (#1680) * Clean stack size warnings in sio_engine (#1687) * Fixes stack size warnings in tcoords.c (#1688) * Address some warnings from casting away of const (#1684) * Fixes stack size warnings in ntypes (#1695) * Fixes stack size warnings in dtransform (#1696) * Fixes stack size warnings in set_extent test (#1698) * Be a bit safer with signed arithmetic, thus quieting some signed-overflow warnings from GCC (#1706) * Avoid a signed overflow: check the range of `entry_ptr->age` before increasing it instead of increasing it and then checking the range. This quiets a GCC warning. * Avoid the potential for signed overflow by rewriting expressions `MAX(0, fwidth - n)` as `MAX(n, fwidth) - n` for various `n`. This change quiets some GCC warnings. * Change some local variables that cannot take sensible negative values from signed to unsigned. This quiets GCC warnings about potential signed overflow. * In a handful of instances, check the range of a signed integer before increasing/decreasing it, just in case the increase/decrease overflows. This quiets a handful of GCC signed-overflow warnings. * Committing clang-format changes Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com> * Fix object size warnings in cache.c test (#1701) * Fix some const cast and stack/static object size warnings (#1700) * Fix various warnings * Move HDfree_const to H5private.h for wider use * Print output from all ranks in parallel tests on allocation failure * Move const pointer freeing macro to h5test.h for now * Stop lying about H5S_t const-ness (#1209) Hyperslabs can be reworked inside several H5S callbacks, making H5S_t non-const in some places where it is marked const. This change switches these incorrectly const H5S_t pointer parameters and variables to non-const where appropriate. * Fix a few warnings after recent H5S const-related changes (#1225) * Adjustments for HDF5 1.12 Co-authored-by: Dana Robinson <43805+derobins@users.noreply.github.com> Co-authored-by: David Young <dyoung@hdfgroup.org> Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>
* 1.12: Brings changes from testpar over from develop (#1614)Dana Robinson2022-04-071-4/+4
|
* Update URL in source file Copyright headers for web copy of COPYINGLarry Knox2021-02-201-1/+1
| | | | file - files not in src or test.
* Source formattedAllen Byrne2020-10-011-313/+305
|
* Whitespace changesAllen Byrne2020-05-131-1/+1
|
* revert type castkmu2020-02-281-2/+2
|
* Merge pull request #2312 in HDFFV/hdf5 from ~LRKNOX/hdf5_lrk:develop to developLarry Knox2020-02-011-1/+1
| | | | | * commit '1c812fcd5d475afb7e65ed28d84990f96eefa649': Restore fix of test assert fail eror undone by later commit.
* Merge pull request #2306 in HDFFV/hdf5 from ~KMU/hdf5:develop to developKimmy Mu2020-01-281-1/+1
| | | | | * commit 'a152ffafb322551033adb594112f53836f7eb240': fix assert fail
* Merge pull request #2300 in HDFFV/hdf5 from ~KMU/hdf5:squashed_cast to developKimmy Mu2020-01-281-5/+5
| | | | | | * commit 'af5c33afabdae2e39bb45eb1b3e9c8366da01145': remove unnecessary stuff squash cast warning fix
* fix test assert fail errorkmu2020-01-101-2/+2
|
* Merge pull request #2071 in HDFFV/hdf5 from ~KMU/hdf5:bugfix/intel_warnings ↵Kimmy Mu2020-01-081-5/+5
| | | | | | | | | | | | | | | | | | | | | | to develop * commit '0a2bb11b248df6841daabca3970df5d8504adfc7': address problems from comments fix and address comments change according to previous comments add missing piece remove unnecessary check macro fix intel compile warnings Revert "fix warnings from Intel compiler" Revert "fix warnings and some text alignment" Revert "let hdf5 pick up the right compiler in Intel environment" Revert "fix issues from previous PR comments" Revert "using a different MACRO" using a different MACRO fix issues from previous PR comments let hdf5 pick up the right compiler in Intel environment fix warnings and some text alignment fix warnings from Intel compiler
* OESS-29 Update HD prefix mostlyAllen Byrne2019-08-241-96/+96
|
* HDFFV-10845 C_FLAGS need set correctly for win32 vs mingwAllen Byrne2019-07-171-5/+5
|
* fix var nameAllen Byrne2019-07-091-1/+1
|
* Fix compile error with ifdef H5_HAVE_GETHOSTNAMEAllen Byrne2019-07-081-786/+757
|
* Merge pull request #426 in HDFFV/hdf5 from ~LRKNOX/hdf5_lrk:hdf5_1_10 to ↵Larry Knox2017-04-251-6/+4
| | | | | | | | | | hdf5_1_10 * commit '54957d37f5aa73912763dbb6e308555e863c43f4': Commit copyright header change for src/H5PLpkg.c which was added after running script to make changes. Add new files in release_docs to MANIFEST. Cimmit changes to Makefile.in(s) and H5PL.c that resulted from running autogen.sh. Merge pull request #407 in HDFFV/hdf5 from ~LRKNOX/hdf5_lrk:hdf5_1_10_1 to hdf5_1_10_1 Change copyright headers to replace url referring to file to be removed and replace it with new url for COPYING file.
* [svn-r28040] avoid use of MPI deprecated routines.Mohamad Chaarawi2015-10-121-9/+9
|
* [svn-r27768] Description:Quincey Koziol2015-09-141-1/+1
| | | | | | | | | | | | | | | | | | | | | | Complete revamp of package initialization/shutdown mechanism in the library. Each package now has a single init/term routine. This new way should avoid packages being re-initialized during library shutdown and is also be _much_ more proactive about giving feedback for resource leaks internal to the library. Introduces a new "module" header file for packages in the library (e.g src/H5Fmodule.h) which sets up some necessary package configuration macros for the FUNC_ENTER/LEAVE macros. (The VFL drivers have their own slightly modified version of this header, src/H5FDdrvr_module.h) Also cleaned up a bunch of resources leaks all across the library and tests, along with addressing many warnings, as I encountered them. Tested on: MacOSX/64 10.10.5 (amazon) w/serial & parallel Linux/64 3.10.x (kituo) w/serial & parallel Linux/64 2.6.x (ostrich) w/serial
* [svn-r27271] fix a few warnings in t_mpi.cMohamad Chaarawi2015-06-231-4/+3
|
* [svn-r25398] add a check to make sure MPI_File_get_size() returns correct ↵Mohamad Chaarawi2014-07-091-1/+2
| | | | value (HDFFV-8856).
* [svn-r25397] - remove configure checks for MPI_File_get_size and BigMohamad Chaarawi2014-07-071-19/+1
| | | | | MPI_File_set_size as they are supported by most MPI implementations. - fix bug in t_mpi.c (HDFFV-8856)
* [svn-r24380] The parallel test code now uses HDfree, HDmalloc, and HDcallocDana Robinson2013-10-311-1/+1
| | | | | | | | | | | exclusively. Part of the preparation for a fix for HDFFV-8551. Tested on: 32-bit LE linux (jam) w/ parallel and Fortran. There are no behavior changes, so testing was minimal.
* [svn-r22388] remove unbalanced bracketAllen Byrne2012-05-221-1/+1
|
* [svn-r22385] Skip tests using sparse file writes on Windows. JIRA issue ↵Allen Byrne2012-05-211-1/+5
| | | | (JIRA HDDFV-8064) created.
* [svn-r21334] HDFFV-7687: add HD macro prefrix to functions and use h5_stat_t ↵Allen Byrne2011-08-301-2/+2
| | | | | | where necessary reviewed
* [svn-r21101] HDFFV-7639Mike McGreevy2011-07-141-159/+102
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Purpose: Remove H5_MPI_SPECIAL_COLLECTIVE_IO_WORKS and H5_MPI_COMPLEX_DERIVED_DATATYPE_WORKS #defines from source. Description: Two advanced parallel functionalities, special collective IO and complex derived datatypes, are not supported by older implementations of mpi, and thus our code limits the use of these features with #ifdefs and has checks in configure to set them (or not). Unfortunately, configure can't actually run a parallel check to see if these features are working (nor not) so it resorts to looking in the config files where they are explicity enabled or disabled based on versions of mpi, sytems being built on, or for no documented reason at all (i.e. just set to on or off as some 'default'). Overriding these settings is easy if need be, provided it is known that it needs to be done to get improved performance, and oftentimes it is not. Most new MPI implementations successfully handle the functionality requested when these #defines are set, and many of the "turn these features off" cases in the config files are for old (> 5 years) versions of MPI and retired systems (such as NCSA's tungsten). Therefore, the decision has been made to remove the support for these old versions of MPI and systems that cannot handle these behaviors. The #ifdefs and supporting setup in the config/ files and configure script has been removed, and the code executed when these options were not set removed from the source. In passing, this commit also cleans up some whitespace issues in both t_mpi.c and H5Dmpio.c. Furthermore, in t_mpi.c, the special collective IO test was not getting regularly run due to it being written to work only with four processes (we regularly test with six, previously with three), and thus it failed when actually run due to an out of bounds data buffer assignment. It has been modified to run at any number of processes greater than four, and the memory problem has been fixed so the test passes. Tested: jam, h5committest, ember
* [svn-r19959] Bug: the VRFY macro, for some reason, called MPI_Finalize() ↵Albert Cheng2011-01-161-0/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | when it encountered error and wanted to exit the test program. This was not good since if only a subset of processes called MPI_Finalize(), the other processes will likely hang. That happened in AIX that it would waited till the alarm signal to kill the processes. Definitely a waste of time. Solution: Changed it to call MPI_Abort. That showed another problem. HDF5 has setup atexit post-process to try to close unclose objects, release resources, etc. But if the MPI processes have encountered an error and has been aborted, it is not likely any more MPI calls can function properly. E.g., it would attempt to free some communicators in the HDF5 MPIO file handle. It would again hang. Solution: need to call H5dont_atexit() to disable any atexit post-processing. This must be done early, like before calling H5open. This is added to each parallel test main program. testphdf5.h: Changed macros VRFY and MESG. Added comments too. testphdf5.c: t_mpi.c: t_cache.c: t_shapesame.c: Added H5dont_atexit. Tested: h5committest.
* [svn-r17654] Purpose:Mike McGreevy2009-10-151-4/+4
| | | | | | | | | | | | Configure cache-val cleanup Description: Corrected some misnamed cache values in configure / config files. Tested: h5committest
* [svn-r17318] Did some code cleanup.Albert Cheng2009-08-111-2/+4
| | | | | | | | | | t_file.c: replace the old variable, color, with a more meaningfule name of is_old. t_mpi.c: use the official mpi_file_delete, instead of remove, to delete a file in MPI environment. Tested: Only jam in parallel as changes are trivial.
* [svn-r16560] Description:Quincey Koziol2009-03-101-8/+6
| | | | | | | | | | Remove another call to H5E_clear_stack() from within the library. Clean up lots of compiler warnings. Tested on: Mac OS X/32 10.5.6 (amazon) (followup on other platforms forthcoming)
* [svn-r15628] Description:Quincey Koziol2008-09-161-4/+4
| | | | | | | | | | | | | Remove trailing whitespace from C/C++ source files, with the following script: foreach f (*.[ch] *.cpp) sed 's/[[:blank:]]*$//' $f > sed.out && mv sed.out $f end Tested on: Mac OS X/32 10.5.5 (amazon) No need for h5committest, just whitespace changes...
* [svn-r13829] Some systems (only SGI Altix ProPack 4 discovered so far) ↵Raymond Lu2007-06-021-0/+40
| | | | | | | | | | | | doesn't return correct file size from MPI_File_get_size. Bypass this problem by replacing it with stat. Add an option --disable-mpi-size in configure to indicate this function doesn't work properly. Add a test in testpar/t_mpi.c, too. If it returns wrong file size, print out a warning. Tested on kagiso (parallel) because already tested the same change to v1.6 on several platforms (kagiso, cobalt, copper, and sol).
* [svn-r13253] Updated all C and C++ style source code files with the THG ↵Albert Cheng2007-02-071-2/+3
| | | | | | | | | copyright notice. Tested platform: Kagiso only since it is only a comment block change. If it works in one machine, it should work in all, I hope. Still need to check the parallel build on copper.
* [svn-r12440] Purpose:Quincey Koziol2006-06-271-9/+9
| | | | | | | | | | | Code cleanup Description: Trim trailing whitespace in Makefile.am and C/C++ source files to make diffing changes easier. Platforms tested: None necessary, whitespace only change
* [svn-r12044] Purpose:MuQun Yang2006-03-091-25/+235
| | | | | | | | | | | | | | | Add new tests Description: Collective IO doesn't work for some platforms/mpio packages when more than one process has no contributions to IO. Solution: 1. Add a collective IO test to verify the correctness of the library when more than one process has no contributions to IO. 2. Add the similar MPI-IO test in t_mpi to help us maintain in more platforms. Platforms tested: heping, mir, copper Misc. update:
* [svn-r11483] Purpose:Leon Arber2005-09-291-7/+7
| | | | | | | | | | | | | | | | bug fix. Description: The complex derived datatype test assumed that the fill value would be 0. This is not the case on all systems. Solution: Modified the test to check against a known value in the outbuf array, instead of the fill value. Platforms tested: heping and MCR. Misc. update:
* [svn-r11481] Purpose:Leon Arber2005-09-291-1/+1
| | | | | | | | | | | | | | | bug fix. Description: Fixed typo in a comment. The word "file" was supposed to be "fill." The explanation of how the complex derived datatype test works is much clearer now. Solution: Platforms tested: minor change. Misc. update:
* [svn-r11459] Purpose:MuQun Yang2005-09-221-2/+3
| | | | | | | | | | | | | | | | | | A bug fix Description: MPI_Status_IGNORE is treated as a NULL pointer for mpich 1.2.4 or similar MPI packages. It caused segmentation fault for MPI derived datatype test. Solution: Define MPI_STATUS status, and pass &status into MPI_File_read and MPI_File_write. Platforms tested: too trivial to test. Misc. update:
* [svn-r11458] Purpose:Albert Cheng2005-09-221-13/+30
| | | | | | | | | | | | | | | | | | Improvement. Description: The test may hang if there are system failures that some processors are not working. Solution: Added the ALARM calls to limit all tests be done with the default alarm time. So, even if a process is hanging, the ALARM signal would terminate the process. Platforms tested: tg-login2 of NCSA. Misc. update:
* [svn-r11292] Purpose:Albert Cheng2005-08-231-3/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | "bug fix" Description: The test_mpio_derived_dtype() often hangs when fails. So it was not run by default to avoid hanging the daily tests or confusing users. But then new system or new codes in collective chunk fails while one can't tell for sure if it is because of the complicated derived type failures or something else. Solution: Changed the logic so it is skipped only if it is known that the complicated MPI derived type does not work. (This is indicated by macro H5_MPI_COMPLEX_DERIVED_DATATYPE_WORKS NOT defined. Platforms tested: heping pp (where it is tested by default). Modi4 pp (where it is SKIPPED by default.) I also forced modi4 to test it and modi4 said it actually is working and should change the setting of H5_MPI_COMPLEX_DERIVED_DATATYPE_WORKS to working?!! Misc. update:
* [svn-r11291] Purpose:Albert Cheng2005-08-231-33/+32
| | | | | | | | | | | | | | | | | | Code cleaning. Description: The block of code that is conditioned by the H5_MPI_COMPLEX_DERIVED_DATATYPE_WORKS and manipulates the return code of test_mpio_derived_dtype, does not really belong in main. If return code of test_mpio_derived_dtype needs to be adjusted, it should be done in test_mpio_derived_dtype. Solution: Moved that block of code test_mpio_derived_dtype. Platforms tested: heping PP. Misc. update:
* [svn-r11245] Purpose:Quincey Koziol2005-08-131-30/+30
| | | | | | | | | | | | | | | | | | | | Code cleanup Description: Trim trailing whitespace, which is making 'diff'ing the two branches difficult. Solution: Ran this script in each directory: foreach f (*.[ch] *.cpp) sed 's/[[:blank:]]*$//' $f > sed.out && mv sed.out $f end Platforms tested: FreeBSD 4.11 (sleipnir) Too minor to require h5committest
* [svn-r10510] Purpose:Albert Cheng2005-03-301-31/+39
| | | | | | | | | | | | | | Improvement. Description: The derived datatype test often hangs when it fails. This blocks daily test or automatic build. Run it only when hi verbose mode is used. Platforms tested: Tested in eirene pp. Misc. update:
* [svn-r10390] Purpose:MuQun Yang2005-03-231-4/+60
| | | | | | | | | | | | | | Add detailed comments for MPI derived data type test. Description: Solution: Platforms tested: heping(Only comments and printf messages were added, no need to test on all platforms.) Misc. update:
* [svn-r10204] Purpose:MuQun Yang2005-03-111-0/+189
| | | | | | | | | | | | | | | | | | | | | | Provide a way to warn users on the usage of collective IO because of potential MPI-IO bugs for some platforms. Also provide a way for users to give us feedback if the vendor has already fixed the problem so that we can turn off the hard-code macro in our configure description file. Description: See above. Solution: Use a simple MPI complicated derived datatype program to check whether derived data type works for this MPI-IO package. Print out some messages to warn users if not working as we had expected. Platforms tested: AIX 5.1(copper) and Linux 2.4(heping) Misc. update:
* [svn-r10125] Purpose:Albert Cheng2005-03-031-1/+1
| | | | | | | | | | | | | | | Feature--to provide a standalone mode for t_mpi.c so that it can be built outside of PHDF5 environment. Description: Move definitions that are common to all parallel test programs to a new header file called testpar.h. Leave only Parallel HDF5 tests related definitions in testphdf5.h. Platforms tested: heping (pp) and modi4(PP). Copper was down. Misc. update:
* [svn-r9727] Purpose:Quincey Koziol2004-12-291-6/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bug Fix/Code Cleanup/Doc Cleanup/Optimization/Branch Sync :-) Description: Generally speaking, this is the "signed->unsigned" change to selections. However, in the process of merging code back, things got stickier and stickier until I ended up doing a big "sync the two branches up" operation. So... I brought back all the "infrastructure" fixes from the development branch to the release branch (which I think were actually making some improvement in performance) as well as fixed several bugs which had been fixed in one branch, but not the other. I've also tagged the repository before making this checkin with the label "before_signed_unsigned_changes". Platforms tested: FreeBSD 4.10 (sleipnir) w/parallel & fphdf5 FreeBSD 4.10 (sleipnir) w/threadsafe FreeBSD 4.10 (sleipnir) w/backward compatibility Solaris 2.7 (arabica) w/"purify options" Solaris 2.8 (sol) w/FORTRAN & C++ AIX 5.x (copper) w/parallel & FORTRAN IRIX64 6.5 (modi4) w/FORTRAN Linux 2.4 (heping) w/FORTRAN & C++ Misc. update:
* [svn-r9453] Purpose:Albert Cheng2004-10-221-1/+1
| | | | | | | | | | | | | Improvement. Description: Made all processes print hostname() by default so that it is easier to spot problems. Platforms tested: Tested in copper only. It is a trivial small change. Misc. update:
generated by cgit v0.12 at 2024-07-15 21:10:40 (GMT)