Changeset 183788f1 in mainline for uspace/tester/thread/thread1.c

Timestamp:

2007-04-09T16:17:25Z (18 years ago)

Author:

Jakub Jermar <jakub@…>

Branches:

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

Children:

36f19c0

Parents:

de33dab

Message:

Remove printf() serialization from thread1 test in tester.
The agreement among developers seems to be that:

• there _must_ be a futex to serialize access to printf()
• there _must_ be pseudo thread serialization in printf()
• the best place for this is vprintf(), the one that goes to console

File:

• uspace/tester/thread/thread1.c (modified) (4 diffs)

uspace/tester/thread/thread1.c

@@ -34 +34 @@
 #include <stdio.h>
 #include <unistd.h>
-#include <futex.h>
 #include "../tester.h"
 
@@ -41 +40 @@
 static bool sh_quiet;
 
-static atomic_t srlz = FUTEX_INITIALIZER;
-
 static void threadtest(void *data)
 {
@@ -48 +45 @@
 
         while (atomic_get(&finish)) {
-                if (!sh_quiet) {
-                        futex_down(&srlz);
+                if (!sh_quiet)
                         printf("%llu ", thread_get_id());
-                        futex_up(&srlz);
-                }
                 usleep(100000);
         }
@@ -75 +69 @@
         }
         
-        if (!quiet) {
-                futex_down(&srlz);
+        if (!quiet)
                 printf("Running threads for 10 seconds...\n");
-                futex_up(&srlz);
-        }
         sleep(10);