Changeset 46577995 in mainline for kernel/generic/src/ipc/ipc.c


Ignore:
Timestamp:
2018-01-04T20:50:52Z (8 years ago)
Author:
Jiří Zárevúcky <zarevucky.jiri@…>
Children:
e211ea04
Parents:
facacc71
git-author:
Jiří Zárevúcky <zarevucky.jiri@…> (2018-01-04 20:47:53)
git-committer:
Jiří Zárevúcky <zarevucky.jiri@…> (2018-01-04 20:50:52)
Message:

Use errno_t in all uspace and kernel code.

Change type of every variable, parameter and return value that holds an
<errno.h> constant to either errno_t (the usual case), or sys_errno_t
(some places in kernel). This is for the purpose of self-documentation,
as well as for type-checking with a bit of type definition hackery.

After this commit, HelenOS is free of code that mixes error codes with non-error
values on the assumption that error codes are negative.

File:
1 edited

Legend:

Unmodified
Added
Removed

  • kernel/generic/src/ipc/ipc.c

    rfacacc71 r46577995  
    208208 *
    209209 */
    210 int ipc_call_sync(phone_t *phone, call_t *request)
     210errno_t ipc_call_sync(phone_t *phone, call_t *request)
    211211{
    212212        answerbox_t *mybox = slab_alloc(answerbox_cache, 0);
    … …  
    216216        request->callerbox = mybox;
    217217       
    218         int rc = ipc_call(phone, request);
     218        errno_t rc = ipc_call(phone, request);
    219219        if (rc != EOK) {
    220220                slab_free(answerbox_cache, mybox);
    … …  
    371371 *
    372372 */
    373 void ipc_backsend_err(phone_t *phone, call_t *call, int err)
     373void ipc_backsend_err(phone_t *phone, call_t *call, errno_t err)
    374374{
    375375        _ipc_call_actions_internal(phone, call, false);
    … …  
    415415 *
    416416 */
    417 int ipc_call(phone_t *phone, call_t *call)
     417errno_t ipc_call(phone_t *phone, call_t *call)
    418418{
    419419        mutex_lock(&phone->lock);
    … …  
    448448 *
    449449 */
    450 int ipc_phone_hangup(phone_t *phone)
     450errno_t ipc_phone_hangup(phone_t *phone)
    451451{
    452452        mutex_lock(&phone->lock);
    … …  
    495495 *
    496496 */
    497 int ipc_forward(call_t *call, phone_t *newphone, answerbox_t *oldbox,
     497errno_t ipc_forward(call_t *call, phone_t *newphone, answerbox_t *oldbox,
    498498    unsigned int mode)
    499499{
    … …  
    535535        uint64_t answer_cnt = 0;
    536536        uint64_t call_cnt = 0;
    537         int rc;
     537        errno_t rc;
    538538       
    539539restart:
Note: See TracChangeset for help on using the changeset viewer.