These are related to the Aint is not void* in size. With the introduction of MPIR_Pint, these should simply use the explicit cast rather than the now unnecessary macro (that macro can have only one expansion, so there's no need to define it). The rules are In the patch, replace MPI_AINT_CAST_TO_VOID_PTR with (void*)(MPIR_Puint) replace MPI_VOID_PTR_CAST_TO_MPI_AINT with (MPIR_Puint) And do not use MPI_AINT_CAST_TO_LONG_LONG (architecture-specific hack for printing). Bill On Jul 8, 2008, at 10:21 PM, Pavan Balaji wrote:
Hi Bill,
Some parts of the build.patch don't seem to have been committed. For example, MPI_AINT_CAST_TO_VOID_PTR doesn't seem to be in. Was there a change in this part?
-- Pavan
On 07/01/2008 03:50 PM, William Gropp wrote:
Yes, I committed it to the trunk.
Bill
On Jun 29, 2008, at 12:04 AM, Pavan Balaji wrote:
Hi Bill,
Did you already commit this patch into the trunk? If you are still working on it, can you send out your current (temporarily workable) version of the patch? We can use it to compile the patches we are working on and check for errors.
Thanks.
-- Pavan
William Gropp wrote:
I'm in the final stages of testing the changes to build.patch . I've attached an item-by-item summary.
Bill
------------------------------------------------------------------- -----
On Jun 23, 2008, at 8:12 PM, Pavan Balaji wrote:
Hi all,
Now that the discussion with IBM is done, we should work on integrating the BG patches into the MPICH2 trunk.
build.patch -- Bill datatype.patch -- Pavan dcmfd.patch -- Pavan debugger.patch -- Darius ignore.patch -- Pavan mpe.patch -- Anthony other.patch -- Rajeev romio-bgl.patch -- RobL romio.patch -- RobL test-build.patch -- Dave test-source.patch -- Dave
I've already integrated the dcmfd patch and am working on integrating the datatype and ignore patches. Whoever has looked at the above patches, can you spend some time to integrate these into the trunk? From the discussion with IBM, the debugger patch is delayed for now and a part of the test suite patches are not being integrated as well. Everything else should go in.
Once the build patch is in, we can start testing the stack on BG/P, so that will probably be the critical requirement for this integration.
Thanks.
-- Pavan
-- Pavan Balaji http://www.mcs.anl.gov/~balaji
William Gropp Paul and Cynthia Saylor Professor of Computer Science University of Illinois Urbana-Champaign
-- Pavan Balaji http://www.mcs.anl.gov/~balaji
William Gropp Paul and Cynthia Saylor Professor of Computer Science University of Illinois Urbana-Champaign
-- Pavan Balaji http://www.mcs.anl.gov/~balaji
William Gropp Paul and Cynthia Saylor Professor of Computer Science University of Illinois Urbana-Champaign