Changeset 712c4ba in mainline for kernel/generic/src/printf/vprintf.c

Timestamp:

2011-05-20T16:09:24Z (13 years ago)

Author:

Martin Decky <martin@…>

Branches:

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

Children:

b6f3e7e

Parents:

326bf65

Message:

avoid spinlocks to be taken in the direct code path to physically outputing kconsole characters to the framebuffer (this fixes ticket #243)
the spinlock in printf() (kernel) was sacrificed, this might lead potentially to scrambled kconsole output
the hack of ignoring spinlocks whose names start with "*" in the livelock detector can be removed

File:

• kernel/generic/src/printf/vprintf.c (modified) (2 diffs)

kernel/generic/src/printf/vprintf.c

@@ -41 +41 @@
 #include <typedefs.h>
 #include <str.h>
-
-IRQ_SPINLOCK_STATIC_INITIALIZE_NAME(printf_lock, "*printf_lock");
 
 static int vprintf_str_write(const char *str, size_t size, void *data)
@@ -93 +91 @@
         };
         
-        irq_spinlock_lock(&printf_lock, true);
-        int ret = printf_core(fmt, &ps, ap);
-        irq_spinlock_unlock(&printf_lock, true);
-        
-        return ret;
+        return printf_core(fmt, &ps, ap);
 }