Changeset 874621f in mainline for generic

Timestamp:

2006-06-06T07:40:51Z (19 years ago)

Author:

Ondrej Palkovsky <ondrap@…>

Branches:

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

Children:

0dbc4e7

Parents:

6f9a9bc

Message:

Added kernel circular buffer klog.
Added automatic killing of tasks raising inappropriate exceptions.
TODO Fix vsnprintf return value(and behaviour according to specs) and remove workaround in klog.

Location:

generic

Files:

• include/console/klog.h (added)
• include/interrupt.h (modified) (2 diffs)
• include/ipc/irq.h (modified) (2 diffs)
• include/ipc/sysipc.h (modified) (1 diff)
• include/proc/thread.h (modified) (1 diff)
• include/stackarg.h (modified) (1 diff)
• include/stdarg.h (modified) (1 diff)
• src/console/klog.c (added)
• src/interrupt/interrupt.c (modified) (1 diff)
• src/ipc/ipc.c (modified) (1 diff)
• src/ipc/irq.c (modified) (5 diffs)
• src/ipc/sysipc.c (modified) (2 diffs)
• src/main/main.c (modified) (2 diffs)
• src/proc/thread.c (modified) (1 diff)

generic/include/interrupt.h

@@ -33 +33 @@
 #include <typedefs.h>
 #include <arch/types.h>
+#include <proc/task.h>
+#include <proc/thread.h>
+#include <arch.h>
+#include <console/klog.h>
+#include <ipc/irq.h>
 
 #ifndef IVT_ITEMS
@@ -42 +47 @@
 #endif
 
