Changeset 0108984a in mainline


Ignore:
Timestamp:
2008-09-17T13:38:18Z (16 years ago)
Author:
Jiri Svoboda <jirik.svoboda@…>
Branches:
lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export
Children:
c9a29d6
Parents:
fb9b0b0
Message:

Use LOG() instead of printf() for debug messages.

Location:
kernel/generic/src
Files:
5 edited

Legend:

Unmodified
Added
Removed

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

    rfb9b0b0 r0108984a  
    4040#include <arch.h>
    4141#include <errno.h>
    42 #include <print.h>
     42#include <debug.h>
    4343#include <udebug/udebug_ipc.h>
    4444#include <ipc/ipc_kbox.h>
    … …  
    6666       
    6767        if (have_kb_thread) {
    68                 printf("join kb_thread..\n");
     68                LOG("join kb_thread..\n");
    6969                thread_join(TASK->kb_thread);
    7070                thread_detach(TASK->kb_thread);
    71                 printf("join done\n");
     71                LOG("join done\n");
    7272                TASK->kb_thread = NULL;
    7373        }
    … …  
    8989
    9090        (void)arg;
    91         printf("kbox_thread_proc()\n");
     91        LOG("kbox_thread_proc()\n");
    9292        done = false;
    9393
    9494        while (!done) {
    95                 //printf("kbox: wait for call\n");
    9695                call = ipc_wait_for_call(&TASK->kernel_box, SYNCH_NO_TIMEOUT,
    9796                        SYNCH_FLAGS_NONE);
    … …  
    105104
    106105                        if (method == IPC_M_PHONE_HUNGUP) {
    107                                 printf("kbox: handle hangup message\n");
     106                                LOG("kbox: handle hangup message\n");
    108107
    109108                                /* Was it our debugger, who hung up? */
    110109                                if (call->sender == TASK->udebug.debugger) {
    111110                                        /* Terminate debugging session (if any) */
    112                                         printf("kbox: terminate debug session\n");
     111                                        LOG("kbox: terminate debug session\n");
    113112                                        ipl = interrupts_disable();
    114113                                        spinlock_lock(&TASK->lock);
    … …  
    117116                                        interrupts_restore(ipl);
    118117                                } else {
    119                                         printf("kbox: was not debugger\n");
     118                                        LOG("kbox: was not debugger\n");
    120119                                }
    121120
    122                                 printf("kbox: continue with hangup message\n");
     121                                LOG("kbox: continue with hangup message\n");
    123122                                IPC_SET_RETVAL(call->data, 0);
    124123                                ipc_answer(&TASK->kernel_box, call);
    … …  
    131130                                        TASK->kb_thread = NULL;
    132131                                        done = true;
    133                                         printf("phone list is empty\n");
     132                                        LOG("phone list is empty\n");
    134133                                }
    135134                                spinlock_unlock(&TASK->answerbox.lock);
    … …  
    140139        }
    141140
    142         printf("kbox: finished\n");
     141        LOG("kbox: finished\n");
    143142}
    144143

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

    rfb9b0b0 r0108984a  
    4646#include <udebug/udebug_ipc.h>
    4747#include <arch/interrupt.h>
    48 #include <print.h>
    4948#include <syscall/copy.h>
    5049#include <security/cap.h>
    … …  
    900899                return (unative_t) rc;
    901900
    902         printf("sys_ipc_connect_kbox(%lld, %d)\n", taskid_arg.value);
     901        LOG("sys_ipc_connect_kbox(%" PRIu64 ")\n", taskid_arg.value);
    903902
    904903        return ipc_connect_kbox(taskid_arg.value);

  • kernel/generic/src/udebug/udebug.c

    rfb9b0b0 r0108984a  
    4949 
    5050#include <synch/waitq.h>
    51 #include <print.h>
     51#include <debug.h>
    5252#include <udebug/udebug.h>
    5353#include <errno.h>
    … …  
    332332        mutex_lock(&THREAD->udebug.lock);
    333333
    334         printf("udebug_thread_b_event\n");
    335         printf("- check state\n");
     334        LOG("udebug_thread_b_event\n");
     335        LOG("- check state\n");
    336336
    337337        /* Must only generate events when in debugging session */
    338338        if (THREAD->udebug.debug_active != true) {
    339                 printf("- debug_active: %s, udebug.stop: %s\n",
     339                LOG("- debug_active: %s, udebug.stop: %s\n",
    340340                        THREAD->udebug.debug_active ? "yes(+)" : "no(-)",
    341341                        THREAD->udebug.stop ? "yes(-)" : "no(+)");
    … …  
    345345        }
    346346
    347         printf("- trigger event\n");
     347        LOG("- trigger event\n");
    348348
    349349        call = THREAD->udebug.go_call;
    … …  
    366366        mutex_unlock(&TASK->udebug.lock);
    367367
    368         printf("- sleep\n");
     368        LOG("- sleep\n");
    369369        udebug_wait_for_go(&THREAD->udebug.go_wq);
    370370
    … …  
    381381        mutex_lock(&THREAD->udebug.lock);
    382382
    383 //      printf("udebug_thread_e_event\n");
    384 //      printf("- check state\n");
     383        LOG("udebug_thread_e_event\n");
     384        LOG("- check state\n");
    385385
    386386        /* Must only generate events when in debugging session */
    … …  
    394394        }
    395395
    396 //      printf("- trigger event\n");
     396        LOG("- trigger event\n");
    397397
    398398        call = THREAD->udebug.go_call;
    … …  
    428428        ipl_t ipl;
    429429
    430         printf("udebug_task_cleanup()\n");
    431         printf("task %llu\n", ta->taskid);
     430        LOG("udebug_task_cleanup()\n");
     431        LOG("task %" PRIu64 "\n", ta->taskid);
    432432
    433433        udebug_int_lock();
    … …  
    435435        if (ta->udebug.dt_state != UDEBUG_TS_BEGINNING &&
    436436            ta->udebug.dt_state != UDEBUG_TS_ACTIVE) {
    437                 printf("udebug_task_cleanup(): task not being debugged\n");
     437                LOG("udebug_task_cleanup(): task not being debugged\n");
    438438                return EINVAL;
    439439        }
    … …  
    468468
    469469                                /* Answer GO call */
    470                                 printf("answer GO call with EVENT_FINISHED\n");
     470                                LOG("answer GO call with EVENT_FINISHED\n");
    471471                                IPC_SET_RETVAL(t->udebug.go_call->data, 0);
    472472                                IPC_SET_ARG1(t->udebug.go_call->data, UDEBUG_EVENT_FINISHED);

  • kernel/generic/src/udebug/udebug_ipc.c

    rfb9b0b0 r0108984a  
    3636 */
    3737 
    38 #include <print.h>
    3938#include <proc/task.h>
    4039#include <proc/thread.h>
    … …  
    103102        int rc;
    104103
    105         //printf("debug_go()\n");
    106 
    107104        t = (thread_t *)IPC_GET_ARG2(call->data);
    108105
    … …  
    119116        thread_t *t;
    120117        int rc;
    121 
    122         printf("debug_stop()\n");
    123118
    124119        t = (thread_t *)IPC_GET_ARG2(call->data);

  • kernel/generic/src/udebug/udebug_ops.c

    rfb9b0b0 r0108984a  
    3636 */
    3737 
    38 #include <print.h>
     38#include <debug.h>
    3939#include <proc/task.h>
    4040#include <proc/thread.h>
    … …  
    163163        link_t *cur;
    164164
    165         printf("udebug_begin()\n");
    166 
    167         mutex_lock(&TASK->udebug.lock);
    168         printf("debugging task %llu\n", TASK->taskid);
     165        LOG("udebug_begin()\n");
     166
     167        mutex_lock(&TASK->udebug.lock);
     168        LOG("debugging task %llu\n", TASK->taskid);
    169169
    170170        if (TASK->udebug.dt_state != UDEBUG_TS_INACTIVE) {
    171171                mutex_unlock(&TASK->udebug.lock);
    172                 printf("udebug_begin(): busy error\n");
     172                LOG("udebug_begin(): busy error\n");
    173173
    174174                return EBUSY;
    … …  
    200200        mutex_unlock(&TASK->udebug.lock);
    201201
    202         printf("udebug_begin() done (%s)\n",
     202        LOG("udebug_begin() done (%s)\n",
    203203            reply ? "reply" : "stoppability wait");
    204204
    … …  
    210210        int rc;
    211211
    212         printf("udebug_end()\n");
    213 
    214         mutex_lock(&TASK->udebug.lock);
    215         printf("task %llu\n", TASK->taskid);
     212        LOG("udebug_end()\n");
     213
     214        mutex_lock(&TASK->udebug.lock);
     215        LOG("task %" PRIu64 "\n", TASK->taskid);
    216216
    217217        rc = udebug_task_cleanup(TASK);
    … …  
    224224int udebug_set_evmask(udebug_evmask_t mask)
    225225{
    226         printf("udebug_set_mask()\n");
    227 
    228         printf("debugging task %llu\n", TASK->taskid);
     226        LOG("udebug_set_mask()\n");
    229227
    230228        mutex_lock(&TASK->udebug.lock);
    … …  
    232230        if (TASK->udebug.dt_state != UDEBUG_TS_ACTIVE) {
    233231                mutex_unlock(&TASK->udebug.lock);
    234                 printf("udebug_set_mask(): not active debuging session\n");
     232                LOG("udebug_set_mask(): not active debuging session\n");
    235233
    236234                return EINVAL;
    … …  
    248246{
    249247        int rc;
    250 
    251 //      printf("udebug_go()\n");
    252248
    253249        /* On success, this will lock t->udebug.lock */
    … …  
    275271        int rc;
    276272
    277         printf("udebug_stop()\n");
     273        LOG("udebug_stop()\n");
    278274        mutex_lock(&TASK->udebug.lock);
    279275
    … …  
    299295         * Answer GO call
    300296         */
    301         printf("udebug_stop - answering go call\n");
     297        LOG("udebug_stop - answering go call\n");
    302298
    303299        /* Make sure nobody takes this call away from us */
    … …  
    307303        IPC_SET_RETVAL(call->data, 0);
    308304        IPC_SET_ARG1(call->data, UDEBUG_EVENT_STOP);
    309         printf("udebug_stop/ipc_answer\n");
     305        LOG("udebug_stop/ipc_answer\n");
    310306
    311307        THREAD->udebug.cur_event = UDEBUG_EVENT_STOP;
    … …  
    316312        mutex_unlock(&TASK->udebug.lock);
    317313
    318         printf("udebog_stop/done\n");
     314        LOG("udebog_stop/done\n");
    319315        return 0;
    320316}
    … …  
    331327        size_t max_ids;
    332328
    333         printf("udebug_thread_read()\n");
     329        LOG("udebug_thread_read()\n");
    334330
    335331        /* Allocate a buffer to hold thread IDs */
    … …  
    386382        unative_t *arg_buffer;
    387383
    388 //      printf("udebug_args_read()\n");
    389 
    390384        /* Prepare a buffer to hold the arguments */
    391385        arg_buffer = malloc(6 * sizeof(unative_t), 0);
    … …  
    428422        data_buffer = malloc(n, 0);
    429423
    430 //      printf("udebug_mem_read: src=%u, size=%u\n", uspace_addr, n);
    431 
    432424        /* NOTE: this is not strictly from a syscall... but that shouldn't
    433425         * be a problem */
Note: See TracChangeset for help on using the changeset viewer.