Changeset a46e56b in mainline for uspace/srv/vfs


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

Location:
uspace/srv/vfs
Files:
4 edited

Legend:

Unmodified
Added
Removed

  • uspace/srv/vfs/vfs.c

    r3e242d2 ra46e56b  
    5555#define NAME  "vfs"
    5656
    57 static void vfs_pager(cap_call_handle_t iid, ipc_call_t *icall, void *arg)
     57static void vfs_pager(cap_call_handle_t icall_handle, ipc_call_t *icall, void *arg)
    5858{
    59         async_answer_0(iid, EOK);
     59        async_answer_0(icall_handle, EOK);
    6060
    6161        while (true) {
    6262                ipc_call_t call;
    63                 cap_call_handle_t callid = async_get_call(&call);
     63                cap_call_handle_t chandle = async_get_call(&call);
    6464
    6565                if (!IPC_GET_IMETHOD(call))
    … …  
    6868                switch (IPC_GET_IMETHOD(call)) {
    6969                case IPC_M_PAGE_IN:
    70                         vfs_page_in(callid, &call);
     70                        vfs_page_in(chandle, &call);
    7171                        break;
    7272                default:
    73                         async_answer_0(callid, ENOTSUP);
     73                        async_answer_0(chandle, ENOTSUP);
    7474                        break;
    7575                }

  • uspace/srv/vfs/vfs.h

    r3e242d2 ra46e56b  
    233233extern errno_t vfs_rdwr_internal(int, aoff64_t, bool, rdwr_io_chunk_t *);
    234234
    235 extern void vfs_connection(cap_call_handle_t iid, ipc_call_t *icall, void *arg);
     235extern void vfs_connection(cap_call_handle_t icall_handle, ipc_call_t *icall, void *arg);
    236236
    237237#endif

  • uspace/srv/vfs/vfs_ipc.c

    r3e242d2 ra46e56b  
    5050        service_id_t service_id = (service_id_t) IPC_GET_ARG1(*request);
    5151        char *fs_name = NULL;
    52         cap_call_handle_t callid;
     52        cap_call_handle_t chandle;
    5353        vfs_fs_probe_info_t info;
    5454        size_t len;
    … …  
    7272
    7373        /* Now we should get a read request */
    74         if (!async_data_read_receive(&callid, &len))
     74        if (!async_data_read_receive(&chandle, &len))
    7575                goto out;
    7676
    7777        if (len > sizeof(info))
    7878                len = sizeof(info);
    79         (void) async_data_read_finalize(callid, &info, len);
     79        (void) async_data_read_finalize(chandle, &info, len);
    8080
    8181out:
    … …  
    8585static void vfs_in_fstypes(cap_call_handle_t rid, ipc_call_t *request)
    8686{
    87         cap_call_handle_t callid;
     87        cap_call_handle_t chandle;
    8888        size_t len;
    8989        vfs_fstypes_t fstypes;
    … …  
    100100
    101101        /* Now we should get a read request */
    102         if (!async_data_read_receive(&callid, &len))
     102        if (!async_data_read_receive(&chandle, &len))
    103103                goto out;
    104104
    105105        if (len > fstypes.size)
    106106                len = fstypes.size;
    107         (void) async_data_read_finalize(callid, fstypes.buf, len);
     107        (void) async_data_read_finalize(chandle, fstypes.buf, len);
    108108
    109109out:
    … …  
    315315}
    316316
    317 void vfs_connection(cap_call_handle_t iid, ipc_call_t *icall, void *arg)
     317void vfs_connection(cap_call_handle_t icall_handle, ipc_call_t *icall, void *arg)
    318318{
    319319        bool cont = true;
    … …  
    323323         * This call needs to be answered.
    324324         */
    325         async_answer_0(iid, EOK);
     325        async_answer_0(icall_handle, EOK);
    326326
    327327        while (cont) {
    328328                ipc_call_t call;
    329                 cap_call_handle_t callid = async_get_call(&call);
     329                cap_call_handle_t chandle = async_get_call(&call);
    330330
    331331                if (!IPC_GET_IMETHOD(call))
    … …  
    334334                switch (IPC_GET_IMETHOD(call)) {
    335335                case VFS_IN_CLONE:
    336                         vfs_in_clone(callid, &call);
     336                        vfs_in_clone(chandle, &call);
    337337                        break;
    338338                case VFS_IN_FSPROBE:
    339                         vfs_in_fsprobe(callid, &call);
     339                        vfs_in_fsprobe(chandle, &call);
    340340                        break;
    341341                case VFS_IN_FSTYPES:
    342                         vfs_in_fstypes(callid, &call);
     342                        vfs_in_fstypes(chandle, &call);
    343343                        break;
    344344                case VFS_IN_MOUNT:
    345                         vfs_in_mount(callid, &call);
     345                        vfs_in_mount(chandle, &call);
    346346                        break;
    347347                case VFS_IN_OPEN:
    348                         vfs_in_open(callid, &call);
     348                        vfs_in_open(chandle, &call);
    349349                        break;
    350350                case VFS_IN_PUT:
    351                         vfs_in_put(callid, &call);
     351                        vfs_in_put(chandle, &call);
    352352                        break;
    353353                case VFS_IN_READ:
    354                         vfs_in_read(callid, &call);
     354                        vfs_in_read(chandle, &call);
    355355                        break;
    356356                case VFS_IN_REGISTER:
    357                         vfs_register(callid, &call);
     357                        vfs_register(chandle, &call);
    358358                        cont = false;
    359359                        break;
    360360                case VFS_IN_RENAME:
    361                         vfs_in_rename(callid, &call);
     361                        vfs_in_rename(chandle, &call);
    362362                        break;
    363363                case VFS_IN_RESIZE:
    364                         vfs_in_resize(callid, &call);
     364                        vfs_in_resize(chandle, &call);
    365365                        break;
    366366                case VFS_IN_STAT:
    367                         vfs_in_stat(callid, &call);
     367                        vfs_in_stat(chandle, &call);
    368368                        break;
    369369                case VFS_IN_STATFS:
    370                         vfs_in_statfs(callid, &call);
     370                        vfs_in_statfs(chandle, &call);
    371371                        break;
    372372                case VFS_IN_SYNC:
    373                         vfs_in_sync(callid, &call);
     373                        vfs_in_sync(chandle, &call);
    374374                        break;
    375375                case VFS_IN_UNLINK:
    376                         vfs_in_unlink(callid, &call);
     376                        vfs_in_unlink(chandle, &call);
    377377                        break;
    378378                case VFS_IN_UNMOUNT:
    379                         vfs_in_unmount(callid, &call);
     379                        vfs_in_unmount(chandle, &call);
    380380                        break;
    381381                case VFS_IN_WAIT_HANDLE:
    382                         vfs_in_wait_handle(callid, &call);
     382                        vfs_in_wait_handle(chandle, &call);
    383383                        break;
    384384                case VFS_IN_WALK:
    385                         vfs_in_walk(callid, &call);
     385                        vfs_in_walk(chandle, &call);
    386386                        break;
    387387                case VFS_IN_WRITE:
    388                         vfs_in_write(callid, &call);
     388                        vfs_in_write(chandle, &call);
    389389                        break;
    390390                default:
    391                         async_answer_0(callid, ENOTSUP);
     391                        async_answer_0(chandle, ENOTSUP);
    392392                        break;
    393393                }

  • uspace/srv/vfs/vfs_register.c

    r3e242d2 ra46e56b  
    195195
    196196        size_t size;
    197         cap_call_handle_t callid;
    198         if (!async_share_in_receive(&callid, &size)) {
     197        cap_call_handle_t chandle;
     198        if (!async_share_in_receive(&chandle, &size)) {
    199199                dprintf("Unexpected call\n");
    200200                list_remove(&fs_info->fs_link);
    … …  
    202202                async_hangup(fs_info->sess);
    203203                free(fs_info);
    204                 async_answer_0(callid, EINVAL);
     204                async_answer_0(chandle, EINVAL);
    205205                async_answer_0(rid, EINVAL);
    206206                return;
    … …  
    216216                async_hangup(fs_info->sess);
    217217                free(fs_info);
    218                 async_answer_0(callid, EINVAL);
     218                async_answer_0(chandle, EINVAL);
    219219                async_answer_0(rid, EINVAL);
    220220                return;
    … …  
    224224         * Commit to read-only sharing the PLB with the client.
    225225         */
    226         (void) async_share_in_finalize(callid, plb,
     226        (void) async_share_in_finalize(chandle, plb,
    227227            AS_AREA_READ | AS_AREA_CACHEABLE);
    228228
Note: See TracChangeset for help on using the changeset viewer.