[PATCH] Warn about MPI not supporting cross-context particles

Jeffrey D. Oldham oldham at codesourcery.com
Mon Jan 19 19:04:31 UTC 2004


Richard Guenther wrote:
> Hi!
> 
> This patch aborts on use of cross-context particles with MPI, which is not
> supported.
> 
> Ok?
> 
> Richard.
> 
> 
> 2004Jan18  Richard Guenther <richard.guenther at uni-tuebingen.de>
> 
> 	* src/Particles/PatchSwapLayout.cpp: abort on cross-context
> 	particles with MPI.
> 
> Index: src/Particles/PatchSwapLayout.cpp
> ===================================================================
> RCS file: /home/pooma/Repository/r2/src/Particles/PatchSwapLayout.cpp,v
> retrieving revision 1.15
> diff -u -u -r1.15 PatchSwapLayout.cpp
> --- src/Particles/PatchSwapLayout.cpp	8 Jun 2000 22:16:23 -0000	1.15
> +++ src/Particles/PatchSwapLayout.cpp	18 Jan 2004 13:38:28 -0000
> @@ -545,6 +545,8 @@
>      Pooma::particleSwapHandler()->send(toContext, tag, buf);
>    }
> 
> +#elif POOMA_MPI
> +  PInsist(false, "Cross-context particles not supported for MPI");
>  #endif // POOMA_CHEETAH

Is the POOMA_CHEETAH comment correct?  I think it is probably correct 
since the first #if is probably for Cheetah, but is it misleading?  Is 
there a better comment?

>  }
> 
> @@ -621,6 +623,8 @@
>    while (layout_m.patchInfo(lid).msgReceived() < remotePatches)
>      Pooma::poll();
> 
> +#elif POOMA_MPI
> +  PInsist(false, "Cross-context particles not supported for MPI");
>  #endif // POOMA_CHEETAH

Likewise here.

>  }
> 

Thanks for tightening the code.  OK to commit.

-- 
Jeffrey D. Oldham
oldham at codesourcery.com




More information about the pooma-dev mailing list