Changeset 4888

Show
Ignore:
Timestamp:
07/02/09 09:58:01 (5 months ago)
Author:
buntinas
Message:

squashed more warnings

Location:
mpich2/trunk
Files:
18 modified

Legend:

Unmodified
Added
Removed

  • mpich2/trunk/confdb/aclocal_cc.m4

    r4878 r4888  
    11271127        -Waggregate-return 
    11281128        -Wold-style-definition 
    1129         -Wmissing-noreturn 
    11301129        -Wno-multichar 
    11311130        -Wno-deprecated-declarations 

  • mpich2/trunk/src/include/pmi.h

    r4377 r4888  
    633633typedef struct PMI_keyval_t 
    634634{ 
    635     char * key; 
     635    const char * key; 
    636636    char * val; 
    637637} PMI_keyval_t; 

  • mpich2/trunk/src/mpi/datatype/type_create_hindexed.c

    r3177 r4888  
    7474        MPID_BEGIN_ERROR_CHECKS; 
    7575        { 
    76             int i; 
     76            int j; 
    7777            MPID_Datatype *datatype_ptr = NULL; 
    7878 
    … …  
    9191                MPID_Datatype_valid_ptr(datatype_ptr, mpi_errno); 
    9292            } 
    93             for (i=0; i < count; i++) { 
    94                 MPIR_ERRTEST_ARGNEG(blocklengths[i], "blocklen", mpi_errno); 
     93            for (j=0; j < count; j++) { 
     94                MPIR_ERRTEST_ARGNEG(blocklengths[j], "blocklen", mpi_errno); 
    9595            } 
    9696            if (mpi_errno != MPI_SUCCESS) goto fn_fail; 

  • mpich2/trunk/src/mpi/datatype/type_create_struct.c

    r3177 r4888  
    7575        MPID_BEGIN_ERROR_CHECKS; 
    7676        { 
    77             int i; 
     77            int j; 
    7878            MPID_Datatype *datatype_ptr = NULL; 
    7979 
    … …  
    8888            } 
    8989 
    90             for (i=0; i < count; i++) { 
    91                 MPIR_ERRTEST_ARGNEG(array_of_blocklengths[i], "blocklen", mpi_errno); 
    92                 MPIR_ERRTEST_DATATYPE(array_of_types[i], "datatype[i]", 
     90            for (j=0; j < count; j++) { 
     91                MPIR_ERRTEST_ARGNEG(array_of_blocklengths[j], "blocklen", mpi_errno); 
     92                MPIR_ERRTEST_DATATYPE(array_of_types[j], "datatype[j]", 
    9393                                      mpi_errno); 
    9494                if (mpi_errno != MPI_SUCCESS) goto fn_fail; 
    9595                         
    96                 if (array_of_types[i] != MPI_DATATYPE_NULL && HANDLE_GET_KIND(array_of_types[i]) != HANDLE_KIND_BUILTIN) { 
    97                     MPID_Datatype_get_ptr(array_of_types[i], datatype_ptr); 
     96                if (array_of_types[j] != MPI_DATATYPE_NULL && HANDLE_GET_KIND(array_of_types[j]) != HANDLE_KIND_BUILTIN) { 
     97                    MPID_Datatype_get_ptr(array_of_types[j], datatype_ptr); 
    9898                    MPID_Datatype_valid_ptr(datatype_ptr, mpi_errno); 
    9999                    if (mpi_errno != MPI_SUCCESS) goto fn_fail; 

  • mpich2/trunk/src/mpi/datatype/type_hindexed.c

    r3177 r4888  
    9999        MPID_BEGIN_ERROR_CHECKS; 
    100100        { 
    101             int i; 
     101            int j; 
    102102            MPID_Datatype *datatype_ptr = NULL; 
    103103 
    … …  
    114114                } 
    115115                /* verify that all blocklengths are >= 0 */ 
    116                 for (i=0; i < count; i++) { 
    117                     MPIR_ERRTEST_ARGNEG(blocklens[i], "blocklen", mpi_errno); 
     116                for (j=0; j < count; j++) { 
     117                    MPIR_ERRTEST_ARGNEG(blocklens[j], "blocklen", mpi_errno); 
    118118                } 
    119119            } 

  • mpich2/trunk/src/mpi/datatype/type_indexed.c

    r3177 r4888  
    9797        MPID_BEGIN_ERROR_CHECKS; 
    9898        { 
    99             int i; 
     99            int j; 
    100100            MPID_Datatype *datatype_ptr = NULL; 
    101101 
    … …  
    112112                } 
    113113                /* verify that all blocklengths are >= 0 */ 
    114                 for (i=0; i < count; i++) { 
    115                     MPIR_ERRTEST_ARGNEG(blocklens[i], "blocklen", mpi_errno); 
     114                for (j=0; j < count; j++) { 
     115                    MPIR_ERRTEST_ARGNEG(blocklens[j], "blocklen", mpi_errno); 
    116116                } 
    117117            } 

  • mpich2/trunk/src/mpi/errhan/dynerrutil.c

    r3717 r4888  
    195195  This routine should be run within a SINGLE_CS in the multithreaded case. 
    196196*/ 
    197 int MPIR_Err_add_class() 
     197int MPIR_Err_add_class(void) 
    198198{ 
    199199    int new_class; 

  • mpich2/trunk/src/mpid/ch3/channels/nemesis/nemesis/include/mpid_nem_impl.h

    r4707 r4888  
    105105    MPID_nem_pkt_lmt_cookie_t lmt_cookie; 
    106106} MPIDI_CH3_nem_pkt_t; 
     107 
     108 
     109/* MPID_PKT_DECL_CAST(pkt_u_var, s_pkt_type, s_pkt_p_var) 
     110   To avoid strict aliasing warnings when doing something like: 
     111       pkt_t upkt; 
     112       rts_pkt_t * const rts_pkt = (rts_pkt_t *)&upkt; 
     113   this macro does the same but through a union. 
     114 
     115   pkt_u_var -- variable name of the packet union 
     116   s_pkt_type -- type of the sub-packet 
     117   s_pkt_p_var -- variable name of the sub-packet pointer 
     118 */ 
     119#define MPID_PKT_DECL_CAST(pkt_u_var, s_pkt_type, s_pkt_p_var)  \ 
     120    union                                                       \ 
     121    {                                                           \ 
     122        MPIDI_CH3_Pkt_t p;                                      \ 
     123        s_pkt_type s;                                           \ 
     124    } pkt_u_var;                                                \ 
     125    s_pkt_type * const s_pkt_p_var = &pkt_u_var.s 
    107126 
    108127     
    … …  
    153172 
    154173#define MPID_nem_lmt_send_CTS(vc, rreq, r_cookie_buf, r_cookie_len) do {                                \ 
    155         MPIDI_CH3_Pkt_t _upkt;                                                                          \ 
    156         MPID_nem_pkt_lmt_cts_t * const _cts_pkt = (MPID_nem_pkt_lmt_cts_t *)&_upkt;                     \ 
     174        MPID_PKT_DECL_CAST(_upkt, MPID_nem_pkt_lmt_cts_t, _cts_pkt);                                    \ 
    157175        MPID_Request *_cts_req;                                                                         \ 
    158176        MPID_IOV _iov[2];                                                                               \ 
    … …  
    187205{ 
    188206    int mpi_errno = MPI_SUCCESS; 
    189     MPIDI_CH3_Pkt_t _upkt; 
    190     MPID_nem_pkt_lmt_cookie_t * const cookie_pkt = (MPID_nem_pkt_lmt_cookie_t *)&_upkt; 
     207    MPID_PKT_DECL_CAST(_upkt, MPID_nem_pkt_lmt_cookie_t, cookie_pkt); 
    191208    MPID_Request *cookie_req; 
    192209    MPID_IOV iov[2]; 
    … …  
    232249         
    233250#define MPID_nem_lmt_send_DONE(vc, rreq) do {                                                                   \ 
    234         MPIDI_CH3_Pkt_t _upkt;                                                                                  \ 
    235         MPID_nem_pkt_lmt_done_t * const _done_pkt = (MPID_nem_pkt_lmt_done_t *)&_upkt;                          \ 
     251        MPID_PKT_DECL_CAST(_upkt, MPID_nem_pkt_lmt_done_t, _done_pkt);                                          \ 
    236252        MPID_Request *_done_req;                                                                                \ 
    237253                                                                                                                \ 
    … …  
    240256        _done_pkt->req_id = (rreq)->ch.lmt_req_id;                                                              \ 
    241257                                                                                                                \ 
    242         mpi_errno = MPIDI_CH3_iStartMsg((vc), _done_pkt, sizeof(*_done_pkt), &_done_req);                        \ 
     258        mpi_errno = MPIDI_CH3_iStartMsg((vc), _done_pkt, sizeof(*_done_pkt), &_done_req);                       \ 
    243259        MPIU_ERR_CHKANDJUMP(mpi_errno, mpi_errno, MPI_ERR_OTHER, "**donepkt");                                  \ 
    244260        if (_done_req != NULL)                                                                                  \ 

  • mpich2/trunk/src/mpid/ch3/channels/nemesis/nemesis/include/mpid_nem_inline.h

    r4707 r4888  
    537537{ 
    538538    MPID_nem_cell_ptr_t el; 
    539     MPIDI_msg_sz_t datalen;     
     539    MPIDI_msg_sz_t datalen; 
    540540    int my_rank; 
    541541    MPIDI_msg_sz_t last; 
    … …  
    576576 
    577577            /* copy header */ 
    578             MPIU_Memcpy((char *)pbox->cell.pkt.mpich2.payload, header, header_sz); 
    579  
     578            { 
     579                void * p = (void *)pbox->cell.pkt.mpich2.payload; 
     580                MPIU_Memcpy(p, header, header_sz); 
     581            /* MPIU_Memcpy((void *)pbox->cell.pkt.mpich2.payload, header, header_sz); */ 
     582            } 
     583             
    580584            /* copy data */ 
    581585            last = segment_size; 
    … …  
    622626 
    623627    /* copy header */ 
    624     MPIU_Memcpy(el->pkt.mpich2.payload, header, header_sz); 
     628    MPIU_Memcpy((void *)el->pkt.mpich2.payload, header, header_sz); 
    625629     
    626630    /* copy data */ 

  • mpich2/trunk/src/mpid/ch3/channels/nemesis/nemesis/netmod/tcp/socksm.c

    r4865 r4888  
    10041004{ 
    10051005    int mpi_errno = MPI_SUCCESS; 
    1006     MPID_NEM_TCP_SOCK_STATUS_t stat; 
     1006    MPID_NEM_TCP_SOCK_STATUS_t status; 
    10071007    MPIDI_STATE_DECL(MPID_STATE_STATE_TC_C_CNTING_HANDLER); 
    10081008 
    10091009    MPIDI_FUNC_ENTER(MPID_STATE_STATE_TC_C_CNTING_HANDLER); 
    10101010    
    1011     stat = MPID_nem_tcp_check_sock_status(plfd); 
    1012  
    1013     if (stat == MPID_NEM_TCP_SOCK_CONNECTED) { 
     1011    status = MPID_nem_tcp_check_sock_status(plfd); 
     1012 
     1013    if (status == MPID_NEM_TCP_SOCK_CONNECTED) { 
    10141014        CHANGE_STATE(sc, CONN_STATE_TC_C_CNTD); 
    10151015    } 
    1016     else if (stat == MPID_NEM_TCP_SOCK_ERROR_EOF) { 
     1016    else if (status == MPID_NEM_TCP_SOCK_ERROR_EOF) { 
    10171017        MPIU_DBG_MSG_FMT(NEM_SOCK_DET, VERBOSE, (MPIU_DBG_FDEST, "state_tc_c_cnting_handler(): changing to " 
    10181018              "quiescent")); 
    … …  
    10201020        /* FIXME: retry 'n' number of retries before signalling an error to VC layer. */ 
    10211021    } 
    1022     else { /* stat == MPID_NEM_TCP_SOCK_NOEVENT */ 
     1022    else { /* status == MPID_NEM_TCP_SOCK_NOEVENT */ 
    10231023        /* 
    10241024          Still connecting... let it. While still connecting, even if 
    … …  
    11831183{ 
    11841184    int mpi_errno = MPI_SUCCESS; 
    1185     MPID_NEM_TCP_SOCK_STATUS_t stat; 
     1185    MPID_NEM_TCP_SOCK_STATUS_t status; 
    11861186    int got_sc_eof = 0; 
    11871187    MPIDI_STATE_DECL(MPID_STATE_STATE_L_CNTD_HANDLER); 
    … …  
    11891189    MPIDI_FUNC_ENTER(MPID_STATE_STATE_L_CNTD_HANDLER); 
    11901190 
    1191     stat = MPID_nem_tcp_check_sock_status(plfd); 
    1192     if (stat == MPID_NEM_TCP_SOCK_ERROR_EOF) { 
     1191    status = MPID_nem_tcp_check_sock_status(plfd); 
     1192    if (status == MPID_NEM_TCP_SOCK_ERROR_EOF) { 
    11931193        MPIU_DBG_MSG_FMT(NEM_SOCK_DET, VERBOSE, (MPIU_DBG_FDEST, "state_l_cntd_handler() 1: changing to " 
    11941194            "quiescent")); 
    … …  
    12791279{ 
    12801280    int mpi_errno = MPI_SUCCESS; 
    1281     MPID_NEM_TCP_SOCK_STATUS_t stat; 
     1281    MPID_NEM_TCP_SOCK_STATUS_t status; 
    12821282    sockconn_t *fnd_sc = NULL; 
    12831283    int snd_nak = FALSE; 
    … …  
    12861286    MPIDI_FUNC_ENTER(MPID_STATE_STATE_L_RANKRCVD_HANDLER); 
    12871287 
    1288     stat = MPID_nem_tcp_check_sock_status(plfd); 
    1289     if (stat == MPID_NEM_TCP_SOCK_ERROR_EOF) { 
     1288    status = MPID_nem_tcp_check_sock_status(plfd); 
     1289    if (status == MPID_NEM_TCP_SOCK_ERROR_EOF) { 
    12901290        MPIU_DBG_MSG_FMT(NEM_SOCK_DET, VERBOSE, (MPIU_DBG_FDEST, "state_l_rankrcvd_handler() 1: changing to quiescent")); 
    12911291        CHANGE_STATE(sc, CONN_STATE_TS_D_QUIESCENT); 
    … …  
    13361336{ 
    13371337    int mpi_errno = MPI_SUCCESS; 
    1338     MPID_NEM_TCP_SOCK_STATUS_t stat; 
     1338    MPID_NEM_TCP_SOCK_STATUS_t status; 
    13391339    int snd_nak = FALSE; 
    13401340    MPIDI_STATE_DECL(MPID_STATE_STATE_L_TMPVCRCVD_HANDLER); 
    … …  
    13421342    MPIDI_FUNC_ENTER(MPID_STATE_STATE_L_TMPVCRCVD_HANDLER); 
    13431343 
    1344     stat = MPID_nem_tcp_check_sock_status(plfd); 
    1345     if (stat == MPID_NEM_TCP_SOCK_ERROR_EOF) { 
     1344    status = MPID_nem_tcp_check_sock_status(plfd); 
     1345    if (status == MPID_NEM_TCP_SOCK_ERROR_EOF) { 
    13461346        CHANGE_STATE(sc, CONN_STATE_TS_D_QUIESCENT); 
    13471347        goto fn_exit; 

  • mpich2/trunk/src/mpid/ch3/channels/nemesis/nemesis/src/mpid_nem_lmt.c

    r4707 r4888  
    6666{ 
    6767    int mpi_errno = MPI_SUCCESS; 
    68     MPIDI_CH3_Pkt_t upkt; 
    69     MPID_nem_pkt_lmt_rts_t * const rts_pkt = (MPID_nem_pkt_lmt_rts_t *)&upkt; 
     68    MPID_PKT_DECL_CAST(upkt, MPID_nem_pkt_lmt_rts_t, rts_pkt); 
    7069    MPIDI_VC_t *vc; 
    7170    MPID_Request *sreq =*sreq_p; 
    … …  
    10099    MPIDI_Request_set_seqnum(sreq, seqnum); 
    101100 
    102     mpi_errno = ((MPIDI_CH3I_VC *)vc->channel_private)->lmt_initiate_lmt(vc, &upkt, sreq); 
     101    mpi_errno = ((MPIDI_CH3I_VC *)vc->channel_private)->lmt_initiate_lmt(vc, &upkt.p, sreq); 
    103102    if (mpi_errno) MPIU_ERR_POP(mpi_errno); 
    104103 

  • mpich2/trunk/src/mpid/ch3/channels/nemesis/src/ch3_finalize.c

    r792 r4888  
    1111#undef FCNAME 
    1212#define FCNAME MPIDI_QUOTE(FUNCNAME) 
    13 int MPIDI_CH3_Finalize() 
     13int MPIDI_CH3_Finalize(void) 
    1414{ 
    1515    int mpi_errno = MPI_SUCCESS; 

  • mpich2/trunk/src/mpid/ch3/channels/nemesis/src/ch3_init.c

    r4411 r4888  
    310310#undef FCNAME 
    311311#define FCNAME MPIDI_QUOTE(FUNCNAME) 
    312 int MPIDI_CH3_InitCompleted() 
     312int MPIDI_CH3_InitCompleted(void) 
    313313{ 
    314314    int mpi_errno = MPI_SUCCESS; 

  • mpich2/trunk/src/mpid/ch3/channels/nemesis/src/ch3i_comm.c

    r4538 r4888  
    311311#undef FCNAME 
    312312#define FCNAME MPIDI_QUOTE(FUNCNAME) 
    313 int MPID_nem_coll_barrier_init () 
     313int MPID_nem_coll_barrier_init(void) 
    314314{ 
    315315    int mpi_errno = MPI_SUCCESS; 

  • mpich2/trunk/src/mpid/ch3/src/ch3u_eager.c

    r4411 r4888  
    385385            } 
    386386            else { 
    387                 MPIDI_msg_sz_t data_sz, last; 
     387                MPIDI_msg_sz_t recv_data_sz, last; 
    388388                /* user buffer is not contiguous.  Use the segment 
    389389                   code to unpack it, handling various errors and  
    … …  
    397397                                  rreq->dev.datatype, rreq->dev.segment_ptr, 0); 
    398398 
    399                 data_sz = rreq->dev.recv_data_sz; 
    400                 last    = data_sz; 
     399                recv_data_sz = rreq->dev.recv_data_sz; 
     400                last    = recv_data_sz; 
    401401                MPID_Segment_unpack( rreq->dev.segment_ptr, 0,  
    402402                                     &last, eagershort_pkt->data ); 
    403                 if (last != data_sz) { 
     403                if (last != recv_data_sz) { 
    404404                    /* --BEGIN ERROR HANDLING-- */ 
    405405                    /* There are two cases:  a datatype mismatch (could 
    … …  
    417417        } 
    418418        else { 
    419             MPIDI_msg_sz_t data_sz; 
     419            MPIDI_msg_sz_t recv_data_sz; 
    420420            /* This is easy; copy the data into a temporary buffer. 
    421421               To begin with, we use the same temporary location as 
    … …  
    429429               a buffer that we've allocated). */ 
    430430            /* printf( "Allocating into tmp\n" ); fflush(stdout); */ 
    431             data_sz = rreq->dev.recv_data_sz; 
    432             rreq->dev.tmpbuf = MPIU_Malloc(data_sz); 
     431            recv_data_sz = rreq->dev.recv_data_sz; 
     432            rreq->dev.tmpbuf = MPIU_Malloc(recv_data_sz); 
    433433            if (!rreq->dev.tmpbuf) { 
    434434                MPIU_ERR_SETANDJUMP(mpi_errno,MPI_ERR_OTHER,"**nomem"); 
    435435            } 
    436             rreq->dev.tmpbuf_sz = data_sz; 
    437             /* Copy the payload. We could optimize this if data_sz & 0x3 == 0  
    438                (copy (data_sz >> 2) ints, inline that since data size is  
     436            rreq->dev.tmpbuf_sz = recv_data_sz; 
     437            /* Copy the payload. We could optimize this if recv_data_sz & 0x3 == 0  
     438               (copy (recv_data_sz >> 2) ints, inline that since data size is  
    439439               currently limited to 4 ints */ 
    440440            { 
    … …  
    444444                    (unsigned char *)rreq->dev.tmpbuf; 
    445445                int i; 
    446                 for (i=0; i<data_sz; i++) { 
     446                for (i=0; i<recv_data_sz; i++) { 
    447447                    *bufp++ = *p++; 
    448448                } 

  • mpich2/trunk/src/mpid/ch3/src/ch3u_rma_ops.c

    r4062 r4888  
    475475            int vec_len, i, type_size, count; 
    476476            MPI_Datatype type; 
    477             MPID_Datatype *dtp; 
    478477            MPI_Aint true_lb, true_extent, extent; 
    479478            void *tmp_buf=NULL, *source_buf, *target_buf; 

  • mpich2/trunk/src/pmi/simple/simple_pmi.c

    r4377 r4888  
    13121312    int pid, rc; 
    13131313    int singinit_listen_sock, stdin_sock, stdout_sock, stderr_sock; 
    1314     char *newargv[8], charpid[8], port_c[8]; 
     1314    const char *newargv[8]; 
     1315    char charpid[8], port_c[8]; 
    13151316    struct sockaddr_in sin; 
    13161317    socklen_t len; 
    … …  
    13461347        newargv[5] = charpid; 
    13471348        newargv[6] = NULL; 
    1348         rc = execvp(newargv[0],newargv); 
     1349        rc = execvp(newargv[0], (char **)newargv); 
    13491350        perror("PMII_singinit: execv failed"); 
    13501351        PMIU_printf(1, "  This singleton init program attempted to access some feature\n"); 

  • mpich2/trunk/src/util/dbg/dbg_printf.c

    r4629 r4888  
    353353    mpiu_dbg_initialized = MPIU_DBG_UNINIT; 
    354354static char filePatternBuf[MAXPATHLEN] = ""; 
    355 static char *filePattern = "-stdout-"; /* "log%d.log"; */ 
    356 static char *defaultFilePattern = "dbg@W%w-@%d@T-%t@.log"; 
     355static const char *filePattern = "-stdout-"; /* "log%d.log"; */ 
     356static const char *defaultFilePattern = "dbg@W%w-@%d@T-%t@.log"; 
    357357static char temp_filename[MAXPATHLEN] = ""; 
    358358static int worldNum  = 0; 
    … …  
    907907#endif 
    908908    static char worldNumAsChar[10] = "0"; 
    909     char *pDest, *p; 
     909    char *pDest; 
     910    const char *p; 
    910911 
    911912    /* FIXME: This is a hack to handle the common case of two worlds */