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

[Pavan Balaji]

>> "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.

We can ifdef out this code when the PM is explicitly defined to none in 
the configure (though I think we'll still need some trivial function in 
its place for the SMP collectives code to not freak out). I'll get to 
this next week and try to have it fixed for 1.1a3.

  -- Pavan

-- 
Pavan Balaji
http://www.mcs.anl.gov/~balaji