Changeset de33dab in mainline for uspace/libc/generic/io/printf_core.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/printf_core.c (modified) (3 diffs)

uspace/libc/generic/io/printf_core.c

@@ -41 +41 @@
 #include <ctype.h>
 #include <string.h>
+#include <async.h>      /* for pseudo thread serialization */
 
 #define __PRINTF_FLAG_PREFIX            0x00000001      /**< show prefixes 0x or 0*/
@@ -443 +444 @@
         int width, precision;
         uint64_t flags;
+        
+        /* Don't let other pseudo threads interfere. */
+        async_serialize_start();
         
         counter = 0;
@@ -675 +679 @@
         }
         
+        async_serialize_end();
         return counter;
 minus_out:
+        async_serialize_end();
         return -counter;
 }