+1

(meaning: yes -- INVALID_HANDLE :-) )

On Jul 26, 2013, at 2:27 AM, "Schulz, Martin" <schulzm@llnl.gov> wrote:

Yes, sorry - that was a typo on my side: it should be INVALID_HANDLE. I think we are all in agreement.

Martin


On Jul 25, 2013, at 6:29 PM, Anh Vo <Anh.Vo@microsoft.com>
wrote:

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@lists.mpi-forum.org [mailto:mpi3-tools-bounces@lists.mpi-forum.org] On Behalf Of Kathryn Mohror
Sent: Thursday, July 25, 2013 5:37 PM
To: mpi3-tools@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@cisco.com> wrote:


+1

On Jul 25, 2013, at 6:15 PM, "Schulz, Martin" <schulzm@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@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@llnl.gov, http://people.llnl.gov/mohror1
CASC @ Lawrence Livermore National Laboratory, Livermore, CA, USA




_______________________________________________
Mpi3-tools mailing list
Mpi3-tools@lists.mpi-forum.org
http://lists.mpi-forum.org/mailman/listinfo.cgi/mpi3-tools

________________________________________________________________________
Martin Schulz, schulzm@llnl.gov, http://people.llnl.gov/schulzm
CASC @ Lawrence Livermore National Laboratory, Livermore, USA




_______________________________________________
Mpi3-tools mailing list
Mpi3-tools@lists.mpi-forum.org
http://lists.mpi-forum.org/mailman/listinfo.cgi/mpi3-tools


--
Jeff Squyres
jsquyres@cisco.com
For corporate legal information go to: http://www.cisco.com/web/about/doing_business/legal/cri/


_______________________________________________
Mpi3-tools mailing list
Mpi3-tools@lists.mpi-forum.org
http://lists.mpi-forum.org/mailman/listinfo.cgi/mpi3-tools

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




_______________________________________________
Mpi3-tools mailing list
Mpi3-tools@lists.mpi-forum.org
http://lists.mpi-forum.org/mailman/listinfo.cgi/mpi3-tools

________________________________________________________________________
Martin Schulz, schulzm@llnl.gov, http://people.llnl.gov/schulzm
CASC @ Lawrence Livermore National Laboratory, Livermore, USA




_______________________________________________
Mpi3-tools mailing list
Mpi3-tools@lists.mpi-forum.org
http://lists.mpi-forum.org/mailman/listinfo.cgi/mpi3-tools

--
Jeff Squyres
jsquyres@cisco.com
For corporate legal information go to: http://www.cisco.com/web/about/doing_business/legal/cri/


_______________________________________________
Mpi3-tools mailing list
Mpi3-tools@lists.mpi-forum.org
http://lists.mpi-forum.org/mailman/listinfo.cgi/mpi3-tools