Changeset 1affcdf3 in mainline for uspace/app/trace/trace.c

Timestamp:

2011-06-10T19:33:41Z (13 years ago)

Author:

Maurizio Lombardi <m.lombardi85@…>

Branches:

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

Children:

1878386

Parents:

13ecdac9 (diff), 79a141a (diff)
Note: this is a merge changeset, the changes displayed below correspond to the merge itself.
Use the (diff) links above to see all the changes relative to each parent.

Message:

Merge mainline changes

File:

• uspace/app/trace/trace.c (modified) (21 diffs)

uspace/app/trace/trace.c

@@ -72 +72 @@
 ipc_call_t thread_ipc_req[THBUF_SIZE];
 
-int phoneid;
+async_sess_t *sess;
 bool abort_trace;
 
@@ -81 +81 @@
 
 static bool cev_valid;
-static console_event_t cev;
+static kbd_event_t cev;
 
 void thread_trace_start(uintptr_t thread_hash);
@@ -146 +146 @@
 static int connect_task(task_id_t task_id)
 {
-        int rc;
-
-        rc = async_connect_kbox(task_id);
-
-        if (rc == ENOTSUP) {
-                printf("You do not have userspace debugging support "
-                    "compiled in the kernel.\n");
-                printf("Compile kernel with 'Support for userspace debuggers' "
-                    "(CONFIG_UDEBUG) enabled.\n");
-                return rc;
-        }
-
-        if (rc < 0) {
+        async_sess_t *ksess = async_connect_kbox(task_id);
+        
+        if (!ksess) {
+                if (errno == ENOTSUP) {
+                        printf("You do not have userspace debugging support "
+                            "compiled in the kernel.\n");
+                        printf("Compile kernel with 'Support for userspace debuggers' "
+                            "(CONFIG_UDEBUG) enabled.\n");
+                        return errno;
+                }
+                
                 printf("Error connecting\n");
-                printf("ipc_connect_task(%" PRIu64 ") -> %d ", task_id, rc);
-                return rc;
-        }
-
-        phoneid = rc;
-
-        rc = udebug_begin(phoneid);
+                printf("ipc_connect_task(%" PRIu64 ") -> %d ", task_id, errno);
+                return errno;
+        }
+        
+        int rc = udebug_begin(ksess);
         if (rc < 0) {
                 printf("udebug_begin() -> %d\n", rc);
                 return rc;
         }
-
-        rc = udebug_set_evmask(phoneid, UDEBUG_EM_ALL);
+        
+        rc = udebug_set_evmask(ksess, UDEBUG_EM_ALL);
         if (rc < 0) {
                 printf("udebug_set_evmask(0x%x) -> %d\n ", UDEBUG_EM_ALL, rc);
                 return rc;
         }
-
+        
+        sess = ksess;
         return 0;
 }
@@ -188 +185 @@
         int i;
 
