Open MPI logo

Open MPI User's Mailing List Archives

  |   Home   |   Support   |   FAQ   |   all Open MPI User's mailing list

Subject: Re: [OMPI users] Prototypes for Fortran MPI_ commands using 64-bit indexing
From: Jeff Squyres (jsquyres) (jsquyres_at_[hidden])
Date: 2013-12-11 09:01:23


On Dec 10, 2013, at 10:42 AM, Dave Love <d.love_at_[hidden]> wrote:

> This doesn't seem to have been fixed, and I think it's going to bite
> here. Is this the right change?

Thanks for reminding us.

> --- openmpi-1.6.5/ompi/config/ompi_setup_mpi_fortran.m4~ 2012-04-03 15:30:24.000000000 +0100
> +++ openmpi-1.6.5/ompi/config/ompi_setup_mpi_fortran.m4 2013-12-10 12:23:54.232854527 +0000
> @@ -127,8 +127,8 @@
> AC_MSG_RESULT([skipped (no Fortran bindings)])
> else
> bytes=`expr 4 \* $ac_cv_sizeof_int + $ac_cv_sizeof_size_t`
> - num_integers=`expr $bytes / $OMPI_SIZEOF_FORTRAN_INTEGER`
> - sanity=`expr $num_integers \* $OMPI_SIZEOF_FORTRAN_INTEGER`
> + num_integers=`expr $bytes / $ac_cv_sizeof_int`
> + sanity=`expr $num_integers \* $ac_cv_sizeof_int`

I think this is right, but it is has different implications for different series:

1. No more releases are planned for the v1.6 series. We can commit this fix over there, and it will be available via nightly tarballs. There are also ABI implications -- see #2, below.

2. This fix changes the ABI for the 1.5/1.6 and 1.7/1.8 series (separately, of course). As such, we will need to make this a non-default configure option. E.g., only do this new behavior if --enable-abi-breaking-fortran-status-i8-fix is specified (or some name like that). By default, we have to keep the ABI for the entire 1.5/1.6 and 1.7/1.8 series -- so if you specify this switch, you acknowledge that you're breaking ABI for the -i8 case.

3. For the v1.9 series (i.e., currently the SVN trunk), we can make this be the default, and the --enable-abi-breaking... switch will not exist.

Sound ok?

-- 
Jeff Squyres
jsquyres_at_[hidden]
For corporate legal information go to: http://www.cisco.com/web/about/doing_business/legal/cri/