[Mpi3-tools] Bad handle return error code

[Kathryn Mohror]

I suggested MPI_T_ERR_INVALID_HANDLE not SESSION. Is that what you meant?

Kathryn

On Jul 25, 2013, at 4:38 PM, Jeff Squyres (jsquyres) <jsquyres at cisco.com> wrote:

> +1
> 
> On Jul 25, 2013, at 6:15 PM, "Schulz, Martin" <schulzm at llnl.gov> wrote:
> 
>> Yes, I agree - the handle is invalid in the session referenced in the argument. Therefore MPI_T_ERR_INVALID_SESSION seems reasonable to me.
>> 
>> Martin
>> 
>> 
>> On Jul 25, 2013, at 2:03 PM, Kathryn Mohror <kathryn at llnl.gov>
>> wrote:
>> 
>>> Hi everybody,
>>> 
>>> I got asked a question from Nathan:
>>> 
>>> What should (MPI_T_pvar_read, _write, _reset, etc) return if bad handle is given?
>>> 
>>> He gave this example of erroneous code:
>>> {
>>> MPI_T_pvar_session session1, session2;
>>> MPI_T_pvar_handle handle;
>>> char buffer[256];
>>> int count;
>>> 
>>> MPI_T_pvar_session_create (&session1);
>>> MPI_T_pvar_session_create (&session2);
>>> 
>>> MPI_T_pvar_handle_alloc (session1, 0, NULL, &handle, &count);
>>> 
>>> MPI_T_pvar_start (session1, handle);
>>> 
>>> /* erroneus statement. should return MPI_T_ERR_INVALID_SESSION or MPI_T_ERR_INVALID_HANDLE */
>>> MPI_T_pvar_read (session2, handle, buffer);
>>> }
>>> 
>>> I argue it should return MPI_T_ERR_INVALID_HANDLE since the session is good, it's just that the handle doesn’t match.
>>> 
>>> Do you all agree? If so, we should specify it since it's ambiguous. I believe this is an errata item.
>>> 
>>> Kathryn
>>> ______________________________________________________________
>>> Kathryn Mohror, kathryn at llnl.gov, http://people.llnl.gov/mohror1
>>> CASC @ Lawrence Livermore National Laboratory, Livermore, CA, USA
>>> 
>>> 
>>> 
>>> 
>>> _______________________________________________
>>> Mpi3-tools mailing list
>>> Mpi3-tools at lists.mpi-forum.org
>>> http://lists.mpi-forum.org/mailman/listinfo.cgi/mpi3-tools
>> 
>> ________________________________________________________________________
>> Martin Schulz, schulzm at llnl.gov, http://people.llnl.gov/schulzm
>> CASC @ Lawrence Livermore National Laboratory, Livermore, USA
>> 
>> 
>> 
>> 
>> _______________________________________________
>> Mpi3-tools mailing list
>> Mpi3-tools at lists.mpi-forum.org
>> http://lists.mpi-forum.org/mailman/listinfo.cgi/mpi3-tools
> 
> 
> -- 
> Jeff Squyres
> jsquyres at cisco.com
> For corporate legal information go to: http://www.cisco.com/web/about/doing_business/legal/cri/
> 
> 
> _______________________________________________
> Mpi3-tools mailing list
> Mpi3-tools at lists.mpi-forum.org
> http://lists.mpi-forum.org/mailman/listinfo.cgi/mpi3-tools

______________________________________________________________
Kathryn Mohror, kathryn at llnl.gov, http://people.llnl.gov/mohror1
CASC @ Lawrence Livermore National Laboratory, Livermore, CA, USA




-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.mpi-forum.org/pipermail/mpiwg-tools/attachments/20130725/46ff1494/attachment-0001.html>