+#define fault_if_from_uspace(istate, cmd, ...) \
+{ \
+        if (istate_from_uspace(istate)) { \
+                klog_printf(cmd, ##__VA_ARGS__); \
+                klog_printf("Task %lld got exception at PC:%P. Task killed.", TASK->taskid, istate_get_pc(istate)); \
+                task_kill(TASK->taskid); \
+                thread_exit(); \
+        } \
+}
+
+
 extern iroutine exc_register(int n, const char *name, iroutine f);
 extern void exc_dispatch(int n, istate_t *t);

generic/include/ipc/irq.h

@@ -30 +30 @@
 #define __IRQ_H__
 
+/** Maximum length of IPC IRQ program */
 #define IRQ_MAX_PROG_SIZE 10
+
+/** Reserved 'virtual' messages for kernel notifications */
+#define IPC_IRQ_RESERVED_VIRTUAL 10
+
+#define IPC_IRQ_KLOG  (-1)
 
 typedef enum {
@@ -60 +66 @@
 #ifdef KERNEL
 
+#include <ipc/ipc.h>
+
 extern void ipc_irq_make_table(int irqcount);
 extern int ipc_irq_register(answerbox_t *box, int irq, irq_code_t *ucode);
 extern void ipc_irq_send_notif(int irq);
+extern void ipc_irq_send_msg(int irq, __native a2, __native a3);
 extern void ipc_irq_unregister(answerbox_t *box, int irq);
 extern void irq_ipc_bind_arch(__native irq);

generic/include/ipc/sysipc.h

@@ -48 +48 @@
                               __native method, __native arg1);
 __native sys_ipc_hangup(int phoneid);
-__native sys_ipc_register_irq(__native irq, irq_code_t *ucode);
-__native sys_ipc_unregister_irq(__native irq);
+__native sys_ipc_register_irq(int irq, irq_code_t *ucode);
+__native sys_ipc_unregister_irq(int irq);
 
 #endif

generic/include/proc/thread.h

@@ -151 +151 @@
 extern thread_t *thread_create(void (* func)(void *), void *arg, task_t *task, int flags, char *name);
 extern void thread_ready(thread_t *t);
-extern void thread_exit(void);
+extern void thread_exit(void) __attribute__((noreturn));
 
 #ifndef thread_create_arch

generic/include/stackarg.h

@@ -50 +50 @@
         (*((type *)((ap).last + ((ap).pos  += sizeof(type) ) - sizeof(type))))
 
+#define va_copy(dst,src)       dst=src
 #define va_end(ap)
 

generic/include/stdarg.h

@@ -41 +41 @@
 #define va_arg(ap, type)                __builtin_va_arg(ap, type)
 #define va_end(ap)                      __builtin_va_end(ap)
+#define va_copy(dst,src)                __builtin_va_copy(dst,src)
 
 #endif

generic/src/interrupt/interrupt.c

@@ -91 +91 @@
 static void exc_undef(int n, istate_t *istate)
 {
+        fault_if_from_uspace(istate, "Unhandled exception %d.", n);
         panic("Unhandled exception %d.", n);
 }

generic/src/ipc/ipc.c

@@ -336 +336 @@
                 list_append(&request->link, &box->dispatched_calls);
         } else {
-                /* This can happen regularly after ipc_cleanup, remove
-                 * the warning in the future when the IPC is
-                 * more debugged */
-                printf("WARNING: Spurious IPC wakeup.\n");
+                /* This can happen regularly after ipc_cleanup */
                 spinlock_unlock(&box->lock);
                 goto restart;

generic/src/ipc/irq.c

@@ -157 +157 @@
 {
         ipl_t ipl;
+        int mq = irq + IPC_IRQ_RESERVED_VIRTUAL;
 
         ipl = interrupts_disable();
-        spinlock_lock(&irq_conns[irq].lock);
-        if (irq_conns[irq].box == box) {
-                irq_conns[irq].box = NULL;
-                code_free(irq_conns[irq].code);
-                irq_conns[irq].code = NULL;
-        }
-
-        spinlock_unlock(&irq_conns[irq].lock);
+        spinlock_lock(&irq_conns[mq].lock);
+        if (irq_conns[mq].box == box) {
+                irq_conns[mq].box = NULL;
+                code_free(irq_conns[mq].code);
+                irq_conns[mq].code = NULL;
+        }
+
+        spinlock_unlock(&irq_conns[mq].lock);
         interrupts_restore(ipl);
 }
@@ -175 +176 @@
         ipl_t ipl;
         irq_code_t *code;
+        int mq = irq + IPC_IRQ_RESERVED_VIRTUAL;
 
         ASSERT(irq_conns);
@@ -186 +188 @@
 
         ipl = interrupts_disable();
-        spinlock_lock(&irq_conns[irq].lock);
-
-        if (irq_conns[irq].box) {
-                spinlock_unlock(&irq_conns[irq].lock);
+        spinlock_lock(&irq_conns[mq].lock);
+
+        if (irq_conns[mq].box) {
+                spinlock_unlock(&irq_conns[mq].lock);
                 interrupts_restore(ipl);
                 code_free(code);
                 return EEXISTS;
         }
-        irq_conns[irq].box = box;
-        irq_conns[irq].code = code;
-        atomic_set(&irq_conns[irq].counter, 0);
-        spinlock_unlock(&irq_conns[irq].lock);
+        irq_conns[mq].box = box;
+        irq_conns[mq].code = code;
+        atomic_set(&irq_conns[mq].counter, 0);
+        spinlock_unlock(&irq_conns[mq].lock);
         interrupts_restore(ipl);
 
@@ -203 +205 @@
 }
 
-/** Notify process that an irq had happend
- *
- * We expect interrupts to be disabled
- */
-void ipc_irq_send_notif(int irq)
+/** Add call to proper answerbox queue
+ *
+ * Assume irq_conns[mq].lock is locked */
+static void send_call(int mq, call_t *call)
+{
+        spinlock_lock(&irq_conns[mq].box->irq_lock);
+        list_append(&call->link, &irq_conns[mq].box->irq_notifs);
+        spinlock_unlock(&irq_conns[mq].box->irq_lock);
+                
+        waitq_wakeup(&irq_conns[mq].box->wq, 0);
+}
+
+/** Send notification message
+ *
+ */
+void ipc_irq_send_msg(int irq, __native a2, __native a3)
 {
         call_t *call;
-
-        ASSERT(irq_conns);
-        spinlock_lock(&irq_conns[irq].lock);
-
-        if (irq_conns[irq].box) {
+        int mq = irq + IPC_IRQ_RESERVED_VIRTUAL;
+
+        spinlock_lock(&irq_conns[mq].lock);
+
+        if (irq_conns[mq].box) {
                 call = ipc_call_alloc(FRAME_ATOMIC);
                 if (!call) {
-                        spinlock_unlock(&irq_conns[irq].lock);
+                        spinlock_unlock(&irq_conns[mq].lock);
                         return;
                 }
@@ -223 +236 @@
                 IPC_SET_METHOD(call->data, IPC_M_INTERRUPT);
                 IPC_SET_ARG1(call->data, irq);
-                IPC_SET_ARG3(call->data, atomic_preinc(&irq_conns[irq].counter));
+                IPC_SET_ARG2(call->data, a2);
+                IPC_SET_ARG3(call->data, a3);
+                
+                send_call(mq, call);
+        }
+        spinlock_unlock(&irq_conns[mq].lock);
+}
+
+/** Notify process that an irq had happend
+ *
+ * We expect interrupts to be disabled
+ */
+void ipc_irq_send_notif(int irq)
+{
+        call_t *call;
+        int mq = irq + IPC_IRQ_RESERVED_VIRTUAL;
+
+        ASSERT(irq_conns);
+        spinlock_lock(&irq_conns[mq].lock);
+
+        if (irq_conns[mq].box) {
+                call = ipc_call_alloc(FRAME_ATOMIC);
+                if (!call) {
+                        spinlock_unlock(&irq_conns[mq].lock);
+                        return;
+                }
+                call->flags |= IPC_CALL_NOTIF;
+                IPC_SET_METHOD(call->data, IPC_M_INTERRUPT);
+                IPC_SET_ARG1(call->data, irq);
+                IPC_SET_ARG3(call->data, atomic_preinc(&irq_conns[mq].counter));
 
                 /* Execute code to handle irq */
-                code_execute(call, irq_conns[irq].code);
-
-                spinlock_lock(&irq_conns[irq].box->irq_lock);
-                list_append(&call->link, &irq_conns[irq].box->irq_notifs);
-                spinlock_unlock(&irq_conns[irq].box->irq_lock);
-
-                waitq_wakeup(&irq_conns[irq].box->wq, 0);
-        }
+                code_execute(call, irq_conns[mq].code);
                 
-        spinlock_unlock(&irq_conns[irq].lock);
-}
-
-
-/** Initialize table of interrupt handlers */
+                send_call(mq, call);
+        }
+                
+        spinlock_unlock(&irq_conns[mq].lock);
+}
+
+
+/** Initialize table of interrupt handlers
+ *
+ * @param irqcount Count of required hardware IRQs to be supported
+ */
 void ipc_irq_make_table(int irqcount)
 {
         int i;
+
+        irqcount +=  IPC_IRQ_RESERVED_VIRTUAL;
 
         irq_conns_size = irqcount;

generic/src/ipc/sysipc.c

@@ -554 +554 @@
 
 /** Connect irq handler to task */
-__native sys_ipc_register_irq(__native irq, irq_code_t *ucode)
+__native sys_ipc_register_irq(int irq, irq_code_t *ucode)
 {
         if (!(cap_get(TASK) & CAP_IRQ_REG))
                 return EPERM;
 
-        if (irq >= IRQ_COUNT)
+        if (irq >= IRQ_COUNT || irq <= -IPC_IRQ_RESERVED_VIRTUAL)
                 return (__native) ELIMIT;
-
+        
         irq_ipc_bind_arch(irq);
 
@@ -568 +568 @@
 
 /* Disconnect irq handler from task */
-__native sys_ipc_unregister_irq(__native irq)
+__native sys_ipc_unregister_irq(int irq)
 {
         if (!(cap_get(TASK) & CAP_IRQ_REG))
                 return EPERM;
 
-        if (irq >= IRQ_COUNT)
+        if (irq >= IRQ_COUNT || irq <= -IPC_IRQ_RESERVED_VIRTUAL)
                 return (__native) ELIMIT;
 

generic/src/main/main.c

@@ -75 +75 @@
 #include <macros.h>
 #include <adt/btree.h>
+#include <console/klog.h>
 
 #ifdef CONFIG_SMP
@@ -219 +220 @@
         thread_init();
         futex_init();
+        klog_init();
         
         for (i = 0; i < init.cnt; i++)

generic/src/proc/thread.c

@@ -389 +389 @@
         spinlock_unlock(&THREAD->lock);
         scheduler();
+
+        /* Not reached */
+        while (1)
+                ;
 }