diff options
| author | Nick Mathewson <nickm@torproject.org> | 2017-07-12 11:52:13 -0400 |
|---|---|---|
| committer | Nick Mathewson <nickm@torproject.org> | 2017-07-27 16:28:59 -0400 |
| commit | bddea78dede010cdf0115265546cdca0c47d64d0 (patch) | |
| tree | ab61ce1cf104d05c5942d091e784132b60302a0d | |
| parent | efadebf7c37d6ed309a2e54f4679fbfe3af0bcdf (diff) | |
| download | tor-bddea78dede010cdf0115265546cdca0c47d64d0.tar.gz tor-bddea78dede010cdf0115265546cdca0c47d64d0.zip | |
Fix a pair of stale comments in workqueue.c
These comments said that each thread had a separate queue, but we
haven't been using that design for some while.
| -rw-r--r-- | src/common/workqueue.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/src/common/workqueue.c b/src/common/workqueue.c index a372501f11..f44f566d21 100644 --- a/src/common/workqueue.c +++ b/src/common/workqueue.c @@ -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 * |