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 d7de532de5e6a24217fe34005552c165f1f8ee2c (commit) via c0b1427748378f0469f42232cdcf58b32db11c2b (commit) via 74a3e759e004e87a00212fbefa2d3d7455174ebf (commit) from b9ce079a6621faed2a8ac560af2fd8e0ab09a242 (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/d7de532de5e6a24217fe34005552c165f1... commit d7de532de5e6a24217fe34005552c165f1f8ee2c Author: Rob Latham <[email protected]> Date: Mon Apr 14 13:40:34 2014 +0000 Bugfix: detected any fs as gpfs if GPFS enabled, we were declaring all file systems GPFS. This holdover from the old BlueGene-specific days does not belong here. Signed-off-by: Paul Coffman <[email protected]> diff --git a/src/mpi/romio/adio/common/ad_fstype.c b/src/mpi/romio/adio/common/ad_fstype.c index e492b9f..f3f59e2 100644 --- a/src/mpi/romio/adio/common/ad_fstype.c +++ b/src/mpi/romio/adio/common/ad_fstype.c @@ -351,9 +351,10 @@ static void ADIO_FileSysType_fncall(const char *filename, int *fstype, int *erro #ifdef ROMIO_GPFS - - *fstype = ADIO_GPFS; - return; + if (fsbuf.f_type == GPFS_SUPER_MAGIC) { + *fstype = ADIO_GPFS; + return; + } #endif /* FPRINTF(stderr, "%d\n", fsbuf.f_type);*/ http://git.mpich.org/mpich.git/commitdiff/c0b1427748378f0469f42232cdcf58b32d... commit c0b1427748378f0469f42232cdcf58b32db11c2b Author: Rob Latham <[email protected]> Date: Mon Apr 14 13:38:50 2014 +0000 more comments for crazy ROMIO multi-platform stat 20 years of portability workarounds make for quite a tangle. Only one of the four methods is ever used in practice now. Provide comments about the older approaches and indicate which one we expect to be the common case. Signed-off-by: Paul Coffman <[email protected]> diff --git a/src/mpi/romio/adio/common/ad_fstype.c b/src/mpi/romio/adio/common/ad_fstype.c index cabf82d..e492b9f 100644 --- a/src/mpi/romio/adio/common/ad_fstype.c +++ b/src/mpi/romio/adio/common/ad_fstype.c @@ -253,6 +253,7 @@ static void ADIO_FileSysType_fncall(const char *filename, int *fstype, int *erro *error_code = MPI_SUCCESS; #ifdef ROMIO_HAVE_STRUCT_STATVFS_WITH_F_BASETYPE + /* rare: old solaris machines */ retry_cnt=0; do { err = statvfs(filename, &vfsbuf); @@ -311,6 +312,8 @@ static void ADIO_FileSysType_fncall(const char *filename, int *fstype, int *erro #endif /* STATVFS APPROACH */ #if defined(HAVE_STRUCT_STATFS) && defined(HAVE_STATFS) + /* common automagic fs-detection logic for any modern POSX-compliant + * environment */ retry_cnt = 0; do { err = statfs(filename, &fsbuf); @@ -339,6 +342,7 @@ static void ADIO_FileSysType_fncall(const char *filename, int *fstype, int *erro /* --END ERROR HANDLING-- */ # ifdef ROMIO_HAVE_STRUCT_STATFS_WITH_F_FSTYPENAME + /* uncommon: maybe only on Darwin ? */ if ( !strncmp("nfs",fsbuf.f_fstypename,3) ) { *fstype = ADIO_NFS; return; @@ -425,6 +429,7 @@ static void ADIO_FileSysType_fncall(const char *filename, int *fstype, int *erro #endif /* STATFS APPROACH */ #ifdef ROMIO_HAVE_STRUCT_STAT_WITH_ST_FSTYPE + /* rare: maybe old NEC SX or SGI IRIX machines */ retry_cnt = 0; do { err = stat(filename, &sbuf); http://git.mpich.org/mpich.git/commitdiff/74a3e759e004e87a00212fbefa2d3d7455... commit 74a3e759e004e87a00212fbefa2d3d7455174ebf Author: Rob Latham <[email protected]> Date: Mon Apr 14 13:36:56 2014 +0000 clean up stray declrations of GPFS_SUPER_MAGIC only ad_fstype.c uses GPFS_SUPER_MAGIC now. Also, GPFS_SUPER_MAGIC is properly provided by gpfs.h, but we can fall back to old way of declaring it ourselves if gpfs.h is not around. Signed-off-by: Paul Coffman <[email protected]> diff --git a/src/mpi/romio/adio/ad_gpfs/ad_gpfs_aggrs.h b/src/mpi/romio/adio/ad_gpfs/ad_gpfs_aggrs.h index e5b666e..4d0b0da 100644 --- a/src/mpi/romio/adio/ad_gpfs/ad_gpfs_aggrs.h +++ b/src/mpi/romio/adio/ad_gpfs/ad_gpfs_aggrs.h @@ -19,9 +19,6 @@ #include "adio.h" #include <sys/stat.h> -#if !defined(GPFS_SUPER_MAGIC) - #define GPFS_SUPER_MAGIC (0x47504653) -#endif /* overriding ADIOI_Calc_file_domains() to apply 'aligned file domain partitioning'. */ void ADIOI_GPFS_Calc_file_domains(ADIO_File fd, diff --git a/src/mpi/romio/adio/ad_gpfs/bg/ad_bg_aggrs.h b/src/mpi/romio/adio/ad_gpfs/bg/ad_bg_aggrs.h index cdd0497..d6baaed 100644 --- a/src/mpi/romio/adio/ad_gpfs/bg/ad_bg_aggrs.h +++ b/src/mpi/romio/adio/ad_gpfs/bg/ad_bg_aggrs.h @@ -20,9 +20,6 @@ #include "adio.h" #include <sys/stat.h> -#if !defined(GPFS_SUPER_MAGIC) - #define GPFS_SUPER_MAGIC (0x47504653) -#endif /* generate a list of I/O aggregators that utilizes BG-PSET orginization. */ int ADIOI_BG_gen_agg_ranklist(ADIO_File fd, int n_aggrs_per_pset); diff --git a/src/mpi/romio/adio/common/ad_fstype.c b/src/mpi/romio/adio/common/ad_fstype.c index 26ad614..cabf82d 100644 --- a/src/mpi/romio/adio/common/ad_fstype.c +++ b/src/mpi/romio/adio/common/ad_fstype.c @@ -30,6 +30,10 @@ #include "zoidfs.h" #endif +#ifdef HAVE_GPFS_H +#include "gpfs.h" +#endif + /* Notes on detection process: * * There are three more "general" mechanisms that we use for detecting @@ -74,6 +78,10 @@ #define PVFS2_SUPER_MAGIC (0x20030528) #endif +#if defined(ROMIO_GPFS) && !defined(GPFS_SUPER_MAGIC) +# define GPFS_SUPER_MAGIC 0x47504653 +#endif + #ifdef ROMIO_HAVE_STRUCT_STATVFS_WITH_F_BASETYPE # ifdef HAVE_SYS_STATVFS_H # include <sys/statvfs.h> ----------------------------------------------------------------------- Summary of changes: src/mpi/romio/adio/ad_gpfs/ad_gpfs_aggrs.h | 3 --- src/mpi/romio/adio/ad_gpfs/bg/ad_bg_aggrs.h | 3 --- src/mpi/romio/adio/common/ad_fstype.c | 20 +++++++++++++++++--- 3 files changed, 17 insertions(+), 9 deletions(-) hooks/post-receive -- MPICH primary repository