Changeset de33dab in mainline for uspace/libc/generic/io/vprintf.c

Timestamp:

2007-04-09T13:53:57Z (18 years ago)

Author:

Jakub Jermar <jakub@…>

Branches:

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

Children:

183788f1

Parents:

d0b1443

Message:

Revert some of the changes introduced in revision 2209.
I think it is not correct to remove serialization of pseudo threads in printf_core.c.
With thread-level futex serialization, several pseudo threads running in one thread could easily deadlock
the task.

Add a dedicated futex serialization to thread1.c test only.

File:

• uspace/libc/generic/io/vprintf.c (modified) (2 diffs)

uspace/libc/generic/io/vprintf.c

@@ -37 +37 @@
 #include <unistd.h>
 #include <io/printf_core.h>
-#include <futex.h>
-
-atomic_t printf_futex = FUTEX_INITIALIZER;
 
 static int vprintf_write(const char *str, size_t count, void *unused)
@@ -55 +52 @@
         struct printf_spec ps = {(int(*)(void *, size_t, void *)) vprintf_write, NULL};
         
-        futex_down(&printf_futex);
         int ret = printf_core(fmt, &ps, ap);
-        futex_up(&printf_futex);
         
         return ret;