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-mai...@open-mpi.org 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(-)

Reply via email to