Changeset b584cd4 in mainline

Timestamp:

2010-06-11T23:56:16Z (14 years ago)

Author:

Martin Decky <martin@…>

Branches:

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

Children:

fd3a631f

Parents:

214ec25c

Message:

Update thread's last_cycle seems to be actually needed to properly account kernel processing time.

Current limitation: The exception handler which executes clock() (and eventually scheduler()) gets charged with all the cycles of the other threads which are being scheduled as a consequence of the preemption.
TODO: Examine the "measure2" branch whether it has a solution or look for a solution.

File:

• kernel/generic/src/interrupt/interrupt.c (modified) (2 diffs)

kernel/generic/src/interrupt/interrupt.c

@@ -103 +103 @@
 #endif
         
-        uint64_t begin_cycle = get_cycle();
-        
         /* Account user cycles */
         if (THREAD) {
                 irq_spinlock_lock(&THREAD->lock, false);
-                THREAD->ucycles += begin_cycle - THREAD->last_cycle;
+                thread_update_accounting(true);
                 irq_spinlock_unlock(&THREAD->lock, false);
         }
+        
+        uint64_t begin_cycle = get_cycle();
         
 #ifdef CONFIG_UDEBUG
@@ -130 +130 @@
         /* Account exception handling */
         uint64_t end_cycle = get_cycle();
+        
         exc_table[n].cycles += end_cycle - begin_cycle;
         exc_table[n].count++;