Open MPI logo

Open MPI Development Mailing List Archives

  |   Home   |   Support   |   FAQ   |   all Development mailing list

Subject: Re: [OMPI devel] [OMPI svn-full] svn:open-mpi r29055 - in trunk/ompi/mca: btl btl/smcuda common/cuda pml/ob1
From: George Bosilca (bosilca_at_[hidden])
Date: 2013-08-22 11:32:25


I'm not very keen of seeing BTL modification tainting the PML. I would have expected support for IPC between GPU must be a BTL-level decision, no a special path in the PML.

Is there a reason IPC support cannot be hidden down in the SMCUDA BTL?

  Thanks,
    George.

On Aug 21, 2013, at 23:00 , svn-commit-mailer_at_[hidden] wrote:

> Author: rolfv (Rolf Vandevaart)
> Date: 2013-08-21 17:00:09 EDT (Wed, 21 Aug 2013)
> New Revision: 29055
> URL: https://svn.open-mpi.org/trac/ompi/changeset/29055
>
> Log:
> Fix support in smcuda btl so it does not blow up when there is no CUDA IPC support between two GPUs. Also make it so CUDA IPC support is added dynamically.
> Fixes ticket 3531.
>
> Added:
> trunk/ompi/mca/btl/smcuda/README
> Text files modified:
> trunk/ompi/mca/btl/btl.h | 2
> trunk/ompi/mca/btl/smcuda/README | 113 ++++++++++++++++++++++
> trunk/ompi/mca/btl/smcuda/btl_smcuda.c | 104 ++++++++++++++++++++
> trunk/ompi/mca/btl/smcuda/btl_smcuda.h | 28 +++++
> trunk/ompi/mca/btl/smcuda/btl_smcuda_component.c | 200 +++++++++++++++++++++++++++++++++++++++
> trunk/ompi/mca/btl/smcuda/btl_smcuda_endpoint.h | 5 +
> trunk/ompi/mca/common/cuda/common_cuda.c | 29 +++++
> trunk/ompi/mca/common/cuda/common_cuda.h | 3
> trunk/ompi/mca/pml/ob1/pml_ob1.c | 11 ++
> trunk/ompi/mca/pml/ob1/pml_ob1_cuda.c | 42 ++++++++
> trunk/ompi/mca/pml/ob1/pml_ob1_recvreq.c | 6
> 11 files changed, 535 insertions(+), 8 deletions(-)