Skip to content

Commit cbfb668

Browse files
committed
Small cosmetic fixes to improve logging and prevent valgrind from bickering.
1 parent f106292 commit cbfb668

File tree

1 file changed

+8
-9
lines changed

1 file changed

+8
-9
lines changed

src/mpi/mpi_caf.c

Lines changed: 8 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1180,7 +1180,7 @@ PREFIX(register) (size_t size, caf_register_t type, caf_token_t *token,
11801180
/* Create or allocate a slave token. */
11811181
mpi_caf_slave_token_t *slave_token;
11821182
#ifdef EXTRA_DEBUG_OUTPUT
1183-
MPI_Aint mpi_address;
1183+
MPI_Aint mpi_address = 0;
11841184
#endif
11851185
CAF_Win_unlock_all(global_dynamic_win);
11861186
if (type == CAF_REGTYPE_COARRAY_ALLOC_REGISTER_ONLY)
@@ -4680,17 +4680,17 @@ PREFIX(get_by_ref) (caf_token_t token, int image_index,
46804680
src = (gfc_descriptor_t *)&src_desc;
46814681
if (access_desc_through_global_win)
46824682
{
4683-
dprint("remote desc fetch from %p, offset = %zd\n",
4684-
remote_base_memptr, desc_offset);
4683+
dprint("remote desc fetch from %p, offset = %zd, ref_rank = %d\n",
4684+
remote_base_memptr, desc_offset, ref_rank);
46854685
MPI_Get(src, sizeof_desc_for_rank(ref_rank), MPI_BYTE, global_dynamic_win_rank,
46864686
MPI_Aint_add((MPI_Aint)remote_base_memptr, desc_offset),
46874687
sizeof_desc_for_rank(ref_rank), MPI_BYTE,
46884688
global_dynamic_win);
46894689
}
46904690
else
46914691
{
4692-
dprint("remote desc fetch from win %d, offset = %zd\n",
4693-
mpi_token->memptr_win, desc_offset);
4692+
dprint("remote desc fetch from win %d, offset = %zd, ref_rank = %d\n",
4693+
mpi_token->memptr_win, desc_offset, ref_rank);
46944694
MPI_Get(src, sizeof_desc_for_rank(ref_rank), MPI_BYTE, memptr_win_rank,
46954695
desc_offset, sizeof_desc_for_rank(ref_rank), MPI_BYTE,
46964696
mpi_token->memptr_win);
@@ -4701,8 +4701,7 @@ PREFIX(get_by_ref) (caf_token_t token, int image_index,
47014701
src = mpi_token->desc;
47024702

47034703
#ifdef EXTRA_DEBUG_OUTPUT
4704-
dprint("remote desc rank: %zd (ref_rank: %zd)\n",
4705-
GFC_DESCRIPTOR_RANK(src), ref_rank);
4704+
dprint("remote desc rank: %zd\n", GFC_DESCRIPTOR_RANK(src));
47064705
for (i = 0; i < GFC_DESCRIPTOR_RANK(src); ++i)
47074706
{
47084707
dprint("remote desc dim[%zd] = (lb=%zd, ub=%zd, stride=%zd)\n",
@@ -4783,8 +4782,8 @@ case kind: \
47834782
caf_internal_error(unknownarrreftype, stat, NULL, 0);
47844783
return;
47854784
}
4786-
dprint("i = %zd, array_ref = %s, delta = %ld\n", i,
4787-
caf_array_ref_str[array_ref], delta);
4785+
dprint("i = %zd, array_ref = %s, delta = %ld, in_array_ref = %d, arr_ext_fixed = %d, realloc_required = %d\n", i,
4786+
caf_array_ref_str[array_ref], delta, in_array_ref, array_extent_fixed, realloc_required);
47884787
if (delta <= 0)
47894788
return;
47904789
/* Check the various properties of the destination array.

0 commit comments

Comments
 (0)