Changeset f0fcb04 in mainline for kernel/generic/include

Timestamp:

2012-07-29T19:26:32Z (13 years ago)

Author:

Adam Hraska <adam.hraska+hos@…>

Branches:

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

Children:

d4d36f9

Parents:

5b03a72

Message:

rcu: Replaced checking three variables to see if the detector needs to be notified in rcu_read_unlock() with a single bool cpu.signal_unlock.

Location:

kernel/generic/include/synch

Files:

• rcu.h (modified) (1 diff)
• rcu_types.h (modified) (1 diff)

kernel/generic/include/synch/rcu.h

@@ -200 +200 @@
                 /* 
                  * The thread was preempted while in a critical section or 
-                 * the detector is eagerly waiting for this cpu's reader 
-                 * to finish. 
-                 * 
-                 * Note that THREAD may be 0 in scheduler() and not just during boot.
+                 * the detector is eagerly waiting for this cpu's reader to finish. 
                  */
-                if ((THREAD && THREAD->rcu.was_preempted) || CPU->rcu.is_delaying_gp) {
+                if (CPU->rcu.signal_unlock) {
                         /* Rechecks with disabled interrupts. */
                         _rcu_signal_read_unlock();
                 }
         }
-
         
         preemption_enable();

kernel/generic/include/synch/rcu_types.h

@@ -107 +107 @@
         bool is_delaying_gp;
         
+        /** True if we should signal the detector that we exited a reader section.
+         * 
+         * Equal to (THREAD->rcu.was_preempted || CPU->rcu.is_delaying_gp).
+         */
+        bool signal_unlock;
+        
         /** Positive if there are callbacks pending in arriving_cbs. */
         semaphore_t arrived_flag;