diff mbox series

[bug#39588] gnu: Add mpich, scalapack-mpich, mumps-mpich, pt-scotch-mpich, python-mpi4py-mpich

Message ID 873625m09f.fsf@gnu.org
State Accepted
Headers show
Series [bug#39588] gnu: Add mpich, scalapack-mpich, mumps-mpich, pt-scotch-mpich, python-mpi4py-mpich | expand

Checks

Context Check Description
cbaines/applying patch fail View Laminar job

Commit Message

Ludovic Courtès Oct. 23, 2020, 3:26 p.m. UTC 
Hi Maurice,

Maurice Brémond <Maurice.Bremond@inria.fr> skribis:

> Apparently at the mpich configuration level, using the experimental
> device ch4 instead of ch3 solves the problem : just remove comment on
> "--with-device=ch4:ucx".  Reversely, with mpich 3.4a2 (for which ch4 is
> de default) setting --with-device=ch3 leads to the same failure as with
> 3.3.2.

Nice, we have a way forward.

With the patch below, I have successfully built:

  guix build mumps-openmpi --with-input=openmpi=mpich

and I confirm that despite the name it depends exclusively on MPICH.
:-)

If that’s fine with you I’ll go ahead and commit it; let me know!

> I also checked sock channel for ch3 : with-device=ch3:sock, but then on
> my laptop, scotch tests hang at
>
> mpirun -n 3 ./test_scotch_dgraph_check data/bump.grf
>
> For the moment, there isn't a stable 3.4 version yet for mpich. I had a
> try with the latest 3.4b1 but a test failed...

We’ll see, but having a solution that works with 3.3 and is likely to
work with 3.4 is good.

I guess we should also check whether we’re obtaining the expected
performance.  This builds fine too:

  guix build intel-mpi-benchmarks --with-input=openmpi=mpich

Thank you!

Ludo’.

Comments

Maurice Brémond Oct. 23, 2020, 5:04 p.m. UTC | #1 
>If that’s fine with you I’ll go ahead and commit it; let me know!
It's ok for me and what I do with it.

Bon week-end!
Ludovic Courtès Nov. 2, 2020, 2:02 p.m. UTC | #2 
Salut,

Maurice Brémond <Maurice.Bremond@inria.fr> skribis:

>>If that’s fine with you I’ll go ahead and commit it; let me know!
> It's ok for me and what I do with it.
>
> Bon week-end!

Finally pushed as c73496f433044a76003b33c3855bb35ecd0df87f, thanks!

I’m closing this bug, let’s open a new one if we need to further discuss
MPI support in Guix.

Ludo’.
diff mbox series

Patch

diff --git a/gnu/packages/mpi.scm b/gnu/packages/mpi.scm
index 06a82cce95..9035147441 100644
--- a/gnu/packages/mpi.scm
+++ b/gnu/packages/mpi.scm
@@ -436,7 +436,12 @@  arrays) that expose a buffer interface.")
      `(#:configure-flags
        (list "--disable-silent-rules"             ;let's see what's happening
              "--enable-debuginfo"
-             ;; "--with-device=ch4:ucx" ; --with-device=ch4:ofi segfaults in tests
+
+             ;; Default to "ch4", as will be the case in 3.4.  It also works
+             ;; around issues when running test suites of packages that use
+             ;; MPICH: <https://issues.guix.gnu.org/39588#15>.
+             "--with-device=ch4:ucx" ; --with-device=ch4:ofi segfaults in tests
+
              (string-append "--with-hwloc-prefix="
                             (assoc-ref %build-inputs "hwloc"))