Fixes VFS open bug
[akaros.git] / kern / src / printf.c
index e870dca..43b6d01 100644 (file)
@@ -1,14 +1,18 @@
 // Simple implementation of cprintf console output for the kernel,
 // based on printfmt() and the kernel console's cputchar().
 
+#ifdef __SHARC__
+#pragma nosharc
+#endif
+
 #include <arch/arch.h>
-#include <arch/types.h>
+#include <ros/common.h>
 
 #include <atomic.h>
 #include <stdio.h>
 #include <stdarg.h>
 
-uint32_t output_lock = 0;
+spinlock_t output_lock = SPINLOCK_INITIALIZER_IRQSAVE;
 
 void putch(int ch, int **cnt)
 {
@@ -22,8 +26,8 @@ void putch(int ch, int **cnt)
 void buffered_putch(int ch, int **cnt)
 {
        #define buffered_putch_bufsize 64
-       static char buf[buffered_putch_bufsize];
-       static int buflen = 0;
+       static char LCKD(&output_lock) (RO buf)[buffered_putch_bufsize];
+       static int LCKD(&output_lock) buflen = 0;
 
        if(ch != -1)
        {
@@ -44,16 +48,33 @@ int vcprintf(const char *fmt, va_list ap)
        int *cntp = &cnt;
        volatile int i;
 
-       // lock all output.  this will catch any printfs at line granularity
+       /* lock all output.  this will catch any printfs at line granularity.  when
+        * tracing, we short-circuit the main lock call, so as not to clobber the
+        * results as we print. */
+       #ifdef CONFIG_TRACE_LOCKS
+       int8_t irq_state = 0;
+       disable_irqsave(&irq_state);
+       __spin_lock(&output_lock);
+       #else
        spin_lock_irqsave(&output_lock);
+       #endif
 
        // do the buffered printf
+       #ifdef __DEPUTY__
        vprintfmt(buffered_putch, &cntp, fmt, ap);
+       #else
+       vprintfmt((void*)buffered_putch, (void*)&cntp, fmt, ap);
+       #endif
 
        // write out remaining chars in the buffer
        buffered_putch(-1,&cntp);
 
+       #ifdef CONFIG_TRACE_LOCKS
+       __spin_unlock(&output_lock);
+       enable_irqsave(&irq_state);
+       #else
        spin_unlock_irqsave(&output_lock);
+       #endif
 
        return cnt;
 }
@@ -63,6 +84,9 @@ int cprintf(const char *fmt, ...)
        va_list ap;
        int cnt;
 
+       if (!fmt)
+               return 0;
+
        va_start(ap, fmt);
        cnt = vcprintf(fmt, ap);
        va_end(ap);