Changeset ae5aa90 in mainline for kernel/generic/src/udebug/udebug_ops.c

Timestamp:

2009-05-14T21:29:00Z (15 years ago)

Author:

Jiri Svoboda <jirik.svoboda@…>

Branches:

lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export

Children:

9ba6262

Parents:

8ebc8bf4

Message:

Clean up various log messages, mostly udebug.

File:

• kernel/generic/src/udebug/udebug_ops.c (modified) (10 diffs)

kernel/generic/src/udebug/udebug_ops.c

@@ -182 +182 @@
         link_t *cur;
 
-        LOG("udebug_begin()\n");
-
-        mutex_lock(&TASK->udebug.lock);
-        LOG("debugging task %llu\n", TASK->taskid);
+        LOG("Debugging task %llu", TASK->taskid);
+        mutex_lock(&TASK->udebug.lock);
 
         if (TASK->udebug.dt_state != UDEBUG_TS_INACTIVE) {
                 mutex_unlock(&TASK->udebug.lock);
-                LOG("udebug_begin(): busy error\n");
-
                 return EBUSY;
         }
@@ -218 +214 @@
 
         mutex_unlock(&TASK->udebug.lock);
-
-        LOG("udebug_begin() done (%s)\n", 
-            reply ? "reply" : "stoppability wait");
-
         return reply;
 }
@@ -234 +226 @@
         int rc;
 
-        LOG("udebug_end()\n");
-
-        mutex_lock(&TASK->udebug.lock);
-        LOG("task %" PRIu64 "\n", TASK->taskid);
-
+        LOG("Task %" PRIu64, TASK->taskid);
+
+        mutex_lock(&TASK->udebug.lock);
         rc = udebug_task_cleanup(TASK);
-
         mutex_unlock(&TASK->udebug.lock);
 
@@ -255 +244 @@
 int udebug_set_evmask(udebug_evmask_t mask)
 {
-        LOG("udebug_set_mask()\n");
+        LOG("mask = 0x%x", mask);
 
         mutex_lock(&TASK->udebug.lock);
@@ -261 +250 @@
         if (TASK->udebug.dt_state != UDEBUG_TS_ACTIVE) {
                 mutex_unlock(&TASK->udebug.lock);
-                LOG("udebug_set_mask(): not active debuging session\n");
-
                 return EINVAL;
         }
 
         TASK->udebug.evmask = mask;
-
         mutex_unlock(&TASK->udebug.lock);
 
@@ -318 +304 @@
         int rc;
 
-        LOG("udebug_stop()\n");
+        LOG("udebug_stop()");
 
         /*
@@ -341 +327 @@
          * Answer GO call.
          */
-        LOG("udebug_stop - answering go call\n");
 
         /* Make sure nobody takes this call away from us. */
@@ -349 +334 @@
         IPC_SET_RETVAL(call->data, 0);
         IPC_SET_ARG1(call->data, UDEBUG_EVENT_STOP);
-        LOG("udebug_stop/ipc_answer\n");
 
         THREAD->udebug.cur_event = UDEBUG_EVENT_STOP;
@@ -359 +343 @@
         mutex_unlock(&TASK->udebug.lock);
 
-        LOG("udebog_stop/done\n");
         return 0;
 }
@@ -393 +376 @@
         size_t max_ids;
 
-        LOG("udebug_thread_read()\n");
+        LOG("udebug_thread_read()");
 
         /* Allocate a buffer to hold thread IDs */