Changeset fd1f65e for libcfa/src/concurrency/ready_queue.cfa

Timestamp:

Apr 1, 2021, 8:02:19 PM (3 years ago)

Author:

Thierry Delisle <tdelisle@…>

Branches:

ADT, arm-eh, ast-experimental, enum, forall-pointer-decay, jacob/cs343-translation, master, new-ast-unique-expr, pthread-emulation, qualifiedEnum

Children:

a344425

Parents:

b580bcc

Message:

Stats now keep track of external pushes separately

File:

• libcfa/src/concurrency/ready_queue.cfa (modified) (3 diffs)

libcfa/src/concurrency/ready_queue.cfa

@@ -244 +244 @@
         __cfadbg_print_safe(ready_queue, "Kernel : Pushing %p on cluster %p\n", thrd, cltr);
 
+        #if !defined(__CFA_NO_STATISTICS__)
+                const bool external = cltr == kernelTLS().this_processor->cltr;
+        #endif
+
         // write timestamp
         thrd->link.ts = rdtscl();
@@ -266 +270 @@
                 [i, local] = idx_from_r(r, preferred);
 
+                i %= __atomic_load_n( &lanes.count, __ATOMIC_RELAXED );
+
                 #if !defined(__CFA_NO_STATISTICS__)
-                        if(local) {
-                                __tls_stats()->ready.pick.push.local++;
-                        }
-                #endif
-
-                i %= __atomic_load_n( &lanes.count, __ATOMIC_RELAXED );
-
-                #if !defined(__CFA_NO_STATISTICS__)
-                        __tls_stats()->ready.pick.push.attempt++;
+                        if(external) {
+                                if(local) __atomic_fetch_add(&cltr->stats->ready.pick.ext.local, 1, __ATOMIC_RELAXED);
+                                __atomic_fetch_add(&cltr->stats->ready.pick.ext.attempt, 1, __ATOMIC_RELAXED);
+                        }
+                        else {
+                                if(local) __tls_stats()->ready.pick.push.local++;
+                                __tls_stats()->ready.pick.push.attempt++;
+                        }
                 #endif
 
@@ -310 +315 @@
         // Update statistics
         #if !defined(__CFA_NO_STATISTICS__)
-                #if defined(BIAS)
-                        if( local ) __tls_stats()->ready.pick.push.lsuccess++;
-                #endif
-                __tls_stats()->ready.pick.push.success++;
+                if(external) {
+                        if(local) __atomic_fetch_add(&cltr->stats->ready.pick.ext.lsuccess, 1, __ATOMIC_RELAXED);
+                        __atomic_fetch_add(&cltr->stats->ready.pick.ext.success, 1, __ATOMIC_RELAXED);
+                }
+                else {
+                        if(local) __tls_stats()->ready.pick.push.lsuccess++;
+                        __tls_stats()->ready.pick.push.success++;
+                }
         #endif