Changeset 12765 for source

Timestamp:

Mar 14, 2014, 8:26:33 PM (11 years ago)

Author:

hock@…

Message:

new functionality: SystemQueue::leave() and SystemQueue::join()

( + unit tests)

Location:

source/ariba/utility/system

Files:

• SystemQueue.cpp (modified) (4 diffs)
• SystemQueue.h (modified) (1 diff)

source/ariba/utility/system/SystemQueue.cpp

@@ -76 +76 @@
 void SystemQueue::scheduleEvent( const SystemEvent& event, uint32_t delay )
 {
-    assert ( SysQ->running );  // should we really enforce this?
+//     assert ( SysQ->running );  // should we really enforce this?
+    if ( ! SysQ->running )
+    {
+        logging_debug("/// WARNING: The SystemQueue is NOT RUNNING!");
+    }
     
     // copy
@@ -140 +144 @@
     SysQ.reset( new QueueThread() );
 }
+
+
+void SystemQueue::leave()
+{
+    // signal SysQ to quit (and abort queued events)
+    SysQ->cancel();
+}
+
+void SystemQueue::join()
+{
+    if ( sysq_thread )
+    {
+        logging_debug("/// ... joining SysQ thread");
+        sysq_thread->join();
+    }
+}
+
+
 
 void SystemQueue::dropAll( const SystemEventListener* mlistener)
@@ -303 +325 @@
         if ( timedEventsQ.empty() )
         {
-            logging_debug("/// SysQ is going to sleep.");
+//             logging_debug("/// SysQ is going to sleep.");
             
             this->system_queue_idle.wait( lock );
@@ -310 +332 @@
         else
         {
-            logging_debug( "/// SysQ is going to sleep for "
-                        << ( timedEventsQ.top().deadline - get_clock() ).total_milliseconds()
-                        << "ms. Deadline: " 
-                        << timedEventsQ.top().deadline
-                        << ", Clock: "
-                        << get_clock() );
+//             logging_debug( "/// SysQ is going to sleep for "
+//                         << ( timedEventsQ.top().deadline - get_clock() ).total_milliseconds()
+//                         << "ms. Deadline: " 
+//                         << timedEventsQ.top().deadline
+//                         << ", Clock: "
+//                         << get_clock() );
 
             this->system_queue_idle.timed_wait( lock, timedEventsQ.top().deadline );

source/ariba/utility/system/SystemQueue.h

@@ -117 +117 @@
          * Cancels the system queue and ends the processing after the
          * currently processed event is processed.
+     * 
+     * NOTE: Do not call this function from within a SystemQueue-Event.
+     *   Use SystemQueue::leave() instead.
          *
          * This method is thread-safe.
          */
         void cancel();
+    
+    /**
+     * Like SystemQueue::cancel(), but may only be called from within a SystemQueue-Event.
+     * 
+     * NOTE: In this case some cleanup can not be made. -- If the SystemQueue is 
+     *   restarted, SystemQueue::cancel() must be called before SystemQueue::run()
+     *   can be called again.
+     */
+    void leave();
+    
+    /**
+     * Join the SystemQueue thread -- the current thread is blocked until the
+     * SystemQueue finishes.
+     * 
+     * NOTE: Use this only in combination with SystemQueue::leave()
+     * 
+     * [ There is a possible race condition with SystemQueue::cancel(), but 
+     *   SystemQueue::join() should not be used at the same time as 
+     *   SystemQueue::cancel() anyway. (SystemQueue::leave() is fine, though.)
+     */
+    void join();
 
         /**