Changes in kernel/test/mm/falloc2.c [6eef3c4:9d58539] in mainline

File:

• kernel/test/mm/falloc2.c (modified) (8 diffs)

kernel/test/mm/falloc2.c

@@ -40 +40 @@
 #include <arch.h>
 
-#define MAX_FRAMES  256U
+#define MAX_FRAMES  256
 #define MAX_ORDER   8
 
@@ -51 +51 @@
 static void falloc(void *arg)
 {
+        int order, run, allocated, i;
         uint8_t val = THREAD->tid % THREADS;
+        size_t k;
         
-        void **frames = (void **)
-            malloc(MAX_FRAMES * sizeof(void *), FRAME_ATOMIC);
+        void **frames =  (void **) malloc(MAX_FRAMES * sizeof(void *), FRAME_ATOMIC);
         if (frames == NULL) {
-                TPRINTF("Thread #%" PRIu64 " (cpu%u): "
-                    "Unable to allocate frames\n", THREAD->tid, CPU->id);
+                TPRINTF("Thread #%" PRIu64 " (cpu%u): Unable to allocate frames\n", THREAD->tid, CPU->id);
                 atomic_inc(&thread_fail);
                 atomic_dec(&thread_count);
@@ -65 +65 @@
         thread_detach(THREAD);
         
-        for (unsigned int run = 0; run < THREAD_RUNS; run++) {
-                for (unsigned int order = 0; order <= MAX_ORDER; order++) {
-                        TPRINTF("Thread #%" PRIu64 " (cpu%u): "
-                            "Allocating %u frames blocks ... \n", THREAD->tid,
-                            CPU->id, 1 << order);
+        for (run = 0; run < THREAD_RUNS; run++) {
+                for (order = 0; order <= MAX_ORDER; order++) {
+                        TPRINTF("Thread #%" PRIu64 " (cpu%u): Allocating %d frames blocks ... \n", THREAD->tid, CPU->id, 1 << order);
                         
-                        unsigned int allocated = 0;
-                        for (unsigned int i = 0; i < (MAX_FRAMES >> order); i++) {
-                                frames[allocated] =
-                                    frame_alloc(order, FRAME_ATOMIC | FRAME_KA);
+                        allocated = 0;
+                        for (i = 0; i < (MAX_FRAMES >> order); i++) {
+                                frames[allocated] = frame_alloc(order, FRAME_ATOMIC | FRAME_KA);
                                 if (frames[allocated]) {
                                         memsetb(frames[allocated], FRAME_SIZE << order, val);
@@ -82 +79 @@
                         }
                         
-                        TPRINTF("Thread #%" PRIu64 " (cpu%u): "
-                            "%u blocks allocated.\n", THREAD->tid, CPU->id,
-                            allocated);
-                        TPRINTF("Thread #%" PRIu64 " (cpu%u): "
-                            "Deallocating ... \n", THREAD->tid, CPU->id);
+                        TPRINTF("Thread #%" PRIu64 " (cpu%u): %d blocks allocated.\n", THREAD->tid, CPU->id, allocated);
+                        TPRINTF("Thread #%" PRIu64 " (cpu%u): Deallocating ... \n", THREAD->tid, CPU->id);
                         
-                        for (unsigned int i = 0; i < allocated; i++) {
-                                for (size_t k = 0; k <= (((size_t) FRAME_SIZE << order) - 1);
-                                    k++) {
+                        for (i = 0; i < allocated; i++) {
+                                for (k = 0; k <= (((size_t) FRAME_SIZE << order) - 1); k++) {
                                         if (((uint8_t *) frames[i])[k] != val) {
-                                                TPRINTF("Thread #%" PRIu64 " (cpu%u): "
-                                                    "Unexpected data (%c) in block %p offset %zu\n",
-                                                    THREAD->tid, CPU->id, ((char *) frames[i])[k],
-                                                    frames[i], k);
+                                                TPRINTF("Thread #%" PRIu64 " (cpu%u): Unexpected data (%c) in block %p offset %zu\n",
+                                                    THREAD->tid, CPU->id, ((char *) frames[i])[k], frames[i], k);
                                                 atomic_inc(&thread_fail);
                                                 goto cleanup;
@@ -103 +94 @@
                         }
                         
-                        TPRINTF("Thread #%" PRIu64 " (cpu%u): "
-                            "Finished run.\n", THREAD->tid, CPU->id);
+                        TPRINTF("Thread #%" PRIu64 " (cpu%u): Finished run.\n", THREAD->tid, CPU->id);
                 }
         }
@@ -111 +101 @@
         free(frames);
         
-        TPRINTF("Thread #%" PRIu64 " (cpu%u): Exiting\n",
-            THREAD->tid, CPU->id);
+        TPRINTF("Thread #%" PRIu64 " (cpu%u): Exiting\n", THREAD->tid, CPU->id);
         atomic_dec(&thread_count);
 }
@@ -118 +107 @@
 const char *test_falloc2(void)
 {
+        unsigned int i;
+        
         atomic_set(&thread_count, THREADS);
         atomic_set(&thread_fail, 0);
         
-        for (unsigned int i = 0; i < THREADS; i++) {
-                thread_t *thrd = thread_create(falloc, NULL, TASK,
-                    THREAD_FLAG_NONE, "falloc2");
+        for (i = 0; i < THREADS; i++) {
+                thread_t * thrd = thread_create(falloc, NULL, TASK, 0, "falloc", false);
                 if (!thrd) {
                         TPRINTF("Could not create thread %u\n", i);
@@ -132 +122 @@
         
         while (atomic_get(&thread_count) > 0) {
-                TPRINTF("Threads left: %" PRIua "\n",
-                    atomic_get(&thread_count));
+                TPRINTF("Threads left: %" PRIua "\n", atomic_get(&thread_count));
                 thread_sleep(1);
         }