- commits - mailman3.cels.anl.gov

[mpich] MPICH primary repository branch, master, updated. v3.0.2-6-g2bdb181
by noreply＠mpich.org 05 Feb '13

05 Feb '13

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 2bdb181f2ff8607c3cfa6f7d306e94f604dc94ba (commit) from d8eec5491fa0929d43da3086230795a2fb1c6545 (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/2bdb181f2ff8607c3cfa6f7d306e94f60… commit 2bdb181f2ff8607c3cfa6f7d306e94f604dc94ba Author: Dave Goodell <goodell(a)mcs.anl.gov> Date: Tue Feb 5 10:53:55 2013 -0600 explain external32 limitations in RELEASE_NOTES references ticket #1754. No reviewer. diff --git a/RELEASE_NOTES b/RELEASE_NOTES index 4d44831..49f2e7e 100644 --- a/RELEASE_NOTES +++ b/RELEASE_NOTES @@ -36,7 +36,14 @@ * Support for "external32" data representation is incomplete. This affects the MPI_Pack_external and MPI_Unpack_external routines, as - well the external data representation capabilities of ROMIO. + well the external data representation capabilities of ROMIO. In + particular: noncontiguous user buffers could consume egregious + amounts of memory in the MPI library and any types which vary in + width between the native representation and the external32 + representation will likely cause corruption. The following ticket + contains some additional information: + + http://trac.mpich.org/projects/mpich/ticket/1754 * ch3 has known problems in some cases when threading and dynamic processes are used together on communicators of size greater than ----------------------------------------------------------------------- Summary of changes: RELEASE_NOTES | 9 ++++++++- 1 files changed, 8 insertions(+), 1 deletions(-) hooks/post-receive -- MPICH primary repository

1 0

[mpich] MPICH primary repository branch, master, updated. v3.0.2-5-gd8eec54
by noreply＠mpich.org 05 Feb '13

05 Feb '13

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 d8eec5491fa0929d43da3086230795a2fb1c6545 (commit) from 0b3640682130339f69e5391560d39b771c04bad4 (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/d8eec5491fa0929d43da3086230795a2f… commit d8eec5491fa0929d43da3086230795a2fb1c6545 Author: Dave Goodell <goodell(a)mcs.anl.gov> Date: Mon Feb 4 23:14:50 2013 -0600 tt#1754: fix warnings in ROMIO external32 code This doesn't fix any of the serious bugs or inefficiencies present in the current external32 implementation. But it at least fixes some very valid warnings related to const-ness and passing incorrect pointer types. References ticket #1754 Reviewed-by: robl diff --git a/src/mpi/romio/adio/common/error.c b/src/mpi/romio/adio/common/error.c index ddf78be..15f2376 100644 --- a/src/mpi/romio/adio/common/error.c +++ b/src/mpi/romio/adio/common/error.c @@ -38,7 +38,7 @@ int ADIOI_Error(ADIO_File fd, int error_code, char *string) /* Returns an MPI error code corresponding to "my_errno", for function "myname" * and the given file, "filename". */ -int ADIOI_Err_create_code(char *myname, char *filename, int my_errno) +int ADIOI_Err_create_code(const char *myname, const char *filename, int my_errno) { int error_code = MPI_SUCCESS; if(!my_errno) return MPI_SUCCESS; diff --git a/src/mpi/romio/adio/include/adioi.h b/src/mpi/romio/adio/include/adioi.h index d430da8..063b39c 100644 --- a/src/mpi/romio/adio/include/adioi.h +++ b/src/mpi/romio/adio/include/adioi.h @@ -561,7 +561,7 @@ int ADIOI_End_call(MPI_Comm comm, int keyval, void *attribute_val, void *extra_s int MPIR_Status_set_bytes(MPI_Status *status, MPI_Datatype datatype, int nbytes); int ADIOI_Uses_generic_read(ADIO_File fd); int ADIOI_Uses_generic_write(ADIO_File fd); -int ADIOI_Err_create_code(char *myname, char *filename, int my_errno); +int ADIOI_Err_create_code(const char *myname, const char *filename, int my_errno); int ADIOI_FAKE_IODone(ADIO_Request *request, ADIO_Status *status, diff --git a/src/mpi/romio/adio/include/mpiu_external32.h b/src/mpi/romio/adio/include/mpiu_external32.h index eb5eec7..4302373 100644 --- a/src/mpi/romio/adio/include/mpiu_external32.h +++ b/src/mpi/romio/adio/include/mpiu_external32.h @@ -8,7 +8,7 @@ #ifndef _MPIU_EXTERNAL32_H_ #define _MPIU_EXTERNAL32_H_ -int MPIU_write_external32_conversion_fn (void *userbuf, MPI_Datatype datatype, +int MPIU_write_external32_conversion_fn (const void *userbuf, MPI_Datatype datatype, int count, void *filebuf); int MPIU_read_external32_conversion_fn(void *userbuf, MPI_Datatype datatype, int count, void *filebuf); @@ -17,6 +17,6 @@ int MPIU_datatype_full_size(MPI_Datatype datatype, MPI_Aint *size); /* given a buffer, count, and datatype, return an apropriately sized and * * external32-formatted buffer, suitable for handing off to a subsequent write * * routine */ -int MPIU_external32_buffer_setup(void * buf, int count, MPI_Datatype type, void **newbuf); +int MPIU_external32_buffer_setup(const void * buf, int count, MPI_Datatype type, void **newbuf); #endif diff --git a/src/mpi/romio/mpi-io/mpiu_external32.c b/src/mpi/romio/mpi-io/mpiu_external32.c index 8e9d467..75b4c27 100644 --- a/src/mpi/romio/mpi-io/mpiu_external32.c +++ b/src/mpi/romio/mpi-io/mpiu_external32.c @@ -12,7 +12,7 @@ #include "mpioprof.h" #endif -int MPIU_write_external32_conversion_fn (void *userbuf, MPI_Datatype datatype, +int MPIU_write_external32_conversion_fn (const void *userbuf, MPI_Datatype datatype, int count, void *filebuf) { int position_i = 0; @@ -136,7 +136,7 @@ fn_exit: /* given a buffer, count, and datatype, return an apropriately allocated, sized * and external32-formatted buffer, suitable for handing off to a subsequent * write routine. Caller is responsible for freeing 'newbuf' */ -int MPIU_external32_buffer_setup(void * buf, int count, MPI_Datatype type, void **newbuf) +int MPIU_external32_buffer_setup(const void * buf, int count, MPI_Datatype type, void **newbuf) { MPI_Aint datatype_size=0, bytes=0; diff --git a/src/mpi/romio/mpi-io/read.c b/src/mpi/romio/mpi-io/read.c index c7fb25d..029db3b 100644 --- a/src/mpi/romio/mpi-io/read.c +++ b/src/mpi/romio/mpi-io/read.c @@ -124,11 +124,12 @@ int MPIOI_File_read(MPI_File fh, xbuf = buf; if (adio_fh->is_external32) { - error_code = MPIU_datatype_full_size(datatype, &datatype_size); + MPI_Aint e32_size = 0; + error_code = MPIU_datatype_full_size(datatype, &e32_size); if (error_code != MPI_SUCCESS) goto fn_exit; - e32_buf = ADIOI_Malloc(datatype_size*count); + e32_buf = ADIOI_Malloc(e32_size*count); xbuf = e32_buf; } diff --git a/src/mpi/romio/mpi-io/read_all.c b/src/mpi/romio/mpi-io/read_all.c index 8d412a3..b387fe9 100644 --- a/src/mpi/romio/mpi-io/read_all.c +++ b/src/mpi/romio/mpi-io/read_all.c @@ -107,11 +107,12 @@ int MPIOI_File_read_all(MPI_File fh, xbuf = buf; if (adio_fh->is_external32) { - error_code = MPIU_datatype_full_size(datatype, &datatype_size); + MPI_Aint e32_size = 0; + error_code = MPIU_datatype_full_size(datatype, &e32_size); if (error_code != MPI_SUCCESS) goto fn_exit; - e32_buf = ADIOI_Malloc(datatype_size*count); + e32_buf = ADIOI_Malloc(e32_size*count); xbuf = e32_buf; } diff --git a/src/mpi/romio/mpi-io/read_allb.c b/src/mpi/romio/mpi-io/read_allb.c index 2ba4b3a..ecc9088 100644 --- a/src/mpi/romio/mpi-io/read_allb.c +++ b/src/mpi/romio/mpi-io/read_allb.c @@ -104,11 +104,12 @@ int MPIOI_File_read_all_begin(MPI_File fh, xbuf = buf; if (adio_fh->is_external32) { - error_code = MPIU_datatype_full_size(datatype, &datatype_size); + MPI_Aint e32_size = 0; + error_code = MPIU_datatype_full_size(datatype, &e32_size); if (error_code != MPI_SUCCESS) goto fn_exit; - e32_buf = ADIOI_Malloc(datatype_size*count); + e32_buf = ADIOI_Malloc(e32_size*count); xbuf = e32_buf; } diff --git a/src/mpi/romio/mpi-io/read_ordb.c b/src/mpi/romio/mpi-io/read_ordb.c index 6944e68..324fa7f 100644 --- a/src/mpi/romio/mpi-io/read_ordb.c +++ b/src/mpi/romio/mpi-io/read_ordb.c @@ -102,11 +102,12 @@ int MPI_File_read_ordered_begin(MPI_File fh, void *buf, int count, xbuf = buf; if (adio_fh->is_external32) { - error_code = MPIU_datatype_full_size(datatype, &datatype_size); + MPI_Aint e32_size = 0; + error_code = MPIU_datatype_full_size(datatype, &e32_size); if (error_code != MPI_SUCCESS) goto fn_exit; - e32_buf = ADIOI_Malloc(datatype_size*count); + e32_buf = ADIOI_Malloc(e32_size*count); xbuf = e32_buf; } diff --git a/src/mpi/romio/mpi-io/read_sh.c b/src/mpi/romio/mpi-io/read_sh.c index 8ee0e63..1536838 100644 --- a/src/mpi/romio/mpi-io/read_sh.c +++ b/src/mpi/romio/mpi-io/read_sh.c @@ -99,11 +99,12 @@ int MPI_File_read_shared(MPI_File fh, void *buf, int count, xbuf = buf; if (adio_fh->is_external32) { - error_code = MPIU_datatype_full_size(datatype, &datatype_size); + MPI_Aint e32_size = 0; + error_code = MPIU_datatype_full_size(datatype, &e32_size); if (error_code != MPI_SUCCESS) goto fn_exit; - e32_buf = ADIOI_Malloc(datatype_size*count); + e32_buf = ADIOI_Malloc(e32_size*count); xbuf = e32_buf; } diff --git a/src/mpi/romio/mpi-io/write.c b/src/mpi/romio/mpi-io/write.c index 4221eb3..02867c2 100644 --- a/src/mpi/romio/mpi-io/write.c +++ b/src/mpi/romio/mpi-io/write.c @@ -74,7 +74,8 @@ int MPIOI_File_write(MPI_File fh, int datatype_size; ADIO_Offset off; ADIO_File adio_fh; - void *e32buf=NULL, *xbuf=NULL; + void *e32buf=NULL; + const void *xbuf=NULL; MPIU_THREAD_CS_ENTER(ALLFUNC,); @@ -121,7 +122,7 @@ int MPIOI_File_write(MPI_File fh, ADIOI_TEST_DEFERRED(adio_fh, myname, &error_code); - xbuf = (void *)buf; + xbuf = buf; if (adio_fh->is_external32) { error_code = MPIU_external32_buffer_setup(buf, count, datatype, &e32buf); if (error_code != MPI_SUCCESS) diff --git a/src/mpi/romio/mpi-io/write_all.c b/src/mpi/romio/mpi-io/write_all.c index 28cd31a..2e1533c 100644 --- a/src/mpi/romio/mpi-io/write_all.c +++ b/src/mpi/romio/mpi-io/write_all.c @@ -74,7 +74,8 @@ int MPIOI_File_write_all(MPI_File fh, { int error_code, datatype_size; ADIO_File adio_fh; - void *e32buf=NULL, *xbuf=NULL; + void *e32buf=NULL; + const void *xbuf=NULL; MPIU_THREAD_CS_ENTER(ALLFUNC,); @@ -104,7 +105,7 @@ int MPIOI_File_write_all(MPI_File fh, MPIO_CHECK_COUNT_SIZE(adio_fh, count, datatype_size, myname, error_code); /* --END ERROR HANDLING-- */ - xbuf = (void *)buf; + xbuf = buf; if (adio_fh->is_external32) { error_code = MPIU_external32_buffer_setup(buf, count, datatype, &e32buf); if (error_code != MPI_SUCCESS) diff --git a/src/mpi/romio/mpi-io/write_allb.c b/src/mpi/romio/mpi-io/write_allb.c index 3ee56ca..21e928e 100644 --- a/src/mpi/romio/mpi-io/write_allb.c +++ b/src/mpi/romio/mpi-io/write_allb.c @@ -60,7 +60,8 @@ int MPIOI_File_write_all_begin(MPI_File fh, { int error_code, datatype_size; ADIO_File adio_fh; - void *e32buf=NULL, *xbuf=NULL; + void *e32buf=NULL; + const void *xbuf=NULL; MPIU_THREAD_CS_ENTER(ALLFUNC,); @@ -100,7 +101,7 @@ int MPIOI_File_write_all_begin(MPI_File fh, /* --END ERROR HANDLING-- */ - xbuf = (void *)buf; + xbuf = buf; if (adio_fh->is_external32) { error_code = MPIU_external32_buffer_setup(buf, count, datatype, &e32buf); if (error_code != MPI_SUCCESS) diff --git a/src/mpi/romio/mpi-io/write_ord.c b/src/mpi/romio/mpi-io/write_ord.c index 2dc7b91..d0fe63f 100644 --- a/src/mpi/romio/mpi-io/write_ord.c +++ b/src/mpi/romio/mpi-io/write_ord.c @@ -48,7 +48,8 @@ int MPI_File_write_ordered(MPI_File fh, const void *buf, int count, static char myname[] = "MPI_FILE_WRITE_ORDERED"; ADIO_Offset shared_fp; ADIO_File adio_fh; - void *e32buf=NULL, *xbuf; + void *e32buf=NULL; + const void *xbuf; MPIU_THREAD_CS_ENTER(ALLFUNC,); diff --git a/src/mpi/romio/mpi-io/write_ordb.c b/src/mpi/romio/mpi-io/write_ordb.c index f11b1df..87ee213 100644 --- a/src/mpi/romio/mpi-io/write_ordb.c +++ b/src/mpi/romio/mpi-io/write_ordb.c @@ -44,7 +44,8 @@ int MPI_File_write_ordered_begin(MPI_File fh, const void *buf, int count, static char myname[] = "MPI_FILE_WRITE_ORDERED_BEGIN"; ADIO_Offset shared_fp; ADIO_File adio_fh; - void *e32buf = NULL, *xbuf=NULL; + void *e32buf = NULL; + const void *xbuf=NULL; MPIU_THREAD_CS_ENTER(ALLFUNC,); diff --git a/src/mpi/romio/mpi-io/write_sh.c b/src/mpi/romio/mpi-io/write_sh.c index 23f81b7..de7d21d 100644 --- a/src/mpi/romio/mpi-io/write_sh.c +++ b/src/mpi/romio/mpi-io/write_sh.c @@ -47,7 +47,8 @@ int MPI_File_write_shared(MPI_File fh, const void *buf, int count, int datatype_size, incr; ADIO_Offset off, shared_fp; ADIO_File adio_fh; - void *e32buf = NULL, *xbuf = NULL; + void *e32buf = NULL; + const void *xbuf = NULL; MPIU_THREAD_CS_ENTER(ALLFUNC,); ----------------------------------------------------------------------- Summary of changes: src/mpi/romio/adio/common/error.c | 2 +- src/mpi/romio/adio/include/adioi.h | 2 +- src/mpi/romio/adio/include/mpiu_external32.h | 4 ++-- src/mpi/romio/mpi-io/mpiu_external32.c | 4 ++-- src/mpi/romio/mpi-io/read.c | 5 +++-- src/mpi/romio/mpi-io/read_all.c | 5 +++-- src/mpi/romio/mpi-io/read_allb.c | 5 +++-- src/mpi/romio/mpi-io/read_ordb.c | 5 +++-- src/mpi/romio/mpi-io/read_sh.c | 5 +++-- src/mpi/romio/mpi-io/write.c | 5 +++-- src/mpi/romio/mpi-io/write_all.c | 5 +++-- src/mpi/romio/mpi-io/write_allb.c | 5 +++-- src/mpi/romio/mpi-io/write_ord.c | 3 ++- src/mpi/romio/mpi-io/write_ordb.c | 3 ++- src/mpi/romio/mpi-io/write_sh.c | 3 ++- 15 files changed, 36 insertions(+), 25 deletions(-) hooks/post-receive -- MPICH primary repository

1 0

[mpich] MPICH primary repository branch, master, updated. v3.0.2-4-g0b36406
by noreply＠mpich.org 05 Feb '13

05 Feb '13

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 0b3640682130339f69e5391560d39b771c04bad4 (commit) from f71dd6e42390021aaf91814008e7208ad3e65f64 (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/0b3640682130339f69e5391560d39b771… commit 0b3640682130339f69e5391560d39b771c04bad4 Author: James Dinan <dinan(a)mcs.anl.gov> Date: Tue Feb 5 08:43:47 2013 -0600 BUGFIX: Unlock piggybacking for Get_accumulate GACC operations were both piggybacking the unlock message to the origin, and sending back a PT done packet. This was causing the origin to be unlocked twice. When another lock operation was performed between the GACC and PT done unlock operations, there was a synchronization race. diff --git a/src/mpid/ch3/src/ch3u_handle_recv_req.c b/src/mpid/ch3/src/ch3u_handle_recv_req.c index 2eaa530..2352da7 100644 --- a/src/mpid/ch3/src/ch3u_handle_recv_req.c +++ b/src/mpid/ch3/src/ch3u_handle_recv_req.c @@ -82,6 +82,7 @@ int MPIDI_CH3_ReqHandler_PutAccumRespComplete( MPIDI_VC_t *vc, int *complete ) { int mpi_errno = MPI_SUCCESS; + int is_gacc_op = 0; MPID_Win *win_ptr; MPIU_CHKPMEM_DECL(1); MPIDI_STATE_DECL(MPID_STATE_MPIDI_CH3_REQHANDLER_PUTACCUMRESPCOMPLETE); @@ -96,6 +97,7 @@ int MPIDI_CH3_ReqHandler_PutAccumRespComplete( MPIDI_VC_t *vc, MPID_Request *resp_req; MPID_IOV iov[MPID_IOV_LIMIT]; + is_gacc_op = 1; MPIDI_Pkt_init(get_accum_resp_pkt, MPIDI_CH3_PKT_GET_ACCUM_RESP); get_accum_resp_pkt->request_handle = rreq->dev.resp_request_handle; @@ -170,7 +172,7 @@ int MPIDI_CH3_ReqHandler_PutAccumRespComplete( MPIDI_VC_t *vc, win_ptr->my_counter -= 1; } else { - if ((win_ptr->current_lock_type == MPI_LOCK_SHARED) || + if ((win_ptr->current_lock_type == MPI_LOCK_SHARED && !is_gacc_op) || (rreq->dev.single_op_opt == 1)) { mpi_errno = MPIDI_CH3I_Send_pt_rma_done_pkt(vc, win_ptr, rreq->dev.source_win_handle); diff --git a/src/mpid/ch3/src/ch3u_rma_sync.c b/src/mpid/ch3/src/ch3u_rma_sync.c index 3ffb73e..6ce857a 100644 --- a/src/mpid/ch3/src/ch3u_rma_sync.c +++ b/src/mpid/ch3/src/ch3u_rma_sync.c @@ -4710,6 +4710,7 @@ int MPIDI_CH3_PktHandler_PtRMADone( MPIDI_VC_t *vc, MPIDI_CH3_Pkt_t *pkt, *buflen = sizeof(MPIDI_CH3_Pkt_t); MPID_Win_get_ptr(pt_rma_done_pkt->source_win_handle, win_ptr); + MPIU_Assert(win_ptr->targets[pt_rma_done_pkt->target_rank].remote_lock_state != MPIDI_CH3_WIN_LOCK_NONE); if (win_ptr->targets[pt_rma_done_pkt->target_rank].remote_lock_state == MPIDI_CH3_WIN_LOCK_FLUSH) win_ptr->targets[pt_rma_done_pkt->target_rank].remote_lock_state = MPIDI_CH3_WIN_LOCK_GRANTED; ----------------------------------------------------------------------- Summary of changes: src/mpid/ch3/src/ch3u_handle_recv_req.c | 4 +++- src/mpid/ch3/src/ch3u_rma_sync.c | 1 + 2 files changed, 4 insertions(+), 1 deletions(-) hooks/post-receive -- MPICH primary repository

1 0

[mpich] MPICH primary repository branch, master, updated. v3.0.2-3-gf71dd6e
by noreply＠mpich.org 04 Feb '13

04 Feb '13

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 f71dd6e42390021aaf91814008e7208ad3e65f64 (commit) from 34249593d1229da56c14beaa5fb8287b70aab08e (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/f71dd6e42390021aaf91814008e7208ad… commit f71dd6e42390021aaf91814008e7208ad3e65f64 Author: Rob Latham <robl(a)mcs.anl.gov> Date: Tue Jan 22 08:33:20 2013 -0600 feature-encode NTFS quirk NTFS does not have the POSIX "file exists until everyone closes it" property, so introduce the feature mechanism to NTFS and let ADIO know NTFS cannot rely on that property. diff --git a/src/mpi/romio/adio/ad_ntfs/ad_ntfs.c b/src/mpi/romio/adio/ad_ntfs/ad_ntfs.c index 696ba85..ff95110 100644 --- a/src/mpi/romio/adio/ad_ntfs/ad_ntfs.c +++ b/src/mpi/romio/adio/ad_ntfs/ad_ntfs.c @@ -34,5 +34,5 @@ struct ADIOI_Fns_struct ADIO_NTFS_operations = { ADIOI_NTFS_Flush, /* Flush */ ADIOI_NTFS_Resize, /* Resize */ ADIOI_GEN_Delete, /* Delete */ - ADIOI_GEN_Feature /* Features */ + ADIOI_NTFS_Feature /* Features */ }; diff --git a/src/mpi/romio/adio/ad_ntfs/ad_ntfs_feature.c b/src/mpi/romio/adio/ad_ntfs/ad_ntfs_feature.c new file mode 100644 index 0000000..691c224 --- /dev/null +++ b/src/mpi/romio/adio/ad_ntfs/ad_ntfs_feature.c @@ -0,0 +1,20 @@ +#include "adio.h" + +int ADIOI_NTFS_Feature(ADIO_File fd, int flag) +{ + switch(flag) { + /* supported features */ + case ADIO_LOCKS: + case ADIO_SHARED_FP: + case ADIO_ATOMIC_MODE: + case ADIO_DATA_SIEVING_WRITES: + return 1; + break; + /* unsupported features */ + case ADIO_SCALABLE_OPEN: + case ADIO_UNLINK_AFTER_CLOSE: + default: + return 0; + break; + } +} ----------------------------------------------------------------------- Summary of changes: src/mpi/romio/adio/ad_ntfs/ad_ntfs.c | 2 +- .../ad_features.c => ad_ntfs/ad_ntfs_feature.c} | 6 ++++-- 2 files changed, 5 insertions(+), 3 deletions(-) copy src/mpi/romio/adio/{common/ad_features.c => ad_ntfs/ad_ntfs_feature.c} (70%) hooks/post-receive -- MPICH primary repository

1 0

[mpich] MPICH primary repository branch, master, updated. v3.0.2-2-g3424959
by noreply＠mpich.org 04 Feb '13

04 Feb '13

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 34249593d1229da56c14beaa5fb8287b70aab08e (commit) from 7e56bfc9f79ce102c7b907e7397efa1a278bfd74 (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/34249593d1229da56c14beaa5fb8287b7… commit 34249593d1229da56c14beaa5fb8287b70aab08e Author: James Dinan <dinan(a)mcs.anl.gov> Date: Mon Feb 4 12:37:37 2013 -0600 Fix type incompat. for size_t in BSend degug msgs This patch updates BSend debugging messages to be compatible with new size_t arguments added in [7fa7e2cf]. Format strings are updated and size_t args are cast to unsigned long long. Upconverting to the largest unsinged integer type is the only safe way to deal with this in pre-C99. If we were coding to C99, we could have avoided the type cast and used '%z' directly in the format strings. This closes ticket #1786. Reviewer: goodell diff --git a/src/mpi/pt2pt/bsendutil.c b/src/mpi/pt2pt/bsendutil.c index 1776b1b..6c2dda0 100644 --- a/src/mpi/pt2pt/bsendutil.c +++ b/src/mpi/pt2pt/bsendutil.c @@ -334,11 +334,12 @@ static void MPIR_Bsend_free_segment( MPIR_Bsend_data_t *p ) MPIR_Bsend_data_t *prev = p->prev, *avail = BsendBuffer.avail, *avail_prev; MPIU_DBG_MSG_FMT(BSEND,TYPICAL,(MPIU_DBG_FDEST, - "Freeing bsend segment at %p of size %d, next at %p", - p,p->size, ((char *)p)+p->total_size)); + "Freeing bsend segment at %p of size %llu, next at %p", + p, (unsigned long long) p->size, ((char *)p)+p->total_size)); MPIU_DBG_MSG_D(BSEND,TYPICAL, - "At the begining of free_segment with size %d:", p->total_size ); + "At the begining of free_segment with size %llu:", + (unsigned long long) p->total_size ); MPIU_DBG_STMT(BSEND,TYPICAL,MPIR_Bsend_dump()); /* Remove the segment from the active list */ @@ -530,8 +531,8 @@ static void MPIR_Bsend_take_buffer( MPIR_Bsend_data_t *p, int size ) allocate for this buffer. */ MPIU_DBG_MSG_FMT(BSEND,TYPICAL,(MPIU_DBG_FDEST, - "Taking %d bytes from a block with %d bytes\n", - alloc_size, p->total_size )); + "Taking %d bytes from a block with %llu bytes\n", + alloc_size, (unsigned long long) p->total_size )); /* Is there enough space left to create a new block? */ if (alloc_size + (int)BSENDDATA_HEADER_TRUE_SIZE + MIN_BUFFER_BLOCK <= p->size) { @@ -559,8 +560,9 @@ static void MPIR_Bsend_take_buffer( MPIR_Bsend_data_t *p, int size ) p->size = p->total_size - BSENDDATA_HEADER_TRUE_SIZE; MPIU_DBG_MSG_FMT(BSEND,TYPICAL,(MPIU_DBG_FDEST, - "broken blocks p (%d) and new (%d)\n", - p->total_size, newp->total_size )); + "broken blocks p (%llu) and new (%llu)\n", + (unsigned long long) p->total_size, + (unsigned long long) newp->total_size )); } /* Remove p from the avail list and add it to the active list */ @@ -610,12 +612,13 @@ static void MPIR_Bsend_dump( void ) { MPIR_Bsend_data_t *a = BsendBuffer.avail; - MPIU_DBG_MSG_D(BSEND,TYPICAL,"Total size is %d",BsendBuffer.buffer_size ); + MPIU_DBG_MSG_D(BSEND, TYPICAL, "Total size is %llu", + (unsigned long long) BsendBuffer.buffer_size); MPIU_DBG_MSG(BSEND,TYPICAL,"Avail list is:" ); while (a) { - MPIU_DBG_MSG_FMT(BSEND,TYPICAL,(MPIU_DBG_FDEST, - "[%p] totalsize = %d(%x)", a, a->total_size, - a->total_size )); + MPIU_DBG_MSG_FMT(BSEND, TYPICAL, (MPIU_DBG_FDEST, "[%p] totalsize = %llu(%llx)", + a, (unsigned long long) a->total_size, + (unsigned long long) a->total_size)); if (a == a->next) { MPIU_DBG_MSG(BSEND,TYPICAL, "@@@Corrupt list; avail block points at itself" ); @@ -627,9 +630,9 @@ static void MPIR_Bsend_dump( void ) MPIU_DBG_MSG(BSEND,TYPICAL,"Active list is:" ); a = BsendBuffer.active; while (a) { - MPIU_DBG_MSG_FMT(BSEND,TYPICAL,(MPIU_DBG_FDEST, - "[%p] totalsize = %d(%x)", a, a->total_size, - a->total_size )); + MPIU_DBG_MSG_FMT(BSEND, TYPICAL, (MPIU_DBG_FDEST, "[%p] totalsize = %llu(%llx)", + a, (unsigned long long) a->total_size, + (unsigned long long) a->total_size)); if (a == a->next) { MPIU_DBG_MSG(BSEND,TYPICAL, "@@@Corrupt list; active block points at itself" ); ----------------------------------------------------------------------- Summary of changes: src/mpi/pt2pt/bsendutil.c | 31 +++++++++++++++++-------------- 1 files changed, 17 insertions(+), 14 deletions(-) hooks/post-receive -- MPICH primary repository

1 0

[mpich] MPICH primary repository branch, master, updated. v3.0.2-1-g7e56bfc
by noreply＠mpich.org 31 Jan '13

31 Jan '13

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 7e56bfc9f79ce102c7b907e7397efa1a278bfd74 (commit) from faa18dd1b7ce815aa57f4a93c8da6e409a303735 (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/7e56bfc9f79ce102c7b907e7397efa1a2… commit 7e56bfc9f79ce102c7b907e7397efa1a278bfd74 Author: Dave Goodell <goodell(a)mcs.anl.gov> Date: Thu Jan 31 14:37:22 2013 -0600 clmake: support more patterns diff --git a/maint/clmake.in b/maint/clmake.in index d12ea06..726675c 100644 --- a/maint/clmake.in +++ b/maint/clmake.in @@ -38,6 +38,7 @@ $rootSrcDir = ""; "[A-Za-z0-9_\/-]*\/icc", "[A-Za-z0-9_\/-]*\/install-sh", "/usr/bin/ar", "mkdir", + "/bin/mkdir", "/usr/ucb/ld", "mpicc", "mpicxx", "compiling ROMIO in", @@ -57,6 +58,7 @@ $rootSrcDir = ""; "Copying Upshot", "Cleaning directory", "[*][*][*][*] Making .*\.\.\.\.", + "Making .* in .*", "Making upshot", "\\s*\$", ); # OtherNoise is an array of patterns that describe blocks of ----------------------------------------------------------------------- Summary of changes: maint/clmake.in | 2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) hooks/post-receive -- MPICH primary repository

1 0

[mpich] MPICH primary repository annotated tag, v3.0.2, created. v3.0.2
by noreply＠mpich.org 31 Jan '13

31 Jan '13

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 annotated tag, v3.0.2 has been created at f632b19f4626d5d41b096fb4a07108617d921511 (tag) tagging faa18dd1b7ce815aa57f4a93c8da6e409a303735 (commit) replaces v3.0.1 tagged by Dave Goodell on Wed Jan 30 17:37:11 2013 -0600 - Log ----------------------------------------------------------------- tagging 'v3.0.2' Darius Buntinas (4): Added gtags files to .gitignore Merge branch 'master' of git.mpich.org:mpich Replaced printf with DBG_MSG added DBG message for memcopy Dave Goodell (50): delete "relative" svn:externals dirs delete src/openpa before re-adding via git-subtree Squashed 'src/openpa/' content from commit b48b33c Merge commit 'db40bb8aa2168ffad8f9a681723dff9f8d6c426c' as 'src/openpa' Merge branch 'openpa-subtree' into post-svn-fixes add root-level .gitignore add hydra .gitignore remove SVN version check from autogen.sh convert release.pl to use git instead of svn use _LAYOUT_COMPATIBLE macros for MPI pair-types guard mpi.h clang annotation types fix automake-1.13+ recursive var warning update hydra .gitignore for automake-1.13+ update openpa .gitignore for automake-1.13+ tt#1776: fix platforms missing LLONG_MAX tt#1763: IN_PLACE checking for MPI_Reduce_local fixup for b31a0a6 (reduce_local error test) add a .mailmap file autogen.sh: use `cp -pPR` instead of `cp -a` check accumulate datatype-op compatibility release.pl: more updates for git autogen.sh: just copy "externals" release.pl: properly use "git describe" Revert "check accumulate datatype-op compatibility" clmake: screen more silent-rules patterns add preliminary TAP support index TAP test lines release.pl: chdir out of $tdir runtests: improve TAP output format tt#1784: fix MPL_VG_ macros with valgrind 3.6.0 tt#1785: NBC progress issues with ch3:sock tt#1785: add a symbol for configure testing tt#1786: set MPID_MAX_THREAD_LEVEL in ch3:sock correctly fix TAP output when test prog fails to build TAP: add "# SKIP" directive support runtests: permit arbitrary capitalization in args TAP: add "xfail" support add `xfail=ticketXYZ` annotations to failing tests move MPI_T tests to their own subdir in "manual" fixup for new pvar build system code move new MPI_T tests to manual/mpi_t dir fixups for new MPI_T tests always define stat vars Merge branch 'nemesis-mpit' TAP: missing newline in test plan line TAP: delete summary.tap in "make clean" mark perf/dtpack as xfail update CHANGES for the 3.0.2 release update version.m4 for 3.0.2 release update RELEASE_NOTES for the 3.0.2 release David Goodell (5): [svn-r10799] ROMIO: check whether datatypes are committed [svn-r10800] ROMIO: check for consistent prealloc sizes [svn-r10801] ROMIO: error checking for MPI_Comm and MPI_Info objects [svn-r10820] fix dylib creation on OS X if fortran is disabled [svn-r10821] tt#1771: allow `+` in `--prefix=`, etc. James Dinan (15): [svn-r10808] Removed stale mpix header inclusion in test case [svn-r10809] Removed duplicate declaration from mpi.h.in Linked list construction benchmark test cases Renamed MPI_MUTEX_TAG to MPIX_MUTEX_TAG Removed unnecessary goto statements Implemented interprocess shared memory RMA ops Sock implementation of interprocess shm mutex Shr. Mem. RMA: Flush is now a nop RMA test case: MCS mutex lock/unlock Updated MCS mutex test to also use shr. mem win. Fixed error checks in shr. mem. win. creation Minor cleanup to shr. mem. win error checks Fix error handling bug for struct datatypes Fix poor error handling in RMA code Updated CHANGES for 3.0.2 release Pavan Balaji (9): [svn-r10804] Missed out some mpich2 --> mpich renames. [svn-r10816] Redo the twovec test to get rid of the MPI_Wtick() test which makes no [svn-r10817] Upgrade to hwloc-1.6.1rc1. [svn-r10818] Merge hydra-specific hwloc patches from r10740. [svn-r10819] Added CHANGES log for mpich-3.0.1 and mpich-3.0.2. Upgrade to hwloc-1.6.1. Merge hydra-specific hwloc patches from Update CHANGES file with the hwloc upgrade. Provide a web URL to see the changes in v3.0.1. Qi QC Zhang (1): ensure non-negative counts in MPIC_ funcs Ralf Gunter (3): MPI_T_pvar_read/write now respect the handle creator's count variable Add MPIT variables and corresponding tests. Add configure args to selectively enable MPIT pvars. William Gropp (1): [svn-r10802] Add necessary checks for long long support ----------------------------------------------------------------------- hooks/post-receive -- MPICH primary repository

1 0

[mpich] MPICH primary repository branch, master, updated. v3.0.1-88-gfaa18dd
by noreply＠mpich.org 30 Jan '13

30 Jan '13

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 faa18dd1b7ce815aa57f4a93c8da6e409a303735 (commit) from 2726919fb15aab469342332458e85a78ff661822 (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/faa18dd1b7ce815aa57f4a93c8da6e409… commit faa18dd1b7ce815aa57f4a93c8da6e409a303735 Author: Dave Goodell <goodell(a)mcs.anl.gov> Date: Wed Jan 30 13:45:08 2013 -0600 update RELEASE_NOTES for the 3.0.2 release diff --git a/RELEASE_NOTES b/RELEASE_NOTES index 3bd579d..4d44831 100644 --- a/RELEASE_NOTES +++ b/RELEASE_NOTES @@ -1,15 +1,12 @@ ---------------------------------------------------------------------- - Experimental Features + KNOWN ISSUES ---------------------------------------------------------------------- - * New MPI-3 RMA functionality has been included in this release; - however, it is experimental and not yet supported. Users may try - out this new feature, but they should not rely on its correctness - or performance. +### Large counts ----------------------------------------------------------------------- - KNOWN ISSUES ----------------------------------------------------------------------- + * The new MPI-3 "large count" routines (e.g., MPI_Type_size_x) do not + work correctly due to 64-bit to 32-bit truncations occurring inside + the MPICH library. We expect to fix this in upcoming releases. ### Known runtime failures @@ -48,12 +45,6 @@ ### Build Platforms - * Builds using the native "make" program on OpenSolaris fail unknown - reasons. A workaround is to use GNU Make instead. See the following - ticket for more information: - - https://trac.mpich.org/projects/mpich/ticket/1122 - * Build fails with Intel compiler suite 13.0, because of weak symbol issues in the compiler. A workaround is to disable weak symbol support by passing --disable-weak-symbols to configure. See the @@ -64,6 +55,10 @@ ### Process Managers + * Hydra has a bug related to stdin handling: + + https://trac.mpich.org/projects/mpich/ticket/1782 + * The MPD process manager can only handle relatively small amounts of data on stdin and may also have problems if there is data on stdin that is not consumed by the program. ----------------------------------------------------------------------- Summary of changes: RELEASE_NOTES | 23 +++++++++-------------- 1 files changed, 9 insertions(+), 14 deletions(-) hooks/post-receive -- MPICH primary repository

1 0

[mpich] MPICH primary repository branch, master, updated. v3.0.1-87-g2726919
by noreply＠mpich.org 30 Jan '13

30 Jan '13

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 2726919fb15aab469342332458e85a78ff661822 (commit) from d20626f7e331759564b612729966271c083ba539 (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/2726919fb15aab469342332458e85a78f… commit 2726919fb15aab469342332458e85a78ff661822 Author: Dave Goodell <goodell(a)mcs.anl.gov> Date: Wed Jan 30 13:24:49 2013 -0600 update version.m4 for 3.0.2 release ABI string is bumped from c:r:a of 10:1:0 to 10:2:1. Though changes were made to mpi.h.in, none of them affect "interfaces" in the libtool sense. They merely affect some clang compile-time checking. No reviewer. diff --git a/maint/version.m4 b/maint/version.m4 index 871f140..03dbaab 100644 --- a/maint/version.m4 +++ b/maint/version.m4 @@ -14,7 +14,7 @@ # changing this by playing with diversions, but then we would probably be # playing with autotools-fire. -m4_define([MPICH_VERSION_m4],[3.0.1])dnl +m4_define([MPICH_VERSION_m4],[3.0.2])dnl m4_define([MPICH_RELEASE_DATE_m4],[unreleased development copy])dnl # For libtool ABI versioning rules see: @@ -31,6 +31,6 @@ m4_define([MPICH_RELEASE_DATE_m4],[unreleased development copy])dnl # # 4. If any interfaces have been removed since the last public # release, then set age to 0. -m4_define([libmpich_so_version_m4],[10:1:0])dnl +m4_define([libmpich_so_version_m4],[10:2:0])dnl [#] end of __file__ ----------------------------------------------------------------------- Summary of changes: maint/version.m4 | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) hooks/post-receive -- MPICH primary repository

1 0

[mpich] MPICH primary repository branch, master, updated. v3.0.1-86-gd20626f
by noreply＠mpich.org 30 Jan '13

30 Jan '13

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 d20626f7e331759564b612729966271c083ba539 (commit) from ae997a616c48d1ea2db2b43e5a6cc54a3cf3e0d7 (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/d20626f7e331759564b612729966271c0… commit d20626f7e331759564b612729966271c083ba539 Author: Dave Goodell <goodell(a)mcs.anl.gov> Date: Wed Jan 30 13:20:10 2013 -0600 update CHANGES for the 3.0.2 release diff --git a/CHANGES b/CHANGES index 8b0d064..946a654 100644 --- a/CHANGES +++ b/CHANGES @@ -4,12 +4,34 @@ # PM/PMI: Upgrade to hwloc-1.6.1 - # Performance enhancements for shared memory windows. + # RMA: Performance enhancements for shared memory windows. + + # COMPILER INTEGRATION: minor improvements and fixes to the clang static type + checking annotation macros. + + # MPI-IO (ROMIO): improved error checking for user errors, contributed by IBM. + + # MPI-3 TOOLS INTERFACE: new MPI_T performance variables providing information + about nemesis communication behavior and and CH3 message matching queues. + + # TEST SUITE: "make testing" now also outputs a "summary.tap" file that can be + interpreted with standard TAP consumer libraries and tools. The + "summary.xml" format remains unchanged. + + # GIT: This is the first release built from the new git repository at + git.mpich.org. A few build system mechanisms have changed because of this + switch. + + # BUG FIX: resolved a compilation error related to LLONG_MAX that affected + several users (ticket #1776). + + # BUG FIX: nonblocking collectives now properly make progress when MPICH is + configured with the ch3:sock channel (ticket #1785). # Several other minor bug fixes, memory leak fixes, and code cleanup. A full list of changes is available at the following link: - http://git.mpich.org/mpich.git/shortlog/v3.0..v3.0.1 + http://git.mpich.org/mpich.git/shortlog/v3.0.1..v3.0.2 =============================================================================== ----------------------------------------------------------------------- Summary of changes: CHANGES | 26 ++++++++++++++++++++++++-- 1 files changed, 24 insertions(+), 2 deletions(-) hooks/post-receive -- MPICH primary repository

1 0