Changeset 1433ecda in mainline for kernel/test

Timestamp:

2018-04-04T15:42:37Z (7 years ago)

Author:

Jiri Svoboda <jiri@…>

Branches:

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

Children:

2c4e1cc

Parents:

47b2d7e3

Message:

Fix cstyle: make ccheck-fix and commit only files where all the changes are good.

Location:

kernel/test

Files:

• avltree/avltree1.c (modified) (2 diffs)
• debug/mips1.c (modified) (1 diff)
• mm/slab1.c (modified) (1 diff)
• mm/slab2.c (modified) (2 diffs)
• smpcall/smpcall1.c (modified) (9 diffs)
• synch/workqueue2.c (modified) (1 diff)

kernel/test/avltree/avltree1.c

@@ -50 +50 @@
 static avltree_node_t *test_tree_parents(avltree_node_t *node);
 static void print_tree_structure_flat (avltree_node_t *node, int level)
-    __attribute__ ((used));
+    __attribute__((used));
 static avltree_node_t *alloc_avltree_node(void);
 
@@ -72 +72 @@
                         TPRINTF("Bad parent pointer key: %" PRIu64
                             ", address: %p\n",
-                            tmp->key,node->rgt);
+                            tmp->key, node->rgt);
                 }
         }

kernel/test/debug/mips1.c

@@ -42 +42 @@
 
         asm volatile (
-                "break\n"
+            "break\n"
         );
 

kernel/test/mm/slab1.c

@@ -132 +132 @@
         for (j = 0; j < 10; j++) {
                 for (i = 0; i < THR_MEM_COUNT; i++)
-                        thr_data[offs][i] = slab_alloc(thr_cache,0);
+                        thr_data[offs][i] = slab_alloc(thr_cache, 0);
                 for (i = 0; i < THR_MEM_COUNT / 2; i++)
                         slab_free(thr_cache, thr_data[offs][i]);

kernel/test/mm/slab2.c

@@ -141 +141 @@
 
         mutex_lock(&starter_mutex);
-        condvar_wait(&thread_starter,&starter_mutex);
+        condvar_wait(&thread_starter, &starter_mutex);
         mutex_unlock(&starter_mutex);
 
@@ -209 +209 @@
 
         thr_cache = slab_cache_create("thread_cache", size, 0, NULL, NULL, 0);
-        semaphore_initialize(&thr_sem,0);
+        semaphore_initialize(&thr_sem, 0);
         for (i = 0; i < THREADS; i++) {
                 if (!(t = thread_create(slabtest, NULL, TASK, THREAD_FLAG_NONE, "slabtest"))) {

kernel/test/smpcall/smpcall1.c

@@ -53 +53 @@
         assert(interrupts_disabled());
 
-        size_t *pcall_cnt = (size_t*)p;
+        size_t *pcall_cnt = (size_t *)p;
         /*
          * No synchronization. Tests if smp_calls makes changes
@@ -64 +64 @@
 static void test_thread(void *p)
 {
-        size_t *pcall_cnt = (size_t*)p;
+        size_t *pcall_cnt = (size_t *)p;
         smp_call_t call_info[MAX_CPUS];
 
@@ -84 +84 @@
                  * same counter would clobber it without additional synchronization.
                  */
-                size_t local_cnt[MAX_CPUS] = {0};
+                size_t local_cnt[MAX_CPUS] = { 0 };
 
                 /* Now start asynchronous calls. */
@@ -110 +110 @@
 {
         /* Number of received calls that were sent by cpu[i]. */
-        size_t call_cnt[MAX_CPUS] = {0};
+        size_t call_cnt[MAX_CPUS] = { 0 };
         thread_t *thread[MAX_CPUS] = { NULL };
 
@@ -121 +121 @@
         for (unsigned int id = 0; id < cpu_count; ++id) {
                 thread[id] = thread_create(test_thread, &call_cnt[id], TASK,
-                        THREAD_FLAG_NONE, "smp-call-test");
+                    THREAD_FLAG_NONE, "smp-call-test");
 
                 if (thread[id]) {
@@ -135 +135 @@
 
         TPRINTF("Running %zu wired threads. Expecting %zu calls. Be patient.\n",
-                running_thread_cnt, exp_calls_sum);
+            running_thread_cnt, exp_calls_sum);
 
         for (unsigned int i = 0; i < cpu_count; ++i) {
@@ -161 +161 @@
                                 ok = false;
                                 TPRINTF("Error: %zu instead of %zu cpu%zu's calls were"
-                                        " acknowledged.\n", call_cnt[i], exp_calls, i);
+                                    " acknowledged.\n", call_cnt[i], exp_calls, i);
                         }
                 }
@@ -170 +170 @@
         if (calls_sum != exp_calls_sum) {
                 TPRINTF("Error: total acknowledged sum: %zu instead of %zu.\n",
-                        calls_sum, exp_calls_sum);
+                    calls_sum, exp_calls_sum);
 
                 ok = false;
@@ -177 +177 @@
         if (ok) {
                 TPRINTF("Success: number of received smp_calls is as expected (%zu).\n",
-                        exp_calls_sum);
+                    exp_calls_sum);
                 return NULL;
         } else

kernel/test/synch/workqueue2.c

@@ -112 +112 @@
 {
         TPRINTF("Stress testing a custom queue. Stops prematurely. "
-                "Errors are expected.\n");
+            "Errors are expected.\n");
         test_custom_workq_impl(true, "test-workq-stop");
         /* Errors are expected. */