[mpich-discuss] MPICH compilation error

Min Si msi at anl.gov
Fri Nov 17 14:01:19 CST 2017


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/63385b77c6edf3fe10c9f4002ee1292fd7715c62

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/2f8fe118fed12357e6f9401e644cc650e21a4e69
>
>
> --- 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     discuss at mpich.org
> To manage subscription options or unsubscribe:
> https://lists.mpich.org/mailman/listinfo/discuss

_______________________________________________
discuss mailing list     discuss at mpich.org
To manage subscription options or unsubscribe:
https://lists.mpich.org/mailman/listinfo/discuss


More information about the discuss mailing list