[mpich-discuss] [mpich2-announce] Announcing the availability of MPICH2-1.1a2

[Dave Goodell]

On Nov 20, 2008, at 2:24 PM, Joe Ratterman wrote:

> Hi,  I am working on merging the latest changes from this alpha in  
> the the BG/P code.  It fully compiles now--I haven't run the tests-- 
> but I had a few issues that I wanted to mention.  Maybe someone  
> already is working on solutions or can otherwise be of some help.
>
> 1)
> PAC_CC_FUNCTION_NAME_SYMBOL (configure.in) doesn't work at all in  
> cross-compilation environments, though I don't know of a good  
> solution to that. [...]

It looks like David Gingold came to the same conclusion here: https:// 
trac.mcs.anl.gov/projects/mpich2/ticket/300

Sorry for the breakage, we don't cross compile as often as you do and  
we didn't catch this one before the release.  I haven't had the  
chance to dig in and fully grok this change yet, but I'm sure we can  
come up with a fix soon-ish.

> 2)
> "src/util/procmap/local_proc.c" seems a bit troubling for us.  We  
> don't use a PMI device, and specify "MPID_NO_PMI=yes" in src/mpid/ 
> dcmfd/mpich2prereq.  However, this file calls  
> PMI_KVS_Get_key_length_max() from MPIU_Get_local_procs(). That did  
> compile because C doesn't care too much, but it wouldn't link, even  
> though we never call MPIU_Get_local_procs().  This is because the  
> file also defines MPIU_Get_intranode_rank(), which is uses by both  
> src/mpi/coll/reduce.c & src/mpi/coll/bcast.c.  I ended up simply  
> deleting the entire MPIU_Get_local_procs() function to solve the  
> problem.  I am sure that isn't the answer, but I don't know what is  
> the correct version.

This code is in need of a good dose of cleanup and improvement.  It's  
not where I'd like it to be but we elected to put it out there to see  
how people felt about the feature.  Don't worry, this won't be the  
final version of this code.  In your situation I suspect removing the  
code is what makes the most sense for now.

> 3)
> This one might be my problem, but I couldn't compile all the tests  
> because neither the F77 nor F90 versions of f*/init/checksizes.c  
> had actually been generated by the test/mpi/configure script.  I  
> don't know why not, but I had to copy them out of the configure.in  
> script.  There is no reference to them in the logs, and they are  
> not part of the config.status script to be re-generated.  I'll be  
> looking into that one more, after I get the system running properly.

Bill has been making a bunch of changes in this area to clean up the  
test script and we might have accidentally excluded one of his  
changes from the release.  I've filed a ticket to track this here:  
https://trac.mcs.anl.gov/projects/mpich2/ticket/301

-Dave