Hi, Thanks for the bug report. This looks like a mistake when we picked the commit from the master branch. Here is a fix patch based on the mpich-3.2.1 release. https://github.com/pmodels/mpich/pull/2872/commits/63385b77c6edf3fe10c9f4002... Regards, Min On 11/17/17 5:04 AM, Marcus Calhoun-Lopez wrote:
In [1], the signature of pkt_CTS_handler wash changed in the file src/mpid/ch3/channels/nemesis/src/mpid_nem_lmt.c. However, the declaration and definition seem to be inconsistent: +static int pkt_CTS_handler(MPIDI_VC_t *vc, MPIDI_CH3_Pkt_t *pkt, void *data, MPIDI_msg_sz_t *buflen, MPID_Request **rreqp); +static int pkt_CTS_handler(MPIDI_VC_t *vc, MPIDI_CH3_Pkt_t *pkt, void *data, intptr_t *buflen, MPID_Request **rreqp)
Is the patch below the correct solution?
[1] http://git.mpich.org/mpich.git/commitdiff/2f8fe118fed12357e6f9401e644cc650e2...
--- src/mpid/ch3/channels/nemesis/src/mpid_nem_lmt.c.orig 2017-11-10 19:19:44.000000000 -0700 +++ src/mpid/ch3/channels/nemesis/src/mpid_nem_lmt.c 2017-11-17 04:23:52.000000000 -0700 @@ -299,7 +299,7 @@ #define FUNCNAME pkt_CTS_handler #undef FCNAME #define FCNAME MPL_QUOTE(FUNCNAME) -static int pkt_CTS_handler(MPIDI_VC_t *vc, MPIDI_CH3_Pkt_t *pkt, void *data, intptr_t *buflen, MPID_Request **rreqp) +static int pkt_CTS_handler(MPIDI_VC_t *vc, MPIDI_CH3_Pkt_t *pkt, void *data, MPIDI_msg_sz_t *buflen, MPID_Request **rreqp) { MPID_nem_pkt_lmt_cts_t * const cts_pkt = (MPID_nem_pkt_lmt_cts_t *)pkt; MPID_Request *sreq; _______________________________________________ discuss mailing list [email protected] To manage subscription options or unsubscribe: https://lists.mpich.org/mailman/listinfo/discuss
_______________________________________________ discuss mailing list [email protected] To manage subscription options or unsubscribe: https://lists.mpich.org/mailman/listinfo/discuss