[mpich-commits] [mpich] MPICH primary repository branch, master, updated. v3.1.2-135-g55e8311

Service Account noreply at mpich.org
Tue Sep 2 09:55:16 CDT 2014


This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "MPICH primary repository".

The branch, master has been updated
       via  55e8311365392a259f407bbe5344e546cf8ead17 (commit)
      from  1735645322ec6aeddedada28e48f1df6a2354d83 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
http://git.mpich.org/mpich.git/commitdiff/55e8311365392a259f407bbe5344e546cf8ead17

commit 55e8311365392a259f407bbe5344e546cf8ead17
Author: Igor Ivanov <Igor.Ivanov at itseez.com>
Date:   Tue Sep 2 15:57:36 2014 +0300

    netmod/mxm: Fix issue with req->dev.tmpbuf corruption
    
    Do not reset temp buffer. Upper layer(MPID_nem_send_iov) might allocate
    a temp buffer(MPIDI_CH3U_SRBuf_alloc) and set a flag.  This buffer will
    get freed at the time of request free.
    Removed assertion is not valid for RMA operations.
    
    Signed-off-by: Devendar Bureddy <devendar at mellanox.com>
    Signed-off-by: Igor Ivanov <Igor.Ivanov at itseez.com>

diff --git a/src/mpid/ch3/channels/nemesis/netmod/mxm/mxm_send.c b/src/mpid/ch3/channels/nemesis/netmod/mxm/mxm_send.c
index 6a6a9a3..fe13044 100644
--- a/src/mpid/ch3/channels/nemesis/netmod/mxm/mxm_send.c
+++ b/src/mpid/ch3/channels/nemesis/netmod/mxm/mxm_send.c
@@ -41,7 +41,6 @@ int MPID_nem_mxm_iSendContig(MPIDI_VC_t * vc, MPID_Request * sreq, void *hdr, MP
     MPIDI_DBG_Print_packet((MPIDI_CH3_Pkt_t *) hdr);
 
     MPIU_Memcpy(&(sreq->dev.pending_pkt), (char *) hdr, sizeof(MPIDI_CH3_Pkt_t));
-    sreq->dev.tmpbuf = NULL;
 
     REQ_FIELD(sreq, ctx) = sreq;
     REQ_FIELD(sreq, iov_buf) = REQ_FIELD(sreq, tmp_buf);
@@ -153,7 +152,6 @@ int MPID_nem_mxm_SendNoncontig(MPIDI_VC_t * vc, MPID_Request * sreq, void *hdr,
     MPIU_DBG_MSG(CH3_CHANNEL, VERBOSE, "MPID_nem_mxm_iSendNoncontig");
 
     MPIU_Memcpy(&(sreq->dev.pending_pkt), (char *) hdr, sizeof(MPIDI_CH3_Pkt_t));
-    sreq->dev.tmpbuf = NULL;
 
     _dbg_mxm_output(5,
                     "SendNoncontig ========> Sending ADI msg (to=%d type=%d) for req %p (data_size %d, %d) \n",
@@ -629,7 +627,6 @@ static void _mxm_send_completion_cb(void *context)
     MPID_Request *req = (MPID_Request *) context;
 
     MPIU_Assert(req);
-    MPIU_Assert((req->kind == MPID_REQUEST_SEND) || (req->kind == MPID_PREQUEST_SEND));
     _dbg_mxm_out_req(req);
 
     _mxm_to_mpi_status(REQ_FIELD(req, mxm_req->item.base.error), &req->status);

-----------------------------------------------------------------------

Summary of changes:
 .../ch3/channels/nemesis/netmod/mxm/mxm_send.c     |    3 ---
 1 files changed, 0 insertions(+), 3 deletions(-)


hooks/post-receive
-- 
MPICH primary repository


More information about the commits mailing list