]> git.ipfire.org Git - thirdparty/tor.git/commitdiff
Fix a pair of stale comments in workqueue.c
authorNick Mathewson <nickm@torproject.org>
Wed, 12 Jul 2017 15:52:13 +0000 (11:52 -0400)
committerNick Mathewson <nickm@torproject.org>
Thu, 27 Jul 2017 20:28:59 +0000 (16:28 -0400)
These comments said that each thread had a separate queue, but we
haven't been using that design for some while.

src/common/workqueue.c

index a372501f11fc2977f95ff8e30481da062b8efa53..f44f566d21b5873ef893e8a182f6576725634aaa 100644 (file)
@@ -113,9 +113,7 @@ struct replyqueue_s {
   alert_sockets_t alert;
 };
 
-/** A worker thread represents a single thread in a thread pool.  To avoid
- * contention, each gets its own queue. This breaks the guarantee that that
- * queued work will get executed strictly in order. */
+/** A worker thread represents a single thread in a thread pool. */
 typedef struct workerthread_s {
   /** Which thread it this?  In range 0..in_pool->n_threads-1 */
   int index;
@@ -368,7 +366,7 @@ workerthread_new(int32_t lower_priority_chance,
  * take from the queued work with the highest prioirity, but will occasionally
  * visit lower-priority queues to keep them from starving completely.
  *
- * Note that because each thread has its own work queue, work items may not
+ * Note that because of priorities and thread behavior, work items may not
  * be executed strictly in order.
  */
 workqueue_entry_t *