Open MPI logo

Open MPI Development Mailing List Archives

  |   Home   |   Support   |   FAQ   |   all Development mailing list

Subject: Re: [OMPI devel] RFC: Adding OMPI_CHECK_WITHDIR checks
From: Matney Sr, Kenneth D. (matneykdsr_at_[hidden])
Date: 2009-02-02 08:10:09


Assuming that this update went through, there were no negative
side-effects for portals.

-- 
Ken
 
-----Original Message-----
From: devel-bounces_at_[hidden] [mailto:devel-bounces_at_[hidden]] On
Behalf Of Jeff Squyres
Sent: Tuesday, January 27, 2009 11:51 AM
To: Open MPI Developers
Subject: Re: [OMPI devel] RFC: Adding OMPI_CHECK_WITHDIR checks
There was no disagreement about this issue on the teleconf today.  So  
if no one disagrees here on the list, I'll commit this stuff to the  
trunk at COB on Thursday 29 Jan 2009 (i.e., the RFC timeout).
On Jan 20, 2009, at 7:44 PM, Jeff Squyres wrote:
> What: Adding OMPI_CHECK_WITHDIR checks in various .m4 files
>
> Why: Help prevent user errors via --with-<foo>=DIR configure options
>
> Where: config/*m4 and */mca/*/*/configure.m4 files, affecting the  
> following environments:
> - bproc (***)
> - gm (***)
> - loadleveler (***)
> - lsf
> - mx (***)
> - open fabrics
> - portals (***)
> - psm (***)
> - tm
> - udapl
> - elan (***)
> - sctp
> - blcr (***)
> - libnuma
> - valgrind
> ===> I could not easily test the (***) environments
>
> When: For OMPI v1.4 (could be convinced to make it for v1.3.1)
>
> Timeout: COB Thursday, Jan 29, 2009
>
> ------------
>
> The intent for OMPI v1.3's new OMPI_CHECK_WITHDIR m4 macro was to  
> fix a case where a user was doing the following:
>
>  ./configure --with-openib=/path/to/nonexistent/OFED/installation
>
> ...but configure succeeded anyway because the sysadmins had  
> installed OFED into /usr.  Hence, the user was getting something  
> unexpected.
>
> OMPI_CHECK_WITHDIR does a very basic sanity check on directories  
> provided by --with-<foo>=DIR configure options.  Specifically, it  
> checks if the directory exists and if a token file exists in that  
> directory (specifically, it calls "ls <token_file>", so wildcards  
> are acceptable).  If either of those tests fail, configure aborts  
> with an appropriate error message.  This macro was used in the  
> openib BTL configure stuff, but we didn't add it anywhere else.  I'm  
> now adding it everywhere we have a --with-<foo>=DIR, which are in  
> various .m4 files in the environments described above.
>
> Here's the hg where I added OMPI_CHECK_WITHDIR to all the  
> environments listed above, but was unable to test the (***)  
> environments:
>
>    http://www.open-mpi.org/hg/hgwebdir.cgi/jsquyres/ 
> ompi_check_withdir/
>
> We could bring this patch to v1.3.1 or we could wait until v1.4.  I  
> don't really care either way.
>
> I plan to bring this work into the trunk next Thursday COB; it would  
> be great if those who have the (*) environments could pull down the  
> hg tree before then and give it a whirl so we can fix any problems  
> beforehand.
>
> -- 
> Jeff Squyres
> Cisco Systems
>
> _______________________________________________
> devel mailing list
> devel_at_[hidden]
> http://www.open-mpi.org/mailman/listinfo.cgi/devel
-- 
Jeff Squyres
Cisco Systems
_______________________________________________
devel mailing list
devel_at_[hidden]
http://www.open-mpi.org/mailman/listinfo.cgi/devel