1
1

Back out some debugging stuff from a careless r8643 commit (only

intended to include the OMPI_DEBUG_ZERO call).

These debugging statements should not have affected correcteness
because the value of 78 will be overridden in the read() and the
assert()/abort() stuff will only be triggered on an error which should
never happen (i.e., the error should have been handled by the prior if
conditional).  But still, thise code should not be there.

This commit was SVN r8649.

The following SVN revision numbers were found above:
  r8643 --> open-mpi/ompi@a6b869ed68
Этот коммит содержится в:
Jeff Squyres 2006-01-05 14:44:10 +00:00
родитель 268a5f5716
Коммит b2de55d72e

Просмотреть файл

@ -468,7 +468,6 @@ static void mca_oob_tcp_recv_handler(int sd, short flags, void* user)
OMPI_DEBUG_ZERO(hdr);
/* recv the process identifier */
hdr.msg_src.cellid = 78;
while((rc = recv(sd, (char *)&hdr, sizeof(hdr), 0)) != sizeof(hdr)) {
if(rc >= 0) {
if(mca_oob_tcp_component.tcp_debug > 1) {
@ -485,10 +484,6 @@ static void mca_oob_tcp_recv_handler(int sd, short flags, void* user)
return;
}
}
assert(rc == sizeof(hdr));
if (rc != sizeof(hdr)) {
abort();
}
MCA_OOB_TCP_HDR_NTOH(&hdr);
/* dispatch based on message type */