Project

General

Profile

Bug #2708

gromacs-2019 fails to build with openmpi-2.0.2

Added by Christoph Junghans about 1 year ago. Updated about 1 year ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
-
Target version:
Affected version - extra info:
515e6b44b8f13b6ca49842af6ae38bbe0e0b5b7d
Affected version:
Difficulty:
uncategorized
Close

Description

/var/tmp/portage/sci-chemistry/gromacs-2019.9999/work/gromacs-2019.9999/src/gromacs/pulling/pullutil.cpp:118:26: error: cannot convert ‘std::vector<float>’ to ‘void*’ for argument ‘2’ to ‘int MPI_Allreduce(const void*, void*, int, MPI_Datatype, MPI_Op, MPI_Comm)’
             MPI_Allreduce(data, buf, n, mpiDatatype(data), MPI_SUM,
             ~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
                           comm->mpi_comm_com);
                           ~~~~~~~~~~~~~~~~~~~
/var/tmp/portage/sci-chemistry/gromacs-2019.9999/work/gromacs-2019.9999/src/gromacs/pulling/pullutil.cpp: In instantiation of ‘void pullAllReduce(const t_commrec*, pull_comm_t*, int, T*) [with T = double]’:
/var/tmp/portage/sci-chemistry/gromacs-2019.9999/work/gromacs-2019.9999/src/gromacs/pulling/pullutil.cpp:327:50:   required from here
/var/tmp/portage/sci-chemistry/gromacs-2019.9999/work/gromacs-2019.9999/src/gromacs/pulling/pullutil.cpp:118:26: error: cannot convert ‘std::vector<double>’ to ‘void*’ for argument ‘2’ to ‘int MPI_Allreduce(const void*, void*, int, MPI_Datatype, MPI_Op, MPI_Comm)’

Associated revisions

Revision 21717fef (diff)
Added by Paul Bauer about 1 year ago

Fix MPI w/o MPI_IN_PLACE

Fixes #2708

Change-Id: Ib251797c11cb5aa344bbc2d7083eee18487c4062

History

#1 Updated by Gerrit Code Review Bot about 1 year ago

Gerrit received a related patchset '1' for Issue #2708.
Uploader: Paul Bauer ()
Change-Id: gromacs~release-2019~Ib251797c11cb5aa344bbc2d7083eee18487c4062
Gerrit URL: https://gerrit.gromacs.org/8591

#2 Updated by Paul Bauer about 1 year ago

Looks like this only happened when MPI_IN_PLACE was not available. I uploaded a patch that should address it (it compiles when I force a situation w/o MPI_IN_PLACE)

#3 Updated by Paul Bauer about 1 year ago

  • Status changed from New to Resolved

#4 Updated by Paul Bauer about 1 year ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF