Changeset d70fc74 in mainline for kernel/test/smpcall/smpcall1.c

Timestamp:

2012-07-06T12:58:58Z (12 years ago)

Author:

Adam Hraska <adam.hraska+hos@…>

Branches:

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

Children:

ef1603b

Parents:

1f8c11f

Message:

smp_call: Minor fixes and comments.

File:

• kernel/test/smpcall/smpcall1.c (modified) (9 diffs)

kernel/test/smpcall/smpcall1.c

@@ -14 +14 @@
 /* 
  * Maximum total number of smp_calls in the system is: 
- *  128000 == 8^2 * 1000 * 2 
+ *  162000 == 9^2 * 1000 * 2 
  *  == MAX_CPUS^2 * ITERATIONS * EACH_CPU_INC_PER_ITER
  */
-#define MAX_CPUS   8
+#define MAX_CPUS   9
 #define ITERATIONS 1000
 #define EACH_CPU_INC_PER_ITER 2
@@ -40 +40 @@
         smp_call_t call_info[MAX_CPUS];
         
-        size_t cpu_count = min(config.cpu_count, MAX_CPUS);
+        unsigned int cpu_count = min(config.cpu_active, MAX_CPUS);
         
         for (int iter = 0; iter < ITERATIONS; ++iter) {
@@ -75 +75 @@
 }
 
+static size_t calc_exp_calls(size_t thread_cnt)
+{
+        return thread_cnt * ITERATIONS * EACH_CPU_INC_PER_ITER;
+}
 
 const char *test_smpcall1(void)
@@ -82 +86 @@
         thread_t *thread[MAX_CPUS] = {0};
         
-        unsigned int cpu_count = min(config.cpu_count, MAX_CPUS);
+        unsigned int cpu_count = min(config.cpu_active, MAX_CPUS);
         size_t running_thread_cnt = 0;
 
@@ -100 +104 @@
         }
 
-        TPRINTF("Running %u wired threads.\n", running_thread_cnt);
+        size_t exp_calls = calc_exp_calls(running_thread_cnt);
+        size_t exp_calls_sum = exp_calls * cpu_count;
+        
+        TPRINTF("Running %zu wired threads. Expecting %zu calls. Be patient.\n", 
+                running_thread_cnt, exp_calls_sum);
 
         for (unsigned int i = 0; i < cpu_count; ++i) {
@@ -112 +120 @@
                 if (thread[i] != NULL) {
                         thread_join(thread[i]);
+                        thread_detach(thread[i]);
                 }
         }
 
         TPRINTF("Threads finished. Checking number of smp_call()s.\n");
-        
-        size_t exp_calls = running_thread_cnt * ITERATIONS * EACH_CPU_INC_PER_ITER;
-        size_t exp_calls_sum = exp_calls * cpu_count;
         
         bool ok = true;
@@ -127 +133 @@
                         if (call_cnt[i] != exp_calls) {
                                 ok = false;
-                                TPRINTF("Error: %u instead of %u cpu%u's calls were"
+                                TPRINTF("Error: %zu instead of %zu cpu%u's calls were"
                                         " acknowledged.\n", call_cnt[i], exp_calls, i);
                         } 
@@ -136 +142 @@
         
         if (calls_sum != exp_calls_sum) {
-                TPRINTF("Error: total acknowledged sum: %u instead of %u.\n",
+                TPRINTF("Error: total acknowledged sum: %zu instead of %zu.\n",
                         calls_sum, exp_calls_sum);
                 
@@ -143 +149 @@
         
         if (ok) {
-                TPRINTF("Success: number of received smp_calls is as expected (%u).\n",
+                TPRINTF("Success: number of received smp_calls is as expected (%zu).\n",
                         exp_calls_sum);
                 return NULL;