Changeset a46e56b in mainline for uspace/srv/volsrv/volsrv.c


Ignore:
Timestamp:
2018-03-22T06:49:35Z (7 years ago)
Author:
Jakub Jermar <jakub@…>
Branches:
lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export
Children:
77f0a1d
Parents:
3e242d2
git-author:
Jakub Jermar <jakub@…> (2018-03-21 23:29:06)
git-committer:
Jakub Jermar <jakub@…> (2018-03-22 06:49:35)
Message:

Prefer handle over ID in naming handle variables

File:
1 edited

Legend:

Unmodified
Added
Removed

  • uspace/srv/volsrv/volsrv.c

    r3e242d2 ra46e56b  
    8585}
    8686
    87 static void vol_get_parts_srv(cap_call_handle_t iid, ipc_call_t *icall)
    88 {
    89         cap_call_handle_t callid;
     87static void vol_get_parts_srv(cap_call_handle_t icall_handle, ipc_call_t *icall)
     88{
     89        cap_call_handle_t chandle;
    9090        size_t size;
    9191        size_t act_size;
    9292        errno_t rc;
    9393
    94         if (!async_data_read_receive(&callid, &size)) {
    95                 async_answer_0(callid, EREFUSED);
    96                 async_answer_0(iid, EREFUSED);
     94        if (!async_data_read_receive(&chandle, &size)) {
     95                async_answer_0(chandle, EREFUSED);
     96                async_answer_0(icall_handle, EREFUSED);
    9797                return;
    9898        }
    … …  
    100100        service_id_t *id_buf = (service_id_t *) malloc(size);
    101101        if (id_buf == NULL) {
    102                 async_answer_0(callid, ENOMEM);
    103                 async_answer_0(iid, ENOMEM);
     102                async_answer_0(chandle, ENOMEM);
     103                async_answer_0(icall_handle, ENOMEM);
    104104                return;
    105105        }
    … …  
    107107        rc = vol_part_get_ids(id_buf, size, &act_size);
    108108        if (rc != EOK) {
    109                 async_answer_0(callid, rc);
    110                 async_answer_0(iid, rc);
    111                 return;
    112         }
    113 
    114         errno_t retval = async_data_read_finalize(callid, id_buf, size);
     109                async_answer_0(chandle, rc);
     110                async_answer_0(icall_handle, rc);
     111                return;
     112        }
     113
     114        errno_t retval = async_data_read_finalize(chandle, id_buf, size);
    115115        free(id_buf);
    116116
    117         async_answer_1(iid, retval, act_size);
    118 }
    119 
    120 static void vol_part_add_srv(cap_call_handle_t iid, ipc_call_t *icall)
     117        async_answer_1(icall_handle, retval, act_size);
     118}
     119
     120static void vol_part_add_srv(cap_call_handle_t icall_handle, ipc_call_t *icall)
    121121{
    122122        service_id_t sid;
    … …  
    127127        rc = vol_part_add(sid);
    128128        if (rc != EOK) {
    129                 async_answer_0(iid, rc);
    130                 return;
    131         }
    132 
    133         async_answer_0(iid, EOK);
    134 }
    135 
    136 static void vol_part_info_srv(cap_call_handle_t iid, ipc_call_t *icall)
     129                async_answer_0(icall_handle, rc);
     130                return;
     131        }
     132
     133        async_answer_0(icall_handle, EOK);
     134}
     135
     136static void vol_part_info_srv(cap_call_handle_t icall_handle, ipc_call_t *icall)
    137137{
    138138        service_id_t sid;
    … …  
    146146        rc = vol_part_find_by_id(sid, &part);
    147147        if (rc != EOK) {
    148                 async_answer_0(iid, ENOENT);
     148                async_answer_0(icall_handle, ENOENT);
    149149                return;
    150150        }
    … …  
    152152        rc = vol_part_get_info(part, &pinfo);
    153153        if (rc != EOK) {
    154                 async_answer_0(iid, EIO);
    155                 return;
    156         }
    157 
    158         cap_call_handle_t callid;
     154                async_answer_0(icall_handle, EIO);
     155                return;
     156        }
     157
     158        cap_call_handle_t chandle;
    159159        size_t size;
    160         if (!async_data_read_receive(&callid, &size)) {
    161                 async_answer_0(callid, EREFUSED);
    162                 async_answer_0(iid, EREFUSED);
     160        if (!async_data_read_receive(&chandle, &size)) {
     161                async_answer_0(chandle, EREFUSED);
     162                async_answer_0(icall_handle, EREFUSED);
    163163                return;
    164164        }
    165165
    166166        if (size != sizeof(vol_part_info_t)) {
    167                 async_answer_0(callid, EINVAL);
    168                 async_answer_0(iid, EINVAL);
    169                 return;
    170         }
    171 
    172         rc = async_data_read_finalize(callid, &pinfo,
     167                async_answer_0(chandle, EINVAL);
     168                async_answer_0(icall_handle, EINVAL);
     169                return;
     170        }
     171
     172        rc = async_data_read_finalize(chandle, &pinfo,
    173173            min(size, sizeof(pinfo)));
    174174        if (rc != EOK) {
    175                 async_answer_0(callid, rc);
    176                 async_answer_0(iid, rc);
    177                 return;
    178         }
    179 
    180         async_answer_0(iid, EOK);
    181 }
    182 
    183 static void vol_part_empty_srv(cap_call_handle_t iid, ipc_call_t *icall)
     175                async_answer_0(chandle, rc);
     176                async_answer_0(icall_handle, rc);
     177                return;
     178        }
     179
     180        async_answer_0(icall_handle, EOK);
     181}
     182
     183static void vol_part_empty_srv(cap_call_handle_t icall_handle, ipc_call_t *icall)
    184184{
    185185        service_id_t sid;
    … …  
    192192        rc = vol_part_find_by_id(sid, &part);
    193193        if (rc != EOK) {
    194                 async_answer_0(iid, ENOENT);
     194                async_answer_0(icall_handle, ENOENT);
    195195                return;
    196196        }
    … …  
    198198        rc = vol_part_empty_part(part);
    199199        if (rc != EOK) {
    200                 async_answer_0(iid, EIO);
    201                 return;
    202         }
    203 
    204         async_answer_0(iid, EOK);
    205 }
    206 
    207 static void vol_part_get_lsupp_srv(cap_call_handle_t iid, ipc_call_t *icall)
     200                async_answer_0(icall_handle, EIO);
     201                return;
     202        }
     203
     204        async_answer_0(icall_handle, EOK);
     205}
     206
     207static void vol_part_get_lsupp_srv(cap_call_handle_t icall_handle, ipc_call_t *icall)
    208208{
    209209        vol_fstype_t fstype;
    … …  
    217217        volsrv_part_get_lsupp(fstype, &vlsupp);
    218218
    219         cap_call_handle_t callid;
     219        cap_call_handle_t chandle;
    220220        size_t size;
    221         if (!async_data_read_receive(&callid, &size)) {
    222                 async_answer_0(callid, EREFUSED);
    223                 async_answer_0(iid, EREFUSED);
     221        if (!async_data_read_receive(&chandle, &size)) {
     222                async_answer_0(chandle, EREFUSED);
     223                async_answer_0(icall_handle, EREFUSED);
    224224                return;
    225225        }
    226226
    227227        if (size != sizeof(vol_label_supp_t)) {
    228                 async_answer_0(callid, EINVAL);
    229                 async_answer_0(iid, EINVAL);
    230                 return;
    231         }
    232 
    233         rc = async_data_read_finalize(callid, &vlsupp,
     228                async_answer_0(chandle, EINVAL);
     229                async_answer_0(icall_handle, EINVAL);
     230                return;
     231        }
     232
     233        rc = async_data_read_finalize(chandle, &vlsupp,
    234234            min(size, sizeof(vlsupp)));
    235235        if (rc != EOK) {
    236                 async_answer_0(callid, rc);
    237                 async_answer_0(iid, rc);
    238                 return;
    239         }
    240 
    241         async_answer_0(iid, EOK);
    242 }
    243 
    244 
    245 static void vol_part_mkfs_srv(cap_call_handle_t iid, ipc_call_t *icall)
     236                async_answer_0(chandle, rc);
     237                async_answer_0(icall_handle, rc);
     238                return;
     239        }
     240
     241        async_answer_0(icall_handle, EOK);
     242}
     243
     244
     245static void vol_part_mkfs_srv(cap_call_handle_t icall_handle, ipc_call_t *icall)
    246246{
    247247        service_id_t sid;
    … …  
    257257            0, NULL);
    258258        if (rc != EOK) {
    259                 async_answer_0(iid, rc);
     259                async_answer_0(icall_handle, rc);
    260260                return;
    261261        }
    … …  
    267267        if (rc != EOK) {
    268268                free(label);
    269                 async_answer_0(iid, ENOENT);
     269                async_answer_0(icall_handle, ENOENT);
    270270                return;
    271271        }
    … …  
    274274        if (rc != EOK) {
    275275                free(label);
    276                 async_answer_0(iid, rc);
     276                async_answer_0(icall_handle, rc);
    277277                return;
    278278        }
    279279
    280280        free(label);
    281         async_answer_0(iid, EOK);
    282 }
    283 
    284 static void vol_client_conn(cap_call_handle_t iid, ipc_call_t *icall, void *arg)
     281        async_answer_0(icall_handle, EOK);
     282}
     283
     284static void vol_client_conn(cap_call_handle_t icall_handle, ipc_call_t *icall, void *arg)
    285285{
    286286        log_msg(LOG_DEFAULT, LVL_DEBUG, "vol_client_conn()");
    287287
    288288        /* Accept the connection */
    289         async_answer_0(iid, EOK);
     289        async_answer_0(icall_handle, EOK);
    290290
    291291        while (true) {
    292292                ipc_call_t call;
    293                 cap_call_handle_t callid = async_get_call(&call);
     293                cap_call_handle_t chandle = async_get_call(&call);
    294294                sysarg_t method = IPC_GET_IMETHOD(call);
    295295
    296296                if (!method) {
    297297                        /* The other side has hung up */
    298                         async_answer_0(callid, EOK);
     298                        async_answer_0(chandle, EOK);
    299299                        return;
    300300                }
    … …  
    302302                switch (method) {
    303303                case VOL_GET_PARTS:
    304                         vol_get_parts_srv(callid, &call);
     304                        vol_get_parts_srv(chandle, &call);
    305305                        break;
    306306                case VOL_PART_ADD:
    307                         vol_part_add_srv(callid, &call);
     307                        vol_part_add_srv(chandle, &call);
    308308                        break;
    309309                case VOL_PART_INFO:
    310                         vol_part_info_srv(callid, &call);
     310                        vol_part_info_srv(chandle, &call);
    311311                        break;
    312312                case VOL_PART_EMPTY:
    313                         vol_part_empty_srv(callid, &call);
     313                        vol_part_empty_srv(chandle, &call);
    314314                        break;
    315315                case VOL_PART_LSUPP:
    316                         vol_part_get_lsupp_srv(callid, &call);
     316                        vol_part_get_lsupp_srv(chandle, &call);
    317317                        break;
    318318                case VOL_PART_MKFS:
    319                         vol_part_mkfs_srv(callid, &call);
     319                        vol_part_mkfs_srv(chandle, &call);
    320320                        break;
    321321                default:
    322                         async_answer_0(callid, EINVAL);
     322                        async_answer_0(chandle, EINVAL);
    323323                }
    324324        }
Note: See TracChangeset for help on using the changeset viewer.