| commit | aa81006ac32c29c10fd8bd3ef8669bd8338d7802 | |
| author | Berk Hess <hess@kth.se> | |
| Thu, 22 Feb 2018 12:40:45 +0000 (22 13:40 +0100) | ||
| committer | Berk Hess <hess@kth.se> | |
| Thu, 22 Feb 2018 21:50:22 +0000 (22 22:50 +0100) | ||
| tree | 9dda6caefc1b4ef6cbbf9fe29cf2fb53e8d723ee | treesnapshot (tar.gz zip) |
| parent | 09828eddcf3a6281c50bdf42be59ef9f72e3d04b | commitdiff |
Simplify mdrun tMPI thread spawning
During spawning of the tMPI threads in mdrunner, the mdrunner object
and filename pointer were copied twice. Now they are only copied once
and a const object/pointer is passed around.
Also made thread-MPI take a const argument to enable this and to
ensure correctness, as thread-MPI makes a copy of the argument pointer
for every thread which could cause problems without const.
Added an explicit MPI_Barrier after spawning, with documentation,
in preparation of removing the implicit barrier in MPI_Comm_split
in reinitialize_commrec_for_this_thread().
Change-Id: I31af8a8ff72aec28366c4aa20b357ed96bf95087
During spawning of the tMPI threads in mdrunner, the mdrunner object
and filename pointer were copied twice. Now they are only copied once
and a const object/pointer is passed around.
Also made thread-MPI take a const argument to enable this and to
ensure correctness, as thread-MPI makes a copy of the argument pointer
for every thread which could cause problems without const.
Added an explicit MPI_Barrier after spawning, with documentation,
in preparation of removing the implicit barrier in MPI_Comm_split
in reinitialize_commrec_for_this_thread().
Change-Id: I31af8a8ff72aec28366c4aa20b357ed96bf95087