Changeset eb13ef8 in mainline for kernel/generic/src/ipc/ops


Ignore:
Timestamp:
2019-02-06T13:25:12Z (6 years ago)
Author:
Jiří Zárevúcky <zarevucky.jiri@…>
Branches:
lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export
Children:
fafb8e5
Parents:
bb97118
git-author:
Jiří Zárevúcky <zarevucky.jiri@…> (2019-02-02 14:04:02)
git-committer:
Jiří Zárevúcky <zarevucky.jiri@…> (2019-02-06 13:25:12)
Message:

Change IPC_GET_* and IPC_SET_* to accept pointer instead of lvalue

Location:
kernel/generic/src/ipc/ops
Files:
9 edited

Legend:

Unmodified
Added
Removed

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

    rbb97118 reb13ef8  
    5959
    6060        /* Remember the handle */
    61         IPC_SET_ARG5(call->data, (sysarg_t) phandle);
     61        IPC_SET_ARG5(&call->data, (sysarg_t) phandle);
    6262
    6363        return EOK;
    … …  
    6666static errno_t request_forget(call_t *call)
    6767{
    68         cap_phone_handle_t phandle = (cap_handle_t) IPC_GET_ARG5(call->data);
     68        cap_phone_handle_t phandle = (cap_handle_t) IPC_GET_ARG5(&call->data);
    6969
    7070        if (cap_handle_raw(phandle) < 0)
    … …  
    8888
    8989        /* Set the recipient-assigned label */
    90         pobj->phone->label = IPC_GET_ARG5(answer->data);
     90        pobj->phone->label = IPC_GET_ARG5(&answer->data);
    9191
    9292        /* Restore phone handle in answer's ARG5 */
    93         IPC_SET_ARG5(answer->data, IPC_GET_ARG5(*olddata));
     93        IPC_SET_ARG5(&answer->data, IPC_GET_ARG5(olddata));
    9494
    9595        /* If the user accepted the call, connect */
    96         if (IPC_GET_RETVAL(answer->data) == EOK) {
     96        if (IPC_GET_RETVAL(&answer->data) == EOK) {
    9797                /* Hand over reference from pobj to the answerbox */
    9898                (void) ipc_phone_connect(pobj->phone, &TASK->answerbox);
    … …  
    107107static errno_t answer_process(call_t *answer)
    108108{
    109         cap_phone_handle_t phandle = (cap_handle_t) IPC_GET_ARG5(answer->data);
     109        cap_phone_handle_t phandle = (cap_handle_t) IPC_GET_ARG5(&answer->data);
    110110        /* Move the reference from answer->priv to pobj */
    111111        kobject_t *pobj = (kobject_t *) answer->priv;
    112112        answer->priv = 0;
    113113
    114         if (IPC_GET_RETVAL(answer->data)) {
     114        if (IPC_GET_RETVAL(&answer->data)) {
    115115                if (cap_handle_raw(phandle) >= 0) {
    116116                        /*

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

    rbb97118 reb13ef8  
    4949                 * Set the sender-assigned label to the new phone.
    5050                 */
    51                 pobj->phone->label = IPC_GET_ARG5(call->data);
     51                pobj->phone->label = IPC_GET_ARG5(&call->data);
    5252        }
    5353        call->priv = (sysarg_t) pobj;
    54         IPC_SET_ARG5(call->data, cap_handle_raw(phandle));
     54        IPC_SET_ARG5(&call->data, cap_handle_raw(phandle));
    5555        return 0;
    5656}
    … …  
    5858static errno_t answer_cleanup(call_t *answer, ipc_data_t *olddata)
    5959{
    60         cap_phone_handle_t phandle = (cap_handle_t) IPC_GET_ARG5(*olddata);
     60        cap_phone_handle_t phandle = (cap_handle_t) IPC_GET_ARG5(olddata);
    6161        kobject_t *pobj = (kobject_t *) answer->priv;
    6262
    … …  
    7171static errno_t answer_preprocess(call_t *answer, ipc_data_t *olddata)
    7272{
    73         cap_phone_handle_t phandle = (cap_handle_t) IPC_GET_ARG5(*olddata);
     73        cap_phone_handle_t phandle = (cap_handle_t) IPC_GET_ARG5(olddata);
    7474        kobject_t *pobj = (kobject_t *) answer->priv;
    7575
    76         if (IPC_GET_RETVAL(answer->data) != EOK) {
     76        if (IPC_GET_RETVAL(&answer->data) != EOK) {
    7777                /* The connection was not accepted */
    7878                answer_cleanup(answer, olddata);
    … …  
    9494                } else {
    9595                        /* The answerbox is shutting down. */
    96                         IPC_SET_RETVAL(answer->data, ENOENT);
     96                        IPC_SET_RETVAL(&answer->data, ENOENT);
    9797                        answer_cleanup(answer, olddata);
    9898                }
    9999        } else {
    100                 IPC_SET_RETVAL(answer->data, ELIMIT);
     100                IPC_SET_RETVAL(&answer->data, ELIMIT);
    101101        }
    102102

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

    rbb97118 reb13ef8  
    4343static errno_t request_preprocess(call_t *call, phone_t *phone)
    4444{
    45         size_t size = IPC_GET_ARG2(call->data);
     45        size_t size = IPC_GET_ARG2(&call->data);
    4646
    4747        if (size > DATA_XFER_LIMIT) {
    48                 int flags = IPC_GET_ARG3(call->data);
     48                int flags = IPC_GET_ARG3(&call->data);
    4949
    5050                if (flags & IPC_XF_RESTRICT)
    51                         IPC_SET_ARG2(call->data, DATA_XFER_LIMIT);
     51                        IPC_SET_ARG2(&call->data, DATA_XFER_LIMIT);
    5252                else
    5353                        return ELIMIT;
    … …  
    6161        assert(!answer->buffer);
    6262
    63         if (!IPC_GET_RETVAL(answer->data)) {
     63        if (!IPC_GET_RETVAL(&answer->data)) {
    6464                /* The recipient agreed to send data. */
    65                 uintptr_t src = IPC_GET_ARG1(answer->data);
    66                 uintptr_t dst = IPC_GET_ARG1(*olddata);
    67                 size_t max_size = IPC_GET_ARG2(*olddata);
    68                 size_t size = IPC_GET_ARG2(answer->data);
     65                uintptr_t src = IPC_GET_ARG1(&answer->data);
     66                uintptr_t dst = IPC_GET_ARG1(olddata);
     67                size_t max_size = IPC_GET_ARG2(olddata);
     68                size_t size = IPC_GET_ARG2(&answer->data);
    6969
    7070                if (size && size <= max_size) {
    … …  
    7373                         * information is not lost.
    7474                         */
    75                         IPC_SET_ARG1(answer->data, dst);
     75                        IPC_SET_ARG1(&answer->data, dst);
    7676
    7777                        answer->buffer = malloc(size);
    7878                        if (!answer->buffer) {
    79                                 IPC_SET_RETVAL(answer->data, ENOMEM);
     79                                IPC_SET_RETVAL(&answer->data, ENOMEM);
    8080                                return EOK;
    8181                        }
    … …  
    8383                            (void *) src, size);
    8484                        if (rc) {
    85                                 IPC_SET_RETVAL(answer->data, rc);
     85                                IPC_SET_RETVAL(&answer->data, rc);
    8686                                /*
    8787                                 * answer->buffer will be cleaned up in
    … …  
    9191                        }
    9292                } else if (!size) {
    93                         IPC_SET_RETVAL(answer->data, EOK);
     93                        IPC_SET_RETVAL(&answer->data, EOK);
    9494                } else {
    95                         IPC_SET_RETVAL(answer->data, ELIMIT);
     95                        IPC_SET_RETVAL(&answer->data, ELIMIT);
    9696                }
    9797        }
    … …  
    103103{
    104104        if (answer->buffer) {
    105                 uintptr_t dst = IPC_GET_ARG1(answer->data);
    106                 size_t size = IPC_GET_ARG2(answer->data);
     105                uintptr_t dst = IPC_GET_ARG1(&answer->data);
     106                size_t size = IPC_GET_ARG2(&answer->data);
    107107                errno_t rc;
    108108
    109109                rc = copy_to_uspace((void *) dst, answer->buffer, size);
    110110                if (rc)
    111                         IPC_SET_RETVAL(answer->data, rc);
     111                        IPC_SET_RETVAL(&answer->data, rc);
    112112        }
    113113

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

    rbb97118 reb13ef8  
    4343static errno_t request_preprocess(call_t *call, phone_t *phone)
    4444{
    45         uintptr_t src = IPC_GET_ARG1(call->data);
    46         size_t size = IPC_GET_ARG2(call->data);
     45        uintptr_t src = IPC_GET_ARG1(&call->data);
     46        size_t size = IPC_GET_ARG2(&call->data);
    4747
    4848        if (size > DATA_XFER_LIMIT) {
    49                 int flags = IPC_GET_ARG3(call->data);
     49                int flags = IPC_GET_ARG3(&call->data);
    5050
    5151                if (flags & IPC_XF_RESTRICT) {
    5252                        size = DATA_XFER_LIMIT;
    53                         IPC_SET_ARG2(call->data, size);
     53                        IPC_SET_ARG2(&call->data, size);
    5454                } else
    5555                        return ELIMIT;
    … …  
    7575        assert(answer->buffer);
    7676
    77         if (!IPC_GET_RETVAL(answer->data)) {
     77        if (!IPC_GET_RETVAL(&answer->data)) {
    7878                /* The recipient agreed to receive data. */
    79                 uintptr_t dst = (uintptr_t)IPC_GET_ARG1(answer->data);
    80                 size_t size = (size_t)IPC_GET_ARG2(answer->data);
    81                 size_t max_size = (size_t)IPC_GET_ARG2(*olddata);
     79                uintptr_t dst = (uintptr_t)IPC_GET_ARG1(&answer->data);
     80                size_t size = (size_t)IPC_GET_ARG2(&answer->data);
     81                size_t max_size = (size_t)IPC_GET_ARG2(olddata);
    8282
    8383                if (size <= max_size) {
    … …  
    8585                            answer->buffer, size);
    8686                        if (rc)
    87                                 IPC_SET_RETVAL(answer->data, rc);
     87                                IPC_SET_RETVAL(&answer->data, rc);
    8888                } else {
    89                         IPC_SET_RETVAL(answer->data, ELIMIT);
     89                        IPC_SET_RETVAL(&answer->data, ELIMIT);
    9090                }
    9191        }

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

    rbb97118 reb13ef8  
    4747{
    4848        if (answer->buffer) {
    49                 uintptr_t dst = IPC_GET_ARG1(answer->data);
    50                 size_t size = IPC_GET_ARG2(answer->data);
     49                uintptr_t dst = IPC_GET_ARG1(&answer->data);
     50                size_t size = IPC_GET_ARG2(&answer->data);
    5151                errno_t rc;
    5252
    5353                rc = copy_to_uspace((void *) dst, answer->buffer, size);
    5454                if (rc)
    55                         IPC_SET_RETVAL(answer->data, rc);
     55                        IPC_SET_RETVAL(&answer->data, rc);
    5656        }
    5757

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

    rbb97118 reb13ef8  
    6868                return EOK;
    6969
    70         if (!IPC_GET_RETVAL(answer->data)) {
     70        if (!IPC_GET_RETVAL(&answer->data)) {
    7171
    7272                pte_t pte;
    … …  
    7474
    7575                page_table_lock(AS, true);
    76                 bool found = page_mapping_find(AS, IPC_GET_ARG1(answer->data),
     76                bool found = page_mapping_find(AS, IPC_GET_ARG1(&answer->data),
    7777                    false, &pte);
    7878                if (found & PTE_PRESENT(&pte)) {
    … …  
    8686                                frame_reference_add(ADDR2PFN(frame));
    8787                        }
    88                         IPC_SET_ARG1(answer->data, frame);
     88                        IPC_SET_ARG1(&answer->data, frame);
    8989                } else {
    90                         IPC_SET_RETVAL(answer->data, ENOENT);
     90                        IPC_SET_RETVAL(&answer->data, ENOENT);
    9191                }
    9292                page_table_unlock(AS, true);

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

    rbb97118 reb13ef8  
    4343static errno_t answer_preprocess(call_t *answer, ipc_data_t *olddata)
    4444{
    45         if (!IPC_GET_RETVAL(answer->data)) {
     45        if (!IPC_GET_RETVAL(&answer->data)) {
    4646                irq_spinlock_lock(&answer->sender->lock, true);
    4747                as_t *as = answer->sender->as;
    … …  
    4949
    5050                uintptr_t dst_base = (uintptr_t) -1;
    51                 errno_t rc = as_area_share(AS, IPC_GET_ARG1(answer->data),
    52                     IPC_GET_ARG1(*olddata), as, IPC_GET_ARG2(answer->data),
    53                     &dst_base, IPC_GET_ARG2(*olddata));
    54                 IPC_SET_ARG5(answer->data, dst_base);
    55                 IPC_SET_RETVAL(answer->data, rc);
     51                errno_t rc = as_area_share(AS, IPC_GET_ARG1(&answer->data),
     52                    IPC_GET_ARG1(olddata), as, IPC_GET_ARG2(&answer->data),
     53                    &dst_base, IPC_GET_ARG2(olddata));
     54                IPC_SET_ARG5(&answer->data, dst_base);
     55                IPC_SET_RETVAL(&answer->data, rc);
    5656        }
    5757

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

    rbb97118 reb13ef8  
    4444static errno_t request_preprocess(call_t *call, phone_t *phone)
    4545{
    46         size_t size = as_area_get_size(IPC_GET_ARG1(call->data));
     46        size_t size = as_area_get_size(IPC_GET_ARG1(&call->data));
    4747
    4848        if (!size)
    4949                return EPERM;
    50         IPC_SET_ARG2(call->data, size);
     50        IPC_SET_ARG2(&call->data, size);
    5151
    5252        return EOK;
    … …  
    5757        errno_t rc = EOK;
    5858
    59         if (!IPC_GET_RETVAL(answer->data)) {
     59        if (!IPC_GET_RETVAL(&answer->data)) {
    6060                /* Accepted, handle as_area receipt */
    6161
    … …  
    6565
    6666                uintptr_t dst_base = (uintptr_t) -1;
    67                 rc = as_area_share(as, IPC_GET_ARG1(*olddata),
    68                     IPC_GET_ARG2(*olddata), AS, IPC_GET_ARG3(*olddata),
    69                     &dst_base, IPC_GET_ARG1(answer->data));
     67                rc = as_area_share(as, IPC_GET_ARG1(olddata),
     68                    IPC_GET_ARG2(olddata), AS, IPC_GET_ARG3(olddata),
     69                    &dst_base, IPC_GET_ARG1(&answer->data));
    7070
    7171                if (rc == EOK) {
    72                         rc = copy_to_uspace((void *) IPC_GET_ARG2(answer->data),
     72                        rc = copy_to_uspace((void *) IPC_GET_ARG2(&answer->data),
    7373                            &dst_base, sizeof(dst_base));
    7474                }
    7575
    76                 IPC_SET_RETVAL(answer->data, rc);
     76                IPC_SET_RETVAL(&answer->data, rc);
    7777        }
    7878

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

    rbb97118 reb13ef8  
    4646
    4747        kobject_t *sender_obj = kobject_get(TASK,
    48             (cap_handle_t) IPC_GET_ARG5(call->data), KOBJECT_TYPE_PHONE);
     48            (cap_handle_t) IPC_GET_ARG5(&call->data), KOBJECT_TYPE_PHONE);
    4949        if (!sender_obj)
    5050                return ENOENT;
    … …  
    6262
    6363        /* Remember the third party task hash. */
    64         IPC_SET_ARG5(call->data, (sysarg_t) other_task_s);
     64        IPC_SET_ARG5(&call->data, (sysarg_t) other_task_s);
    6565
    6666        kobject_put(sender_obj);
    … …  
    7272        errno_t rc = EOK;
    7373
    74         if (!IPC_GET_RETVAL(answer->data)) {
     74        if (!IPC_GET_RETVAL(&answer->data)) {
    7575                /* The recipient authorized the change of state. */
    7676                task_t *other_task_s;
    … …  
    7878
    7979                kobject_t *recipient_obj = kobject_get(TASK,
    80                     (cap_handle_t) IPC_GET_ARG1(answer->data),
     80                    (cap_handle_t) IPC_GET_ARG1(&answer->data),
    8181                    KOBJECT_TYPE_PHONE);
    8282                if (!recipient_obj) {
    83                         IPC_SET_RETVAL(answer->data, ENOENT);
     83                        IPC_SET_RETVAL(&answer->data, ENOENT);
    8484                        return ENOENT;
    8585                }
    … …  
    8888                if (recipient_obj->phone->state != IPC_PHONE_CONNECTED) {
    8989                        mutex_unlock(&recipient_obj->phone->lock);
    90                         IPC_SET_RETVAL(answer->data, EINVAL);
     90                        IPC_SET_RETVAL(&answer->data, EINVAL);
    9191                        kobject_put(recipient_obj);
    9292                        return EINVAL;
    … …  
    9494
    9595                other_task_r = recipient_obj->phone->callee->task;
    96                 other_task_s = (task_t *) IPC_GET_ARG5(*olddata);
     96                other_task_s = (task_t *) IPC_GET_ARG5(olddata);
    9797
    9898                /*
    … …  
    101101                 */
    102102                if (other_task_r != other_task_s) {
    103                         IPC_SET_RETVAL(answer->data, EINVAL);
     103                        IPC_SET_RETVAL(&answer->data, EINVAL);
    104104                        rc = EINVAL;
    105105                } else {
    106106                        rc = event_task_notify_5(other_task_r,
    107107                            EVENT_TASK_STATE_CHANGE, false,
    108                             IPC_GET_ARG1(*olddata),
    109                             IPC_GET_ARG2(*olddata),
    110                             IPC_GET_ARG3(*olddata),
     108                            IPC_GET_ARG1(olddata),
     109                            IPC_GET_ARG2(olddata),
     110                            IPC_GET_ARG3(olddata),
    111111                            LOWER32(olddata->task_id),
    112112                            UPPER32(olddata->task_id));
    113                         IPC_SET_RETVAL(answer->data, rc);
     113                        IPC_SET_RETVAL(&answer->data, rc);
    114114                }
    115115
Note: See TracChangeset for help on using the changeset viewer.