Changes in kernel/generic/src/ipc/ipc.c [5378f99:e186eb5] in mainline

File:

• kernel/generic/src/ipc/ipc.c (modified) (5 diffs)

kernel/generic/src/ipc/ipc.c

@@ -295 +295 @@
                 atomic_inc(&phone->active_calls);
                 call->data.phone = phone;
-                call->data.task = TASK;
         }
         
@@ -407 +406 @@
                         call->caller_phone = call->data.phone;
                 call->data.phone = newphone;
-                call->data.task = TASK;
         }
         
@@ -690 +688 @@
         irq_spinlock_exchange(&tasks_lock, &task->lock);
         
-        printf("[phone id] [calls] [state\n");
+        /* Print opened phones & details */
+        printf("PHONE:\n");
         
         size_t i;
         for (i = 0; i < IPC_MAX_PHONES; i++) {
                 if (SYNCH_FAILED(mutex_trylock(&task->phones[i].lock))) {
-                        printf("%-10zu (mutex busy)\n", i);
+                        printf("%zu: mutex busy\n", i);
                         continue;
                 }
                 
                 if (task->phones[i].state != IPC_PHONE_FREE) {
-                        printf("%-10zu %7" PRIun " ", i,
-                            atomic_get(&task->phones[i].active_calls));
+                        printf("%zu: ", i);
                         
                         switch (task->phones[i].state) {
                         case IPC_PHONE_CONNECTING:
-                                printf("connecting");
+                                printf("connecting ");
                                 break;
                         case IPC_PHONE_CONNECTED:
-                                printf("connected to %" PRIu64 " (%s)",
-                                    task->phones[i].callee->task->taskid,
-                                    task->phones[i].callee->task->name);
+                                printf("connected to: %p (%" PRIu64 ") ",
+                                    task->phones[i].callee,
+                                    task->phones[i].callee->task->taskid);
                                 break;
                         case IPC_PHONE_SLAMMED:
-                                printf("slammed by %p",
+                                printf("slammed by: %p ", 
                                     task->phones[i].callee);
                                 break;
                         case IPC_PHONE_HUNGUP:
-                                printf("hung up by %p",
+                                printf("hung up - was: %p ", 
                                     task->phones[i].callee);
                                 break;
@@ -724 +722 @@
                         }
                         
-                        printf("\n");
+                        printf("active: %" PRIun "\n",
+                            atomic_get(&task->phones[i].active_calls));
                 }
                 
@@ -732 +731 @@
         irq_spinlock_lock(&task->answerbox.lock, false);
         
-#ifdef __32_BITS__
-        printf("[call id ] [method] [arg1] [arg2] [arg3] [arg4] [arg5]"
-            " [flags] [sender\n");
-#endif
-        
-#ifdef __64_BITS__
-        printf("[call id         ] [method] [arg1] [arg2] [arg3] [arg4]"
-            " [arg5] [flags] [sender\n");
-#endif
-        
         link_t *cur;
         
-        printf(" --- incomming calls ---\n");
+        /* Print answerbox - calls */
+        printf("ABOX - CALLS:\n");
         for (cur = task->answerbox.calls.next; cur != &task->answerbox.calls;
             cur = cur->next) {
                 call_t *call = list_get_instance(cur, call_t, link);
-                
-#ifdef __32_BITS__
-                printf("%10p ", call);
-#endif
-                
-#ifdef __64_BITS__
-                printf("%18p ", call);
-#endif
-                
-                printf("%-8" PRIun " %-6" PRIun " %-6" PRIun " %-6" PRIun
-                    " %-6" PRIun " %-6" PRIun " %-7x %" PRIu64 " (%s)\n",
+                printf("Callid: %p Srctask:%" PRIu64 " M:%" PRIun 
+                    " A1:%" PRIun " A2:%" PRIun " A3:%" PRIun
+                    " A4:%" PRIun " A5:%" PRIun " Flags:%x\n", call,
+                    call->sender->taskid,
                     IPC_GET_IMETHOD(call->data), IPC_GET_ARG1(call->data),
                     IPC_GET_ARG2(call->data), IPC_GET_ARG3(call->data),
                     IPC_GET_ARG4(call->data), IPC_GET_ARG5(call->data),
-                    call->flags, call->sender->taskid, call->sender->name);
-        }
-        
-        printf(" --- dispatched calls ---\n");
+                    call->flags);
+        }
+        
+        /* Print answerbox - dispatched calls */
+        printf("ABOX - DISPATCHED CALLS:\n");
         for (cur = task->answerbox.dispatched_calls.next;
             cur != &task->answerbox.dispatched_calls;
             cur = cur->next) {
                 call_t *call = list_get_instance(cur, call_t, link);
-                
-#ifdef __32_BITS__
-                printf("%10p ", call);
-#endif
-                
-#ifdef __64_BITS__
-                printf("%18p ", call);
-#endif
-                
-                printf("%-8" PRIun " %-6" PRIun " %-6" PRIun " %-6" PRIun
-                    " %-6" PRIun " %-6" PRIun " %-7x %" PRIu64 " (%s)\n",
+                printf("Callid: %p Srctask:%" PRIu64 " M:%" PRIun
+                    " A1:%" PRIun " A2:%" PRIun " A3:%" PRIun
+                    " A4:%" PRIun " A5:%" PRIun " Flags:%x\n", call,
+                    call->sender->taskid,
                     IPC_GET_IMETHOD(call->data), IPC_GET_ARG1(call->data),
                     IPC_GET_ARG2(call->data), IPC_GET_ARG3(call->data),
                     IPC_GET_ARG4(call->data), IPC_GET_ARG5(call->data),
-                    call->flags, call->sender->taskid, call->sender->name);
-        }
-        
-        printf(" --- outgoing answers ---\n");
+                    call->flags);
+        }
+        
+        /* Print answerbox - answers */
+        printf("ABOX - ANSWERS:\n");
         for (cur = task->answerbox.answers.next;
             cur != &task->answerbox.answers;
             cur = cur->next) {
                 call_t *call = list_get_instance(cur, call_t, link);
-                
-#ifdef __32_BITS__
-                printf("%10p ", call);
-#endif
-                
-#ifdef __64_BITS__
-                printf("%18p ", call);
-#endif
-                
-                printf("%-8" PRIun " %-6" PRIun " %-6" PRIun " %-6" PRIun
-                    " %-6" PRIun " %-6" PRIun " %-7x %" PRIu64 " (%s)\n",
-                    IPC_GET_IMETHOD(call->data), IPC_GET_ARG1(call->data),
+                printf("Callid:%p M:%" PRIun " A1:%" PRIun " A2:%" PRIun
+                    " A3:%" PRIun " A4:%" PRIun " A5:%" PRIun " Flags:%x\n",
+                    call, IPC_GET_IMETHOD(call->data), IPC_GET_ARG1(call->data),
                     IPC_GET_ARG2(call->data), IPC_GET_ARG3(call->data),
                     IPC_GET_ARG4(call->data), IPC_GET_ARG5(call->data),
-                    call->flags, call->sender->taskid, call->sender->name);
+                    call->flags);
         }