-        rc = udebug_thread_read(phoneid, thread_hash_buf,
+        rc = udebug_thread_read(sess, thread_hash_buf,
                 THBUF_SIZE*sizeof(unsigned), &tb_copied, &tb_needed);
         if (rc < 0) {
@@ -314 +311 @@
 
         memset(&call, 0, sizeof(call));
-        rc = udebug_mem_read(phoneid, &call.args, sc_args[1], sizeof(call.args));
+        rc = udebug_mem_read(sess, &call.args, sc_args[1], sizeof(call.args));
 
         if (rc >= 0) {
@@ -325 +322 @@
         ipc_call_t question, reply;
         int rc;
-        int phoneidx;
-
-//      printf("sc_ipc_call_sync_fast()\n");
-        phoneidx = sc_args[0];
+        int phoneid;
+
+        phoneid = sc_args[0];
 
         IPC_SET_IMETHOD(question, sc_args[1]);
@@ -337 +333 @@
         IPC_SET_ARG5(question, 0);
 
-//      printf("memset\n");
         memset(&reply, 0, sizeof(reply));
-//      printf("udebug_mem_read(phone=%d, buffer_ptr=%u, src_addr=%d, n=%d\n",
-//              phoneid, &reply.args, sc_args[5], sizeof(reply.args));
-        rc = udebug_mem_read(phoneid, &reply.args, sc_args[5], sizeof(reply.args));
-//      printf("dmr->%d\n", rc);
-        if (rc < 0) return;
-
-//      printf("call ipc_call_sync\n");
-        ipcp_call_sync(phoneidx, &question, &reply);
+        rc = udebug_mem_read(sess, &reply.args, sc_args[5], sizeof(reply.args));
+        if (rc < 0)
+                return;
+        
+        ipcp_call_sync(phoneid, &question, &reply);
 }
 
@@ -355 +347 @@
 
         memset(&question, 0, sizeof(question));
-        rc = udebug_mem_read(phoneid, &question.args, sc_args[1],
+        rc = udebug_mem_read(sess, &question.args, sc_args[1],
             sizeof(question.args));
 
@@ -372 +364 @@
 
         memset(&reply, 0, sizeof(reply));
-        rc = udebug_mem_read(phoneid, &reply.args, sc_args[2],
+        rc = udebug_mem_read(sess, &reply.args, sc_args[2],
             sizeof(reply.args));
 
@@ -391 +383 @@
 
         memset(&call, 0, sizeof(call));
-        rc = udebug_mem_read(phoneid, &call, sc_args[0], sizeof(call));
-//      printf("udebug_mem_read(phone %d, dest %d, app-mem src %d, size %d -> %d\n",
-//              phoneid, (int)&call, sc_args[0], sizeof(call), rc);
-
-        if (rc >= 0) {
+        rc = udebug_mem_read(sess, &call, sc_args[0], sizeof(call));
+        
+        if (rc >= 0)
                 ipcp_call_in(&call, sc_rc);
-        }
 }
 
@@ -407 +396 @@
 
         /* Read syscall arguments */
-        rc = udebug_args_read(phoneid, thread_hash, sc_args);
-
-        async_serialize_start();
-
-//      printf("[%d] ", thread_id);
+        rc = udebug_args_read(sess, thread_hash, sc_args);
 
         if (rc < 0) {
                 printf("error\n");
-                async_serialize_end();
                 return;
         }
@@ -432 +416 @@
                 break;
         }
-
-        async_serialize_end();
 }
 
@@ -444 +426 @@
 
         /* Read syscall arguments */
-        rc = udebug_args_read(phoneid, thread_hash, sc_args);
-
-        async_serialize_start();
+        rc = udebug_args_read(sess, thread_hash, sc_args);
 
 //      printf("[%d] ", thread_id);
@@ -452 +432 @@
         if (rc < 0) {
                 printf("error\n");
-                async_serialize_end();
                 return;
         }
@@ -481 +460 @@
                 break;
         }
-
-        async_serialize_end();
 }
 
 static void event_thread_b(uintptr_t hash)
 {
-        async_serialize_start();
         printf("New thread, hash %p\n", (void *) hash);
-        async_serialize_end();
-
         thread_trace_start(hash);
 }
@@ -527 +501 @@
 
                 /* Run thread until an event occurs */
-                rc = udebug_go(phoneid, thread_hash,
+                rc = udebug_go(sess, thread_hash,
                     &ev_type, &val0, &val1);
 
@@ -656 +630 @@
 {
         (void) arg;
-
+        
+        console_ctrl_t *console = console_init(stdin, stdout);
+        
         while (true) {
                 fibril_mutex_lock(&state_lock);
@@ -662 +638 @@
                         fibril_condvar_wait(&state_cv, &state_lock);
                 fibril_mutex_unlock(&state_lock);
-
-                if (!console_get_event(fphone(stdin), &cev))
+                
+                if (!console_get_kbd_event(console, &cev))
                         return -1;
-
+                
                 fibril_mutex_lock(&state_lock);
                 cev_valid = true;
                 fibril_condvar_broadcast(&state_cv);
-                fibril_mutex_unlock(&state_lock);               
+                fibril_mutex_unlock(&state_lock);
         }
 }
@@ -675 +651 @@
 static void trace_task(task_id_t task_id)
 {
-        console_event_t ev;
+        kbd_event_t ev;
         bool done;
         int i;
@@ -727 +703 @@
                 case KC_P:
                         printf("Pause...\n");
-                        rc = udebug_stop(phoneid, thash);
+                        rc = udebug_stop(sess, thash);
                         if (rc != EOK)
                                 printf("Error: stop -> %d\n", rc);
@@ -743 +719 @@
         printf("\nTerminate debugging session...\n");
         abort_trace = true;
-        udebug_end(phoneid);
-        async_hangup(phoneid);
+        udebug_end(sess);
+        async_hangup(sess);
 
         ipcp_cleanup();