[Mpi3-tools] Bad handle return error code

[Anh Vo]

Yeah, I was a bit confused, too.
I agree with Kathryn. Each session has a bunch of handles that were allocated for it, and thus any inquiry should first reference the session, and then the handle. In this case the scope is session2, and handle is not part of session2's scope, so a search in session2 should not be able to locate the handle, thus MPI_T_ERR_INVALID_HANDLE

--Anh

From: mpi3-tools-bounces at lists.mpi-forum.org [mailto:mpi3-tools-bounces at lists.mpi-forum.org] On Behalf Of Kathryn Mohror
Sent: Thursday, July 25, 2013 5:37 PM
To: mpi3-tools at lists.mpi-forum.org
Subject: Re: [Mpi3-tools] Bad handle return error code

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<mailto:jsquyres at cisco.com>> wrote:


+1

On Jul 25, 2013, at 6:15 PM, "Schulz, Martin" <schulzm at llnl.gov<mailto: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<mailto: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<mailto: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<mailto:Mpi3-tools at lists.mpi-forum.org>
http://lists.mpi-forum.org/mailman/listinfo.cgi/mpi3-tools

________________________________________________________________________
Martin Schulz, schulzm at llnl.gov<mailto: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<mailto:Mpi3-tools at lists.mpi-forum.org>
http://lists.mpi-forum.org/mailman/listinfo.cgi/mpi3-tools


--
Jeff Squyres
jsquyres at cisco.com<mailto: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<mailto:Mpi3-tools at lists.mpi-forum.org>
http://lists.mpi-forum.org/mailman/listinfo.cgi/mpi3-tools

______________________________________________________________
Kathryn Mohror, kathryn at llnl.gov<mailto: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/20130726/b26606bc/attachment-0001.html>